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

Fix base image build having a selinux dependency collision #662

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

gerrod3
Copy link
Collaborator

@gerrod3 gerrod3 commented Sep 17, 2024

Honestly have no idea what I am doing, but the error with sed: File not found is because buildah was failing to install due to seeing selinux-policy in the system. https://github.com/pulp/pulp-oci-images/actions/runs/10894878722/job/30232255402#step:8:5020

Error: 
 Problem: package buildah-2:1.37.2-1.el9.aarch64 from appstream requires containers-common >= 2:1-2, but none of the providers can be installed
  - package containers-common-2:1-13.el9.noarch from appstream requires (container-selinux >= 2:2.162.1 if selinux-policy), but none of the providers can be installed
  - package containers-common-2:1-21.el9.noarch from appstream requires (container-selinux >= 2:2.162.1 if selinux-policy), but none of the providers can be installed
  - package containers-common-2:1-52.el9.aarch64 from appstream requires (container-selinux >= 2:2.162.1 if selinux-policy), but none of the providers can be installed
  - package containers-common-2:1-55.el9.aarch64 from appstream requires (container-selinux >= 2:2.162.1 if selinux-policy), but none of the providers can be installed
  - package containers-common-2:1-61.el9.aarch64 from appstream requires (container-selinux >= 2:2.162.1 if selinux-policy), but none of the providers can be installed
  - package containers-common-2:1-8.el9.noarch from appstream requires (container-selinux >= 2:2.162.1 if selinux-policy), but none of the providers can be installed
  - package containers-common-2:1-90.el9.aarch64 from appstream requires (container-selinux >= 2:2.162.1 if selinux-policy), but none of the providers can be installed
(try to add '--skip-broken' to skip uninstallable packages or '--nobest' to use not only best candidate packages)
  - package containers-common-2:1-91.el9.aarch64 from appstream requires (container-selinux >= 2:2.162.1 if selinux-policy), but none of the providers can be installed
  - problem with installed package selinux-policy-38.1.44-1.el9.noarch
  - cannot install the best candidate for the job
  - package container-selinux-3:2.228.1-1.el9.noarch from appstream is filtered out by exclude filtering
  - package container-selinux-3:2.229.0-1.el9.noarch from appstream is filtered out by exclude filtering
  - package container-selinux-3:2.230.0-1.el9.noarch from appstream is filtered out by exclude filtering
  - package container-selinux-3:2.231.0-1.el9.noarch from appstream is filtered out by exclude filtering
  - package container-selinux-3:2.232.1-1.el9.noarch from appstream is filtered out by exclude filtering

After reading this linked article in the containerfile (https://www.redhat.com/sysadmin/podman-inside-container) I guessed we don't want selinux in order to do podman inside podman. So I found an earlier install that was dragging in selinux, openldap-devel, and excluded it from install. Maybe it would be better to switch the order of the installs and have podman/buildah installed first? Would like some suggestions on the best approach.

@gerrod3 gerrod3 force-pushed the selinux-makes-no-sense branch from d92e994 to 5e02394 Compare September 17, 2024 19:26
@gerrod3 gerrod3 force-pushed the selinux-makes-no-sense branch from 5e02394 to 81c476e Compare September 17, 2024 19:27
@gerrod3 gerrod3 changed the title WIP Fix base image build having a selinux dependency collision Fix base image build having a selinux dependency collision Sep 17, 2024
@gerrod3 gerrod3 marked this pull request as ready for review September 17, 2024 21:40
Copy link
Member

@lubosmj lubosmj left a comment

Choose a reason for hiding this comment

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

Interesting. If it helps... Cannot give you suggestions with regard to the install order.

@lubosmj lubosmj merged commit de9b8e2 into pulp:latest Sep 18, 2024
15 of 16 checks passed
@gerrod3 gerrod3 deleted the selinux-makes-no-sense branch September 18, 2024 13:10
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