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

Switch to ubuntu-24.04 #1139

Merged
merged 1 commit into from
Jul 31, 2024
Merged

Switch to ubuntu-24.04 #1139

merged 1 commit into from
Jul 31, 2024

Conversation

bdunne
Copy link
Member

@bdunne bdunne commented Jul 31, 2024

ubuntu-latest still points to 22.04 which has podman 3.4.4 and run in that version does not support bind mounting volumes. Upgrading to 24.04 gets us podman 4.9.3

Error:

error resolving mountpoints for container "4349440300a7f09982c00e0a8d5a010141b5eb912483ea6646bacaf54c080194": invalid mount type "bind"

ubuntu-latest still points to 22.04 which has podman 3.4.4 and run in that
version does not support bind mounting volumes.  Upgrading to 24.04 gets
us podman 4.9.3

Error:
error resolving mountpoints for container "4349440300a7f09982c00e0a8d5a010141b5eb912483ea6646bacaf54c080194": invalid mount type "bind"
@Fryguy
Copy link
Member

Fryguy commented Jul 31, 2024

LGTM - I wonder if there's a way we can know when latest bumps to this version. I feel like we need something to trigger us to undo this change at some point or we're going to forget (maybe it's as simple as just opening an issue and letting it go stale)

@Fryguy
Copy link
Member

Fryguy commented Jul 31, 2024

@miq-bot
Copy link
Member

miq-bot commented Jul 31, 2024

Checked commit bdunne@b679e50 with ruby 3.1.5, rubocop 1.56.3, haml-lint 0.51.0, and yamllint
1 file checked, 1 offense detected

**

  • 💣 💥 🔥 🚒 - Linter/Yaml - missing config files

@bdunne
Copy link
Member Author

bdunne commented Jul 31, 2024

@bdunne Do we also need changes in https://github.com/ManageIQ/manageiq/blob/80ff67c710d7d081622a6b9b7bd0f70a2e929fca/.github/workflows/build_monolith.yaml#L10 ?

I don't think so. There's no bind mount there.

@Fryguy Fryguy merged commit b150122 into ManageIQ:master Jul 31, 2024
2 checks passed
@bdunne bdunne deleted the podman4 branch July 31, 2024 21:32
@Fryguy
Copy link
Member

Fryguy commented Aug 15, 2024

Backported to radjabov via merge of master into radjabov branch

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