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]: cert-csi documentation clarity #1503

Open
dancohen21 opened this issue Oct 4, 2024 · 4 comments
Open

[BUG]: cert-csi documentation clarity #1503

dancohen21 opened this issue Oct 4, 2024 · 4 comments
Assignees
Labels
area/cert-csi Issues pertaining to Cert CSI area/csm-docs Issue pertains to the CSM Documentation type/bug Something isn't working. This is the default label associated with a bug issue.
Milestone

Comments

@dancohen21
Copy link

Bug Description

The driver specific examples in the documentation are great
https://dell.github.io/csm-docs/docs/support/cert-csi/

For PowerStore, as an example, the values (true/false) are setup properly for valid use cases.

in contrast,
https://github.com/dell/cert-csi/example-certify-config.yaml seems incomplete
example-certify-config.yaml

requests:

  • include driver specific example in the github repository
  • remove the example-certify-config.yaml from the github repository
  • include critical comments in the driver specific tests yaml for any test requirements. For example, the "waitforfirstconsumer" requirement for the CapacityTracking test, and in OpenShift, any labels that need to be applied to the CSI driver

Logs

n/a

Screenshots

No response

Additional Environment Information

No response

Steps to Reproduce

n/a

Expected Behavior

n/a

CSM Driver(s)

powerstore 2.10.1

Installation Type

No response

Container Storage Modules Enabled

No response

Container Orchestrator

OpenShift 4.14

Operating System

OpenShift 4.14

@dancohen21 dancohen21 added needs-triage Issue requires triage. type/bug Something isn't working. This is the default label associated with a bug issue. labels Oct 4, 2024
@csmbot
Copy link
Collaborator

csmbot commented Oct 4, 2024

@dancohen21: Thank you for submitting this issue!

The issue is currently awaiting triage. Please make sure you have given us as much context as possible.

If the maintainers determine this is a relevant issue, they will remove the needs-triage label and respond appropriately.


We want your feedback! If you have any questions or suggestions regarding our contributing process/workflow, please reach out to us at container.storage.modules@dell.com.

@rajkumar-palani
Copy link

Hi Rajendra, please do triage and create internal ticket for tracking of this issue.

@suryagupta4 suryagupta4 added area/csm-docs Issue pertains to the CSM Documentation area/cert-csi Issues pertaining to Cert CSI and removed needs-triage Issue requires triage. labels Nov 6, 2024
@suryagupta4
Copy link

/sync

@csmbot
Copy link
Collaborator

csmbot commented Nov 6, 2024

link: 29751

@suryagupta4 suryagupta4 added this to the v1.13.0 milestone Nov 6, 2024
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 area/csm-docs Issue pertains to the CSM Documentation 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

5 participants