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.

Sign up for GitHub

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

Jump to bottom

[6.15.z] fix parameterisation issue to run test for rhel7/8/9 #15538

Merged
merged 1 commit into from
Jul 1, 2024
Merged

[6.15.z] fix parameterisation issue to run test for rhel7/8/9 #15538

merged 1 commit into from
Jul 1, 2024

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #15537

Problem Statement

Small mistake in PR #15428 related to paramerization. So test will only run for rhel9. @damoore044 notice this when 6.15.z CP PR ran only one test case.

Solution

Update marker for test

PRT test Cases example

trigger: test-robottelo
pytest: tests/foreman/cli/test_host.py -k 'test_host_registration_with_capsule_using_content_coherence'

Note

@damoore044 Please Ack and merge this PR once PRT pass for all 6 rhel versions.

@Satellite-QE Satellite-QE requested a review from a team as a code owner June 28, 2024 18:16
@Satellite-QE Satellite-QE added 6.15.z Introduced in or relating directly to Satellite 6.15 Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels Jun 28, 2024
@Satellite-QE
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_host.py -k 'test_host_registration_with_capsule_using_content_coherence'

@Satellite-QE Satellite-QE added the AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing label Jun 28, 2024
@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 7577
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_host.py -k test_host_registration_with_capsule_using_content_coherence --external-logging
Test Result : ========= 6 passed, 88 deselected, 451 warnings in 2818.28s (0:46:58) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jun 28, 2024
@damoore044 damoore044 enabled auto-merge (squash) June 28, 2024 19:31
@damoore044 damoore044 added AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing and removed AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing labels Jun 29, 2024
@damoore044 damoore044 disabled auto-merge June 29, 2024 16:35
@Gauravtalreja1
Copy link
Collaborator

pre-commit.ci autofix

@vijaysawant vijaysawant force-pushed the cherry-pick-6.15.z-06138d98c9538cdcb22a8ae05065ea68565a5124 branch from 7145cae to 290b905 Compare July 1, 2024 09:26
@Gauravtalreja1 Gauravtalreja1 merged commit 001fc38 into 6.15.z Jul 1, 2024
7 of 8 checks passed
@Gauravtalreja1 Gauravtalreja1 deleted the cherry-pick-6.15.z-06138d98c9538cdcb22a8ae05065ea68565a5124 branch July 1, 2024 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@damoore044 damoore044 damoore044 approved these changes

@ColeHiggins2 ColeHiggins2 ColeHiggins2 approved these changes

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

Assignees

@vijaysawant vijaysawant

Labels
6.15.z Introduced in or relating directly to Satellite 6.15 Auto_Cherry_Picked Automatically cherrypicked PR using GHA AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Satellite-QE @Gauravtalreja1 @ColeHiggins2 @damoore044 @vijaysawant