NFVBENCH-104 Specifying a hypervisor does not work

Need to add a leading ":" to the hypervisor name

Change-Id: Id5d02e14a73a2e278848ffccd7a0c60165dbf509
Signed-off-by: ahothan <ahothan@cisco.com>

diff --git a/nfvbench/chaining.py b/nfvbench/chaining.py
index a5ae680..fa9b799 100644 (file)
--- a/nfvbench/chaining.py
+++ b/nfvbench/chaining.py
@@ -715,8 +715,8 @@ class InstancePlacer(object):
         if req_az:
             self.required_az = req_az + ':' + self.requested_hyp
         else:
-            # no ":" needed
-            self.required_az = self.requested_hyp if req_hyp else ''
+            # need to insert a ':' so nova knows this is the hypervisor name
+            self.required_az = ':' + self.requested_hyp if req_hyp else ''
         # placement is resolved when both AZ and hypervisor names are known and set
         self.resolved = self.requested_az != '' and self.requested_hyp != ''
 

diff --git a/nfvbench/compute.py b/nfvbench/compute.py
index d5a8119..97fd166 100644 (file)
--- a/nfvbench/compute.py
+++ b/nfvbench/compute.py
@@ -107,6 +107,7 @@ class Compute(object):
             security_groups = None
 
         # Also attach the created security group for the test
+        LOG.info('Creating instance %s with AZ %s', vmname, avail_zone)
         instance = self.novaclient.servers.create(name=vmname,
                                                   image=image,
                                                   flavor=flavor,

diff --git a/test/test_chains.py b/test/test_chains.py
index e952eb8..36a29dd 100644 (file)
--- a/test/test_chains.py
+++ b/test/test_chains.py
@@ -298,7 +298,7 @@ def test_placer_user_az():
 
 def test_placer_user_hyp():
     """Test placement when user provides a hypervisor."""
-    check_placer(None, 'comp1', 'comp1')
+    check_placer(None, 'comp1', ':comp1')
     check_placer('nova', 'comp1', 'nova:comp1', resolved=True)
     check_placer(None, 'nova:comp1', 'nova:comp1', resolved=True)
     # hyp overrides az