summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJenkins <jenkins@review.openstack.org>2015-08-28 13:42:13 +0000
committerGerrit Code Review <review@openstack.org>2015-08-28 13:42:13 +0000
commit8699410998b72a513ef7764fe6d81f6a6a5982de (patch)
tree2b4ae72e63051417f7a125ad5fa954f34f09b802
parent3252d458896f7d16897df1408604d01d4b0d835d (diff)
parent2d25fffd29c96b32b44fa350f2038f513ec4b07c (diff)
downloadtuskar-ui-8699410998b72a513ef7764fe6d81f6a6a5982de.tar.gz
Merge "Replace ironic_discoverd.client with ironic_inspector_client"0.4.0
-rw-r--r--requirements.txt2
-rw-r--r--tuskar_ui/api/node.py15
2 files changed, 10 insertions, 7 deletions
diff --git a/requirements.txt b/requirements.txt
index c94775ad..1875d769 100644
--- a/requirements.txt
+++ b/requirements.txt
@@ -2,5 +2,5 @@
# of appearance. Changing the order has an impact on the overall integration
# process, which may cause wedges in the gate later.
os-cloud-config
+python-ironic-inspector-client>=1.0.1
python-ironicclient>=0.6.0
-ironic-discoverd>=1.0.0 # Apache-2.0
diff --git a/tuskar_ui/api/node.py b/tuskar_ui/api/node.py
index 2ad65195..e9fef71b 100644
--- a/tuskar_ui/api/node.py
+++ b/tuskar_ui/api/node.py
@@ -16,12 +16,11 @@ import time
from django.conf import settings
from django.utils.translation import ugettext_lazy as _
from horizon.utils import memoized
-from ironic_discoverd import client as discoverd_client
+from ironic_inspector_client import client as inspector_client
from ironicclient import client as ironic_client
from openstack_dashboard.api import base
from openstack_dashboard.api import glance
from openstack_dashboard.api import nova
-import requests
from tuskar_ui.cached_property import cached_property # noqa
from tuskar_ui.handle_errors import handle_errors # noqa
@@ -265,8 +264,12 @@ class Node(base.APIResourceWrapper):
if not IRONIC_DISCOVERD_URL:
return
for uuid in uuids:
- discoverd_client.introspect(uuid, IRONIC_DISCOVERD_URL,
- request.user.token.id)
+
+ inspector_client.introspect(
+ uuid,
+ base_url=IRONIC_DISCOVERD_URL,
+ auth_token=request.user.token.id)
+
# NOTE(dtantsur): PXE firmware on virtual machines misbehaves when
# a lot of nodes start DHCPing simultaneously: it ignores NACK from
# DHCP server, tries to get the same address, then times out. Work
@@ -357,12 +360,12 @@ class Node(base.APIResourceWrapper):
if not IRONIC_DISCOVERD_URL:
return MAINTENANCE_STATE
try:
- status = discoverd_client.get_status(
+ status = inspector_client.get_status(
uuid=self.uuid,
base_url=IRONIC_DISCOVERD_URL,
auth_token=self._request.user.token.id,
)
- except requests.HTTPError as e:
+ except inspector_client.ClientError as e:
if getattr(e.response, 'status_code', None) == 404:
return MAINTENANCE_STATE
raise