Skip to content
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

Merged
merged 1 commit into from
Apr 17, 2024
Merged

Conversation

vsedmik
Copy link
Contributor

@vsedmik vsedmik commented Apr 12, 2024

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

@vsedmik vsedmik added the No-CherryPick PR doesnt need CherryPick to previous branches label Apr 12, 2024
@vsedmik vsedmik self-assigned this Apr 12, 2024
@vsedmik
Copy link
Contributor Author

vsedmik commented Apr 12, 2024

trigger: test-robottelo
pytest: tests/foreman/destructive/test_katello_agent.py -k test_positive_upgrade_warning

@Gauravtalreja1 Gauravtalreja1 added the 6.14.z Introduced in or relating directly to Satellite 6.14 label Apr 12, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6494
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/destructive/test_katello_agent.py -k test_positive_upgrade_warning --external-logging
Test Result : ========== 1 passed, 20 deselected, 37 warnings in 2411.77s (0:40:11) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Apr 12, 2024
@vsedmik vsedmik marked this pull request as draft April 15, 2024 11:00
@vsedmik vsedmik force-pushed the 6.14.z-fix-ka-check branch from b7b3466 to 680d0e5 Compare April 16, 2024 12:29
@vsedmik
Copy link
Contributor Author

vsedmik commented Apr 16, 2024

trigger: test-robottelo
pytest: tests/foreman/destructive/test_katello_agent.py -k test_positive_upgrade_warning

@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Apr 16, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6518
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/destructive/test_katello_agent.py -k test_positive_upgrade_warning --external-logging
Test Result : ========== 1 passed, 20 deselected, 37 warnings in 2310.19s (0:38:30) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Apr 16, 2024
@vsedmik vsedmik requested a review from jameerpathan111 April 16, 2024 13:23
@vsedmik vsedmik marked this pull request as ready for review April 16, 2024 13:29
@@ -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'
Copy link
Contributor

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

Copy link
Contributor Author

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.

Copy link
Contributor

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:)

Copy link
Contributor Author

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.

@vsedmik vsedmik force-pushed the 6.14.z-fix-ka-check branch from 680d0e5 to 0a2ed6f Compare April 17, 2024 09:14
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Apr 17, 2024
@vsedmik
Copy link
Contributor Author

vsedmik commented Apr 17, 2024

trigger: test-robottelo
pytest: tests/foreman/destructive/test_katello_agent.py -k test_positive_upgrade_warning

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6552
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/destructive/test_katello_agent.py -k test_positive_upgrade_warning --external-logging
Test Result : ========== 1 passed, 20 deselected, 37 warnings in 2463.58s (0:41:03) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Apr 17, 2024
@jameerpathan111 jameerpathan111 merged commit b48c273 into SatelliteQE:6.14.z Apr 17, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.14.z Introduced in or relating directly to Satellite 6.14 No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants