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

Add test coverage for BZ:1982698 #15022

Merged
merged 1 commit into from
May 16, 2024
Merged

Add test coverage for BZ:1982698 #15022

merged 1 commit into from
May 16, 2024

Conversation

Gauravtalreja1
Copy link
Collaborator

@Gauravtalreja1 Gauravtalreja1 commented May 9, 2024 •
edited
Loading

Problem Statement

Missing coverage for BZ:1982698, where we verified ansible playbook job template with "hosts: localhost:

Solution

Add test coverage for BZ:1982698, to verifiy ansible-playbook job template with "hosts: localhost"

Dependent PRs:

SatelliteQE/nailgun#1139

@Gauravtalreja1 Gauravtalreja1 added CherryPick PR needs CherryPick to previous branches 6.12.z Introduced in or relating directly to Satellite 6.12 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 May 9, 2024
@Gauravtalreja1 Gauravtalreja1 self-assigned this May 9, 2024
@Gauravtalreja1 Gauravtalreja1 requested a review from a team as a code owner May 9, 2024 18:19
@Gauravtalreja1 Gauravtalreja1 removed the 6.12.z Introduced in or relating directly to Satellite 6.12 label May 9, 2024
@Gauravtalreja1
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::TestAnsibleREX::test_positive_ansible_localhost_job_on_host
nailgun: 1139

@Gauravtalreja1 Gauravtalreja1 added the QETestCoverage Issues and PRs relating to a Satellite bug label May 9, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6871
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_ansible.py::TestAnsibleREX::test_positive_ansible_localhost_job_on_host --external-logging
Test Result : ================= 3 passed, 205 warnings in 1675.04s (0:27:55) =================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label May 9, 2024
@Gauravtalreja1
Add test coverage for BZ:1982698
ce2252b 
Signed-off-by: Gaurav Talreja <[email protected]>
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label May 10, 2024
@Gauravtalreja1 Gauravtalreja1 added the PRT-Passed Indicates that latest PRT run is passed for the PR label May 10, 2024
@Gauravtalreja1 Gauravtalreja1 requested a review from shweta83 May 10, 2024 15:54
shweta83
shweta83 approved these changes May 15, 2024
View reviewed changes
Copy link
Contributor

@shweta83 shweta83 left a comment

Choose a reason for hiding this comment

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

Ack

shubhamsg199
shubhamsg199 approved these changes May 16, 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

@shubhamsg199 shubhamsg199 merged commit c060dd6 into SatelliteQE:master May 16, 2024
10 checks passed
github-actions bot pushed a commit that referenced this pull request May 16, 2024
@Gauravtalreja1 @web-flow
Add test coverage for BZ:1982698 (#15022)
0bc91ee 
Signed-off-by: Gaurav Talreja <[email protected]>
(cherry picked from commit c060dd6)
@Satellite-QE Satellite-QE mentioned this pull request May 16, 2024
Merged
github-actions bot pushed a commit that referenced this pull request May 16, 2024
@Gauravtalreja1 @web-flow
Add test coverage for BZ:1982698 (#15022)
0625904 
Signed-off-by: Gaurav Talreja <[email protected]>
(cherry picked from commit c060dd6)
github-actions bot pushed a commit that referenced this pull request May 16, 2024
@Gauravtalreja1 @web-flow
Add test coverage for BZ:1982698 (#15022)
9a5aaaf 
Signed-off-by: Gaurav Talreja <[email protected]>
(cherry picked from commit c060dd6)
@Satellite-QE Satellite-QE mentioned this pull request May 16, 2024
Merged
@Satellite-QE Satellite-QE mentioned this pull request May 16, 2024
Merged
@Gauravtalreja1 Gauravtalreja1 deleted the ansible-localhost branch May 16, 2024 06:48
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
@Gauravtalreja1 @jyejare
Add test coverage for BZ:1982698 (SatelliteQE#15022)
1b8653b 
Signed-off-by: Gaurav Talreja <[email protected]>
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

@shweta83 shweta83 shweta83 approved these changes

Assignees

@Gauravtalreja1 Gauravtalreja1

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 PRT-Passed Indicates that latest PRT run is passed for the PR QETestCoverage Issues and PRs relating to a Satellite bug
Projects
None yet
Milestone
No milestone
Development

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