openshift-docs icon indicating copy to clipboard operation
openshift-docs copied to clipboard

DOCS OBSDOCS-846 - Logging 5.7.12 Release Notes

Open shreyasiddhartha opened this issue 1 year ago • 2 comments
trafficstars

Change type: Doc update; Logging Z-Stream Release Notes - 5.7.12 Doc JIRA: https://issues.redhat.com/browse/OBSDOCS-846

Fix Version: 4.11, 4.12, 4.13, 4.14

Doc Preview: https://73272--ocpdocs-pr.netlify.app/openshift-enterprise/latest/logging/logging_release_notes/logging-5-7-release-notes#logging-release-notes-5-7-12_logging-5-7-release-notes

SME Review: QE Review: Peer Review:

shreyasiddhartha avatar Mar 18 '24 06:03 shreyasiddhartha

🤖 Thu Mar 28 09:21:21 - Prow CI generated the docs preview: https://73272--ocpdocs-pr.netlify.app

ocpdocs-previewbot avatar Mar 18 '24 06:03 ocpdocs-previewbot

LGTM

anpingli avatar Mar 27 '24 01:03 anpingli

/label peer-review-needed

shreyasiddhartha avatar Mar 27 '24 06:03 shreyasiddhartha

Hello @shreyasiddhartha, Good job on this PR! Had a couple of comments/questions, otherwise LGTM!

mletalie avatar Mar 27 '24 16:03 mletalie

New changes are detected. LGTM label has been removed.

openshift-ci[bot] avatar Mar 28 '24 08:03 openshift-ci[bot]

@shreyasiddhartha: 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/test-infra repository. I understand the commands that are listed here.

openshift-ci[bot] avatar Mar 28 '24 09:03 openshift-ci[bot]

/label merge-review-needed

shreyasiddhartha avatar Mar 28 '24 10:03 shreyasiddhartha

/label merge-review-in-progress

bergerhoffer avatar Mar 28 '24 13:03 bergerhoffer

@shreyasiddhartha Same thing as the other PR I merged for you - let us know if the manual cherry picks also need to be merged by adding the merge-review-needed labels on those.

Same thing about 4.11 being out of support, and if we really do need to publish for 4.11, just need confirmation from @kalexand-rh, etc.

bergerhoffer avatar Mar 28 '24 13:03 bergerhoffer

@shreyasiddhartha Same thing as the other PR I merged for you - let us know if the manual cherry picks also need to be merged by adding the merge-review-needed labels on those.

Same thing about 4.11 being out of support, and if we really do need to publish for 4.11, just need confirmation from @kalexand-rh, etc.

Thanks @bergerhoffer! I would also like to know if we're not updating 4.11. Will refrain from working on it in future. :)

shreyasiddhartha avatar Mar 28 '24 13:03 shreyasiddhartha

@shreyasiddhartha Yeah, we are not supposed to update 4.11 anymore, unless there is an exception to. If there is a z-stream release, that does seem like a valid reason. But I'm not positive, so that's where I'd like a DPM confirmation :)

bergerhoffer avatar Mar 28 '24 13:03 bergerhoffer

@shreyasiddhartha Yeah, we are not supposed to update 4.11 anymore, unless there is an exception to. If there is a z-stream release, that does seem like a valid reason. But I'm not positive, so that's where I'd like a DPM confirmation :)

@bergerhoffer I have added the merge review needed labels to 4.13 and 4.12 PRs. Can you please help with it? :) Thanks for your help.

@briandooley just confirmed that I can skip 4.11.

shreyasiddhartha avatar Mar 28 '24 13:03 shreyasiddhartha

@bergerhoffer I have added the merge review needed labels to 4.13 and 4.12 PRs. Can you please help with it? :) Thanks for your help.

On it!

bergerhoffer avatar Mar 28 '24 15:03 bergerhoffer