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

(RHEL-52326) fix(nfs): include also entries from /usr/lib/{passwd,group} #92

Merged
merged 1 commit into from
Aug 11, 2024

Conversation

pvalena
Copy link
Contributor

@pvalena pvalena commented Aug 11, 2024

as those paths are used by bootc instead of the /etc ones.

(cherry picked from commit 45cdf3c4f24f77f04b264a7747f115d1031b2e67)

Resolves: RHEL-52326

Copy link

github-actions bot commented Aug 11, 2024

Commit validation

Tracker - RHEL-52326

The following commits need an inspection

commit note
444a3b4 - fix(nfs): include also entries from /usr/lib/{passwd,group} Missing upstream reference ‼️

Tracker validation

Success

🟢 Tracker RHEL-52326 has set desired product: rhel-9.4
🟢 Tracker RHEL-52326 has set desired component: dracut
🟢 Tracker RHEL-52326 has been approved


Pull Request validation

Failed

🔴 Failed or pending checks - testing-farm:centos-stream-9-x86_64[failure],test (fedora:latest, 41)[failure],test (fedora:latest, 40)[failure],test (fedora:latest, 36)[failure],test (fedora:latest, 35)[failure],test (fedora:latest, 31)[failure],test (fedora:latest, 30)[failure],test (fedora:latest, 17)[failure],test (fedora:latest, 14)[failure],test (fedora:latest, 13)[failure],test (fedora:latest, 12)[failure],test (fedora:latest, 11)[failure],test (fedora:latest, 10)[failure]
🔴 Review - Missing review from a member

as those paths are used by bootc instead of the /etc ones.

(cherry picked from commit 45cdf3c4f24f77f04b264a7747f115d1031b2e67 from PR#573)

Resolves: RHEL-52326
@github-actions github-actions bot changed the title fix(nfs): include also entries from /usr/lib/{passwd,group} (RHEL-52326) fix(nfs): include also entries from /usr/lib/{passwd,group} Aug 11, 2024
Copy link
Contributor Author

@pvalena pvalena left a comment

Choose a reason for hiding this comment

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

LGTM. Reviewed in upstream PR.

@pvalena
Copy link
Contributor Author

pvalena commented Aug 11, 2024

Fedora tests are timing out, but we don't care about those.

@pvalena pvalena merged commit 1a57cf7 into redhat-plumbers:main Aug 11, 2024
31 of 44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant