DaanHoogland commented on a change in pull request #3154: Copy template to
target KVM host if needed when migrating local <> local storage
URL: https://github.com/apache/cloudstack/pull/3154#discussion_r252276888
##########
File path:
engine/components-api/src/main/java/com/cloud/storage/StorageManager.java
##########
@@ -101,6 +101,9 @@
ConfigKey.Scope.Cluster,
null);
+ ConfigKey<Integer> PRIMARY_STORAGE_DOWNLOAD_WAIT = new
ConfigKey<Integer>("Storage", Integer.class, "primary.storage.download.wait",
"10800",
Review comment:
I'd say keep your new one and find usages of the old one to unify them. If
need be move the key to a more centralised place.
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
