-
Notifications
You must be signed in to change notification settings - Fork 16
feat: Add STIG/FIPS-compliant Network Operator deployment instructions #275
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
base: main
Are you sure you want to change the base?
Conversation
| Platform Support <platform-support.rst> | ||
| Getting Started with Kubernetes <getting-started-with-kubernetes.rst> | ||
| Getting Started with Red Hat OpenShift <getting-started-with-openshift.rst> | ||
| NVIDIA Network Operator Government Ready <install-network-operator-gov-ready.rst> |
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.
Do we need NVIDIA Network Operator in the tree link?
Maybe stick with same wording as Getting Started with
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.
I use the same naming as GPU Operator has. Should we change the title somehow?
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.
I think that here Government Ready is enough in the left tree.
9ee0494 to
9a5829e
Compare
| --set sriov-network-operator.images.sriovConfigDaemon=doca-driver-stig-fips \ | ||
| --set sriov-network-operator.imagePullSecrets={ngc-secret} \ | ||
| --set sriovNetworkOperator.enabled=true | ||
| --set nfd.enabled=true |
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.
should nfd.enabled be false in this case?
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.
Why should we disable NFD? it's STIG/FIPS compliant
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.
You added a section that says users need to deploy nfd separately. AFAIK with nfd.enabled=true in the helm chart we will deploy our own downstream fork chart?
67d9332 to
88a0362
Compare
Documentation previewhttps://mellanox.github.io/network-operator-docs/review/pr-275 |
24575ad to
5953aa4
Compare
Signed-off-by: Ivan Kolodiazhnyi <[email protected]>
No description provided.