atrocitytheme commented on pull request #5358:
URL: https://github.com/apache/cloudstack/pull/5358#issuecomment-904332053
> I think it makes sense to move the check to the `getPreferredStoragePool()
method`:
>
> ```
> private Optional<StoragePool> getPreferredStoragePool(List<StoragePool>
poolList, VirtualMachine vm) {
> String accountStoragePoolUuid = null;
> if (vm != null) {
> accountStoragePoolUuid =
StorageManager.PreferredStoragePool.valueIn(vm.getAccountId());
> }
> ...
> ```
>
> Then the pool ID specified in the global setting (if set) will be picked
instead of the first in the list of pools.
Makes sense, just moved the checker to getPreferredStorge method
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
