openstack
/
python-tripleoclient
Code Issues Proposed changes

Remove printing the introspection status 

As of I892f03b0a703ce4d99c1e0f8caaae5f66dcb0d9b the workflow sends the
introspection status as each node completes. This means that we no
longer need to print out the information at the end as it just
duplicates it.

Closes-Bug: #1711660
Depends-On: I892f03b0a703ce4d99c1e0f8caaae5f66dcb0d9b
Change-Id: Ifc1cf0b959cf7da494d6582070ab0241fcb5e9ac
This commit is contained in:
Dougal Matthews 2017-08-16 16:55:24 +01:00
parent 995acd6834
commit 65898271d8
1 changed files with 1 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
tripleoclient/workflows/baremetal.py
View File

@ -145,7 +145,6 @@ def introspect_manageable_nodes(clients, **workflow_input):
print("Waiting for introspection to finish...")
errors = []
successful_node_uuids = set()
with tripleoclients.messaging_websocket(queue_name) as ws:
execution = base.start_workflow(
@ -164,16 +163,8 @@ def introspect_manageable_nodes(clients, **workflow_input):
if payload['status'] == 'SUCCESS':
introspected_nodes = payload['introspected_nodes'] or {}
for node_uuid, status in introspected_nodes.items():
if status['error'] is None:
print(("Introspection for UUID {0} finished "
"successfully.").format(node_uuid))
successful_node_uuids.add(node_uuid)
else:
print(("Introspection for UUID {0} finished with error"
": {1}").format(node_uuid, status['error']))
if status['error'] is not None:
errors.append("%s: %s" % (node_uuid, status['error']))
if not introspected_nodes:
print("No nodes in manageable state found for introspection.")
else:
raise exceptions.IntrospectionError(
'Exception introspecting nodes: {}'.format(payload['message']))