From 03d5445cf5aad5473bbc0f628d00014aedcd38f4 Mon Sep 17 00:00:00 2001
From: andreaturli <andrea.turli@gmail.com>
Date: Mon, 1 Nov 2010 22:35:01 +0100
Subject: [PATCH] fix runNodeWithTagAndNameAndStoreCredentials: it shoud run
 the domain not only define it

---
 .../compute/strategy/LibvirtComputeServiceAdapter.java      | 6 ++++--
 .../jclouds/libvirt/compute/LibvirtExperimentLiveTest.java  | 2 +-
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/sandbox/libvirt/src/main/java/org/jclouds/libvirt/compute/strategy/LibvirtComputeServiceAdapter.java b/sandbox/libvirt/src/main/java/org/jclouds/libvirt/compute/strategy/LibvirtComputeServiceAdapter.java
index 845564a7c6..7a33dfd6d3 100644
--- a/sandbox/libvirt/src/main/java/org/jclouds/libvirt/compute/strategy/LibvirtComputeServiceAdapter.java
+++ b/sandbox/libvirt/src/main/java/org/jclouds/libvirt/compute/strategy/LibvirtComputeServiceAdapter.java
@@ -117,8 +117,10 @@ public class LibvirtComputeServiceAdapter implements ComputeServiceAdapter<Domai
                String xmlFinal = generateClonedDomainXML(domain.getXMLDesc(0));
                domain = client.domainDefineXML(xmlFinal);
 
+               domain.create();
+               
         	   // store the credentials so that later functions can use them
-               //credentialStore.put(domain.getUUIDString() + "", new Credentials("identity", "credential"));
+               credentialStore.put(domain.getUUIDString() + "", new Credentials("identity", "credential"));
 
             //}
          //}
@@ -260,7 +262,7 @@ public class LibvirtComputeServiceAdapter implements ComputeServiceAdapter<Domai
       newNode.getFirstChild().setNodeValue(UUID.randomUUID().toString());
       builder.getDocument().getDocumentElement().replaceChild(newNode, oldChild);
 
-      builder.xpathFind("//domain/devices/disk/source").a("file", "/var/lib/libvirt/images/ubuntu.img-clone");
+      builder.xpathFind("//domain/devices/disk/source").a("file", "/var/lib/libvirt/images/ttylinux.img-clone");
       // TODO generate valid MAC address
       builder.xpathFind("//domain/devices/interface/mac").a("address", "52:54:00:5c:dd:eb");
       return builder.asString(outputProperties);
diff --git a/sandbox/libvirt/src/test/java/org/jclouds/libvirt/compute/LibvirtExperimentLiveTest.java b/sandbox/libvirt/src/test/java/org/jclouds/libvirt/compute/LibvirtExperimentLiveTest.java
index 9821108638..ca5bff57f1 100644
--- a/sandbox/libvirt/src/test/java/org/jclouds/libvirt/compute/LibvirtExperimentLiveTest.java
+++ b/sandbox/libvirt/src/test/java/org/jclouds/libvirt/compute/LibvirtExperimentLiveTest.java
@@ -83,7 +83,7 @@ public class LibvirtExperimentLiveTest {
      * You can control the default template via overriding a method in standalonecomputeservicexontextmodule
      */
 
-         context.getComputeService().runNodesWithTag("ubuntu", 1/*, defaultTemplate*/);
+         context.getComputeService().runNodesWithTag("ttylinux", 1/*, defaultTemplate*/);
 
 	} catch (RunNodesException e) {
 		// TODO Auto-generated catch block