Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Update fact name for fetching fqdn fact #17308

Merged
merged 1 commit into from
Jan 13, 2025
Merged

Update fact name for fetching fqdn fact #17308

merged 1 commit into from
Jan 13, 2025

Conversation

shweta83
Copy link
Contributor

Problem Statement

Assertion failing on host fqdn while fetching the fact.

Solution

Updated the fact name .

Related Issues

@shweta83 shweta83 added CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 6.16.z Introduced in or relating directly to Satellite 6.16 labels Jan 13, 2025
@shweta83 shweta83 requested a review from a team as a code owner January 13, 2025 10:27
@shweta83
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_fact.py -k test_positive_upload_host_facts

@shweta83 shweta83 added 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 labels Jan 13, 2025
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9835
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_fact.py -k test_positive_upload_host_facts --external-logging
Test Result : ================= 2 passed, 46 warnings in 1379.61s (0:22:59) ==================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jan 13, 2025
@Gauravtalreja1 Gauravtalreja1 merged commit ee9b57d into SatelliteQE:master Jan 13, 2025
13 checks passed
github-actions bot pushed a commit that referenced this pull request Jan 13, 2025
@shweta83 @web-flow
Update fact name for fetching fqdn fact (#17308)
2c39fb4 
(cherry picked from commit ee9b57d)
github-actions bot pushed a commit that referenced this pull request Jan 13, 2025
@shweta83 @web-flow
Update fact name for fetching fqdn fact (#17308)
88b22e3 
(cherry picked from commit ee9b57d)
This was referenced Jan 13, 2025
Merged
Merged
github-actions bot pushed a commit that referenced this pull request Jan 13, 2025
@shweta83 @web-flow
Update fact name for fetching fqdn fact (#17308)
86ea6c7 
(cherry picked from commit ee9b57d)
@Satellite-QE Satellite-QE mentioned this pull request Jan 13, 2025
Closed
github-actions bot pushed a commit that referenced this pull request Jan 13, 2025
@shweta83 @web-flow
Update fact name for fetching fqdn fact (#17308)
6a2412f 
(cherry picked from commit ee9b57d)
@Satellite-QE Satellite-QE mentioned this pull request Jan 13, 2025
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

@LadislavVasina1 LadislavVasina1 LadislavVasina1 approved these changes

Assignees

@shweta83 shweta83

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 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 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.
4 participants
@shweta83 @Satellite-QE @Gauravtalreja1 @LadislavVasina1