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

Fix in CR for import ansible role... #16507

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

Fix in CR for import ansible role... #16507

wants to merge 1 commit into from

Conversation

amolpati30
Copy link
Contributor

@amolpati30 amolpati30 commented Sep 26, 2024 •
edited
Loading

Problem Statement

Sometimes, the import role option does not appear in Satellite, which is causing issues with importing the role

Solution

Instead of importing all roles, I synced only a single role.

@amolpati30 amolpati30 added Easy Fix :) Easiest Fix to review and quick merge request. 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 Sep 26, 2024
@amolpati30 amolpati30 requested a review from a team as a code owner September 26, 2024 13:03
@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Sep 26, 2024
@amolpati30 amolpati30 force-pushed the Fix_CR_importing_role branch 2 times, most recently from 0ed93ab to 1f6b81e Compare September 27, 2024 05:20
@Satellite-QE Satellite-QE added PRT-Passed Indicates that latest PRT run is passed for the PR PRT-Failed Indicates that latest PRT run is failed for the PR and removed PRT-Failed Indicates that latest PRT run is failed for the PR PRT-Passed Indicates that latest PRT run is passed for the PR labels Sep 27, 2024
@amolpati30 amolpati30 force-pushed the Fix_CR_importing_role branch 3 times, most recently from 94e0547 to 1b3f106 Compare October 24, 2024 18:14
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Oct 24, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Oct 24, 2024
@SatelliteQE SatelliteQE deleted a comment from Gauravtalreja1 Oct 24, 2024
@SatelliteQE SatelliteQE deleted a comment from shubhamsg199 Oct 24, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Oct 24, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Oct 24, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Oct 24, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Oct 24, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Oct 24, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Oct 24, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Oct 24, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Oct 24, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Oct 24, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Oct 24, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Oct 24, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Oct 24, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9155
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_computeresource_vmware.py -k test_positive_provision_end_to_end --external-logging --include-onprem-provisioning
Test Result : ========== 4 failed, 12 deselected, 92 warnings in 3935.33s (1:05:35) ==========

@amolpati30 amolpati30 force-pushed the Fix_CR_importing_role branch 2 times, most recently from db65397 to 1b7b067 Compare November 4, 2024 17:48
@devendra104
Copy link
Member

trigger: test-robottelo
pytest: tests/foreman/ui/test_computeresource_vmware.py -k test_positive_provision_end_to_end
provisioning: true
airgun: 1602

@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_computeresource_vmware.py -k test_positive_provision_end_to_end
provisioning: true
airgun: 1602

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9239
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_computeresource_vmware.py -k test_positive_provision_end_to_end --external-logging --include-onprem-provisioning
Test Result : ========== 12 deselected, 39 warnings, 4 errors in 885.38s (0:14:45) ===========

@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_computeresource_vmware.py -k test_positive_provision_end_to_end
provisioning: true
airgun: 1602

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9242
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_computeresource_vmware.py -k test_positive_provision_end_to_end --external-logging --include-onprem-provisioning
Test Result : ========= 4 failed, 12 deselected, 114 warnings in 4922.91s (1:22:02) ==========

@amolpati30 amolpati30 mentioned this pull request Nov 7, 2024
Closed
@Gauravtalreja1
Copy link
Collaborator

@amolpati30 Any updates on this PR?

@amolpati30
Copy link
Contributor Author

@amolpati30 Any updates on this PR?

I tried multiple changes, but it is still failing when creating the compute profile in the datastore. It appears to be fetching an incorrect value, which is causing the failure.

@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_computeresource_vmware.py -k test_positive_provision_end_to_end
provisioning: true

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9370
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_computeresource_vmware.py -k test_positive_provision_end_to_end --external-logging --include-onprem-provisioning
Test Result : ========== 4 failed, 12 deselected, 97 warnings in 5290.00s (1:28:09) ==========

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

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 left review comments

Assignees

@amolpati30 amolpati30

Labels
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 Easy Fix :) Easiest Fix to review and quick merge request. 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.
4 participants
@amolpati30 @Satellite-QE @devendra104 @Gauravtalreja1