Skip to content

Conversation

@bitoku
Copy link
Contributor

@bitoku bitoku commented Jan 29, 2025

(cherry picked from commit 0529655)

cherry picked from #8868

/assign bitoku

What type of PR is this?

/kind bug

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #8815

Special notes for your reviewer:

Does this PR introduce a user-facing change?

Fix log rotation failure during pod termination

@openshift-ci openshift-ci bot added release-note Denotes a PR that will be considered when it comes time to generate release notes. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. kind/bug Categorizes issue or PR as related to a bug. labels Jan 29, 2025
@bitoku bitoku changed the title Avoid using UpdateContainerStatus for ReopenContainerLog and add logs tests [release-1.30] Avoid using UpdateContainerStatus for ReopenContainerLog and add logs tests Jan 29, 2025
@bitoku bitoku force-pushed the release-1.30-reopen-log branch 2 times, most recently from 5a4ec6b to a787d63 Compare February 11, 2025 14:15
… tests

Signed-off-by: Ayato Tokubi <[email protected]>

(cherry picked from commit 0529655)
Signed-off-by: Ayato Tokubi <[email protected]>
@bitoku bitoku force-pushed the release-1.30-reopen-log branch from a787d63 to 3dab91d Compare February 11, 2025 14:18
@sohankunkerkar sohankunkerkar added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 12, 2025
Copy link
Member

@sohankunkerkar sohankunkerkar left a comment

Choose a reason for hiding this comment

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

CI is broken for a while. The change looks good to me.
/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Feb 12, 2025
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 12, 2025

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by: bitoku, sohankunkerkar

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

The pull request process is described 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

@openshift-merge-bot openshift-merge-bot bot merged commit 0af565a into cri-o:release-1.30 Feb 12, 2025
44 of 49 checks passed
@bitoku bitoku deleted the release-1.30-reopen-log branch April 3, 2025 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants