Skip to content

OCPBUGS-64584 Modify DPU Operator Docs to reflect 4.20 code #101591

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 12, 2025
Merged

OCPBUGS-64584 Modify DPU Operator Docs to reflect 4.20 code #101591

merged 1 commit into from
Nov 12, 2025

Conversation

@kquinn1204
Copy link
Contributor

@kquinn1204 kquinn1204 commented Nov 3, 2025 •
edited
Loading

Version(s): 4.20, 4.21 and Main

Issue: https://issues.redhat.com/browse/OCPBUGS-64584

Link to docs preview: https://101591--ocpdocs-pr.netlify.app/openshift-enterprise/latest/networking/networking_operators/dpu-operator/dpu-operator.html

QE review:

  • QE has approved this change.

Additional information:

@openshift-ci openshift-ci bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Nov 3, 2025
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Nov 3, 2025 •
edited
Loading

🤖 Wed Nov 12 14:23:44 - Prow CI generated the docs preview:

https://101591--ocpdocs-pr.netlify.app/
https://101591--ocpdocs-pr.netlify.app/openshift-enterprise/latest/networking/networking_operators/dpu-operator/dpu-operator.html

@openshift-ci openshift-ci bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Nov 3, 2025
bn222
bn222 reviewed Nov 3, 2025
View reviewed changes
bn222
bn222 reviewed Nov 3, 2025
View reviewed changes
bn222
bn222 reviewed Nov 3, 2025
View reviewed changes
bn222
bn222 reviewed Nov 3, 2025
View reviewed changes
bn222
bn222 reviewed Nov 4, 2025
View reviewed changes
@openshift-ci openshift-ci bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Nov 4, 2025
@kquinn1204
Copy link
Contributor Author

kquinn1204 commented Nov 5, 2025

@bn222 @wizhaoredhat I made updates can you PTAL again, tried to consolidate all the feedback and get a better picture adding a new section on status

bn222
bn222 reviewed Nov 5, 2025
View reviewed changes
bn222
bn222 reviewed Nov 5, 2025
View reviewed changes
Copy link

@bn222 bn222 left a comment

Choose a reason for hiding this comment

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

1 comment otherwise lgtm

@zhiqiangf
Copy link

zhiqiangf commented Nov 5, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 5, 2025
wizhaoredhat
wizhaoredhat reviewed Nov 6, 2025
View reviewed changes
wizhaoredhat
wizhaoredhat reviewed Nov 6, 2025
View reviewed changes
wizhaoredhat
wizhaoredhat reviewed Nov 6, 2025
View reviewed changes
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Nov 7, 2025
tshwartz
tshwartz reviewed Nov 9, 2025
View reviewed changes
tshwartz
tshwartz reviewed Nov 9, 2025
View reviewed changes
tshwartz
tshwartz reviewed Nov 9, 2025
View reviewed changes
tshwartz
tshwartz reviewed Nov 9, 2025
View reviewed changes
tshwartz
tshwartz reviewed Nov 9, 2025
View reviewed changes
tshwartz
tshwartz reviewed Nov 9, 2025
View reviewed changes
tshwartz
tshwartz reviewed Nov 9, 2025
View reviewed changes
tshwartz
tshwartz reviewed Nov 9, 2025
View reviewed changes
tshwartz
tshwartz reviewed Nov 10, 2025
View reviewed changes
tshwartz
tshwartz reviewed Nov 10, 2025
View reviewed changes
tshwartz
tshwartz reviewed Nov 10, 2025
View reviewed changes
tshwartz
tshwartz reviewed Nov 10, 2025
View reviewed changes
tshwartz
tshwartz reviewed Nov 10, 2025
View reviewed changes
@wizhaoredhat
Copy link

wizhaoredhat commented Nov 10, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 10, 2025
@zhiqiangf
Copy link

zhiqiangf commented Nov 10, 2025

/lgtm

@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Nov 11, 2025
ocpdocs-vale-bot
ocpdocs-vale-bot reviewed Nov 11, 2025
View reviewed changes
slovern
slovern reviewed Nov 11, 2025
View reviewed changes
Copy link
Contributor

@slovern slovern left a comment

Choose a reason for hiding this comment

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

a few suggestions

@kquinn1204
OCPBUGS-64584 Modify DPU Operator Docs to reflect 4.20 code
f6e32b5 
formatting fix

formatting fix 2

fixing vale errors 5

fixing vale errors 6

fixing vale errors 7
ocpdocs-vale-bot
ocpdocs-vale-bot reviewed Nov 12, 2025
View reviewed changes
modules/nw-dpu-intro-installing-operator.adoc
@@ -0,0 +1,17 @@
// Module included in the following assemblies:
Copy link
Collaborator

@ocpdocs-vale-bot ocpdocs-vale-bot Nov 12, 2025

Choose a reason for hiding this comment

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

🤖 [error] OpenShiftAsciiDoc.ModuleContainsContentType: Module is missing the '_mod-docs-content-type' variable.

@openshift-ci
Copy link

openshift-ci bot commented Nov 12, 2025

@kquinn1204: all tests passed!

Full PR test history. Your PR dashboard.

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.

@wizhaoredhat
Copy link

wizhaoredhat commented Nov 12, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 12, 2025
@slovern slovern added this to the Continuous Release milestone Nov 12, 2025
@slovern slovern merged commit 8fcf0c9 into openshift:main Nov 12, 2025
2 checks passed
@slovern
Copy link
Contributor

slovern commented Nov 12, 2025

/cherrypick enterprise-4.21

@slovern
Copy link
Contributor

slovern commented Nov 12, 2025

/cherrypick enterprise-4.20

@openshift-cherrypick-robot
Copy link

openshift-cherrypick-robot commented Nov 12, 2025

@slovern: new pull request created: #102393

In response to this:

/cherrypick enterprise-4.21

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.

@openshift-cherrypick-robot openshift-cherrypick-robot mentioned this pull request Nov 12, 2025
Merged
@openshift-cherrypick-robot
Copy link

openshift-cherrypick-robot commented Nov 12, 2025

@slovern: new pull request created: #102394

In response to this:

/cherrypick enterprise-4.20

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.

@openshift-cherrypick-robot openshift-cherrypick-robot mentioned this pull request Nov 12, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@slovern slovern slovern left review comments

+4 more reviewers

@bn222 bn222 bn222 left review comments

@wizhaoredhat wizhaoredhat wizhaoredhat left review comments

@tshwartz tshwartz tshwartz left review comments

@ocpdocs-vale-bot ocpdocs-vale-bot ocpdocs-vale-bot left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@zhiqiangf zhiqiangf

@wizhaoredhat wizhaoredhat

Labels

branch/enterprise-4.20 branch/enterprise-4.21 lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Milestone

Continuous Release

Development

Successfully merging this pull request may close these issues.

9 participants

@kquinn1204 @ocpdocs-previewbot @zhiqiangf @wizhaoredhat @slovern @openshift-cherrypick-robot @bn222 @tshwartz @ocpdocs-vale-bot