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

OCPBUGS-49826-RN418: Known issues details added #88874

Merged
merged 1 commit into from
Feb 19, 2025

Conversation

ogradyp
Copy link
Contributor

@ogradyp ogradyp commented Feb 19, 2025

https://issues.redhat.com/browse/OCPBUGS-49826 T-GM announces "Locked" too fast after holdover - Known Issue Release Note Text

Applies to OCP version : 4.18

Preview: Known Issues - See OCPBUGS-49826

Dev review completed by @aneeshkp
Peer review completed by @agantony

Thank you.

@openshift-ci openshift-ci bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Feb 19, 2025
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Feb 19, 2025

🤖 Wed Feb 19 15:52:46 - Prow CI generated the docs preview:

https://88874--ocpdocs-pr.netlify.app/openshift-enterprise/latest/release_notes/ocp-4-18-release-notes.html

Copy link

@aneeshkp aneeshkp left a comment

Choose a reason for hiding this comment

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

/lgtm

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

ogradyp commented Feb 19, 2025

/label peer-review-needed

@openshift-ci openshift-ci bot added the peer-review-needed Signifies that the peer review team needs to review this PR label Feb 19, 2025
Copy link
Contributor

@agantony agantony left a comment

Choose a reason for hiding this comment

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

Added a comment; overall lgtm!

Please squash the commits before merging

@agantony agantony added peer-review-done Signifies that the peer review team has reviewed this PR branch/enterprise-4.18 and removed peer-review-needed Signifies that the peer review team needs to review this PR labels Feb 19, 2025
@agantony agantony added this to the Planned for 4.18 GA milestone Feb 19, 2025
OCPBUGS-49826-RN418: Typo fixed

OCPBUGS-49826-RN418: Peer review feedback applied
@ogradyp ogradyp force-pushed the OCPBUGS-49826-RN418 branch from b77d26d to b24c472 Compare February 19, 2025 15:29
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Feb 19, 2025
Copy link

openshift-ci bot commented Feb 19, 2025

New changes are detected. LGTM label has been removed.

@ogradyp
Copy link
Contributor Author

ogradyp commented Feb 19, 2025

/label merge-review-needed

@openshift-ci openshift-ci bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Feb 19, 2025
Copy link

openshift-ci bot commented Feb 19, 2025

@ogradyp: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@stevsmit stevsmit added merge-review-in-progress Signifies that the merge review team is reviewing this PR lgtm Indicates that a PR is ready to be merged. and removed merge-review-needed Signifies that the merge review team needs to review this PR labels Feb 19, 2025
@stevsmit stevsmit merged commit f7b9569 into openshift:enterprise-4.18 Feb 19, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch/enterprise-4.18 lgtm Indicates that a PR is ready to be merged. merge-review-in-progress Signifies that the merge review team is reviewing this PR peer-review-done Signifies that the peer review team has reviewed this PR size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants