-
Notifications
You must be signed in to change notification settings - Fork 59
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
authselect-1.5.0-4.fc40 & authselect-1.5.0-4.fc41 breaks ext.config.files.fcos_users test #1680
Comments
Was just looking over https://src.fedoraproject.org/rpms/authselect/c/f411c0ecd9eef866fbe13e710867a3fcebaaf87d?branch=rawhide and https://src.fedoraproject.org/rpms/authselect/pull-request/22 caught my eye. Would you be able to test that @jmarrero? |
@jlebon yes sir, Ill give it a try. |
Got the rpm from: Added them to the overrides folder and did once we I got a build, confirmed I have the correct rpm by
Confirmed
Got out of the VM and ran the failing Kola test:
I think your patch is good to go. |
The regex that was given matched the `group` line but not the `passwd` line (which doesn't have that `[SUCCESS=merge]` bit). Generalize the regex instead so that we just take whatever the contents of the conditional was and put it in there. Fixes: coreos/fedora-coreos-tracker#1680 Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=2266344 (Upstreamed from https://src.fedoraproject.org/rpms/authselect/pull-request/22)
The regex that was given matched the `group` line but not the `passwd` line (which doesn't have that `[SUCCESS=merge]` bit). Generalize the regex instead so that we just take whatever the contents of the conditional was and put it in there. Fixes: coreos/fedora-coreos-tracker#1680 Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=2266344 (Upstreamed from https://src.fedoraproject.org/rpms/authselect/pull-request/22)
Until the fix for coreos/fedora-coreos-tracker#1680 finally propagates. Signed-off-by: Colin Walters <walters@verbum.org>
Until the fix for coreos/fedora-coreos-tracker#1680 finally propagates. Signed-off-by: Colin Walters <walters@verbum.org>
xref the BZ: https://bugzilla.redhat.com/show_bug.cgi?id=2266344 |
I think we can unpin this now since https://bodhi.fedoraproject.org/updates/FEDORA-2024-bb02da5f27 is in rawhide. |
The underlying issue should be fixed now. See: - coreos/fedora-coreos-tracker#1680 - https://bugzilla.redhat.com/show_bug.cgi?id=2266344
The underlying issue should be fixed now. See: - coreos/fedora-coreos-tracker#1680 - https://bugzilla.redhat.com/show_bug.cgi?id=2266344
The underlying issue should be fixed now. See: - coreos/fedora-coreos-tracker#1680 - https://bugzilla.redhat.com/show_bug.cgi?id=2266344
dropped in coreos/fedora-coreos-config@5c5017a |
Until the fix for coreos/fedora-coreos-tracker#1680 finally propagates. Signed-off-by: Colin Walters <walters@verbum.org>
Describe the bug
With the recent update of authselect-1.5.0-4 on both f40 and f41 the ext.config.files.fcos_users test fails.
Reproduction steps
kola run ext.config.files.fcos_users
Expected behavior
kola run ext.config.files.fcos_users should pass.
Actual behavior
System details
QEMU Fedora CoreOS 40.20240227.dev.3
also seen in the pipeline:
https://jenkins-fedora-coreos-pipeline.apps.ocp.fedoraproject.org/job/build/2312/
Butane or Ignition config
No response
Additional information
The issue also is seen by looking at
/etc/nsswitch.conf
of a system built with authselect-1.5.0-4(bad) vs authselect-1.5.0-3(good)In the good case we should see a passwd entry like:
However we see:
The text was updated successfully, but these errors were encountered: