nvazquez commented on code in PR #9564:
URL: https://github.com/apache/cloudstack/pull/9564#discussion_r3768233688
##########
engine/components-api/src/main/java/com/cloud/storage/StorageManager.java:
##########
@@ -174,8 +174,8 @@ public interface StorageManager extends StorageService {
"If set to true, the disk is created only when there is a suitable
storage pool that supports the disk provisioning type specified by the
service/disk offering. " +
"If set to false, the disk is created with a disk
provisioning type supported by the pool. Default value is false, and this is
currently supported for VMware only.",
true, ConfigKey.Scope.Zone);
- ConfigKey<String> PreferredStoragePool = new
ConfigKey<String>(String.class, "preferred.storage.pool", "Advanced", "",
- "The UUID of preferred storage pool for allocation.", true,
ConfigKey.Scope.Account, null);
+ ConfigKey<String> PreferredStoragePool = new ConfigKey<>(String.class,
"preferred.storage.pool", "Advanced", "",
Review Comment:
Thanks @weizhouapache - there is now an option to define multiple scopes per
setting:
https://docs.cloudstack.apache.org/en/4.22.0.0/installguide/configuration.html#granular-global-configuration-parameters.
I will explore 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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
