You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by an...@apache.org on 2014/04/24 00:38:03 UTC

[1/2] git commit: updated refs/heads/master to e0e2268

Repository: cloudstack
Updated Branches:
  refs/heads/master ee461f0c6 -> e0e226869


    Applied Tina's patches for removing the xapi jar


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

Branch: refs/heads/master
Commit: 2ff2b6c249a0d6a68df0f3054bb431387e14e188
Parents: ee461f0
Author: Anthony Xu <an...@citrix.com>
Authored: Wed Apr 23 11:02:02 2014 -0700
Committer: Anthony Xu <an...@citrix.com>
Committed: Wed Apr 23 15:32:12 2014 -0700

----------------------------------------------------------------------
 pom.xml | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/2ff2b6c2/pom.xml
----------------------------------------------------------------------
diff --git a/pom.xml b/pom.xml
index a20c40c..ecae71a 100644
--- a/pom.xml
+++ b/pom.xml
@@ -60,7 +60,7 @@
     <cs.ehcache.version>2.6.6</cs.ehcache.version>
     <cs.gson.version>1.7.2</cs.gson.version>
     <cs.guava.version>14.0-rc1</cs.guava.version>
-    <cs.xapi.version>6.2.0-3</cs.xapi.version>
+    <cs.xapi.version>6.2.0-3.1</cs.xapi.version>
     <cs.httpclient.version>3.1</cs.httpclient.version>
     <cs.httpcore.version>4.2.1</cs.httpcore.version>
     <cs.mysql.version>5.1.21</cs.mysql.version>
@@ -448,11 +448,6 @@
       <artifactId>javax.inject</artifactId>
       <version>1</version>
     </dependency>
-    <dependency>
-        <groupId>net.java.dev.vcc.thirdparty</groupId>
-        <artifactId>xen-api</artifactId>
-        <version>${cs.xapi.version}</version>
-    </dependency>
   </dependencies>
 
   <build>


[2/2] git commit: updated refs/heads/master to e0e2268

Posted by an...@apache.org.
CLOUDSTACK-6438
CLOUDSTACK-6442
  XAPI plugins must be copied to XS master first.


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

Branch: refs/heads/master
Commit: e0e226869d83189dcdb69e3c18b24c47e2b8f1fe
Parents: 2ff2b6c
Author: Anthony Xu <an...@citrix.com>
Authored: Wed Apr 23 15:30:36 2014 -0700
Committer: Anthony Xu <an...@citrix.com>
Committed: Wed Apr 23 15:32:36 2014 -0700

----------------------------------------------------------------------
 .../hypervisor/xen/resource/CitrixResourceBase.java | 16 ++++++++++------
 .../xen/resource/Xenserver625Resource.java          |  5 +++--
 2 files changed, 13 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/e0e22686/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
----------------------------------------------------------------------
diff --git a/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java b/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
index ba8be91..65d5a0e 100644
--- a/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
+++ b/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
@@ -4663,8 +4663,16 @@ public abstract class CitrixResourceBase implements ServerResource, HypervisorRe
 
     protected SetupAnswer execute(SetupCommand cmd) {
         Connection conn = getConnection();
-        setupServer(conn);
         try {
+            Map<Pool, Pool.Record> poolRecs = Pool.getAllRecords(conn);
+            if (poolRecs.size() != 1) {
+                throw new CloudRuntimeException("There are " + poolRecs.size() + " pool for host :" + _host.uuid);
+            }
+            Host master = poolRecs.values().iterator().next().master;
+            setupServer(conn, master);
+            Host host = Host.getByUuid(conn, _host.uuid);
+            setupServer(conn, host);
+
             if (!setIptables(conn)) {
                 s_logger.warn("set xenserver Iptable failed");
                 return null;
@@ -4687,7 +4695,6 @@ public abstract class CitrixResourceBase implements ServerResource, HypervisorRe
                 return null;
             }
             cleanupTemplateSR(conn);
-            Host host = Host.getByUuid(conn, _host.uuid);
             try {
                 if (cmd.useMultipath()) {
                     // the config value is set to true
@@ -4801,14 +4808,11 @@ public abstract class CitrixResourceBase implements ServerResource, HypervisorRe
     }
 
     /* return : if setup is needed */
-    protected boolean setupServer(Connection conn) {
+    protected boolean setupServer(Connection conn, Host host) {
         String packageVersion = CitrixResourceBase.class.getPackage().getImplementationVersion();
         String version = this.getClass().getName() + "-" + (packageVersion == null ? Long.toString(System.currentTimeMillis()) : packageVersion);
 
         try {
-            Host host = Host.getByUuid(conn, _host.uuid);
-            /* enable host in case it is disabled somehow */
-            host.enable(conn);
             /* push patches to XenServer */
             Host.Record hr = host.getRecord(conn);
 

http://git-wip-us.apache.org/repos/asf/cloudstack/blob/e0e22686/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625Resource.java
----------------------------------------------------------------------
diff --git a/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625Resource.java b/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625Resource.java
index b1442de..4014364 100644
--- a/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625Resource.java
+++ b/plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Xenserver625Resource.java
@@ -30,6 +30,7 @@ import org.apache.xmlrpc.XmlRpcException;
 import com.xensource.xenapi.Connection;
 import com.xensource.xenapi.Types;
 import com.xensource.xenapi.VM;
+import com.xensource.xenapi.Host;
 
 import org.apache.cloudstack.hypervisor.xenserver.XenServerResourceNewBase;
 
@@ -107,7 +108,7 @@ public class Xenserver625Resource extends XenServerResourceNewBase {
     }
 
     @Override
-    protected boolean setupServer(Connection conn) {
+    protected boolean setupServer(Connection conn,Host host) {
         com.trilead.ssh2.Connection sshConnection = new com.trilead.ssh2.Connection(_host.ip, 22);
         try {
             sshConnection.connect(null, 60000, 60000);
@@ -131,7 +132,7 @@ public class Xenserver625Resource extends XenServerResourceNewBase {
         } finally {
             sshConnection.close();
         }
-        return super.setupServer(conn);
+        return super.setupServer(conn, host);
     }
 
     @Override