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

CA-176801: Work around rpcinfo intricacies to determine nfs versions #268

Closed
wants to merge 2 commits into from

Conversation

siddharthv
Copy link
Contributor

Signed-off-by: Siddharth Vinothkumar siddharth.vinothkumar@citrix.com

Signed-off-by: Siddharth Vinothkumar <siddharth.vinothkumar@citrix.com>
Signed-off-by: Siddharth Vinothkumar <siddharth.vinothkumar@citrix.com>
@sriccio
Copy link

sriccio commented Jul 17, 2015

While changes are being made to nfs.py, would it be possible to allow mount options to be specified for NFS storage repositories. Like for example, not everybody wants to use soft mount. This can avoid virtual machines switching to read only when a disconnection between the hosts and the storage occurs.

Changing these options need nfs.py to be manually edited and it's being overwritten when doing updates.

XenCenter allowed or still allows, I've not really checked recently, to specify mount options, when creating a NFS repository. But nothing in nfs.py is caring about these options :/

Your thoughts about the "issue" ?

ps: there is a thread about this on citrix forum since nov. 2013:

http://discussions.citrix.com/topic/344713-linux-vm-storage-going-read-only-on-nfs-shares-and-a-proposed-fix-in-xenserver/

@siddharthv
Copy link
Contributor Author

@sriccio thanks a lot for your comment.
I can see what you're faced with but it would be best if you could raise a Feature Request at http://bugs.xenserver.org/ to get such a change onto the product.
This change is not trivial in my opinion.

@germanop germanop closed this in 8cb225e Sep 25, 2015
andrey-podko pushed a commit to andrey-podko/sm that referenced this pull request Aug 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants