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

Updating the FDI image to correct discovery issue. #15761

Merged
merged 2 commits into from
Jul 25, 2024

Conversation

amolpati30
Copy link
Contributor

Problem Statement

As we now have the latest version, 8.10, the test included an older version, which caused the issue.

Solution

I have updated the version, and it is now working as expected.

@amolpati30 amolpati30 added Easy Fix :) Easiest Fix to review and quick merge request. No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master labels Jul 24, 2024
@amolpati30 amolpati30 requested review from a team as code owners July 24, 2024 21:45
@amolpati30
Copy link
Contributor Author

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

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7848
Build Status: SUCCESS
PRT Comment: pytest /opt/app-root/src/robottelo/tests/foreman -v -m build_sanity --external-logging
Test Result : ======= 12 passed, 5531 deselected, 5639 warnings in 2081.70s (0:34:41) ========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jul 24, 2024
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Jul 25, 2024
Copy link
Contributor

@shubhamsg199 shubhamsg199 left a comment

Choose a reason for hiding this comment

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

Ack

@shubhamsg199
Copy link
Contributor

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

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7849
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 806.69s (0:13:26) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jul 25, 2024
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Jul 25, 2024
@amolpati30 amolpati30 changed the title Fix for discovery image Updating the FDI image to correct discovery issues. Jul 25, 2024
@amolpati30 amolpati30 changed the title Updating the FDI image to correct discovery issues. Updating the FDI image to correct discovery issue. Jul 25, 2024
@amolpati30
Copy link
Contributor Author

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

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7865
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 736.65s (0:12:16) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jul 25, 2024
@Gauravtalreja1 Gauravtalreja1 merged commit ad0e263 into SatelliteQE:master Jul 25, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Easy Fix :) Easiest Fix to review and quick merge request. No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR Stream Introduced in or relating directly to Satellite Stream/Master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants