cloudstack-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From t..@apache.org
Subject git commit: updated refs/heads/4.2 to 3a41c53
Date Tue, 06 Aug 2013 05:20:20 GMT
Updated Branches:
  refs/heads/4.2 d6fcabc57 -> 3a41c53bd


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>


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/3a41c53b
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/3a41c53b
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/3a41c53b

Branch: refs/heads/4.2
Commit: 3a41c53bd3354d99dc2a1043745bc55aab7c1ee7
Parents: d6fcabc
Author: Prasanna Santhanam <tsp@apache.org>
Authored: Tue Aug 6 10:37:48 2013 +0530
Committer: Prasanna Santhanam <tsp@apache.org>
Committed: Tue Aug 6 10:39:26 2013 +0530

----------------------------------------------------------------------
 engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java   | 2 +-
 server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/3a41c53b/engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java
----------------------------------------------------------------------
diff --git a/engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java b/engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java
index 5537a49..50bb000 100644
--- a/engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java
+++ b/engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java
@@ -395,7 +395,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();
             }

http://git-wip-us.apache.org/repos/asf/cloudstack/blob/3a41c53b/server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java b/server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java
index b639aa8..2caf45d 100755
--- a/server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java
+++ b/server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java
@@ -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;
         }


Mime
View raw message