diff options
author | Jenkins <jenkins@review.openstack.org> | 2015-08-28 13:42:13 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2015-08-28 13:42:13 +0000 |
commit | 8699410998b72a513ef7764fe6d81f6a6a5982de (patch) | |
tree | 2b4ae72e63051417f7a125ad5fa954f34f09b802 | |
parent | 3252d458896f7d16897df1408604d01d4b0d835d (diff) | |
parent | 2d25fffd29c96b32b44fa350f2038f513ec4b07c (diff) | |
download | tuskar-ui-8699410998b72a513ef7764fe6d81f6a6a5982de.tar.gz |
Merge "Replace ironic_discoverd.client with ironic_inspector_client"0.4.0
-rw-r--r-- | requirements.txt | 2 | ||||
-rw-r--r-- | tuskar_ui/api/node.py | 15 |
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 |