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

Set customer scenario to True #15513

Merged
merged 1 commit into from
Jun 26, 2024
Merged

Set customer scenario to True #15513

merged 1 commit into from
Jun 26, 2024

Conversation

shweta83
Copy link
Contributor

Problem Statement

The test is a customer BZ but we missed to add the docstring.

Solution

Added the docstring for customer scenario and BZ.

Related Issues

@shweta83 shweta83 requested a review from a team as a code owner June 26, 2024 13:09
@shweta83
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py -k test_positive_create_delete_variable_with_overrides

@shweta83 shweta83 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.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 Jun 26, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7536
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_ansible.py -k test_positive_create_delete_variable_with_overrides --external-logging
Test Result : ========== 2 passed, 23 deselected, 61 warnings in 772.96s (0:12:52) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jun 26, 2024
@shweta83 shweta83 added the Easy Fix :) Easiest Fix to review and quick merge request. label Jun 26, 2024
@shweta83 shweta83 force-pushed the set_customer_scenario branch from cfed166 to 065c9bf Compare June 26, 2024 13:48
@shweta83
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py -k test_positive_create_delete_variable_with_overrides

@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Jun 26, 2024
shubhamsg199
shubhamsg199 approved these changes Jun 26, 2024
View reviewed changes
Copy link
Contributor

@shubhamsg199 shubhamsg199 left a comment

Choose a reason for hiding this comment

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

Ack, one minor comment

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7538
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_ansible.py -k test_positive_create_delete_variable_with_overrides --external-logging
Test Result : ========== 2 passed, 23 deselected, 61 warnings in 762.10s (0:12:42) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jun 26, 2024
@shweta83 shweta83 force-pushed the set_customer_scenario branch from 065c9bf to 6f66b78 Compare June 26, 2024 14:18
@shweta83
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py -k test_positive_create_delete_variable_with_overrides

@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Jun 26, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7540
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_ansible.py -k test_positive_create_delete_variable_with_overrides --external-logging
Test Result : ========== 2 passed, 23 deselected, 61 warnings in 773.86s (0:12:53) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jun 26, 2024
@Gauravtalreja1 Gauravtalreja1 merged commit 15b8a2c into SatelliteQE:master Jun 26, 2024
11 checks passed
github-actions bot pushed a commit that referenced this pull request Jun 26, 2024
@shweta83 @web-flow
Set customer scenario to True (#15513)
8165a4f 
Set customer scenario to true for ansible test

(cherry picked from commit 15b8a2c)
github-actions bot pushed a commit that referenced this pull request Jun 26, 2024
@shweta83 @web-flow
Set customer scenario to True (#15513)
591bca6 
Set customer scenario to true for ansible test

(cherry picked from commit 15b8a2c)
github-actions bot pushed a commit that referenced this pull request Jun 26, 2024
@shweta83 @web-flow
Set customer scenario to True (#15513)
45d903d 
Set customer scenario to true for ansible test

(cherry picked from commit 15b8a2c)
@Satellite-QE Satellite-QE mentioned this pull request Jun 26, 2024
Merged
This was referenced Jun 26, 2024
Merged
Merged
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
@shweta83 @jyejare
Set customer scenario to True (SatelliteQE#15513)
98bf556 
Set customer scenario to true for ansible test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shubhamsg199 shubhamsg199 shubhamsg199 approved these changes

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

Assignees

@shweta83 shweta83

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

Successfully merging this pull request may close these issues.
4 participants
@shweta83 @Satellite-QE @Gauravtalreja1 @shubhamsg199