etan39
/
cloudstack
mirror of https://github.com/apache/cloudstack.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

CLOUDSTACK-4064: Missed renaming the config for KVM snapshots 

KVM.snapshot.enabled is lowercased by f025db95 to keep the configs
uniformly lower-case. But it missed the upgrade script and the
references in SnapshotManagerImpl. This commit will fix the issue in all
locations

Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 0e216fa7e41bdfe0cc744006bb896c8b00138ca2)
This commit is contained in:
Prasanna Santhanam 2013-08-06 10:37:48 +05:30
parent cfac58493f
commit 0eafc0c0d8
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java
View File

@ -318,7 +318,7 @@ public class Upgrade410to420 implements DbUpgrade {
pstmt.close();
if (numRows > 0){
//Add the configuration flag
pstmt = conn.prepareStatement("UPDATE `cloud`.`configuration` SET value = ? WHERE name = 'KVM.snapshot.enabled'");
pstmt = conn.prepareStatement("UPDATE `cloud`.`configuration` SET value = ? WHERE name = 'kvm.snapshot.enabled'");
pstmt.setString(1, "true");
pstmt.executeUpdate();
}

2
server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java
View File

@ -910,7 +910,7 @@ public class SnapshotManagerImpl extends ManagerBase implements SnapshotManager,
}
//Turn off snapshot by default for KVM, unless it is set in the global flag
boolean snapshotEnabled = Boolean.parseBoolean(_configDao.getValue("KVM.snapshot.enabled"));
boolean snapshotEnabled = Boolean.parseBoolean(_configDao.getValue("kvm.snapshot.enabled"));
if (!snapshotEnabled) {
return false;
}