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

updated curl to register host with name param instead id #17152

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

updated curl to register host with name param instead id #17152

wants to merge 1 commit into from

Conversation

amolpati30
Copy link
Contributor

The host is successfully registering using the label/name instead of the ID, and this applies to the following parameters:

  • Organization
  • Location
  • Host group
  • Operating System

@amolpati30 amolpati30 added QETestCoverage Issues and PRs relating to a Satellite bug CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 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 labels Dec 11, 2024
@amolpati30 amolpati30 requested a review from a team as a code owner December 11, 2024 15:56
@amolpati30 amolpati30 force-pushed the register_host_with_curl_with_name_param branch from 3ee8c57 to 7c974c2 Compare December 11, 2024 15:59
@amolpati30
Copy link
Contributor Author

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

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9592
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_fact.py -k test_positive_upload_host_facts --external-logging
Test Result : ================== 1 passed, 12 warnings in 728.79s (0:12:08) ==================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Dec 11, 2024
pondrejk
pondrejk reviewed Dec 12, 2024
View reviewed changes
tests/foreman/ui/test_fact.py
2. Navigate to the Host page and click on kebab.
3. Verify that the facts option is available and is updated on Satellite.

:expectedresults: Facts option is available on the Host UI and all the facts of the host are updated in Satellite.
:expectedresults:
1. The host is successfully register using the label/name instead of the ID.
Copy link
Contributor

Choose a reason for hiding this comment

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

registered?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pondrejk pondrejk pondrejk left review comments

Assignees

@amolpati30 amolpati30

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 QETestCoverage Issues and PRs relating to a Satellite bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@amolpati30 @Satellite-QE @pondrejk