-
Notifications
You must be signed in to change notification settings - Fork 1.9k
BZ1918456 new changes #30285
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
BZ1918456 new changes #30285
Conversation
|
Deploy preview for osdocs ready! Built with commit 9a2a56f |
|
@kenyis - PTAL. |
|
@wking @sferich888 @kenyis @StephenJamesSmith - not sure who can answer this, but we recently added this release note entry via an exception process (#29835). I am trying to figure out why this entry needs to change so soon after that. Was something missed in the original explanations? Is there someone whose ack we should have taken before publishing it that we didn't? Changing release notes in between releases does require us to notify stakeholders and get acks, and I would like to reduce the paperwork involved. |
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 thought this was going to be:
Customers who are using the kernel-rt and run either {product-title} 4.6 or 4.7.0 will experience a severe impact.
My point is that both folks running 4.6 and 4.7 are already impacted, so this bug has nothing to do with upgrades to 4.7. It might have something to do with updates from 4.5 to 4.6, but that's irrelevant for the 4.7-and-later release notes.
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.
wking didn't have a chance to review during the initial iteration and had some important changes. Sorry about the inconvenience.
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.
@wking Above change has been made.
a13ce2c to
9a2a56f
Compare
wking
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.
/lgtm
Not me personally, but the updates team (which should be more similar to @openshift/openshift-team-cincinnati , I'll run down the difference) would like to review statements about OCP update recommendations. Because if an update is really not recommended, we want to stop recommending that update edge. We want to minimize divergence between human-oriented docs and the Cincinnati API that represents OCP update recommendations. |
|
Anything we can do to help this along? |
|
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
|
Stale issues rot after 30d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle rotten |
|
Rotten issues close after 30d of inactivity. Reopen the issue by commenting /close |
|
@openshift-bot: Closed this PR. DetailsIn 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. |
@wking @[email protected] - Plz verify/approve (lgtm).