Skip to content

Add --ovs-dpdk-cpu-count flag to PPC documentation - #119735

Open
kquinn1204 wants to merge 4 commits into
openshift:mainfrom
kquinn1204:TELCODOCS-2983
Open

Add --ovs-dpdk-cpu-count flag to PPC documentation#119735
kquinn1204 wants to merge 4 commits into
openshift:mainfrom
kquinn1204:TELCODOCS-2983

Conversation

@kquinn1204

@kquinn1204 kquinn1204 commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Updated Performance Profile Creator documentation to include the new --ovs-dpdk-cpu-count flag that was added in OpenShift 5.0.

This flag allows users to specify the number of CPUs to reserve for OVS-DPDK PMD threads, which populates spec.cpu.ovsDpdk in the generated performance profile.

Changes:

  • Added ovs-dpdk-cpu-count to optional arguments reference table
  • Included description of CPU allocation behavior (after reserved/offlined)
  • Added IMPORTANT note about NUMA locality verification requirement
  • Updated help output in both Podman and offline PPC modules
  • Documented default value of 0 (no OVS-DPDK CPUs reserved)

Related to upstream PR: openshift/cluster-node-tuning-operator#1630

Version(s):

Issue:

Link to docs preview:https://119735--ocpdocs-pr.netlify.app/openshift-enterprise/latest/scalability_and_performance/cnf-tuning-low-latency-nodes-with-perf-profile.html

QE review:

  • QE has approved this change.

Additional information:

Updated Performance Profile Creator documentation to include the new
--ovs-dpdk-cpu-count flag that was added in OpenShift 5.0.

This flag allows users to specify the number of CPUs to reserve for
OVS-DPDK PMD threads, which populates spec.cpu.ovsDpdk in the generated
performance profile.

Changes:
- Added ovs-dpdk-cpu-count to optional arguments reference table
- Included description of CPU allocation behavior (after reserved/offlined)
- Added IMPORTANT note about NUMA locality verification requirement
- Updated help output in both Podman and offline PPC modules
- Documented default value of 0 (no OVS-DPDK CPUs reserved)

Related to upstream PR: openshift/cluster-node-tuning-operator#1630

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@openshift-ci openshift-ci Bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 11, 2026
@ocpdocs-previewbot

ocpdocs-previewbot commented Sep 11, 2026

Copy link
Copy Markdown

kquinn1204 and others added 2 commits September 14, 2026 10:57
- Add spec.cpu.ovsDpdk field to YAML template
- Add reference to Performance Profile Creator with --ovs-dpdk-cpu-count
- Add guidance on NUMA placement for OVS-DPDK CPUs

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Add link to "Reserving CPUs for OVS-DPDK" documentation

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Comment thread modules/installation-openstack-ovs-dpdk-performance-profile.adoc Outdated
@kquinn1204

Copy link
Copy Markdown
Contributor Author

Error on preview building valid as PR #116259 not merged yet


* link:https://access.redhat.com/articles/7081587[Understanding workload hints]

* * xref:../scalability_and_performance/cnf-reserving-cpus-for-ovs-dpdk.adoc#cnf-reserving-cpus-for-ovs-dpdk[Reserving CPUs for OVS-DPDK]

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🤖 [error] AsciiDocDITA.RelatedLinks: Content other than links cannot be mapped to DITA related-links.

@openshift-ci

openshift-ci Bot commented Sep 14, 2026

Copy link
Copy Markdown

@kquinn1204: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/validate-portal fb8cd21 link true /test validate-portal

Full PR test history. Your PR dashboard.

Details

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-sigs/prow repository. I understand the commands that are listed here.

@Tal-or Tal-or left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM but I see CI is failing.
Please fix the issues first and i'll apporve

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

Labels

size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants