You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2020/11/19 14:20:36 UTC

[GitHub] [cloudstack] wido opened a new issue #4482: Configure NFS version for Primary Storage

wido opened a new issue #4482:
URL: https://github.com/apache/cloudstack/issues/4482


   #### ISSUE TYPE
    * Improvement Request
   
   
   ##### COMPONENT NAME
   ~~~
   NFS Primary Storage
   ~~~
   
   ##### CLOUDSTACK VERSION
   ~~~
   ALL
   ~~~
   
   ##### CONFIGURATION
   - Primary Storage
   - NFS
   
   
   ##### OS / ENVIRONMENT
   - Ubuntu / CentOS
   
   ##### SUMMARY
   NFS Primary Storage mounts are handled by libvirt.
   
   Currently libvirt defaults to NFS version 3 when mounting while it does support NFS version 4 if provided in the XML definition: https://libvirt.org/formatstorage.html#StoragePoolSource
   
   ~~~
     <source>
       <host name='localhost'/>
       <dir path='/var/lib/libvirt/images'/>
       <format type='nfs'/>
       <protocol ver='4'/>
     </source>
   ~~~
   
   Maybe pass the argument 'nfsvers' to the URL provided to the Management Server and then pass this down to the Hypervisors which generate the XML for libvirt.
   
   ##### EXPECTED RESULTS
   ~~~
   Primary Storage is mounted over NFS version 4
   ~~~
   
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] GabrielBrascher commented on issue #4482: Configure NFS version for Primary Storage

Posted by GitBox <gi...@apache.org>.
GabrielBrascher commented on issue #4482:
URL: https://github.com/apache/cloudstack/issues/4482#issuecomment-1044044619


   @nvazquez not yet, this is still on our "backlog".


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] nvazquez commented on issue #4482: Configure NFS version for Primary Storage

Posted by GitBox <gi...@apache.org>.
nvazquez commented on issue #4482:
URL: https://github.com/apache/cloudstack/issues/4482#issuecomment-1043721267


   Hi @wido @GabrielBrascher is there any update on this?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] DaanHoogland commented on issue #4482: Configure NFS version for Primary Storage

Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on issue #4482:
URL: https://github.com/apache/cloudstack/issues/4482#issuecomment-730430464


   looks great, @wido . plan to put it behind a config option? or in the create primary call? (/me having the impertinence of marking it for 4.16)


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] rhtyd commented on issue #4482: Configure NFS version for Primary Storage

Posted by GitBox <gi...@apache.org>.
rhtyd commented on issue #4482:
URL: https://github.com/apache/cloudstack/issues/4482#issuecomment-923718651


   @GabrielBrascher @wido @nvazquez @sureshanaparti @davidjumani @shwstppr @Pearl1594 @DaanHoogland @weizhouapache - any update on this, who's assigned/working on this? Or should we change milestone? Thanks.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] DaanHoogland commented on issue #4482: Configure NFS version for Primary Storage

Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on issue #4482:
URL: https://github.com/apache/cloudstack/issues/4482#issuecomment-730968655


   does that mean that it can be upgraded to version 4 by updating the URL, appending the nfsvers argument? (url is no argument for updateStoragePool now: https://cloudstack.apache.org/api/apidocs-4.14/apis/updateStoragePool.html)
   or is any other mech envisioned?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] wido commented on issue #4482: Configure NFS version for Primary Storage

Posted by GitBox <gi...@apache.org>.
wido commented on issue #4482:
URL: https://github.com/apache/cloudstack/issues/4482#issuecomment-731204843


   > does that mean that it can be upgraded to version 4 by updating the URL, appending the nfsvers argument? (url is no argument for updateStoragePool now: https://cloudstack.apache.org/api/apidocs-4.14/apis/updateStoragePool.html)
   > or is any other mech envisioned?
   
   I don't think we can do and want to do this for existing primary storages. People could hack the DB and modify it if they really want to.
   
   Otherwise we only support it when adding a new Primary Storage. We just need to parse the URL and store the nfs version in the DB and pass it all the way down to the XML we provide to libvirt.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] wido commented on issue #4482: Configure NFS version for Primary Storage

Posted by GitBox <gi...@apache.org>.
wido commented on issue #4482:
URL: https://github.com/apache/cloudstack/issues/4482#issuecomment-730447207


   > looks great, @wido . plan to put it behind a config option? or in the create primary call? (/me having the impertinence of marking it for 4.16)
   
   See: https://cloudstack.apache.org/api/apidocs-4.14/apis/createStoragePool.html
   
   We provide the URL in this form:
   
   <pre>nfs://my-nfs-server/exports/ps01</pre>
   
   We could change that URL by setting:
   
   <pre>nfs://my-nfs-server/exports/ps01?nfsvers=4</pre>


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] nvazquez commented on issue #4482: Configure NFS version for Primary Storage

Posted by GitBox <gi...@apache.org>.
nvazquez commented on issue #4482:
URL: https://github.com/apache/cloudstack/issues/4482#issuecomment-1058193306


   Ok - moving it tentatively to the 4.18 milestone


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] nvazquez commented on issue #4482: Configure NFS version for Primary Storage

Posted by GitBox <gi...@apache.org>.
nvazquez commented on issue #4482:
URL: https://github.com/apache/cloudstack/issues/4482#issuecomment-925485674


   Moved to the 4.17 milestone


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] DaanHoogland commented on issue #4482: Configure NFS version for Primary Storage

Posted by GitBox <gi...@apache.org>.
DaanHoogland commented on issue #4482:
URL: https://github.com/apache/cloudstack/issues/4482#issuecomment-731206327


   copy


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] GabrielBrascher commented on issue #4482: Configure NFS version for Primary Storage

Posted by GitBox <gi...@apache.org>.
GabrielBrascher commented on issue #4482:
URL: https://github.com/apache/cloudstack/issues/4482#issuecomment-897042173


   @nvazquez this could be an option indeed
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack] nvazquez commented on issue #4482: Configure NFS version for Primary Storage

Posted by GitBox <gi...@apache.org>.
nvazquez commented on issue #4482:
URL: https://github.com/apache/cloudstack/issues/4482#issuecomment-886644855


   @wido @DaanHoogland there is a global setting to control the NFS version for secondary storage 'secstorage.nfs.version' perhaps it's easier to include a new global setting for primary storage and pass the version on the storage pool parameters down to the XML? cc @weizhouapache 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org