Skip to content

Add logging to a file for os must gather task#3246

Merged
amartyasinha merged 1 commit intoopenstack-k8s-operators:mainfrom
danpawlik:improve-logging
Sep 4, 2025
Merged

Add logging to a file for os must gather task#3246
amartyasinha merged 1 commit intoopenstack-k8s-operators:mainfrom
danpawlik:improve-logging

Conversation

@danpawlik
Copy link
Copy Markdown
Contributor

After we drop executing Ansible playbook and we move temporary to Ansible command directly, there is an issue that some tasks are not executed properly.
Add ANSIBLE_LOG_PATH for that task for now, to make clear view what is executed.

@danpawlik danpawlik requested a review from a team as a code owner September 3, 2025 07:01
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Sep 3, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

After we drop executing Ansible playbook and we move temporary to
Ansible command directly, there is an issue that some tasks
are not executed properly.
Add ANSIBLE_LOG_PATH for that task for now, to make clear view
what is executed.

Signed-off-by: Daniel Pawlik <dpawlik@redhat.com>
Comment thread ci/playbooks/e2e-collect-logs.yml
@averdagu
Copy link
Copy Markdown
Contributor

averdagu commented Sep 3, 2025

lgtm :)

@amartyasinha amartyasinha merged commit f6ba4ed into openstack-k8s-operators:main Sep 4, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants