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] verify host owner name after host registration #15402

Open
wants to merge 1 commit into
base: 6.15.z
Choose a base branch
from
Open

[6.15.z] verify host owner name after host registration #15402

wants to merge 1 commit into from

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #15178

Problem Statement

When a host is registered, its Owner is not set to the admin that generated the registration command. It stays set to a hidden API user, Owner is displayed as Anonymous Admin.

The user who actually ran the registration process should be set as owner.

Solution

The following automation script is designed to verify that the host is registered correctly and that it is assigned the expected owner name as specified.

@Satellite-QE Satellite-QE requested a review from a team as a code owner June 13, 2024 15:42
@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 13, 2024
@Satellite-QE
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_registration.py -k test_host_registration_end_to_end

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

PRT Result

Build Number: 7389
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_registration.py -k test_host_registration_end_to_end --external-logging
Test Result : ========== 6 failed, 9 deselected, 522 warnings in 3746.04s (1:02:26) ==========

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Jun 13, 2024
@amolpati30
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/cli/test_registration.py -k test_host_registration_end_to_end

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 7392
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_registration.py -k test_host_registration_end_to_end --external-logging
Test Result : ========== 6 failed, 9 deselected, 533 warnings in 3807.97s (1:03:27) ==========

@amolpati30
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/cli/test_registration.py -k test_host_registration_end_to_end

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 7394
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_registration.py -k test_host_registration_end_to_end --external-logging
Test Result : ========== 6 failed, 9 deselected, 489 warnings in 3369.37s (0:56:09) ==========

@amolpati30
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/cli/test_registration.py -k test_host_registration_end_to_end

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 7436
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_registration.py -k test_host_registration_end_to_end --external-logging
Test Result : ========== 6 failed, 9 deselected, 480 warnings in 3312.08s (0:55:12) ==========

@amolpati30 @Gauravtalreja1
verify host owner name after host registration (#15178)
9815ec0 
* verify host name after host registration

* assertion added for host

(cherry picked from commit 423864f)
@Gauravtalreja1 Gauravtalreja1 force-pushed the cherry-pick-6.15.z-423864f72c811278e59bc2160247018ec2e4b4be branch from 42d880f to 9815ec0 Compare June 21, 2024 19:16
@Gauravtalreja1
Copy link
Collaborator

trigger: test-robottelo
pytest: tests/foreman/cli/test_registration.py -k test_host_registration_end_to_end

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 7480
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_registration.py -k test_host_registration_end_to_end --external-logging
Test Result : ========== 6 failed, 9 deselected, 507 warnings in 3537.93s (0:58:57) ==========

@amolpati30
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/cli/test_registration.py -k test_host_registration_end_to_end

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 7482
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_registration.py -k test_host_registration_end_to_end --external-logging
Test Result : ========== 6 failed, 9 deselected, 492 warnings in 3466.19s (0:57:46) ==========

@Gauravtalreja1
Copy link
Collaborator

@amolpati30 Can you investigate the PRT failures, and update the PR?

@amolpati30
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/cli/test_registration.py -k test_host_registration_end_to_end

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 7670
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_registration.py -k test_host_registration_end_to_end --external-logging
Test Result : ========== 6 failed, 9 deselected, 526 warnings in 3536.43s (0:58:56) ==========

@Gauravtalreja1
Copy link
Collaborator

@amolpati30 Could you please update this PR?

@amolpati30
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/cli/test_registration.py -k test_host_registration_end_to_end

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 8175
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_registration.py -k test_host_registration_end_to_end --external-logging
Test Result : ===== 5 failed, 9 deselected, 781 warnings, 1 error in 6001.05s (1:40:01) ======

@lhellebr
Copy link
Contributor

lhellebr commented Sep 4, 2024

trigger: test-robottelo
pytest: tests/foreman/cli/test_registration.py -k test_host_registration_end_to_end

@shweta83
Copy link
Contributor

@amolpati30 Can you please rebase the PR and check the failure?

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

@amolpati30 amolpati30

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-Failed Indicates that latest PRT run is failed for the PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Satellite-QE @amolpati30 @Gauravtalreja1 @lhellebr @shweta83