Skip to content

Commit

Permalink
Fix error log "Found unknown instance manager"
Browse files Browse the repository at this point in the history
When we found an instance manager with correct type but not in the
running state, don't print out the log "Found unknown instance manager"
because it is misleading

The function wait_for_instance_manager_desire_state() below it already
taking care of waiting for the instance manager to be running.

longhorn-7413

Signed-off-by: Phan Le <[email protected]>
  • Loading branch information
PhanLe1010 authored and David Ko committed Dec 29, 2023
1 parent d9f74dc commit c0d1a4a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion manager/integration/tests/common.py
Original file line number Diff line number Diff line change
Expand Up @@ -3556,7 +3556,7 @@ def wait_for_all_instance_manager_running(client):
node_to_instance_manager_map = {}
try:
for im in instance_managers:
if im.managerType == "aio" and im.currentState == "running":
if im.managerType == "aio":
node_to_instance_manager_map[im.nodeID] = im
else:
print("\nFound unknown instance manager:", im)
Expand Down

0 comments on commit c0d1a4a

Please sign in to comment.