add usermod workaround for openvswitch hugetlbfs group #1318
Conversation
openshift-ci
bot
added
the
approved
|
|
The RPM is now using systemd-sysusers fragments [1] and the RPM scriptlets no longer successfully add the `hugetlbfs` group to the `openvswitch` user [2]. Let's add a workaround for now while we investigate. [1] openshift#1274 (comment) [2] openshift#1274 (comment)
dustymabe
force-pushed
the
dusty-fix-openvswitch-groups-2
branch
from
5e44ef8
to
98e8d99
Compare
ahh. yeah I didn't add it to a shared manifest. Just pushed up a commit moving it to a shared manifest so hopefully it works now. |
|
/retest |
|
It looks like https://src.fedoraproject.org/rpms/systemd/blob/rawhide/f/sysusers.generate-pre.sh and https://pkgs.devel.redhat.com/cgit/rpms/systemd/tree/sysusers.generate-pre.sh?h=rhel-9.2.0 have diverged, and the latter is missing multiple updates to the script. Specifically, this change that is missing in RHEL looks like the culprit to me: https://src.fedoraproject.org/rpms/systemd/c/f27d461663bec17ad64422682f260f0020ccc7f7?branch=rawhide |
|
/retest-required |
1 similar comment
|
/retest-required |
|
/retest
…On Mon, Jun 19, 2023, at 21:26, openshift-ci[bot] wrote:
@dustymabe <https://github.com/dustymabe>: The following test *failed*, say `/retest` to rerun all failed tests or `/retest-required` to rerun all mandatory failed tests:
Test name
Commit
Details
Required
Rerun command
ci/prow/scos-9-build-test-qemu
98e8d99
link <https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_os/1318/pull-ci-openshift-os-master-scos-9-build-test-qemu/1670965559585411072>
true
`/test scos-9-build-test-qemu`
Full PR test history <https://prow.ci.openshift.org/pr-history?org=openshift&repo=os&pr=1318>. Your PR dashboard <https://prow.ci.openshift.org/pr?query=is:pr+state:open+author:dustymabe>.
Instructions for interacting with me using PR comments are available here <https://git.k8s.io/community/contributors/guide/pull-requests.md>. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra <https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:> repository. I understand the commands that are listed here <https://go.k8s.io/bot-commands>.
—
Reply to this email directly, view it on GitHub <#1318 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABCR63RHL64S5PTZL6R3R6TXMD347ANCNFSM6AAAAAAZLDHCT4>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
|
Have no idea how to debug the repo error: |
|
I just don't know why it's trying a repo in a |
It's to avoid entitling the Prow hosts; we pull from a proxy mirror today. |
|
Hmm it's weird that this same repo worked in the rhcos metal job. |
|
Also actually, the scos metal job worked too with the same repo! I wonder if this is a build system health issue; something about the node on which the build landed not being able to contact the proxy service. I think given that ci/prow/scos-9-build-test-metal and the other jobs worked, I'm comfortable |
|
@cgwalters: Overrode contexts on behalf of cgwalters: 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. |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cgwalters, dustymabe 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 |
|
@dustymabe: 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. |
|
/cherry-pick release-4.13 |
|
@dustymabe: new pull request created: #1319 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. |
The RPM is now using systemd-sysusers fragments [1] and the RPM scriptlets no longer successfully add the
hugetlbfsgroup to theopenvswitchuser [2]. Let's add a workaround for now while we investigate.[1] #1274 (comment)
[2] #1274 (comment)