Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Weekly Report of PouchContainer from 2018-08-02 to 2018-08-09 #2067

Closed
pouchrobot opened this issue Aug 9, 2018 · 0 comments
Closed

Weekly Report of PouchContainer from 2018-08-02 to 2018-08-09 #2067

pouchrobot opened this issue Aug 9, 2018 · 0 comments

Comments

@pouchrobot
Copy link
Collaborator

pouchrobot commented Aug 9, 2018

Weekly Report of PouchContainer

This is a weekly report of PouchContainer. It summarizes what have changed in PouchContainer in the passed week, including pr merged, new contributors, and more things in the future.
It is all done by @pouchrobot which is an AI robot. See: https://github.com/pouchcontainer/pouchrobot.

Repo Update

Watch Star Fork Contributors New Issues Closed Issues
232 3004 775 80 0 0

PR Update

Thanks to contributions from community, PouchContainer team merged 25 pull requests in the PouchContainer repositories last week. All these pull requests could be divided into feature, bugfix, doc, test and others:

feature 🆕 🔫

  • feature: support resize the tty of cri exec process (#2063)
  • feature: support event service (#2053)
  • feature: add memory cgroup check when validate config (#1615)

bugfix 🐛 🔪

  • bugfix: makeup logpath from sandbox & container mata (#2054)
  • bugfix: fast fail if the exit code of script is not zero (#2049)
  • bugfix: fix stdin hang when kubectl exec (#2044)
  • bugfix: cpu params json name compatibility with moby (#2041)
  • bugfix: move function getListener() to pkg/net/listener.go (#2040)
  • bugfix: failed to start container when /etc/mtab is symbol link (#2035)
  • bugfix: data race in pkg cri/stream (#1925)
  • bugfix: fix panic for AttachVolume unittests (#1923)

doc 📜 📝

  • docs: update install step in INSTALLATION.md (#2045)

test ✅ ☑️

  • test: add test cases about modifyContainerConfig&containerNetns (#2062)
  • test: add test cases about modifyHostConfig (#2056)
  • test: add test cases about toCNIPortMappings (#2050)
  • test: fix typo in TestAliKernelDiskQuotaWorks (#2048)
  • test: add test cases about toCriContainer (#2039)
  • test: add unit test for generateMountBindings and modifySandboxNamespaceOptions (#1768)

others

  • refactor: move functions from pre-start hook to pouchd (#2064)
  • refactor: add more error information (#2055)
  • refactor: make the API case clear and stable (#2042)
  • refactor: rm deadcode in storage/volume (#2027)
  • refactor: make all quota implementation logic clear (#1736)
  • refactor: merge different validate funtions (#1730)
  • test: add unit-tests for core AttachVolume (#1688)

Code Review Statistics 🐞 🐞 🐞

PouchContainer encourages everyone to participant in code review, in order to improve software quality. Everyweek @pouchrobot would automatically help to count pull request reviews of single github user as the following. So, try to help review code in this project.

Contributor ID Pull Request Reviews
@fuweid 23
@HusterWan 12
@allencloud 11
@Ace-Tang 6
@xiechengsheng 3
@YaoZengzeng 2
@rudyfly 2
@JeromeTan1997 2
@starnop 1

New Contributors 🎖 🎖 🎖

It is PouchContainer team's great honor to have new contributors in Pouch's community. We really appreciate your contributions. Feel free to tell us if you have any opinion and please share PouchContainer with more people if you could. If you hope to be a contributor as well, please start from https://github.com/alibaba/pouch/blob/master/CONTRIBUTING.md . 🎁 👏 🍺
Here is the list of new contributors:
@forienlauo

Thank all of you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants