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

Added downgrade of crun in molecule scenario where podman is needed #489

Conversation

rebtoor
Copy link
Contributor

@rebtoor rebtoor commented Nov 9, 2023

There's a regression in crun 1.11.1 (see containers/crun#1338) that prevents podman to being executed on podman containers in some molecule scenario.

We are downgrade crun to last known version (1.10.1) until the new package will arrive in Centos 9 Stream.

@rebtoor rebtoor requested review from gibizer and karelyatin November 9, 2023 11:21
@openshift-ci openshift-ci bot requested a review from booxter November 9, 2023 11:21
@openshift-ci openshift-ci bot added the approved label Nov 9, 2023
Copy link
Contributor

openshift-ci bot commented Nov 9, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: gibizer, rebtoor

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://review.rdoproject.org/zuul/buildset/8267185e04d14685b69a27a102dff917

✔️ edpm-ansible-molecule-edpm_podman SUCCESS in 6m 01s
✔️ edpm-ansible-molecule-edpm_module_load SUCCESS in 5m 23s
✔️ edpm-ansible-molecule-edpm_kernel SUCCESS in 11m 52s
✔️ edpm-ansible-molecule-edpm_libvirt SUCCESS in 7m 20s
✔️ edpm-ansible-molecule-edpm_nova SUCCESS in 6m 52s
edpm-ansible-molecule-edpm_frr FAILURE in 6m 07s
✔️ edpm-ansible-molecule-edpm_iscsid SUCCESS in 5m 31s
✔️ edpm-ansible-molecule-edpm_ovn_bgp_agent SUCCESS in 8m 09s

@rebtoor rebtoor force-pushed the molecule-downgrade-crun branch from 41c6b6b to 2790c02 Compare November 9, 2023 11:58
@openshift-ci openshift-ci bot removed the lgtm label Nov 9, 2023
Copy link
Contributor

@karelyatin karelyatin left a comment

Choose a reason for hiding this comment

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

/lgtm

containers/crun@7c3393c triggered the issue

@booxter
Copy link
Contributor

booxter commented Nov 9, 2023

/lgtm

@openshift-merge-bot openshift-merge-bot bot merged commit 31459d5 into openstack-k8s-operators:main Nov 9, 2023
29 checks passed
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.

4 participants