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

Workaround for Podman issue #4920

Merged
merged 1 commit into from
Jan 15, 2025
Merged

Workaround for Podman issue #4920

merged 1 commit into from
Jan 15, 2025

Conversation

edewata
Copy link
Contributor

@edewata edewata commented Jan 3, 2025

The tests that use Podman inside Docker have been modified to update the graph driver in the SQLite backend as a workaround for a known issue in Podman on Ubuntu 24 or later.

containers/podman#21683

@edewata edewata requested a review from fmarco76 January 3, 2025 20:53
Copy link
Member

@fmarco76 fmarco76 left a comment

Choose a reason for hiding this comment

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

The CI still fails with the same error related to the overlay

@edewata
Copy link
Contributor Author

edewata commented Jan 13, 2025

Hmm.. it worked fine in my fork:
https://github.com/edewata/pki/actions/runs/12603780508
Let me take a look.

@fmarco76
Copy link
Member

One thing that I noticed is that the main repo uses Ubuntu 24.04.1: https://github.com/dogtagpki/pki/actions/runs/12603791276/job/35129768529#step:1:4 https://github.com/actions/runner-images/blob/main/images/ubuntu/Ubuntu2404-Readme.md

whereas my repo uses Ubuntu 22.04.5: https://github.com/edewata/pki/actions/runs/12603780508/job/35129639188#step:1:4 https://github.com/actions/runner-images/blob/main/images/ubuntu/Ubuntu2204-Readme.md

This is very strange. The workflow is configured to run on latest Ubuntu (

) so I was not expecting to have different versions

To avoid a known issue in Podman on Ubuntu 24 the non-rootless
container tests have been modified to update the graph driver in
the SQLite backend manually. However, this method does not work
for the rootless container test so it has has been hard-coded to
use Ubuntu 22 for now.

containers/podman#21683
@edewata
Copy link
Contributor Author

edewata commented Jan 15, 2025

It looks like GitHub did not release Ubuntu 24 runner consistently to all repos. Regardless, I'm not able to find a working solution for rootless containers on Ubuntu 24 so for that one we need to use Ubuntu 22 for now. Please see the updated patch.

Copy link
Member

@fmarco76 fmarco76 left a comment

Choose a reason for hiding this comment

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

LGTM

@edewata
Copy link
Contributor Author

edewata commented Jan 15, 2025

@fmarco76 Thanks!

@edewata edewata merged commit 919db96 into dogtagpki:master Jan 15, 2025
163 of 171 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants