-
Notifications
You must be signed in to change notification settings - Fork 115
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
hosts cockpit tests fix #12557
hosts cockpit tests fix #12557
Conversation
trigger: test-robottelo |
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.
Looking at the changes, they look sane, but in PRT:
def test_positive_cockpit(self, cockpit_host, class_cockpit_sat, class_org):
[...]
with class_cockpit_sat.ui_session() as session:
session.organization.select(org_name=class_org.name)
session.location.select(loc_name=ANY_CONTEXT['location'])
kill_process = class_cockpit_sat.execute('pkill -f cockpit-ws')
> assert kill_process.status == 0
E AssertionError: assert 1 == 0
E + where 1 = stdout:\n\nstderr:\n(0, b'')\nstatus: 1.status
Use global registration for `contenthosts::cockpit_host` fixture. Also a minor logging message improvement.
dd08b2e
to
aa13454
Compare
trigger: test-robottelo |
Rebasing & re-running PRT. I see in ReportPortal that on with the latest 6.14. only rhel_7 host failed, the rest (rhel 8 & 9) is passing. |
trigger: test-robottelo |
This pull request has not been updated in the past 45 days. |
This pull request is now being closed after stale warnings. |
Use global registration for Cockipt host.
It shoud fix problem with repo setup & Cockpit installation.