Skip to content

Update cri to v1.11.0.#2430

Merged
estesp merged 1 commit intocontainerd:masterfrom
Random-Liu:update-cri-to-v1.11.0
Jun 28, 2018
Merged

Update cri to v1.11.0.#2430
estesp merged 1 commit intocontainerd:masterfrom
Random-Liu:update-cri-to-v1.11.0

Conversation

@Random-Liu
Copy link
Copy Markdown
Member

Signed-off-by: Lantao Liu <[email protected]>
@Random-Liu Random-Liu added this to the 1.2 milestone Jun 28, 2018
@crosbymichael
Copy link
Copy Markdown
Member

geez, 23k lines @Random-Liu ;)

@Random-Liu
Copy link
Copy Markdown
Member Author

Random-Liu commented Jun 28, 2018

@crosbymichael Kubernetes api update is always super huge... A lot of auto generated files... The actual code change in cri is not that big.

@crosbymichael
Copy link
Copy Markdown
Member

@Random-Liu ya, i'm just joking

@codecov-io
Copy link
Copy Markdown

codecov-io commented Jun 28, 2018

Codecov Report

Merging #2430 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #2430   +/-   ##
======================================
  Coverage      45%     45%           
======================================
  Files          92      92           
  Lines        9412    9412           
======================================
  Hits         4236    4236           
  Misses       4493    4493           
  Partials      683     683
Flag Coverage Δ
#linux 49.23% <ø> (ø) ⬆️
#windows 41.3% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e51ddf4...f530a3c. Read the comment docs.

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

@estesp estesp merged commit a044b04 into containerd:master Jun 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants