harikrishna-patnala commented on code in PR #11446:
URL: https://github.com/apache/cloudstack/pull/11446#discussion_r2318199501


##########
server/src/main/java/com/cloud/configuration/Config.java:
##########
@@ -504,7 +504,7 @@ public enum Config {
             "300",
             "The time interval in seconds when the management server polls for 
snapshots to be scheduled.",
             null),
-    KVMSnapshotEnabled("Hidden", SnapshotManager.class, Boolean.class, 
"kvm.snapshot.enabled", "false", "Whether volume snapshot is enabled on running 
instances on a KVM host", null),
+    KVMSnapshotEnabled("Hidden", SnapshotManager.class, Boolean.class, 
"kvm.snapshot.enabled", "true", "Whether volume snapshot is enabled on running 
instances on a KVM host", null),

Review Comment:
   @rohityadavcloud for an upgraded environment this default value won't be 
reflected. either we will need an mysql update statement in the upgrade sql 
file or move this configuration to ConfigKey object.



-- 
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]

Reply via email to