Skip to content
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

Merged
merged 2 commits into from
Mar 12, 2021

Conversation

asifdxtreme
Copy link
Member

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)

@asifdxtreme
Copy link
Member Author

@asifdxtreme
Copy link
Member Author

Preview :

Copy link
Member

@anvithks anvithks left a 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.

@anvithks anvithks added Integration Guide Integration Guide user-guide Feature User Guide labels Mar 12, 2021
@asifdxtreme asifdxtreme force-pushed the csi-docs branch 2 times, most recently from 1f032d3 to 2760ceb Compare March 12, 2021 05:31
anvithks
anvithks previously approved these changes Mar 12, 2021
Copy link
Member

@anvithks anvithks left a 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.

Copy link
Member

@anvithks anvithks left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@kumarashit kumarashit left a comment

Choose a reason for hiding this comment

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

LGTM

@kumarashit kumarashit merged commit 4d074e7 into sodafoundation:master Mar 12, 2021
@sushanthakumar
Copy link
Contributor

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Integration Guide Integration Guide user-guide Feature User Guide
Projects
None yet
Development

Successfully merging this pull request may close these issues.

List of CSI drivers which are supported by Soda CSI PlugNPlay
4 participants