Weekly Report of PouchContainer from 2018-07-26 to 2018-08-02

[pouchrobot]

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
231	2987	771	30	0	0

PR Update

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

feature 🆕 🔫

• feature: add CriEnabled field in api and show this in cli (#1918)
• feature: add memory cgroup check when validate config (#1615)

bugfix 🐛 🔪

• bugfix: cri Test_getSELinuxSecurityOpts (#2031)
• bugfix: fix container can't ping outside and resolve domain names (#2025)
• fix: add more details in the NotFound error (#1960)
• bugfix: add ErrorMessage in jsonstream (#1927)
• bugfix: don't set mount tab when rootfs diskquota is null (#1926)
• bugfix: data race in pkg cri/stream (#1925)
• bugfix: add --name following container name in cmd (#1924)
• bugfix: remove duplicate package in test file (#1896)
• bugfix: avoid task exit channel exit unexpected (#1765)

doc 📜 📝

• doc: fix spelling error (#1961)
• docs: update kubernetes deploying document (#1917)
• docs: add code coverage icon on README (#1893)

test ✅ ☑️

• test: add unit-test for getSELinuxSecurityOpts method which locate on cri/v… (#2023)
• test: add unit-test for toCriSandbox (#2002)
• test: add unit-test for getAppArmorSecurityOpts and getSeccompSecurityOpts (#1948)
• test: unit test for cri/stream/errors.go (#1909)
• test: run with volume whose destination is not empty (#1899)
• test: add test cases about parse object (#1895)
• test: unit test for cri/stream/httpstream/httpstream.go (#1892)
• test: unit test for request cache of stream server (#1767)

others

• bug: fix judgement for SetRootfsDiskQuota (#2028)
• Test_imageToCriImage (#1990)
• BJ266-4: Add unit-test for apis/opts/ports.go, fix [help wanted] add unit-test for apis/opts/ports.go #1906 (#1976)
• add unit test for modifyContainerNamespaceOptions (#1945)
• makefile: simplify makefile output (#1921)
• test: add api tests for some commands (#1747)
• refactor: Extract cni manager as the public part (#1719)

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	15
@allencloud	10
@starnop	7
@Ace-Tang	3
@xiechengsheng	2
@YaoZengzeng	2
@ZouRui89	2
@CodeJuan	1
@HusterWan	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:

@ShallowNight
@kaizizhu
@Coordinate36
@MarkLux
@KevinLeung-nice
@devily

Thank all of you!

[allencloud]

@ZouRui89 The weekly report has incorrect format you see, could you help to fix this?

• contributors number;
• pr update's format

[ZouRui89]

@allencloud Okay. I guess I can check the logic of code review statistic as well. Maybe I have more than 2 reviews. 😭