Skip to content
This repository has been archived by the owner on Oct 13, 2023. It is now read-only.

[18.09] Backport Buildkit fixes for 18.09 #59

Merged
merged 7 commits into from
Sep 22, 2018

Conversation

tonistiigi
Copy link

@andrewhsu @tiborvass @AntaresS

tonistiigi and others added 7 commits September 21, 2018 17:06
Signed-off-by: Tonis Tiigi <[email protected]>
… is expected now with buildkit

Signed-off-by: Tibor Vass <[email protected]>
(cherry picked from commit 5aa222d)
Signed-off-by: Tonis Tiigi <[email protected]>
Signed-off-by: Anda Xu <[email protected]>
(cherry picked from commit 308701f)
(cherry picked from commit b48afc2)
Signed-off-by: Tonis Tiigi <[email protected]>
Signed-off-by: Anda Xu <[email protected]>
(cherry picked from commit 171d51c)
(cherry picked from commit a72752b)
Signed-off-by: Tonis Tiigi <[email protected]>
This allows users to configure the buildkit GC.

The following enables the default GC:
```
{
  "builder": {
    "gc": {
      "enabled": true
    }
  }
}
```

The default GC policy has a simple config:
```
{
  "builder": {
    "gc": {
      "enabled": true,
      "defaultKeepStorage": "30GB"
    }
  }
}
```

A custom GC policy can be used instead by specifying a list of cache prune rules:
```
{
  "builder": {
    "gc": {
      "enabled": true,
      "policy": [
        {"keepStorage": "512MB", "filter": ["unused-for=1400h"]]},
        {"keepStorage": "30GB", "all": true}
      ]
    }
  }
}
```

Signed-off-by: Tibor Vass <[email protected]>
(cherry picked from commit 4a776d0)
Signed-off-by: Tonis Tiigi <[email protected]>
Signed-off-by: Tibor Vass <[email protected]>
(cherry picked from commit d0f00bc)
Signed-off-by: Tonis Tiigi <[email protected]>
Copy link

@andrewhsu andrewhsu left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants