-
Notifications
You must be signed in to change notification settings - Fork 107
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
Pin to libsemanage-2.9-8.el8 #1037
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
With -9.el8, `ext.config.rebuild-selinux-policy` fails: openshift#1036 We need to debug this, but for now let's unblock CI and dev pipelines.
282da86
to
247e64a
Compare
Though thinking more on this, having the policy be recompiled on first boot is really not great and may affect other things, so let's pin instead. I've verified locally that this unbreaks the test. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dustymabe, jlebon, marmijo 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 |
SCOS failures look like distinct issues. The pin added here can only affect RHCOS 8.6, whose associated tests did pass. /override ci/prow/scos-9-build-test-metal |
@jlebon: Overrode contexts on behalf of jlebon: ci/prow/scos-9-build-test-metal, ci/prow/scos-9-build-test-qemu In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@jlebon: all tests passed! Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Same error on 4.11, need to backport to 4.11 |
@HuijingHei: #1037 failed to apply on top of branch "release-4.11":
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
With -9.el8,
ext.config.rebuild-selinux-policy
fails:#1036
We need to debug this, but for now let's unblock CI and dev pipelines.