harikrishna-patnala commented on code in PR #11446:
URL: https://github.com/apache/cloudstack/pull/11446#discussion_r2318553010
##########
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:
Okay. I think it is also good to have consistent values and moving to
ConfigKey. I've the fix, let me please push it here.
--
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]
