Skip to content

Update snapshots docs with garbage collector label#2962

Merged
estesp merged 1 commit intocontainerd:masterfrom
ostenbom:snapshots-docs
Jan 29, 2019
Merged

Update snapshots docs with garbage collector label#2962
estesp merged 1 commit intocontainerd:masterfrom
ostenbom:snapshots-docs

Conversation

@ostenbom
Copy link
Copy Markdown
Contributor

@ostenbom ostenbom commented Jan 29, 2019

Fixes #2941. When following the normal snapshots tutorial, any delay or unpacking large layers lets the garbage collector delete the snapshot in use. Add a no garbage collector label to the docs to make that clear.
Also, calling Commit should remove the active snapshot so no need to do that in the tutorial.

Signed-off-by: Oliver Stenbom [email protected]

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #2962 into master will decrease coverage by 3.64%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2962      +/-   ##
==========================================
- Coverage   47.62%   43.97%   -3.65%     
==========================================
  Files          92      102      +10     
  Lines        8517    10874    +2357     
==========================================
+ Hits         4056     4782     +726     
- Misses       3734     5359    +1625     
- Partials      727      733       +6
Flag Coverage Δ
#linux 47.62% <ø> (ø) ⬆️
#windows 41.19% <ø> (?)
Impacted Files Coverage Δ
snapshots/native/native.go 43.04% <0%> (-9.99%) ⬇️
metadata/snapshot.go 45.8% <0%> (-8.96%) ⬇️
archive/tar.go 43.79% <0%> (-7.07%) ⬇️
metadata/containers.go 47.97% <0%> (-6.62%) ⬇️
content/local/writer.go 57.84% <0%> (-6.36%) ⬇️
content/local/store.go 48.51% <0%> (-5.03%) ⬇️
archive/tar_opts.go 28.57% <0%> (-4.77%) ⬇️
archive/compression/compression.go 58.69% <0%> (-4.7%) ⬇️
metadata/images.go 58.46% <0%> (-4.7%) ⬇️
metadata/buckets.go 56.33% <0%> (-4.6%) ⬇️
... and 58 more

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 f35c352...682fb99. Read the comment docs.

1 similar comment
@codecov-io
Copy link
Copy Markdown

codecov-io commented Jan 29, 2019

Codecov Report

Merging #2962 into master will decrease coverage by 3.64%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2962      +/-   ##
==========================================
- Coverage   47.62%   43.97%   -3.65%     
==========================================
  Files          92      102      +10     
  Lines        8517    10874    +2357     
==========================================
+ Hits         4056     4782     +726     
- Misses       3734     5359    +1625     
- Partials      727      733       +6
Flag Coverage Δ
#linux 47.62% <ø> (ø) ⬆️
#windows 41.19% <ø> (?)
Impacted Files Coverage Δ
snapshots/native/native.go 43.04% <0%> (-9.99%) ⬇️
metadata/snapshot.go 45.8% <0%> (-8.96%) ⬇️
archive/tar.go 43.79% <0%> (-7.07%) ⬇️
metadata/containers.go 47.97% <0%> (-6.62%) ⬇️
content/local/writer.go 57.84% <0%> (-6.36%) ⬇️
content/local/store.go 48.51% <0%> (-5.03%) ⬇️
archive/tar_opts.go 28.57% <0%> (-4.77%) ⬇️
archive/compression/compression.go 58.69% <0%> (-4.7%) ⬇️
metadata/images.go 58.46% <0%> (-4.7%) ⬇️
metadata/buckets.go 56.33% <0%> (-4.6%) ⬇️
... and 58 more

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 f35c352...682fb99. Read the comment docs.

Comment thread snapshots/snapshotter.go Outdated
@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
Copy link
Copy Markdown
Member

estesp commented Jan 29, 2019

merging per comment re: Windows CI flakiness; this is only edits to comments/documentation

@estesp estesp merged commit 77d7b01 into containerd:master Jan 29, 2019
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