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

[6.13z CP] Check dynflow console output for host #1574

Merged
merged 1 commit into from
Sep 27, 2024
Merged

[6.13z CP] Check dynflow console output for host #1574

merged 1 commit into from
Sep 27, 2024

Conversation

amolpati30
Copy link
Contributor

@amolpati30 amolpati30 commented Sep 27, 2024 •
edited
Loading

Cherrypick of PR: #1522

  • When enabling debug mode on Satellite, the task is generating an output log for all hosts present on the Job.

  • The host execution log is showing the debug log for all hosts present on the inventory instead of just the expected host.

Solution : -

Below automation check the host execution log should show the debug log for expected host instead of the all host present in the inventory.

Dependent PR: SatelliteQE/robottelo#16528

Fixes: #1571

@amolpati30 amolpati30 changed the title added views and entities to fetch the dynflow console output Check dynflow console output for host Sep 27, 2024
@amolpati30 amolpati30 added No-CherryPick PR doesnt need CherryPick to previous branches 6.13.z labels Sep 27, 2024
@amolpati30 amolpati30 changed the title Check dynflow console output for host [6.13z CP] Check dynflow console output for host Sep 27, 2024
@amolpati30
Copy link
Contributor Author

amolpati30 commented Sep 27, 2024 •
edited
Loading

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

@amolpati30 amolpati30 added 6.14.z 6.15.z CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing No-CherryPick PR doesnt need CherryPick to previous branches and removed No-CherryPick PR doesnt need CherryPick to previous branches CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing 6.14.z 6.15.z labels 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
robottelo: 16528

@Satellite-QE
Copy link
Contributor

PRT Result

Build Number: 427
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 1083.04s (0:18:03) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Sep 27, 2024
Gauravtalreja1
Gauravtalreja1 approved these changes Sep 27, 2024
View reviewed changes
Copy link
Collaborator

@Gauravtalreja1 Gauravtalreja1 left a comment

Choose a reason for hiding this comment

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

ACK, PRT passed on SatelliteQE/robottelo#16528

@Gauravtalreja1 Gauravtalreja1 merged commit 760287e into SatelliteQE:6.13.z Sep 27, 2024
31 checks passed
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

Assignees

@amolpati30 amolpati30

Labels
6.13.z No-CherryPick PR doesnt need CherryPick to previous branches 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.
3 participants
@amolpati30 @Satellite-QE @Gauravtalreja1