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

Revert "infra: Improve cache usage for container builds" #5085

Merged
merged 1 commit into from
Aug 28, 2023

Conversation

jkonecny12
Copy link
Member

This reverts commit 5df7a8b.

Seems that cache-ttl is a new option for podman. Unfortunately, it wasn't mentioned in the release notes so not sure in which version it was added.

Anyway let's revert this for now.

This will fix container rebuilds which are broken right now.

@jkonecny12 jkonecny12 added infrastructure Changes affecting mostly infrastructure f39 f40 labels Aug 28, 2023
@jkonecny12
Copy link
Member Author

/kickstart-test --waive infra only

@github-actions github-actions bot removed the infrastructure Changes affecting mostly infrastructure label Aug 28, 2023
@VladimirSlavik
Copy link
Contributor

BTW: containers/podman#19773 looks like it's rather about buildah, but never mind, what matters is that it's missing :-/

This reverts commit 5df7a8b.

Seems that cache-ttl is a new option for podman. Unfortunately, it
wasn't mentioned in the release notes so not sure in which version it
was added.

Anyway let's revert this for now.
@jkonecny12
Copy link
Member Author

/kickstart-test --waive infra only

@jkonecny12 jkonecny12 added the infrastructure Changes affecting mostly infrastructure label Aug 28, 2023
@github-actions
Copy link

Infrastructure check failed on these files. Please do a double check of these files before merge!

Makefile.am

@jkonecny12 jkonecny12 merged commit c0d6562 into rhinstaller:master Aug 28, 2023
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
f39 f40 infrastructure Changes affecting mostly infrastructure
Development

Successfully merging this pull request may close these issues.

2 participants