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

We don't see any VM under RHEV after creating the CR #17442

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

We don't see any VM under RHEV after creating the CR #17442

wants to merge 1 commit into from
+6 −0

Conversation

amolpati30
Copy link
Contributor

No description provided.

@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 Stream Introduced in or relating directly to Satellite Stream/Master 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 Jan 28, 2025
@amolpati30 amolpati30 requested a review from a team as a code owner January 28, 2025 06:31
@Gauravtalreja1 Gauravtalreja1 self-requested a review January 28, 2025 11:08
Gauravtalreja1
Gauravtalreja1 requested changes Jan 31, 2025
View reviewed changes
tests/foreman/ui/test_computeresource.py
@@ -164,6 +164,8 @@ def test_positive_list_resource_vms(session, rhev_data):
:parametrized: yes
:expectedresults: VMs listed for provided compute resource
:BlockedBy: SAT-30627
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are you aware if SAT-30627 is issue for all supported Sat versions? if not, and its a regression in stream, then please don't cherrypick this to all sat versions
And, please correct the PR title and commit message to what's being updated in this PR, not a future :D

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It impacts all versions.
And updated title and commit message

@amolpati30 amolpati30 changed the title RHEV is going to be removed as part of "SAT-12005" We don't see any VM under RHEV after creating the CR Jan 31, 2025
@amolpati30 amolpati30 force-pushed the Skip_RHEV_test_due_to_an_existing_bug branch from 37d4327 to f7ac17c Compare January 31, 2025 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 requested changes

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 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.
2 participants
@amolpati30 @Gauravtalreja1