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
NAS-128399 / 24.10 / More nfs failover fixes #13598
Conversation
Fix flake8 complaint. Improve error reporting for service state check.
…about 5 sec to start. Added check for NFS dataset in first boot check in test_001_ssh.py. Some simple refactoring in test_300_nfs.py.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few remaining items 👍
* Rename class NFSPath to NFSServicePathInfo and fixup usage in status.py and local.conf.mako * Move improved name_to_id_conversion to standalone private method * Update NFS state directory initialization
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fantastic, great job! Glad to see this general issue get resolved, finally. Be sure and wait for @anodos325 before merge 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!
This PR has been merged and conversations have been locked. |
Problem
NFS was not configured to properly support HA function. NFS holds client state information in a database that, by default, is located at /var/lib/nfs. To properly support failover we need that to be a shared directory. For TrueNAS that is the system dataset.
Fix
Create and NFS entry in the system dataset. Generate the necessary directory structure and to support upgrade we also initialize it with data from /var/lib/nfs.
In systemd we bind the NFS client tracking daemon to nfs-server to allow the transfer of the client tracking database.
Update the nfs configuration file with the path to the nfs data on the system dataset:
/var/db/system/nfs
Other related items in the PR