-
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
Add coverage for BZ#2173756 #13202
Add coverage for BZ#2173756 #13202
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.
few comments, rebase needed
:customerscenario: true | ||
""" | ||
res = rhel_contenthost.execute('ansible --version') | ||
assert res.status, 'Ansible must not be preinstalled on the host for this test case' |
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.
so what are we asserting here? the existence of the status? I'd rather be specific about the expected value of res.status
), 'Package count available on the host did not meet the expectation' | ||
|
||
res = rhel_contenthost.execute(f'dnf -y install {filtered_pkg}') | ||
assert res.status, 'Installation of filtered package succeeded unexpectedly' |
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.
also here, does it mean res.status != 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.
yes, it means res.status > 0
@pytest.mark.e2e | ||
@pytest.mark.tier3 | ||
@pytest.mark.no_containers | ||
@pytest.mark.rhel_ver_list([8]) |
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.
Why hardcode RHEL 8?
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.
Because we use particular repo to test with too (Ansible Engine for RHEL8). I guess there is no need to test with multiple RHELs where is the repo available.
@pytest.mark.rhel_ver_list([8]) | ||
@pytest.mark.parametrize( | ||
'function_synced_rhel_repo', | ||
['rhae2.9_el8'], |
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.
RHEL8 is also hardcoded here. If the above comment is addressed, this one needs it, too.
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.
Yes, answered above.
88c5967
to
d02d1a1
Compare
trigger: test-robottelo |
PRT seems to fail due to this https://bugzilla.redhat.com/show_bug.cgi?id=2253673 |
trigger: test-robottelo |
d02d1a1
to
555aad2
Compare
Removed old stub, rebased. |
555aad2
to
8feba59
Compare
Resolved conflict and rebased. |
|
54bf1e6
to
238f677
Compare
Resolved conflicts, rebased. |
trigger: test-robottelo |
Looks like the EDIT: It should get in Stream snap 44.0. Stay tuned! |
|
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.
Co-reviewed with @rplevka.
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.
ACK, pending @ogajduse requests for changes (we did the review together)
They are addressed by the previous commit.
238f677
to
15eaa17
Compare
trigger: test-robottelo |
This PR just adds coverage for BZ#2173756 - export/import RH yum repo incrementally (via CV) and consume it on a content host.