dell / csm

Dell Container Storage Modules (CSM)
Apache License 2.0
68 stars 15 forks source link

[BUG]: Documentation for PowerStore Operator secret creation is inconsistent with helm and contains typos #1471

Closed coulof closed 1 week ago

coulof commented 1 month ago

Bug Description

Per the docs here : https://dell.github.io/csm-docs/docs/deployment/csmoperator/drivers/powerstore/#namespace-and-powerstore-api-access-configuration & there : https://dell.github.io/csm-docs/docs/deployment/helm/drivers/installation/powerstore/#user-privileges the way to create secret are different and not very good with Operator.

Moreover the configuration contains typos.

Suggested changes are here : https://github.com/dell/csm-docs/pull/1236

I let the team choose to integrate those in main or differently.

Logs

N/A

Screenshots

No response

Additional Environment Information

No response

Steps to Reproduce

Read the doc

Expected Behavior

No typo, easy creation

CSM Driver(s)

PowerStore

Installation Type

No response

Container Storage Modules Enabled

No response

Container Orchestrator

k8s

Operating System

CoreOS

csmbot commented 1 month ago

@coulof: 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.

donatwork commented 2 weeks ago

link: 29393

mjsdell commented 2 weeks ago

Linking the PR which was opened but not complete. It is now closed. PR can be revisited with the proper context.