Skip to content

kubeai/readme: improve nri optimization instructions#1119

Merged
poussa merged 3 commits intoopea-project:mainfrom
marquiz:devel/kubeai-nri-2
Jul 3, 2025
Merged

kubeai/readme: improve nri optimization instructions#1119
poussa merged 3 commits intoopea-project:mainfrom
marquiz:devel/kubeai-nri-2

Conversation

@marquiz
Copy link
Copy Markdown
Collaborator

@marquiz marquiz commented Jun 12, 2025

Co-authored-by: Malini Bhandaru [email protected]

Co-authored-by: Malini Bhandaru <[email protected]>
Signed-off-by: Markus Lehtonen <[email protected]>
@marquiz marquiz requested review from mkbhanda and poussa as code owners June 12, 2025 10:11
@poussa poussa requested a review from eero-t June 12, 2025 10:38
> install the NRI plugin to an empty node (do it right after node bootstrap, or
> drain the node before installation).
> **NOTE:** To avoid disturbing already running workloads, it is recommended to
> install the NRI plugin on an empty node, that is, before deploying workloads
Copy link
Copy Markdown
Collaborator

@eero-t eero-t Jun 12, 2025

Choose a reason for hiding this comment

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

Admin does not need to install the plugin on the node itself, but can do it using Kubernetes facilities from outside the node, to install it to the node, right?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Yeah, maybe that's better, "onto"?

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.

@marquiz Are you going to update the PR? (I can merge it after that as Sakari has already approved this.)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@marquiz @eero-t ping, ping. Let's get this done now.

@poussa poussa merged commit 6bccecf into opea-project:main Jul 3, 2025
5 checks passed
@marquiz marquiz deleted the devel/kubeai-nri-2 branch July 28, 2025 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants