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
OCPBUGS-3407 - Workload partitioning wrong CRI-O configuration key #53510
OCPBUGS-3407 - Workload partitioning wrong CRI-O configuration key #53510
Conversation
/label telco |
🤖 Updated build preview is available at: Build log: https://circleci.com/gh/ocpdocs-previewbot/openshift-docs/4765 |
/lgtm |
bdaf500
to
091c862
Compare
091c862
to
3983d5b
Compare
---- | ||
<1> The `CPUs` value varies based on the installation. | ||
<1> The `cpuset` value varies based on the installation. | ||
+ | ||
If Hyper-Threading is enabled, specify both threads for each core. The `CPUs` value must match the reserved CPU set specified in the performance profile. |
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.
Does this also need a change CPUs
-> cpuset
? Also, I think we should highlight the word reserved
if we mean to reference the reserved
field in the performance profile so it's clear we mean that field.
3983d5b
to
4d6d4b9
Compare
4d6d4b9
to
edd8991
Compare
/label peer-review-needed |
/lgtm |
/remove-label peer-review-needed |
/remove-label peer-review-in-progress |
/label merge-review-needed |
/cherrypick enterprise-4.12 |
/cherrypick enterprise-4.11 |
/cherrypick enterprise-4.10 |
/cherrypick enterprise-4.9 |
@mjpytlak: new pull request created: #53602 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. |
@mjpytlak: new pull request created: #53603 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. |
@mjpytlak: new pull request created: #53604 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. |
@mjpytlak: #53510 failed to apply on top of branch "enterprise-4.9":
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. |
@rohennes Got a conflict 4.9. PTAL. I will merge up the rest of branches. |
See the manual CP PR with resolved conflicts @mjpytlak - thanks! |
OCPBUGS-3407: There was an incorrect parameter (
CPU
vscpuset
) in a previous documentation update. The module has been updated since the bug was created, soCPU
was not present but the formatting seemed different to the schema. So I updated the yaml to match the schema. And also changed the numbered callout fromCPU
tocpuset
Version(s):
4.9+
Issue:
https://issues.redhat.com/browse/OCPBUGS-3407
Link to docs preview:
http://file.emea.redhat.com/rohennes/OCPBUGS-3407-workload-partitionin-yaml/scalability_and_performance/sno-du-enabling-workload-partitioning-on-single-node-openshift.html
QE review:
Additional information: