Skip to content

Revert get os-must-gather logs#3206

Merged
danpawlik merged 1 commit intomainfrom
move-99-logs-playbook-into-role
Aug 21, 2025
Merged

Revert get os-must-gather logs#3206
danpawlik merged 1 commit intomainfrom
move-99-logs-playbook-into-role

Conversation

@bshewale
Copy link
Copy Markdown
Contributor

@bshewale bshewale commented Aug 20, 2025

Due to this change 1 os-must-gather logs was not logged
because of some var issues.
so reverting that partial part of code in this PR will fix the
issue2.

@bshewale bshewale requested a review from a team as a code owner August 20, 2025 12:04
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Aug 20, 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

@bshewale bshewale force-pushed the move-99-logs-playbook-into-role branch from d4f9467 to a035204 Compare August 20, 2025 14:02
@bshewale bshewale changed the title [DNM] [WIP] Fix to get os-must-gather logs Fix to get os-must-gather logs Aug 20, 2025
@danpawlik danpawlik changed the title Fix to get os-must-gather logs Revert get os-must-gather logs Aug 21, 2025
@danpawlik
Copy link
Copy Markdown
Contributor

so please wait, I will do a quick fix.

Copy link
Copy Markdown
Contributor

@danpawlik danpawlik left a comment

Choose a reason for hiding this comment

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

let me do a quick fix.

@danpawlik
Copy link
Copy Markdown
Contributor

Checking here: #3208

@danpawlik danpawlik merged commit 0e95afa into main Aug 21, 2025
7 checks passed
@danpawlik danpawlik deleted the move-99-logs-playbook-into-role branch August 21, 2025 12:59
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