Skip to content

Conversation

@fuweid
Copy link
Contributor

@fuweid fuweid commented Oct 31, 2018

Signed-off-by: Wei Fu [email protected]

Ⅰ. Describe what this PR did

When we release pouch to yum/apt source, always needs to update builder
script. It will change the git commit id. Sometimes, the changes doesn't
belongs to the release version.

Based on this case, using other repo to handle the package builder is
better than current one.

All related builder scripts are moved to https://github.com/pouchcontainer/project.
The project shows us how to build deb/rpm packages.

Ⅱ. Does this pull request fix one issue?

related issues
fixes #2358
fixes #2357
fixes #2280
fixes #2232

Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗?)

no need

Ⅳ. Describe how to verify it

wait for CI

Ⅴ. Special notes for reviews

When we release pouch to yum/apt source, always needs to update builder
script. It will change the git commit id. Sometimes, the changes doesn't
belongs to the release version.

Based on this case, using other repo to handle the package builder is
better than current one.

Signed-off-by: Wei Fu <[email protected]>
@codecov
Copy link

codecov bot commented Oct 31, 2018

Codecov Report

Merging #2398 into master will increase coverage by 4.34%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2398      +/-   ##
==========================================
+ Coverage   64.14%   68.49%   +4.34%     
==========================================
  Files         275      275              
  Lines       18261    18261              
==========================================
+ Hits        11713    12507     +794     
+ Misses       5275     4336     -939     
- Partials     1273     1418     +145
Flag Coverage Δ
#criv1alpha1test 31.85% <ø> (?)
#criv1alpha2test 35.73% <ø> (-0.2%) ⬇️
#integrationtest 39.78% <ø> (-0.01%) ⬇️
#nodee2etest 33.35% <ø> (+0.02%) ⬆️
#unittest 25.5% <ø> (ø) ⬆️
Impacted Files Coverage Δ
apis/server/utils.go 71.15% <0%> (-3.85%) ⬇️
ctrd/container.go 58.8% <0%> (-0.96%) ⬇️
daemon/mgr/container.go 59.58% <0%> (-0.44%) ⬇️
cri/v1alpha2/cri.go 68.05% <0%> (-0.24%) ⬇️
cri/v1alpha2/cri_utils.go 91.09% <0%> (+0.28%) ⬆️
cri/v1alpha2/cri_wrapper.go 62.4% <0%> (+1.19%) ⬆️
ctrd/client.go 69.23% <0%> (+2.3%) ⬆️
daemon/mgr/events.go 100% <0%> (+3.7%) ⬆️
daemon/containerio/cri_log_file.go 88.23% <0%> (+3.92%) ⬆️
ctrd/watch.go 83.33% <0%> (+4.54%) ⬆️
... and 8 more

@fuweid
Copy link
Contributor Author

fuweid commented Oct 31, 2018

@sunyuan3 @chuanchang @Ace-Tang @CodeJuan PTAL Thanks.

@sunyuan3
Copy link
Contributor

sunyuan3 commented Nov 1, 2018

LGTM

@pouchrobot pouchrobot added the LGTM one maintainer or community participant agrees to merge the pull reuqest. label Nov 1, 2018
@sunyuan3 sunyuan3 merged commit a3b5c25 into AliyunContainerService:master Nov 1, 2018
@fuweid fuweid deleted the tools_remove_package_scripts_from_pouch branch November 1, 2018 02:39
@chuanchang
Copy link
Contributor

@fuweid LGTM

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

Labels

LGTM one maintainer or community participant agrees to merge the pull reuqest. size/XXL

Projects

None yet

4 participants