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

[BUG]: Update Cert-csi documentation for driver certification #914

Closed
vincent1chen opened this issue Jul 28, 2023 · 5 comments
Closed

[BUG]: Update Cert-csi documentation for driver certification #914

vincent1chen opened this issue Jul 28, 2023 · 5 comments
Assignees
Labels
area/cert-csi Issues pertaining to Cert CSI type/bug Something isn't working. This is the default label associated with a bug issue.
Milestone

Comments

@vincent1chen
Copy link

How can the Team help you today?

I could get the CSI cert configuration template from two places both places don't show a full configuration template, like

VolumeHealth: true
RWOP: true
VGS: true

but from documents or CLI help list, I know CSI cert support these tests. please update full template as example in doc

https://dell.github.io/csm-docs/docs/csidriver/installation/test/certcsi/#launching-certification-test-run

https://github.com/dell/cert-csi/blob/main/example-certify-config.yaml

@vincent1chen vincent1chen added the type/question Ask a question. This is the default label associated with a question issue. label Jul 28, 2023
@adarsh-dell
Copy link
Contributor

Hi @vincent1chen Before I proceed to update the documentation, I wanted to confirm with you whether you've tested running these test suites by configuring the fields in the config.yaml file. Have you successfully executed all suites using the certify command?

@adarsh-dell adarsh-dell added the area/cert-csi Issues pertaining to Cert CSI label Aug 9, 2023
@adarsh-dell adarsh-dell self-assigned this Aug 9, 2023
@vincent1chen
Copy link
Author

vincent1chen commented Aug 11, 2023

@adarsh-dell , I use config.yaml as template in my test. it is a full feature list. but there need to make some change as storage class may have some different support feature.

@adarsh-dell
Copy link
Contributor

@vincent1chen If I understood the requirement correctly, you're requesting us to incorporate additional suites (if it is supported but missing in the sample file) into the template and revise the documentation so that they can be executed using the certify command, correct ?

@vincent1chen
Copy link
Author

vincent1chen commented Aug 16, 2023 via email

@adarsh-dell adarsh-dell removed the type/question Ask a question. This is the default label associated with a question issue. label Aug 25, 2023
@adarsh-dell adarsh-dell added this to the v1.8.0 milestone Aug 25, 2023
@adarsh-dell
Copy link
Contributor

Hi @vincent1chen The requested changes have been completed and will be available from CSM1.8 onwards.

@adarsh-dell adarsh-dell added the type/bug Something isn't working. This is the default label associated with a bug issue. label Aug 29, 2023
@shaynafinocchiaro shaynafinocchiaro changed the title [QUESTION]: csi-cert template is not updated in doc [BUG]: Update Cert-csi documentation for driver certification Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/cert-csi Issues pertaining to Cert CSI type/bug Something isn't working. This is the default label associated with a bug issue.
Projects
None yet
Development

No branches or pull requests

2 participants