-
Notifications
You must be signed in to change notification settings - Fork 220
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
Add CSI PNP supported Driver list and Integration Guide #147
Conversation
Preview : |
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.
Please verify naming convention of SODA (Soda and soda at some places.)
The support matrix can go into the support matrix section.
content/guides/integration-guides/csi-integration/soda-csi-pnp/_index.md
Outdated
Show resolved
Hide resolved
1f032d3
to
2760ceb
Compare
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.
Over all looks LGTM.
Some minor comments.
content/guides/integration-guides/csi-integration/soda-csi-pnp/_index.md
Outdated
Show resolved
Hide resolved
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.
LGTM
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.
LGTM
content/guides/integration-guides/csi-integration/soda-csi-pnp/_index.md
Show resolved
Hide resolved
content/guides/integration-guides/csi-integration/soda-csi-pnp/_index.md
Show resolved
Hide resolved
content/guides/integration-guides/csi-integration/soda-csi-pnp/_index.md
Show resolved
Hide resolved
LGTM |
What type of PR is this?
/kind documentation
What this PR does / why we need it:
This PR adds the list of supported driver of CSI Plug-N-Play with their tested deployment and sample application files. This PR also adds the integration guide for CSI PNP.
Which issue(s) this PR fixes:
Fixes sodafoundation/nbp#406
Test Report Added?:
/kind TESTED
Test Report:
Test Reports are attached here sodafoundation/nbp#398 (comment)