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

Add ability to force usage of IP in the nfs mount command #142

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

oe-hbk
Copy link

@oe-hbk oe-hbk commented Sep 29, 2022

Description of changes:
Add a new mount option forceuseip that will cause a DNS lookup as it's currently done for the mount target but use the IP only in the NFS mount.

This is important for machines that run rpc.gssd with the -n argument to provide kerberized NFS to other shares, as without it, NFS mounts will search for a kerberos principal for nfs/mounttarget and fail with a
Key has expired message.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

… address for the EFS fqdn. Useful when Amazon DNS is not used, and when using DNS name for the nfs mount causes issues with kerberos/rpc.gssd
@oe-hbk
Copy link
Author

oe-hbk commented Nov 21, 2022

I am not sure how to assign reviewers here. Apologies for referencing you directly @lshigupt and @Cappuccinuo but could you please look and advise? Thank you

@briantist
Copy link

Hm, I wonder if this would solve my particular use case described here:

@oe-hbk
Copy link
Author

oe-hbk commented Jan 3, 2023

@briantist, yes, I think this should help that situation.

@oe-hbk
Copy link
Author

oe-hbk commented Apr 26, 2023

@mjsoyeon sorry to ping you directly, but saw you do a recent merge, any chance we can get this PR merged?

@oe-hbk
Copy link
Author

oe-hbk commented Dec 19, 2023

@RyanStan any chance you could review this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants