-
Notifications
You must be signed in to change notification settings - Fork 116
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
[Customer Case] Adding test for syncing inventory with missing ip #16547
[Customer Case] Adding test for syncing inventory with missing ip #16547
Conversation
trigger: test-robottelo |
PRT Result
|
for task in task_output: | ||
if task.input.get("organization_ids") is None: | ||
continue | ||
if str(task.input.get("organization_ids")[0]) == str(org.id): | ||
host_status = task.output | ||
break |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we narrow the task search instead of this looping through orgs? Something like
query={'search': f'{inventory_sync_task} and started_at >= "{timestamp}" and organization_id={org.id}}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried that. But could not get it to work... even on the UI itself
assert host_status['host_statuses']['sync'] == 2 | ||
assert host_status['host_statuses']['disconnect'] == 0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some context on these 2
and 0
values might be helpful in the future for anyone who looks here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pulled from the rhcloud_registered_hosts fixture. Two hosts are spun up and registered with insights. In our case, the first one has a missing IP. Both however should be registered and synced to rh cloud without failure.
…telliteQE#16547) * Adding test for syncing inventory with missing ip * update test and docsrings * precommit check
Customer Case automation for SAT-24805. Sync inventory status with missing IP