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

[1.1] ci: drop docker layer caching from release job #3479

Merged
merged 1 commit into from
May 26, 2022

Conversation

kolyshkin
Copy link
Contributor

@kolyshkin kolyshkin commented May 23, 2022

This job is failing with "No space left on device" lately, and this
helps to fix this.

Besides, it seems that caching does not help to shorten execution times.

@kolyshkin kolyshkin marked this pull request as ready for review May 23, 2022 20:01
@kolyshkin
Copy link
Contributor Author

No longer a draft; PTAL @AkihiroSuda @thaJeztah

(will do a forward-port to main branch now)

@kolyshkin kolyshkin force-pushed the 1.1-drop-docker-cache branch from 7899fcb to e463265 Compare May 23, 2022 20:03
@kolyshkin kolyshkin added backport/1.1-todo A PR in main branch which needs to be backported to release-1.1 and removed backport/1.1-todo A PR in main branch which needs to be backported to release-1.1 labels May 24, 2022
@kolyshkin kolyshkin changed the title ci: drop docker layer caching from release job [1.1] ci: drop docker layer caching from release job May 24, 2022
@kolyshkin kolyshkin added this to the 1.1.3 milestone May 24, 2022
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

This job is failing with "No space left on device" lately, and this
helps to fix it.

Besides, it seems that caching does not help to shorten execution times
(validate/release job succeeds in under 8 minutes now; ymmv).

Signed-off-by: Kir Kolyshkin <[email protected]>
@kolyshkin kolyshkin force-pushed the 1.1-drop-docker-cache branch from e463265 to 3a09da6 Compare May 26, 2022 20:50
Copy link
Member

@cyphar cyphar left a comment

Choose a reason for hiding this comment

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

LGTM.

@cyphar cyphar merged commit 9d00472 into opencontainers:release-1.1 May 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants