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

Test Coverage for FDI version verification #13209

Merged

Conversation

adarshdubey-star
Copy link
Contributor

Problem Statement

The versioning of the foreman discovery image should be consistent or greater in the latest version of the satellite.

Solution

Save the FDI version pre_upgrade and post upgrading check the version.

Related Issues

Bugzilla

@adarshdubey-star adarshdubey-star requested a review from a team as a code owner November 29, 2023 10:14
@adarshdubey-star adarshdubey-star self-assigned this Nov 29, 2023
@adarshdubey-star adarshdubey-star added QETestCoverage Issues and PRs relating to a Satellite bug 6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 CherryPick PR needs CherryPick to previous branches labels Nov 29, 2023
tests/upgrades/test_discovery.py Outdated Show resolved Hide resolved
tests/upgrades/test_discovery.py Outdated Show resolved Hide resolved
Copy link
Contributor

@lhellebr lhellebr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm no expert in FDI but a lot of things seem fishy to me, I left quite a few comments.

tests/upgrades/test_discovery.py Outdated Show resolved Hide resolved
tests/upgrades/test_discovery.py Outdated Show resolved Hide resolved
tests/upgrades/test_discovery.py Outdated Show resolved Hide resolved
tests/upgrades/test_discovery.py Outdated Show resolved Hide resolved
tests/upgrades/test_discovery.py Outdated Show resolved Hide resolved
tests/upgrades/test_discovery.py Outdated Show resolved Hide resolved
@adarshdubey-star adarshdubey-star added the in progress This issue is being worked on label Dec 4, 2023
@adarshdubey-star
Copy link
Contributor Author

adarshdubey-star commented Dec 6, 2023

Test result :

Pre-upgrade

pytest --disable-pytest-warnings -m pre_upgrade tests/upgrades/test_discovery.py -sv

========================================================================== test session starts ==========================================================================                                                                                                           tests/upgrades/test_discovery.py::TestDiscoveryImage::test_pre_upgrade_fdi_version PASSED

======================================================== 1 passed, 1 deselected, 1 warning in 129.28s (0:02:09) =========================================================

Post-upgrade

pytest tests/upgrades/test_discovery.py -k test_post_upgrade_fdi_version -m post_upgrade
========================================================================== test session starts ==========================================================================

collected 2 items / 1 deselected / 1 selected                                                                                                                           

tests/upgrades/test_discovery.py .                                                                                                                                [100%]

=========================================================================== warnings summary ============================================================================

============================================================== 1 passed, 1 deselected, 1 warning in 12.37s ==============================================================

@adarshdubey-star adarshdubey-star removed the in progress This issue is being worked on label Dec 7, 2023
@Gauravtalreja1 Gauravtalreja1 added the Upgrades Issues and PRs related to upgrades label Dec 7, 2023
@adarshdubey-star adarshdubey-star force-pushed the discovery_upgrade branch 2 times, most recently from f7f2c4f to 6f92901 Compare December 7, 2023 09:47
@Gauravtalreja1 Gauravtalreja1 merged commit 18acd55 into SatelliteQE:master Dec 7, 2023
5 checks passed
github-actions bot pushed a commit that referenced this pull request Dec 7, 2023
* Test Coverage for FDI version verification

* Removed unwanted code

* Using foreman-maintain for installation

* Using Version

(cherry picked from commit 18acd55)
github-actions bot pushed a commit that referenced this pull request Dec 7, 2023
* Test Coverage for FDI version verification

* Removed unwanted code

* Using foreman-maintain for installation

* Using Version

(cherry picked from commit 18acd55)
github-actions bot pushed a commit that referenced this pull request Dec 7, 2023
* Test Coverage for FDI version verification

* Removed unwanted code

* Using foreman-maintain for installation

* Using Version

(cherry picked from commit 18acd55)
Gauravtalreja1 pushed a commit that referenced this pull request Dec 7, 2023
Test Coverage for FDI version verification (#13209)

* Test Coverage for FDI version verification

* Removed unwanted code

* Using foreman-maintain for installation

* Using Version

(cherry picked from commit 18acd55)

Co-authored-by: Adarsh dubey <[email protected]>
Gauravtalreja1 pushed a commit that referenced this pull request Dec 7, 2023
Test Coverage for FDI version verification (#13209)

* Test Coverage for FDI version verification

* Removed unwanted code

* Using foreman-maintain for installation

* Using Version

(cherry picked from commit 18acd55)

Co-authored-by: Adarsh dubey <[email protected]>
Gauravtalreja1 pushed a commit that referenced this pull request Dec 7, 2023
Test Coverage for FDI version verification (#13209)

* Test Coverage for FDI version verification

* Removed unwanted code

* Using foreman-maintain for installation

* Using Version

(cherry picked from commit 18acd55)

Co-authored-by: Adarsh dubey <[email protected]>
damoore044 pushed a commit to damoore044/robottelo that referenced this pull request Dec 11, 2023
* Test Coverage for FDI version verification

* Removed unwanted code

* Using foreman-maintain for installation

* Using Version
shweta83 pushed a commit to shweta83/robottelo that referenced this pull request Apr 10, 2024
* Test Coverage for FDI version verification

* Removed unwanted code

* Using foreman-maintain for installation

* Using Version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 CherryPick PR needs CherryPick to previous branches QETestCoverage Issues and PRs relating to a Satellite bug Upgrades Issues and PRs related to upgrades
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants