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

fix(nfs): give /run/rpcbind ownership to rpc user and add missing requirements #1783

Merged
merged 2 commits into from
Apr 21, 2022

Conversation

aafeijoo-suse
Copy link
Member

  • parse-nfsroot.sh requires sed, chmod and chown.
# grep -r -n -w -e sed -e chmod -e chown modules.d/95nfs | grep -v module-setup
modules.d/95nfs/parse-nfsroot.sh:92:        sed -i -e \
modules.d/95nfs/parse-nfsroot.sh:96:    # and even again after the sed, in case it was not yet specified
modules.d/95nfs/parse-nfsroot.sh:127:chown rpc:rpc /var/lib/rpcbind
modules.d/95nfs/parse-nfsroot.sh:128:chmod 770 /var/lib/rpcbind
  • Avoid errors when rpcbind tries to write to the /run/rpcbind directory.
Apr 13 11:35:43 localhost systemd[1]: Starting dracut pre-pivot and cleanup hook...
Apr 13 11:33:13 localhost rpc.idmapd[262]: exiting on signal 15
Apr 13 11:33:13 localhost rpcbind[253]: cannot open file = /run/rpcbind/rpcbind.xdr for writing
Apr 13 11:33:13 localhost rpcbind[253]: cannot save any registration
Apr 13 11:33:13 localhost rpcbind[253]: cannot open file = /run/rpcbind/portmap.xdr for writing
Apr 13 11:33:13 localhost rpcbind[253]: cannot save any registration

Checklist

  • I have tested it locally
  • I have reviewed and updated any documentation if relevant
  • I am providing new code and test(s) for it

parse-nfsroot.sh requires sed, chmod and chown.
Avoid errors when rpcbind tries to write to the /run/rpcbind directory.
@github-actions github-actions bot added modules Issue tracker for all modules nfs Issues related to the nfs module labels Apr 20, 2022
Copy link
Collaborator

@johannbg johannbg left a comment

Choose a reason for hiding this comment

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

LGTM

@johannbg johannbg merged commit d615934 into dracutdevs:master Apr 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
modules Issue tracker for all modules nfs Issues related to the nfs module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants