-
Notifications
You must be signed in to change notification settings - Fork 116
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
[6.14.z] Fix KA check #14766
[6.14.z] Fix KA check #14766
Conversation
trigger: test-robottelo |
PRT Result
|
b7b3466
to
680d0e5
Compare
trigger: test-robottelo |
PRT Result
|
@@ -153,8 +154,7 @@ def test_positive_upgrade_warning(sat_with_katello_agent): | |||
|
|||
""" | |||
sat = sat_with_katello_agent | |||
ver = sat.version.split('.') | |||
target_ver = f'{ver[0]}.{int(ver[1]) + 1}' | |||
target_ver = '6.15' |
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.
@vsedmik Do we need to keep this hardcoded? I don't understand the reason for removing the original code related to target_ver
. Or we can use SATELLITE_VERSION
that would be better in my opinion than having another constant here. wdyt?
from robottelo.constants import SATELLITE_VERSION
target_ver = float(SATELLITE_VERSION) + 1
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.
@jameerpathan111 The idea behind is that this test is "6.14.z-only" so the next version is always 6.15. But I don't mind to bring the original code back if you think it's better.
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.
Yeah, but it's always good to keep things consistent across the branches, it avoids complications going forward like in the case of cherry picks. Anyway, you have my ACK:)
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.
This test cannot pass in branches other than 6.14.z
so I wouldn't be afraid of CPs. Anyway, updated for consistency.
680d0e5
to
0a2ed6f
Compare
trigger: test-robottelo |
PRT Result
|
Problem Statement
KA removal warning test is failing since
6.15
scenario has been removed from foreman-maintain nightly.Solution
Use 6.15 version of foreman-maintain instead.
PRT test Cases example
trigger: test-robottelo
pytest: tests/foreman/destructive/test_katello_agent.py -k test_positive_upgrade_warning