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

content host set location to Default Location #14497

Merged
merged 1 commit into from
Apr 11, 2024
Merged

content host set location to Default Location #14497

merged 1 commit into from
Apr 11, 2024

Conversation

yanpliu
Copy link
Contributor

@yanpliu yanpliu commented Mar 22, 2024

Problem Statement

hypervisor guest register to satellite server, the guest is in the Default Location, so locate to Default Location then can list in the content host page
Case : PASS

(robottelo_615) [virtwho@dell-per740-68-vm-04 robottelo]$ pytest ./tests/foreman/virtwho/ui/test_esx_sca.py -k test_positive_last_checkin_status --disable-pytest-warnings -q
.                                                                                                                                                                                                           [100%]
1 passed, 35 deselected, 5 warnings in 282.19s (0:04:42)

@yanpliu yanpliu added CherryPick PR needs CherryPick to previous branches 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 Mar 22, 2024
@yanpliu yanpliu self-assigned this Mar 22, 2024
@yanpliu yanpliu requested a review from a team as a code owner March 22, 2024 14:28
lhellebr
lhellebr approved these changes Mar 26, 2024
View reviewed changes
Copy link
Contributor

@lhellebr lhellebr left a comment

Choose a reason for hiding this comment

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

LGTM, pending PRT results

@Gauravtalreja1 Gauravtalreja1 force-pushed the test_positive_last_checkin_status branch from 2820950 to 415bb8d Compare April 11, 2024 18:40
Gauravtalreja1
Gauravtalreja1 approved these changes Apr 11, 2024
View reviewed changes
Copy link
Collaborator

@Gauravtalreja1 Gauravtalreja1 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

ACK..@lhellebr we don't have PRT support yet to run virtwho test, and @yanpliu has already provided the local test results in the description above.

@Gauravtalreja1 Gauravtalreja1 enabled auto-merge (squash) April 11, 2024 18:44
@Gauravtalreja1 Gauravtalreja1 added the Easy Fix :) Easiest Fix to review and quick merge request. label Apr 11, 2024
@Gauravtalreja1 Gauravtalreja1 merged commit 1a432df into SatelliteQE:master Apr 11, 2024
9 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 11, 2024
@yanpliu @web-flow
content host set location to Default Location (#14497)
7c043b3 
content host set location to Default_Organization

(cherry picked from commit 1a432df)
github-actions bot pushed a commit that referenced this pull request Apr 11, 2024
@yanpliu @web-flow
content host set location to Default Location (#14497)
486ae43 
content host set location to Default_Organization

(cherry picked from commit 1a432df)
This was referenced Apr 11, 2024
Merged
Merged
github-actions bot pushed a commit that referenced this pull request Apr 11, 2024
@yanpliu @web-flow
content host set location to Default Location (#14497)
a5b2359 
content host set location to Default_Organization

(cherry picked from commit 1a432df)
@Satellite-QE Satellite-QE mentioned this pull request Apr 11, 2024
Merged
Gauravtalreja1 pushed a commit that referenced this pull request Apr 11, 2024
@Satellite-QE @yanpliu
[6.15.z] content host set location to Default Location (#14752)
a0683b2 
content host set location to Default Location (#14497)

content host set location to Default_Organization

(cherry picked from commit 1a432df)

Co-authored-by: yanpliu <yanpliu@redhat.com>
Gauravtalreja1 pushed a commit that referenced this pull request Apr 11, 2024
@Satellite-QE @yanpliu
[6.14.z] content host set location to Default Location (#14750)
c5bd1d9 
content host set location to Default Location (#14497)

content host set location to Default_Organization

(cherry picked from commit 1a432df)

Co-authored-by: yanpliu <yanpliu@redhat.com>
Gauravtalreja1 pushed a commit that referenced this pull request Apr 11, 2024
@Satellite-QE @yanpliu
[6.13.z] content host set location to Default Location (#14751)
027d5b3 
content host set location to Default Location (#14497)

content host set location to Default_Organization

(cherry picked from commit 1a432df)

Co-authored-by: yanpliu <yanpliu@redhat.com>
@yanpliu yanpliu deleted the test_positive_last_checkin_status branch May 16, 2024 01:37
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
@yanpliu @jyejare
content host set location to Default Location (SatelliteQE#14497)
85090a0 
content host set location to Default_Organization
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lhellebr lhellebr lhellebr approved these changes

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

Assignees

@yanpliu yanpliu

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 CherryPick PR needs CherryPick to previous branches Easy Fix :) Easiest Fix to review and quick merge request.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@yanpliu @lhellebr @Gauravtalreja1