cli icon indicating copy to clipboard operation
cli copied to clipboard

Documentation for builder prune.

Open adeniyistephen opened this issue 4 years ago • 8 comments

- What I did Documentation of builder prune with its inputs and examples for #2325 - How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

adeniyistephen avatar Dec 06 '20 16:12 adeniyistephen

Codecov Report

Merging #2864 (844643a) into master (dca98e3) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2864   +/-   ##
=======================================
  Coverage   57.08%   57.08%           
=======================================
  Files         297      297           
  Lines       18642    18642           
=======================================
  Hits        10642    10642           
  Misses       7141     7141           
  Partials      859      859           

codecov-io avatar Dec 06 '20 16:12 codecov-io

@thaJeztah @tiborvass PTAL

adeniyistephen avatar Dec 09 '20 15:12 adeniyistephen

@tiborvass PTAL 🤗

thaJeztah avatar Dec 22 '20 15:12 thaJeztah

oh, and probably @tonistiigi ptal

thaJeztah avatar Dec 22 '20 15:12 thaJeztah

@tonistiigi @thaJeztah Could you PTAL?

usha-mandya avatar Jun 30 '21 10:06 usha-mandya

@docker/build PTAL

thaJeztah avatar Jan 20 '22 08:01 thaJeztah

Nice docs! Looking at #3314, I wonder if this should not be moved to https://github.com/docker/buildx/blob/master/docs/reference/buildx_prune.md?

crazy-max avatar Jan 20 '22 08:01 crazy-max

@crazy-max Agree for the prune command. For the garbage collection, maybe somewhere around https://docs.docker.com/engine/reference/commandline/dockerd/ but @thaJeztah knows better.

tonistiigi avatar Jan 20 '22 08:01 tonistiigi

I don't really understand last example.

"gc": {
      "enabled": true,
      "policy": [
            {"keepStorage": "10GB", "filter": ["unused-for=2200h"]},
            {"keepStorage": "50GB", "filter": {"unused-for": {"3300h": true}}},
            {"keepStorage": "100GB", "all": true}
        ]
    }

if the build cache is more than 10GB delete every unused build cache that are more than 92 days old

next rule should remove every cache that are more than 136 days old

If we already deleted every unused build cache that's more than 92 days old, there should not be any caches left which are more than 136 days old. It's confusing.

vbezhenar avatar Dec 01 '22 08:12 vbezhenar

Looks like the GC docs are now already mostly covered on this page in docs; https://docs.docker.com/build/cache/garbage-collection/

Would be nice to merge the description for prune though.

dvdksn avatar Dec 14 '22 08:12 dvdksn

Codecov Report

Merging #2864 (844643a) into master (dca98e3) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 844643a differs from pull request most recent head 9f542d1. Consider uploading reports for the commit 9f542d1 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2864   +/-   ##
=======================================
  Coverage   57.08%   57.08%           
=======================================
  Files         297      297           
  Lines       18642    18642           
=======================================
  Hits        10642    10642           
  Misses       7141     7141           
  Partials      859      859           

codecov-commenter avatar Jan 28 '23 14:01 codecov-commenter

Why is this not merged yet ?

bdrnglm avatar Jul 27 '23 14:07 bdrnglm