Browse Source

Merge "improve error messages during node lookup failures" into stable/stein

stable/stein
Zuul 1 month ago
committed by Gerrit Code Review
parent
commit
86aead2f35
1 changed files with 15 additions and 4 deletions
  1. +15
    -4
      ironic_python_agent/ironic_api_client.py

+ 15
- 4
ironic_python_agent/ironic_api_client.py View File

@@ -148,12 +148,19 @@ class APIClient(object):
'GET', self.lookup_api,
headers=self._get_ironic_api_version_header(),
params=params)
except Exception:
LOG.exception('Lookup failed')
except Exception as err:
LOG.exception(
'Unhandled error looking up node with addresses %r at %s: %s',
params['addresses'], self.api_url, err,
)
return False

if response.status_code != requests.codes.OK:
LOG.warning('Failure status code: %s', response.status_code)
LOG.warning(
'Failed looking up node with addresses %r at %s, '
'status code: %s',
params['addresses'], self.api_url, response.status_code,
)
return False

try:
@@ -164,7 +171,11 @@ class APIClient(object):

# Check for valid response data
if 'node' not in content or 'uuid' not in content['node']:
LOG.warning('Got invalid node data from the API: %s', content)
LOG.warning(
'Got invalid node data in response to query for node '
'with addresses %r from %s: %s',
params['addresses'], self.api_url, content,
)
return False

if 'config' not in content:


Loading…
Cancel
Save