Skip to content

hack: do not cache some stages on release#3543

Merged
tonistiigi merged 1 commit into
moby:masterfrom
crazy-max:ci-nocache
Feb 9, 2023
Merged

hack: do not cache some stages on release#3543
tonistiigi merged 1 commit into
moby:masterfrom
crazy-max:ci-nocache

Conversation

@crazy-max
Copy link
Copy Markdown
Member

@crazy-max crazy-max commented Jan 27, 2023

fixes #3542

Disables cache on release for some stages so packages are always installed and up to date.

Comment thread hack/images Outdated
@crazy-max crazy-max changed the title hack: do not cache buildkit-export stage on ci ci: disable cache on release Jan 31, 2023
Comment thread frontend/dockerfile/cmd/dockerfile-frontend/hack/release Outdated
@crazy-max crazy-max force-pushed the ci-nocache branch 6 times, most recently from 33d29bf to 54aaab9 Compare January 31, 2023 13:23
@crazy-max crazy-max added this to the v0.11.3 milestone Jan 31, 2023
Comment thread .github/workflows/build.yml Outdated
@crazy-max crazy-max force-pushed the ci-nocache branch 2 times, most recently from b00fb23 to 1985212 Compare February 1, 2023 16:14
Copy link
Copy Markdown
Member

@tonistiigi tonistiigi left a comment

Choose a reason for hiding this comment

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

I'm not sure we need to disable cache for releases completely. The cache is already scoped. Having these separate codepaths makes it possible that the CI works fine, we tag a release and that release does not build at all.

@crazy-max crazy-max marked this pull request as draft February 1, 2023 16:50
@crazy-max crazy-max changed the title ci: disable cache on release hack: do not cache some stages on release Feb 2, 2023
@crazy-max crazy-max added area/hack building buildkit itself area/ci labels Feb 2, 2023
@crazy-max crazy-max marked this pull request as ready for review February 2, 2023 19:24
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci area/hack building buildkit itself

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Vulnerabilities issues with latest 0.11.2 image

4 participants