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.

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

Log overcloud installation to log #2659

Closed

Conversation

holser
Copy link

@holser holser commented Jan 16, 2025

In case of failure it would be nice to have a separate log for overcloud deployment in order to be able to analyze it in case of failure. This patch saves overcloud deployment to the log and spot a few last lines to console.

In case of failure it would be nice to have a separate log for overcloud
deployment in order to be able to analyze it in case of failure. This
patch saves overcloud deployment to the log and spot a few last lines to
console.
@holser holser requested a review from a team as a code owner January 16, 2025 15:50
Copy link
Contributor

openshift-ci bot commented Jan 16, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign bshewale for approval. For more information see the Code Review Process.

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

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

Copy link
Contributor

@bogdando bogdando left a comment

Choose a reason for hiding this comment

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

/lgtm

@pablintino
Copy link
Collaborator

@holser @bogdando @ciecierski The task is already using ci_script, that pipes the command output to a file automatically. Have you checked the logs directory?

@holser
Copy link
Author

holser commented Jan 17, 2025

@pablintino without "| tee -a overcloud_install.log" all output is stdout only which is saved to ansible.log only. When job fails it's not enough to analyse it what went wrong during OSP installation.

@holser holser closed this Jan 17, 2025
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