-
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
Remove use of install_katello_ca with register_contenthost #15395
Conversation
trigger: test-robottelo |
PRT Result
|
7e167ed
to
54d2ed0
Compare
trigger: test-robottelo |
PRT Result
|
54d2ed0
to
1f60c73
Compare
trigger: test-robottelo |
PRT Result
|
1f60c73
to
b909d65
Compare
trigger: test-robottelo |
PRT Result
|
b909d65
to
bc4cfb9
Compare
trigger: test-robottelo |
PRT Result
|
a0bb753
to
84fa3fb
Compare
trigger: test-robottelo |
PRT Result
|
84fa3fb
to
5a910c6
Compare
trigger: test-robottelo |
PRT Result
|
63d02d8
to
298a374
Compare
298a374
to
87de06b
Compare
trigger: test-robottelo |
PRT Result
|
trigger: test-robottelo |
PRT Result
|
Ok looking through the failures it seems these are not related to the changes and have been failing for the same reasons according to Ibutsu. |
trigger: test-robottelo |
PRT Result
|
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 for the Endeavour's components
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.
Next steps (future PR) I see are:
- remove
install_katello_ca
method - add
if reg_result.status == 0: self._satellite = target
at the very end of theregister
andapi_register
methods
Co-authored-by: vsedmik <[email protected]>
…QE#15395) * Remove use of install_katello_ca with register_contenthost * Update tests/foreman/cli/test_contentview.py Co-authored-by: vsedmik <[email protected]> --------- Co-authored-by: Jameer Pathan <[email protected]> Co-authored-by: vsedmik <[email protected]>
Problem Statement
We will be removing the katello-ca-consumer RPM by default. Our automation should only use global registration anyway.
Solution
Swap uses of
install_katello_ca
andregister_contenthost
to theregister
method which uses global registration. Use default location and a basic activation key where needed.Related Issues
PRT test Cases example
trigger: test-robottelo
pytest: tests/foreman/cli/test_host.py::test_positive_install_package_via_rex tests/foreman/cli/test_http_proxy.py::test_insights_client_registration_with_http_proxy tests/foreman/ui/test_subscription.py::test_positive_view_vdc_subscription_products tests/foreman/destructive/test_host.py::TestHostCockpit::test_positive_cockpit tests/foreman/cli/test_contentview.py::TestContentView::test_positive_sub_host_with_restricted_user_perm_at_custom_loc tests/foreman/cli/test_contentview.py::TestContentView::test_positive_sub_host_with_restricted_user_perm_at_default_loc tests/foreman/destructive/test_capsule_loadbalancer.py::test_loadbalancer_install_package tests/foreman/endtoend/test_api_endtoend.py::TestEndToEnd::test_positive_end_to_end tests/foreman/endtoend/test_cli_endtoend.py::test_positive_cli_end_to_end tests/foreman/installer/test_installer.py::test_capsule_installation