-
Notifications
You must be signed in to change notification settings - Fork 6.7k
docs: Add release checklist step to reference the release-specific instructions file #25854
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
Conversation
Add documentation PR reminder for next minor version after RC. Signed-off-by: Revital Barletz <[email protected]>
❌ Preview Environment deleted from BunnyshellAvailable commands (reply to this comment):
|
Signed-off-by: Revital Barletz <[email protected]>
|
@reggie-k can you take a look? |
Signed-off-by: Revital Barletz <[email protected]>
Added sections for deprecated items and upgrade instructions. Signed-off-by: Revital Barletz <[email protected]>
reggie-k
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR!
I think that the ISSUE_TEMPLATE is mainly for creating GitHub issues,so it would be better to put this template file in another place, like a new templates folder under docs or under docs/operator-manual.
I also think it is better to put the healthcheck section at the bottom of the page as it will mostly be the longest part of the doc.
Co-authored-by: Peter Jiang <[email protected]> Signed-off-by: Revital Barletz <[email protected]>
Co-authored-by: Peter Jiang <[email protected]> Signed-off-by: Revital Barletz <[email protected]>
Co-authored-by: Peter Jiang <[email protected]> Signed-off-by: Revital Barletz <[email protected]>
Co-authored-by: Peter Jiang <[email protected]> Signed-off-by: Revital Barletz <[email protected]>
Signed-off-by: Revital Barletz <[email protected]>
Signed-off-by: Revital Barletz <[email protected]>
Updated the documentation PR template link for the next minor version in the release checklist. Signed-off-by: Revital Barletz <[email protected]>
|
Thanks @reggie-k & @pjiang-dev , I have addressed your comments, please review again. |
Signed-off-by: Revital Barletz <[email protected]>
Signed-off-by: Revital Barletz <[email protected]>
reggie-k
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left some comments regarding the structure, and we should probably change the PR title to Update release checklist with upgrade instructions template documentation PR step, based on a template
Co-authored-by: Regina Voloshin <[email protected]> Signed-off-by: Revital Barletz <[email protected]>
Reorganized sections for clarity and added details on health checks. Signed-off-by: Revital Barletz <[email protected]>
|
@reggie-k comments addressed. Thank you! |
|
@revitalbarletz One last request and we're good to go! Let's make the description under each header an md comment, so that it is visible only to the contributors who fill those sections and not to the end users. |
Removed comments from the minor version upgrade template. Signed-off-by: Revital Barletz <[email protected]>
|
@reggie-k great idea! Done. |
reggie-k
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! I've realized the step in release.md belongs to the RC1 section as a last step. Could you move it there?
Removed duplicate checklist item for opening documentation PR after creating RC. Signed-off-by: Revital Barletz <[email protected]>
reggie-k
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot! LGTM.
Add documentation PR reminder for next minor version after RC.
Fixes #25492
Checklist: