You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by hu...@apache.org on 2014/09/23 11:09:21 UTC

[1/2] git commit: updated refs/heads/bugfix/CID-1240106 to c20336c

Repository: cloudstack
Updated Branches:
  refs/heads/bugfix/CID-1240106 [created] c20336c1a


CID-1240106 Remove unused variable

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

Branch: refs/heads/bugfix/CID-1240106
Commit: 9ead2046d495db6af496656c1f69ae1dc1a1d1a2
Parents: 49de3ab
Author: Hugo Trippaers <ht...@schubergphilis.com>
Authored: Tue Sep 23 11:02:40 2014 +0200
Committer: Hugo Trippaers <ht...@schubergphilis.com>
Committed: Tue Sep 23 11:02:40 2014 +0200

----------------------------------------------------------------------
 server/src/com/cloud/template/TemplateManagerImpl.java | 5 -----
 1 file changed, 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/9ead2046/server/src/com/cloud/template/TemplateManagerImpl.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/template/TemplateManagerImpl.java b/server/src/com/cloud/template/TemplateManagerImpl.java
index 247ed00..a1ca344 100755
--- a/server/src/com/cloud/template/TemplateManagerImpl.java
+++ b/server/src/com/cloud/template/TemplateManagerImpl.java
@@ -629,13 +629,10 @@ public class TemplateManagerImpl extends ManagerBase implements TemplateManager,
 
         // Event details
         String copyEventType;
-        String createEventType;
         if (template.getFormat().equals(ImageFormat.ISO)) {
             copyEventType = EventTypes.EVENT_ISO_COPY;
-            createEventType = EventTypes.EVENT_ISO_CREATE;
         } else {
             copyEventType = EventTypes.EVENT_TEMPLATE_COPY;
-            createEventType = EventTypes.EVENT_TEMPLATE_CREATE;
         }
 
         TemplateInfo srcTemplate = _tmplFactory.getTemplate(template.getId(), srcSecStore);
@@ -1493,8 +1490,6 @@ public class TemplateManagerImpl extends ManagerBase implements TemplateManager,
     @Override
     @ActionEvent(eventType = EventTypes.EVENT_TEMPLATE_CREATE, eventDescription = "creating template", create = true)
     public VMTemplateVO createPrivateTemplateRecord(CreateTemplateCmd cmd, Account templateOwner) throws ResourceAllocationException {
-        Long userId = CallContext.current().getCallingUserId();
-
         Account caller = CallContext.current().getCallingAccount();
         boolean isAdmin = (_accountMgr.isAdmin(caller.getId()));
 


[2/2] git commit: updated refs/heads/bugfix/CID-1240106 to c20336c

Posted by hu...@apache.org.
CID-1240107 Remove unused variable

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

Branch: refs/heads/bugfix/CID-1240106
Commit: c20336c1ab45ba7b046b1064a8c10d8e405055ee
Parents: 9ead204
Author: Hugo Trippaers <ht...@schubergphilis.com>
Authored: Tue Sep 23 11:06:41 2014 +0200
Committer: Hugo Trippaers <ht...@schubergphilis.com>
Committed: Tue Sep 23 11:06:41 2014 +0200

----------------------------------------------------------------------
 .../cloud/hypervisor/kvm/resource/LibvirtComputingResource.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/c20336c1/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
----------------------------------------------------------------------
diff --git a/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java b/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
index a79526a..1dc1955 100755
--- a/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
+++ b/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
@@ -230,8 +230,8 @@ import com.cloud.hypervisor.kvm.resource.LibvirtVMDef.InterfaceDef;
 import com.cloud.hypervisor.kvm.resource.LibvirtVMDef.InterfaceDef.guestNetType;
 import com.cloud.hypervisor.kvm.resource.LibvirtVMDef.SerialDef;
 import com.cloud.hypervisor.kvm.resource.LibvirtVMDef.TermPolicy;
-import com.cloud.hypervisor.kvm.resource.LibvirtVMDef.VirtioSerialDef;
 import com.cloud.hypervisor.kvm.resource.LibvirtVMDef.VideoDef;
+import com.cloud.hypervisor.kvm.resource.LibvirtVMDef.VirtioSerialDef;
 import com.cloud.hypervisor.kvm.storage.KVMPhysicalDisk;
 import com.cloud.hypervisor.kvm.storage.KVMStoragePool;
 import com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager;
@@ -5261,7 +5261,7 @@ public class LibvirtComputingResource extends ServerResourceBase implements Serv
         String device = Script.runSimpleBashScript("rbd showmapped | grep \""+splitPoolImage[0]+"[ ]*"+splitPoolImage[1]+"\" | grep -o \"[^ ]*[ ]*$\"");
         if(device == null) {
             //If not mapped, map and return mapped device
-            String mapRbd = Script.runSimpleBashScript("rbd map " + disk.getPath() + " --id "+pool.getAuthUserName());
+            Script.runSimpleBashScript("rbd map " + disk.getPath() + " --id " + pool.getAuthUserName());
             device = Script.runSimpleBashScript("rbd showmapped | grep \""+splitPoolImage[0]+"[ ]*"+splitPoolImage[1]+"\" | grep -o \"[^ ]*[ ]*$\"");
         }
         return device;