aboutsummaryrefslogtreecommitdiffstats
path: root/meta-openstack
diff options
context:
space:
mode:
authorVu Tran <vu.tran@windriver.com>2014-07-10 10:23:50 -0400
committerBruce Ashfield <bruce.ashfield@windriver.com>2014-07-17 00:14:05 -0400
commit3d16c9b202967ee9485943568db917d65288b6f3 (patch)
tree8c538a4ce7b0fa4c2b85766806a43a0a7550b43f /meta-openstack
parent526542c36d53d24c786a1d2e481176899f20bdff (diff)
downloadmeta-cloud-services-3d16c9b202967ee9485943568db917d65288b6f3.tar.gz
meta-cloud-services-3d16c9b202967ee9485943568db917d65288b6f3.tar.bz2
meta-cloud-services-3d16c9b202967ee9485943568db917d65288b6f3.zip
rally: remove ironic support
OpenStack ironic is not currently supported, so remove any code in rally that invokes ironicclient. Signed-off-by: Vu Tran <vu.tran@windriver.com>
Diffstat (limited to 'meta-openstack')
-rw-r--r--meta-openstack/recipes-devtools/python/python-rally/remove-ironic-support.patch87
-rw-r--r--meta-openstack/recipes-devtools/python/python-rally_git.bb1
2 files changed, 88 insertions, 0 deletions
diff --git a/meta-openstack/recipes-devtools/python/python-rally/remove-ironic-support.patch b/meta-openstack/recipes-devtools/python/python-rally/remove-ironic-support.patch
new file mode 100644
index 00000000..10df953c
--- /dev/null
+++ b/meta-openstack/recipes-devtools/python/python-rally/remove-ironic-support.patch
@@ -0,0 +1,87 @@
+remove ironic support
+
+OpenStack ironic is not currently supported, so remove
+any code in rally that invokes ironicclient.
+
+Signed-off-by: Vu Tran <vu.tran@windriver.com>
+
+diff --git a/rally/benchmark/context/images.py b/rally/benchmark/context/images.py
+index 9642af4..92abbf6 100644
+--- a/rally/benchmark/context/images.py
++++ b/rally/benchmark/context/images.py
+@@ -21,6 +21,7 @@ from rally import exceptions
+ from rally.openstack.common import log as logging
+ from rally import osclients
+ from rally import utils as rutils
++from rally.openstack.common.gettextutils import _
+
+
+ LOG = logging.getLogger(__name__)
+diff --git a/rally/osclients.py b/rally/osclients.py
+index af916b9..66522cd 100644
+--- a/rally/osclients.py
++++ b/rally/osclients.py
+@@ -19,7 +19,6 @@ from ceilometerclient import client as ceilometer
+ from cinderclient import client as cinder
+ import glanceclient as glance
+ from heatclient import client as heat
+-from ironicclient import client as ironic
+ from keystoneclient import exceptions as keystone_exceptions
+ from keystoneclient.v2_0 import client as keystone
+ from neutronclient.neutron import client as neutron
+@@ -204,16 +203,3 @@ class Clients(object):
+ insecure=CONF.https_insecure,
+ cacert=CONF.https_cacert)
+ return client
+-
+- @memoize('ironic')
+- def ironic(self, version='1.0'):
+- """Returns Ironic client."""
+- client = ironic.Client(version,
+- username=self.endpoint.username,
+- password=self.endpoint.password,
+- tenant_name=self.endpoint.tenant_name,
+- auth_url=self.endpoint.auth_url,
+- timeout=CONF.openstack_client_http_timeout,
+- insecure=CONF.https_insecure,
+- cacert=CONF.https_cacert)
+- return client
+diff --git a/requirements.txt b/requirements.txt
+index 9ab2e61..318bcfe 100644
+--- a/requirements.txt
++++ b/requirements.txt
+@@ -17,7 +17,6 @@ python-neutronclient>=2.3.5,<3
+ python-cinderclient>=1.0.6
+ python-heatclient>=0.2.9
+ python-ceilometerclient>=1.0.6
+-python-ironicclient
+ python-subunit>=0.0.18
+ requests>=1.1
+ SQLAlchemy>=0.7.8,!=0.9.5,<=0.9.99
+diff --git a/tests/test_osclients.py b/tests/test_osclients.py
+index 49cecb8..6ca03f4 100644
+--- a/tests/test_osclients.py
++++ b/tests/test_osclients.py
+@@ -161,22 +161,3 @@ class OSClientsTestCase(test.TestCase):
+ mock_ceilometer.Client.assert_called_once_with("2", **kw)
+ self.assertEqual(self.clients.cache["ceilometer"],
+ fake_ceilometer)
+-
+- @mock.patch("rally.osclients.ironic")
+- def test_ironic(self, mock_ironic):
+- fake_ironic = fakes.FakeIronicClient()
+- mock_ironic.Client = mock.MagicMock(return_value=fake_ironic)
+- self.assertTrue("ironic" not in self.clients.cache)
+- client = self.clients.ironic()
+- self.assertEqual(client, fake_ironic)
+- kw = {
+- "username": self.endpoint.username,
+- "password": self.endpoint.password,
+- "tenant_name": self.endpoint.tenant_name,
+- "auth_url": self.endpoint.auth_url,
+- "timeout": cfg.CONF.openstack_client_http_timeout,
+- "insecure": cfg.CONF.https_insecure,
+- "cacert": cfg.CONF.https_cacert
+- }
+- mock_ironic.Client.assert_called_once_with("1.0", **kw)
+- self.assertEqual(self.clients.cache["ironic"], fake_ironic)
diff --git a/meta-openstack/recipes-devtools/python/python-rally_git.bb b/meta-openstack/recipes-devtools/python/python-rally_git.bb
index 811fd07c..77c140a3 100644
--- a/meta-openstack/recipes-devtools/python/python-rally_git.bb
+++ b/meta-openstack/recipes-devtools/python/python-rally_git.bb
@@ -14,6 +14,7 @@ SRC_URI = "git://github.com/stackforge/${SRCNAME}.git;branch=master \
file://rally.conf \
file://task-example.json \
file://deployment-existing.json \
+ file://remove-ironic-support.patch \
"
SRCREV="b297cf00750f263b8b5bdeb71f6952f672e87f5a"