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

Check dynflow console for expected host #15918

Merged
merged 1 commit into from
Sep 27, 2024
Merged

Check dynflow console for expected host #15918

merged 1 commit into from
Sep 27, 2024

Conversation

amolpati30
Copy link
Contributor

@amolpati30 amolpati30 commented Aug 12, 2024 •
edited
Loading

Problem Statement

After selecting multiple hosts and running an Ansible job, the debug log for a particular host displayed logs of another host with expected host.

Solution

This issue has been resolved, and the automation below validates that when running an Ansible role for multiple hosts, the debug log for a specific host only displays its output and not that of other hosts

Dependent PR: SatelliteQE/airgun#1522

@amolpati30 amolpati30 requested a review from a team as a code owner August 12, 2024 18:44
@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 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 QETestCoverage Issues and PRs relating to a Satellite bug labels Aug 12, 2024
@amolpati30 amolpati30 marked this pull request as draft August 12, 2024 18:45
@amolpati30 amolpati30 added the 6.13.z Introduced in or relating directly to Satellite 6.13 label Aug 20, 2024
@amolpati30 amolpati30 force-pushed the dynflow_console_log branch 2 times, most recently from bb58fc1 to 5f266bd Compare August 20, 2024 17:26
@amolpati30 amolpati30 marked this pull request as ready for review August 20, 2024 17:35
@amolpati30 amolpati30 mentioned this pull request Aug 20, 2024
Merged
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_debug_log_for_host
airgun: 1522

@amolpati30 amolpati30 changed the title Check console log for expected host Check dynflow console for expected host Aug 20, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8189
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_debug_log_for_host --external-logging
Test Result : ========== 1 failed, 16 deselected, 37 warnings in 1300.18s (0:21:40) ==========

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Aug 20, 2024
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_debug_log_for_host
airgun: 1522

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8190
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_debug_log_for_host --external-logging
Test Result : ========== 1 failed, 16 deselected, 37 warnings in 1360.66s (0:22:40) ==========

@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_debug_log_for_host
airgun: 1522

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8203
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_debug_log_for_host --external-logging
Test Result : ========== 1 failed, 16 deselected, 37 warnings in 1101.87s (0:18:21) ==========

@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_debug_log_for_host
airgun: 1522

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8209
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_debug_log_for_host --external-logging
Test Result : ========== 1 failed, 14 deselected, 35 warnings in 1055.87s (0:17:35) ==========

@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_debug_log_for_host
airgun: 1522

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8212
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_debug_log_for_host --external-logging
Test Result : ========== 1 failed, 14 deselected, 35 warnings in 1143.23s (0:19:03) ==========

@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_debug_log_for_host
airgun: 1522

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8217
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_debug_log_for_host --external-logging
Test Result : ========== 1 failed, 14 deselected, 35 warnings in 1187.09s (0:19:47) ==========

@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_debug_log_for_host
airgun: 1522

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8221
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_debug_log_for_host --external-logging
Test Result : ========== 1 failed, 14 deselected, 35 warnings in 1117.36s (0:18:37) ==========

@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_debug_log_for_host
airgun: 1522

@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Sep 27, 2024
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleREX -k test_positive_ansible_job_with_verbose_stdout
airgun: 1522

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8820
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleREX -k test_positive_ansible_job_with_verbose_stdout --external-logging
Test Result : ========== 1 failed, 9 deselected, 30 warnings in 1094.95s (0:18:14) ===========

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Sep 27, 2024
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleREX -k test_positive_ansible_job_with_verbose_stdout
airgun: 1522

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8821
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleREX -k test_positive_ansible_job_with_verbose_stdout --external-logging
Test Result : ========== 1 passed, 9 deselected, 30 warnings in 1069.84s (0:17:49) ===========

@Satellite-QE Satellite-QE added PRT-Passed Indicates that latest PRT run is passed for the PR and removed PRT-Failed Indicates that latest PRT run is failed for the PR labels Sep 27, 2024
Gauravtalreja1
Gauravtalreja1 reviewed Sep 27, 2024
View reviewed changes
tests/foreman/ui/test_ansible.py Show resolved Hide resolved
tests/foreman/ui/test_ansible.py Outdated

:BZ: 1912941

:Verifies: SAT-12267
Copy link
Collaborator

Choose a reason for hiding this comment

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

Also add SAT-27757 here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added.

@amolpati30 amolpati30 mentioned this pull request Sep 27, 2024
Merged
@amolpati30
individual log should show for each host
dc58834 
check debug log for expected host
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Sep 27, 2024
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleREX -k test_positive_ansible_job_with_verbose_stdout
airgun: 1522

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8830
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleREX -k test_positive_ansible_job_with_verbose_stdout --external-logging
Test Result : ========== 1 passed, 9 deselected, 30 warnings in 1025.54s (0:17:05) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Sep 27, 2024
@Gauravtalreja1 Gauravtalreja1 merged commit 33c39d5 into SatelliteQE:master Sep 27, 2024
11 checks passed
github-actions bot pushed a commit that referenced this pull request Sep 27, 2024
@amolpati30 @web-flow
Check dynflow console for expected host (#15918)
826a565 
individual log should show for each host

check debug log for expected host

(cherry picked from commit 33c39d5)
github-actions bot pushed a commit that referenced this pull request Sep 27, 2024
@amolpati30 @web-flow
Check dynflow console for expected host (#15918)
9cde5b8 
individual log should show for each host

check debug log for expected host

(cherry picked from commit 33c39d5)
@Satellite-QE Satellite-QE mentioned this pull request Sep 27, 2024
Merged
github-actions bot pushed a commit that referenced this pull request Sep 27, 2024
@amolpati30 @web-flow
Check dynflow console for expected host (#15918)
54f106e 
individual log should show for each host

check debug log for expected host

(cherry picked from commit 33c39d5)
This was referenced Sep 27, 2024
Merged
Merged
github-actions bot pushed a commit that referenced this pull request Sep 27, 2024
@amolpati30 @web-flow
Check dynflow console for expected host (#15918)
589d640 
individual log should show for each host

check debug log for expected host

(cherry picked from commit 33c39d5)
@Satellite-QE Satellite-QE mentioned this pull request Sep 27, 2024
Merged
Gauravtalreja1 pushed a commit that referenced this pull request Sep 27, 2024
@Satellite-QE @amolpati30
[6.15.z] Check dynflow console for expected host (#16527)
c54d8bb 
Check dynflow console for expected host (#15918)

individual log should show for each host

check debug log for expected host

(cherry picked from commit 33c39d5)

Co-authored-by: amolpati30 <[email protected]>
Gauravtalreja1 pushed a commit that referenced this pull request Sep 27, 2024
@Satellite-QE @amolpati30
[6.13.z] Check dynflow console for expected host (#16528)
4e9de6e 
Check dynflow console for expected host (#15918)

individual log should show for each host

check debug log for expected host

(cherry picked from commit 33c39d5)

Co-authored-by: amolpati30 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

@LadislavVasina1 LadislavVasina1 LadislavVasina1 approved these changes

Assignees

@amolpati30 amolpati30

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 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 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.
5 participants
@amolpati30 @Satellite-QE @Gauravtalreja1 @LadislavVasina1 @jyejare