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

Update verifies jira in FDI validation test #16698

Merged
merged 1 commit into from
Oct 18, 2024
Merged

Update verifies jira in FDI validation test #16698

merged 1 commit into from
Oct 18, 2024

Conversation

Gauravtalreja1
Copy link
Collaborator

Problem Statement

SAT-25275 was closed in favor of SAT-27541, which caused this test to fail for EL9

Solution

Removing SAT-25275 used in is_open, and updating it to use SAT-27541

@Gauravtalreja1 Gauravtalreja1 added Easy Fix :) Easiest Fix to review and quick merge request. CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 6.15.z Introduced in or relating directly to Satellite 6.15 6.16.z Introduced in or relating directly to Satellite 6.16 labels Oct 17, 2024
@Gauravtalreja1 Gauravtalreja1 self-assigned this Oct 17, 2024
@Gauravtalreja1 Gauravtalreja1 requested a review from a team as a code owner October 17, 2024 16:34
@Gauravtalreja1
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_discoveredhost.py -k test_positive_verify_updated_fdi_image

@Gauravtalreja1 Gauravtalreja1 changed the title Update vefifies jira in FDI validation test Update verifies jira in FDI validation test Oct 17, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9006
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_discoveredhost.py -k test_positive_verify_updated_fdi_image --external-logging
Test Result : ========== 1 passed, 17 deselected, 26 warnings in 1175.40s (0:19:35) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Oct 17, 2024
@Gauravtalreja1
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_discoveredhost.py -k test_positive_verify_updated_fdi_image
deploy_rhel_version: 9

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9014
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_discoveredhost.py -k test_positive_verify_updated_fdi_image --external-logging
Test Result : ========== 1 passed, 17 deselected, 26 warnings in 689.51s (0:11:29) ===========

@shubhamsg199 shubhamsg199 merged commit e8f428f into SatelliteQE:master Oct 18, 2024
14 checks passed
github-actions bot pushed a commit that referenced this pull request Oct 18, 2024
@Gauravtalreja1 @web-flow
Update verifies jira in FDI validation test (#16698)
8bb3eef 
Update vefifies jira in FDI validation test

Signed-off-by: Gaurav Talreja <[email protected]>
(cherry picked from commit e8f428f)
This was referenced Oct 18, 2024
Merged
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shubhamsg199 shubhamsg199 shubhamsg199 approved these changes

Assignees

@Gauravtalreja1 Gauravtalreja1

Labels
6.15.z Introduced in or relating directly to Satellite 6.15 6.16.z Introduced in or relating directly to Satellite 6.16 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs CherryPick to previous branches Easy Fix :) Easiest Fix to review and quick merge request. 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.
3 participants
@Gauravtalreja1 @Satellite-QE @shubhamsg199