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

host should successfully reg with setup_insights=true #16606

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

host should successfully reg with setup_insights=true #16606

wants to merge 1 commit into from

Conversation

amolpati30
Copy link
Contributor

@amolpati30 amolpati30 commented Oct 8, 2024 •
edited
Loading

Problem Statement

The global registration method of registration started behaving oddly. The entire process completes but the Host remains in build mode

Solution

Automation: The host has been successfully registered and is no longer in build mode. Additionally, the facts for the host have been processed

Dependent PR: SatelliteQE/airgun#1591

@amolpati30 amolpati30 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.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 Oct 8, 2024
@amolpati30 amolpati30 requested a review from a team as a code owner October 8, 2024 06:29
@amolpati30 amolpati30 mentioned this pull request Oct 8, 2024
Open
@amolpati30 amolpati30 force-pushed the host_reg_with_setup_insight branch 2 times, most recently from ce2afb1 to 4fb3060 Compare October 9, 2024 10:50
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8914
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_registration.py -k test_positive_global_registration_with_setup_insights --external-logging
Test Result : ========== 1 failed, 19 deselected, 34 warnings in 813.27s (0:13:33) ===========

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Oct 9, 2024
@amolpati30 amolpati30 marked this pull request as draft October 10, 2024 08:52
@github-actions github-actions bot added the Stale Stale issue or Pull Request label Nov 25, 2024
@github-actions github-actions bot closed this Dec 2, 2024
@amolpati30 amolpati30 reopened this Dec 2, 2024
@github-actions github-actions bot removed the Stale Stale issue or Pull Request label Dec 3, 2024
@amolpati30 amolpati30 force-pushed the host_reg_with_setup_insight branch 2 times, most recently from 5c8c7a7 to 95224ae Compare January 15, 2025 10:48
@amolpati30 amolpati30 added No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master and removed 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 CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing labels Jan 15, 2025
@amolpati30 amolpati30 marked this pull request as ready for review January 15, 2025 10:59
@amolpati30 amolpati30 force-pushed the host_reg_with_setup_insight branch from 95224ae to 886259b Compare January 15, 2025 11:01
@shweta83
Copy link
Contributor

@amolpati30 Why is it not cherrypicked to older versions? I think the fix is available there too.

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9896
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_registration.py -k test_registering_with_title_using_global_registration_parameter --external-logging
Test Result : ===== 1 passed, 21 deselected, 32 warnings, 1 error in 2297.86s (0:38:17) ======

@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_registration.py -k test_registering_with_title_using_global_registration_parameter
airgun: 1591

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9900
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_registration.py -k test_registering_with_title_using_global_registration_parameter --external-logging
Test Result : ========== 1 passed, 21 deselected, 32 warnings in 1501.22s (0:25:01) ==========

@Satellite-QE Satellite-QE added PRT-Passed Indicates that latest PRT run is passed for the PR and removed PRT-Failed Indicates that latest PRT run is failed for the PR labels Jan 16, 2025
@SatelliteQE SatelliteQE deleted a comment from github-actions bot Jan 16, 2025
@SatelliteQE SatelliteQE deleted a comment from github-actions bot Jan 16, 2025
@amolpati30
Copy link
Contributor Author

@shweta83 The fix is available in version 6.16, but the test case is not included in this version. I will manually cherry-pick the test case for 6.16.

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
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
Milestone
No milestone
Development

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