rhtyd commented on a change in pull request #4228:
URL: https://github.com/apache/cloudstack/pull/4228#discussion_r480975079



##########
File path: 
server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java
##########
@@ -410,6 +410,10 @@
     public final static ConfigKey<Long> IOPS_MAX_WRITE_LENGTH = new 
ConfigKey<Long>(Long.class, "vm.disk.iops.maximum.write.length", "Advanced", 
"0",
             "Maximum IOPS write burst duration (seconds). If '0' (zero) then 
does not check for maximum burst length.", true, ConfigKey.Scope.Global, null);
 
+    public static final ConfigKey<Boolean> ADD_HOST_ON_SERVICE_RESTART = new 
ConfigKey<Boolean>(Boolean.class, "add.host.on.service.restart", "Advanced", 
"true",

Review comment:
       If this is KVM specific can we add kvm in the global setting name 
@ravening ?




----------------------------------------------------------------
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:
[email protected]


Reply via email to