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

update rh satellite capsule version to rhel9 from rhel7 #17421

Merged
merged 2 commits into from
Jan 24, 2025

Conversation

vijaysawant
Copy link
Contributor

Problem Statement

Upgrade test was failing due to unavailability of Red Hat Satellite Capsule 6.16 for RHEL 7 Server RPMs x86_64 repository, as Satellite is running on RHEL8 & RHEL9, no RHSC RHEL7 repo present in reposet

Solution

Update test case and pass rhsc rhel9 details to fetch correct details.

Related Issues

No

PRT test Cases example

trigger: test-robottelo
pytest: tests/foreman/ui/test_package.py -k 'test_positive_rh_repo_search_and_check_file_list'

@vijaysawant vijaysawant added TestFailure Issues and PRs related to a test failing in automation Upgrades Issues and PRs related to upgrades No-CherryPick PR doesnt need CherryPick to previous branches 6.16.z Introduced in or relating directly to Satellite 6.16 labels Jan 23, 2025
@vijaysawant vijaysawant requested a review from a team as a code owner January 23, 2025 14:15
@vijaysawant
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_package.py -k 'test_positive_rh_repo_search_and_check_file_list'

@vijaysawant
Copy link
Contributor Author

Local test execution result:

Launching pytest with arguments -k test_positive_access_with_non_admin_user_with_manifest -vvs /home/visawant/Desktop/workspace/robottelo/tests/foreman/ui/test_subscription.py --no-header --no-summary -q in /home/visawant/Desktop/workspace/robottelo

============================= test session starts ==============================
collecting ... 2025-01-21 13:34:32 - robottelo.collection - INFO - Processing test items to add testimony token markers
collected 8 items / 7 deselected / 1 selected

tests/foreman/ui/test_subscription.py::test_positive_access_with_non_admin_user_with_manifest 

=========== 1 passed, 7 deselected, 3 warnings in 124.06s (0:02:04) ============
PASSED
Process finished with exit code 0

@vijaysawant vijaysawant requested a review from vsedmik January 23, 2025 14:18
Copy link
Contributor

@synkd synkd left a comment

Choose a reason for hiding this comment

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

Since the Red Hat Satellite Capsule 6.16 for RHEL 7 Server repo does not exist, can you please remove the corresponding items from the REPOS and REPOSETS constants in robottelo/constants/init.py as well?

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9970
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_package.py -k test_positive_rh_repo_search_and_check_file_list --external-logging
Test Result : =========== 1 passed, 5 deselected, 16 warnings in 736.02s (0:12:16) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jan 23, 2025
@vijaysawant vijaysawant changed the title update rhsc verstion to rhel9 from rhel7 update rh satellite capsule version to rhel9 from rhel7 Jan 23, 2025
@vijaysawant
Copy link
Contributor Author

Since the Red Hat Satellite Capsule 6.16 for RHEL 7 Server repo does not exist, can you please remove the corresponding items from the REPOS and REPOSETS constants in robottelo/constants/init.py as well?

Sure, but I need to talk with team-phoenix-content before updating anything related to Capsule component.

@vijaysawant vijaysawant requested a review from a team as a code owner January 23, 2025 16:12
@vijaysawant vijaysawant requested a review from synkd January 23, 2025 16:12
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Jan 23, 2025
Copy link
Contributor

@synkd synkd left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for making that change @vijaysawant.

@jyejare jyejare merged commit 611ee7b into SatelliteQE:6.16.z Jan 24, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.16.z Introduced in or relating directly to Satellite 6.16 No-CherryPick PR doesnt need CherryPick to previous branches TestFailure Issues and PRs related to a test failing in automation Upgrades Issues and PRs related to upgrades
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants