-
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
Adapt to changes in broker stderr handling #15442
Adapt to changes in broker stderr handling #15442
Conversation
Since version 0.5 broker accesses the list and decodes the stderr https://github.com/SatelliteQE/broker/blob/master/broker/helpers.py#L512
Can one of the admins verify this patch? |
trigger: test-robottelo |
@JacobCallahan Are the failing PRT tests related to the change? |
It seems that it hit a pipeline error. Let's run it once again. |
trigger: test-robottelo |
Something is going on in our pipeline. I asked @omkarkhatavkar for help. I will post updates in the coming days. |
trigger: test-robottelo |
1 similar comment
trigger: test-robottelo |
Thanks for the patch, @dosas! |
(cherry picked from commit 08f5b26)
(cherry picked from commit 08f5b26)
(cherry picked from commit 08f5b26)
Problem Statement
Since version 0.5 broker accesses the list and
decodes the stderr
https://github.com/SatelliteQE/broker/blob/master/broker/helpers.py#L512
Solution
Simple code adaptation
Related Issues
#15394
relevant tests
tests/foreman/api/test_location.py::TestLocation::test_positive_create_update_and_remove_capsule
tests/foreman/cli/test_organization.py::test_positive_add_and_remove_capsules