Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: do not use caching for docker build #2516

Merged
merged 1 commit into from
May 20, 2022

Conversation

nabokihms
Copy link
Member

Signed-off-by: m.nabokikh [email protected]

Overview

Try to build dex without using cache and see the impact

What this PR does / why we need it

There are a lot of failed docker builds, and their nature is unknown. Most of the time job freezes after preparing the cache

2022-05-06T04:34:03.2162689Z 
2022-05-06T04:34:03.2162791Z #79 exporting cache
2022-05-06T04:34:03.2163057Z #79 preparing build cache for export
2022-05-06T04:35:07.8469443Z #79 preparing build cache for export 64.7s done
2022-05-06T10:06:23.6107579Z ##[error]The operation was canceled.
2022-05-06T10:06:23.6170814Z Post job cleanup.
2022-05-06T10:06:23.7425906Z ##[group]Removing temp folder /tmp/docker-build-push-x2GXuu
2022-05-06T10:06:23.7426866Z ##[endgroup]

I would like to build Dex images without caching and see how it affects building time (and the consistency of building results).

Special notes for your reviewer

Does this PR introduce a user-facing change?

NONE

@nabokihms nabokihms marked this pull request as ready for review May 14, 2022 14:39
Copy link
Member

@sagikazarmark sagikazarmark left a comment

Choose a reason for hiding this comment

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

LGTM, let's try this. I also have a support ticket open, hopefully that will result in a permanent solution.

@sagikazarmark sagikazarmark merged commit 92161ab into dexidp:master May 20, 2022
@nabokihms nabokihms deleted the no-cache-build branch May 20, 2022 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants