You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by mc...@apache.org on 2013/07/16 00:56:14 UTC

git commit: updated refs/heads/4.2 to c1b0228

Updated Branches:
  refs/heads/4.2 bd0cfd9c0 -> c1b0228f1


After upgrade to 4.2.0, restart of system vms and router vm fails with a
NPE

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

Branch: refs/heads/4.2
Commit: c1b0228f13891694c58268f03a411bf60ef4c60c
Parents: bd0cfd9
Author: Min Chen <mi...@citrix.com>
Authored: Mon Jul 15 15:55:53 2013 -0700
Committer: Min Chen <mi...@citrix.com>
Committed: Mon Jul 15 15:55:53 2013 -0700

----------------------------------------------------------------------
 .../src/com/cloud/upgrade/dao/Upgrade410to420.java       | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/c1b0228f/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 96d402a..f89c862 100644
--- a/engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java
+++ b/engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java
@@ -21,7 +21,6 @@ import com.cloud.deploy.DeploymentPlanner;
 import com.cloud.utils.exception.CloudRuntimeException;
 import com.cloud.utils.script.Script;
 
-import org.apache.cloudstack.api.ApiConstants;
 import org.apache.cloudstack.engine.subsystem.api.storage.DataStoreProvider;
 import org.apache.log4j.Logger;
 import java.io.File;
@@ -32,19 +31,13 @@ import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.sql.Types;
 import java.util.ArrayList;
-import java.util.HashMap;
-import java.util.Iterator;
 import java.util.List;
-import java.util.Map;
 import java.util.UUID;
 import com.cloud.network.vpc.NetworkACL;
 
 public class Upgrade410to420 implements DbUpgrade {
 	final static Logger s_logger = Logger.getLogger(Upgrade410to420.class);
 
-//	private Map<Long, Long> host_store_id_map = new HashMap<Long, Long>();
-//	private Map<Long, Long> s3_store_id_map = new HashMap<Long, Long>();
-//	private Map<Long, Long> swift_store_id_map = new HashMap<Long, Long>();
 
 	@Override
 	public String[] getUpgradableVersionRange() {
@@ -421,12 +414,12 @@ public class Upgrade410to420 implements DbUpgrade {
 	    PreparedStatement sql2 = null;
         try {
             sql = conn.prepareStatement("update storage_pool set storage_provider_name = ? , scope = ? where pool_type = 'Filesystem' or pool_type = 'LVM'");
-            sql.setString(1, "ancient primary data store provider");
+            sql.setString(1, DataStoreProvider.DEFAULT_PRIMARY);
             sql.setString(2, "HOST");
             sql.executeUpdate();
 
             sql2 = conn.prepareStatement("update storage_pool set storage_provider_name = ? , scope = ? where pool_type != 'Filesystem' and pool_type != 'LVM'");
-            sql2.setString(1, "ancient primary data store provider");
+            sql2.setString(1, DataStoreProvider.DEFAULT_PRIMARY);
             sql2.setString(2, "CLUSTER");
             sql2.executeUpdate();
         } catch (SQLException e) {