Skip to content

Conversation

@StephenJamesSmith
Copy link
Contributor

@KeYoung Ken Young will be explaining the urgency of including these two as exceptions and why these exceptions should be granted.

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 25, 2021
@netlify
Copy link

netlify bot commented Feb 25, 2021

Deploy preview for osdocs ready!

Built with commit 8625397

https://deploy-preview-29835--osdocs.netlify.app

@kenyis
Copy link

kenyis commented Feb 25, 2021

JUSTIFICATION:
The defects which are documented here are fundamental platform problems which can impact customers severely. BZ#1893281 is a continual kernel panic which makes systems using the Real Time kernel unstable. Customers using a rt-kernel should not upgrade until this problem is corrected. BZ#1930469 will correct the SRIOV operator to enable customers to use 4.7 with DPDK applications on Intel NICs. If 4.7 is used out of the box, this will not work.

Given the impact of these defects at GA, it is important to document these bugs to provide a customer warning of these issues.

@StephenJamesSmith
Copy link
Contributor Author

Updated text from @[email protected] and @[email protected].

Copy link
Contributor

Choose a reason for hiding this comment

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

plug-in does not

Copy link
Contributor Author

Choose a reason for hiding this comment

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

change made.

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't understand what this first sentence is trying to say. You might want to break it up for clarity.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, it is a little problematic, but I was told by Eric that this is understandable to the audience.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@[email protected] Can we change this to "...causes the openvswitch et/openvswitch/flow_table::flow_lookup accessing per-cpu data condition on preemptible (and migratable) sections..."? Or, Am I reading this wrong?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Making the above change as per Eric Rich.

@ahardin-rh ahardin-rh added branch/enterprise-4.7 peer-review-done Signifies that the peer review team has reviewed this PR telco Label for all Telco PRs labels Feb 25, 2021
@StephenJamesSmith StephenJamesSmith force-pushed the 47releasenotes branch 2 times, most recently from 208c0d4 to ed547d8 Compare March 1, 2021 14:38
@ahardin-rh ahardin-rh added this to the Next Release milestone Mar 2, 2021
@zhaozhanqi
Copy link

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Mar 3, 2021
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Mar 3, 2021
@openshift-ci-robot
Copy link

New changes are detected. LGTM label has been removed.

@openshift-ci-robot openshift-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Mar 3, 2021
@StephenJamesSmith
Copy link
Contributor Author

Reopened the PR.

Copy link

@juphoff juphoff left a comment

Choose a reason for hiding this comment

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

LGTM

@ahardin-rh ahardin-rh removed peer-review-done Signifies that the peer review team has reviewed this PR branch/enterprise-4.7 labels Mar 3, 2021
@ahardin-rh ahardin-rh removed this from the Next Release milestone Mar 3, 2021
@openshift-ci-robot openshift-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Mar 3, 2021
@ahardin-rh ahardin-rh added this to the Next Release milestone Mar 3, 2021
@ahardin-rh ahardin-rh merged commit b55a194 into openshift:enterprise-4.7 Mar 3, 2021
+
It is recommended not to upgrade to {product-title} 4.7 until this is fixed.
+
(link:https://https://bugzilla.redhat.com/show_bug.cgi?id=1918456[*BZ#1918456*])
Copy link
Member

Choose a reason for hiding this comment

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

Are the https://https:// stutters intentional (here and below)?

@StephenJamesSmith
Copy link
Contributor Author

StephenJamesSmith commented Mar 5, 2021 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.7 size/S Denotes a PR that changes 10-29 lines, ignoring generated files. telco Label for all Telco PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants