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

Add libdnf workaround for intel-bootc #640

Closed
wants to merge 1 commit into from

Conversation

enriquebelarte
Copy link
Collaborator

Same as in #630 a workaround is applied in order to install needed packages with dnf

@rhatdan
Copy link
Member

rhatdan commented Jun 27, 2024

Is the issue that SELinux labels are attempting to be added?

@enriquebelarte
Copy link
Collaborator Author

Yes, that's the issue.

Copy link
Collaborator

@Gregory-Pereira Gregory-Pereira left a comment

Choose a reason for hiding this comment

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

/lgtm

@rhatdan
Copy link
Member

rhatdan commented Jun 28, 2024

This needs a rebase, but I would really like to remove this hack and figure out how SELinux being enabled is leaking into the container when using Konflux.

@cgwalters
Copy link
Contributor

@rhatdan It's all in https://issues.redhat.com/browse/RHEL-39796 - bug in libdnf

@rhatdan
Copy link
Member

rhatdan commented Jul 1, 2024

Thanks @cgwalters
I was thinking that selinuxenabled was returning true, but it looks by the links that libdnf was not even checking if SELinux was enabled.

LGTM, after rebase.

Signed-off-by: Enrique Belarte Luque <[email protected]>
@enriquebelarte
Copy link
Collaborator Author

Closing in favor of #780

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.

4 participants