[GitHub] rafaelweingartner opened a new pull request #2401: [CLOUDSTACK-10226] CloudStack is not importing Local storage properly

[GitBox <gi...@apache.org>]

rafaelweingartner opened a new pull request #2401: [CLOUDSTACK-10226] CloudStack is not importing Local storage properly
URL: https://github.com/apache/cloudstack/pull/2401
 
 
   CloudStack is importing as Local storage any XenServer SR that is of type LVM or EXT. This causes a problem when one wants to use both Direct attach storage and local storage. Moreover, CloudStack was not importing all of the local storage that a host has available when local storage is enabled. It was only importing the First SR it sees.
   
   To fix the first problem we started ignoring SRs that have the flag shared=true when discovering local storages. SRs configured to be shared are used as direct attached storage, and therefore should not be imported again as local ones.
   
   To fix the second problem, we started loading all Local storage and importing them accordingly to ACS.
   
   Tip for reviewers: the code changes is mainly in `CitrixResourceBase`, method `initializeLocalSR`

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services