Skip to content

[cherry-pick] rootfs: remove upper snapshot after use#1954

Merged
crosbymichael merged 1 commit intocontainerd:release/1.0from
crosbymichael:cherry-diff
Jan 3, 2018
Merged

[cherry-pick] rootfs: remove upper snapshot after use#1954
crosbymichael merged 1 commit intocontainerd:release/1.0from
crosbymichael:cherry-diff

Conversation

@crosbymichael
Copy link
Copy Markdown
Member

cherry-pick of #1953

Signed-off-by: Jess Valarezo [email protected]

@crosbymichael crosbymichael changed the title rootfs: remove upper snapshot after use [cherry-pick] rootfs: remove upper snapshot after use Jan 3, 2018
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #1954 into release/1.0 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           release/1.0   #1954   +/-   ##
===========================================
  Coverage         50.7%   50.7%           
===========================================
  Files               81      81           
  Lines             7161    7161           
===========================================
  Hits              3631    3631           
  Misses            2845    2845           
  Partials           685     685
Flag Coverage Δ
#linux 50.7% <ø> (ø) ⬆️

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 a1a9bda...70bc6db. Read the comment docs.

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

@dmcgowan
Copy link
Copy Markdown
Member

dmcgowan commented Jan 3, 2018

LGTM

1 similar comment
@jessvalarezo
Copy link
Copy Markdown
Contributor

LGTM

@crosbymichael crosbymichael merged commit c59bc7e into containerd:release/1.0 Jan 3, 2018
@crosbymichael crosbymichael deleted the cherry-diff branch January 3, 2018 18:25
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.

5 participants