-
Notifications
You must be signed in to change notification settings - Fork 1.8k
OSDOCS-7625: add GA and update release notes 4.14 #64195
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
@ShaunaDiaz: This pull request references OSDOCS-7625 which is a valid jira issue. In response to this:
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. |
🤖 Updated build preview is available at: Build log: https://circleci.com/gh/ocpdocs-previewbot/openshift-docs/23815 |
@ShaunaDiaz: This pull request references OSDOCS-7625 which is a valid jira issue. In response to this:
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. |
@DanielFroehlich PTAL at the "About this release" section. Do we have any Lifecycle dates? I have one that I grabbed from OCP there as a placeholder. |
@pmtk and @dhellmann PTAL at the Updates section (and anything else you want to). |
a19ce91
to
78dcc9a
Compare
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 don't have anything to add beyond what Daniel had to say.
78dcc9a
to
d0c467b
Compare
/lgtm |
@ShaunaDiaz: This pull request references OSDOCS-7625 which is a valid jira issue. In response to this:
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. |
@DanielFroehlich PTAL and LGTM if ready? Thank you! |
Last comment: In the "Networking changes section", the changes are pretty high level summaries. Can we please add a link to the Networking section doc where this is being explained a bit more detailed? |
@DanielFroehlich The networking release note is being done in a separate PR. So, yes, there is more detail and a link. |
/label peer-review-needed |
Thx, and sorry I did not saw that other PR. Then this |
/lgtm |
/label peer-review-in-progress |
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.
Left a few suggestions, otherwise looks good!
/remove-label peer-review-needed /label peer-review-done |
d0c467b
to
fa252b9
Compare
New changes are detected. LGTM label has been removed. |
/label merge-review-needed |
Note other MicroShift relnote PRs are also in the merge review queue. |
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.
/remove-label merge-review-in-progress
/remove-label merge-review-needed
Version(s):
4.14
Issue:
https://issues.redhat.com/browse/OSDOCS-7625
Link to docs preview:
https://64195--docspreview.netlify.app/microshift/latest/microshift_release_notes/microshift-4-14-release-notes#microshift-4-14-about-this-release
https://64195--docspreview.netlify.app/microshift/latest/microshift_release_notes/microshift-4-14-release-notes#microshift-4-14-updates-supported
QE review: