Skip to content

Conversation

@starnop
Copy link
Contributor

@starnop starnop commented Aug 24, 2018

Signed-off-by: Starnop [email protected]

Ⅰ. Describe what this PR did

docs: add CRI enhancement into document

Ⅱ. Does this pull request fix one issue?

fixes #2150

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

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@codecov-io
Copy link

codecov-io commented Aug 24, 2018

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2159      +/-   ##
==========================================
+ Coverage    64.3%   64.36%   +0.05%     
==========================================
  Files         209      209              
  Lines       16693    16709      +16     
==========================================
+ Hits        10735    10754      +19     
- Misses       4618     4623       +5     
+ Partials     1340     1332       -8
Flag Coverage Δ
#criv1alpha1test 32.84% <ø> (+0.01%) ⬆️
#criv1alpha2test 33.57% <ø> (+0.16%) ⬆️
#integrationtest 39.27% <ø> (-0.03%) ⬇️
#unittest 24.01% <ø> (ø) ⬆️
Impacted Files Coverage Δ
daemon/config/config.go 40.62% <0%> (-8.13%) ⬇️
cri/v1alpha2/cri.go 64.01% <0%> (+0.33%) ⬆️
daemon/mgr/container.go 56.39% <0%> (+0.4%) ⬆️
ctrd/container.go 43.19% <0%> (+1.43%) ⬆️
daemon/logger/jsonfile/utils.go 73.17% <0%> (+1.62%) ⬆️
ctrd/watch.go 80.3% <0%> (+7.57%) ⬆️

@starnop starnop force-pushed the cri-apis-change branch 2 times, most recently from c498abd to 3b3e9b7 Compare August 27, 2018 08:11

### What To Solve?

+ After kubelet performs in-situ upgrade, the container after upgrade cannot delete the anonymous volume inherited.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does in-situ mean?


## CreateContainer

###What To Solve?
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a whitespace before What ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@starnop starnop force-pushed the cri-apis-change branch 2 times, most recently from 483fa3d to 8449a0a Compare August 27, 2018 12:19
@allencloud
Copy link
Collaborator

LGTM

@pouchrobot pouchrobot added the LGTM one maintainer or community participant agrees to merge the pull reuqest. label Aug 27, 2018
@allencloud allencloud merged commit e578938 into AliyunContainerService:master Aug 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

areas/docs areas/kubernetes LGTM one maintainer or community participant agrees to merge the pull reuqest. size/XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[doc] add CRI enhancement into document

4 participants