dell / csm

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

[BUG]: Inconsistent naming convention of secret is misleading in Installation of PowerMax #1566

Open shanmydell opened 2 hours ago

shanmydell commented 2 hours ago

Bug Description

Refer: https://dell.github.io/csm-docs/docs/deployment/csmoperator/drivers/powermax/#install-driver

There is a credential called powermax-creds which does not giving enough clarity with the naming as well as description

There will be different credentials for different unisphere

It would be good to have the naming convention as primary-unisphere-creds and for backup-unisphere-creds so that the users can understand the usage by its name

Logs

Refer to the documentation here https://dell.github.io/csm-docs/docs/deployment/csmoperator/drivers/powermax/#install-driver

Screenshots

No response

Additional Environment Information

No response

Steps to Reproduce

Secret name can be found in this location https://dell.github.io/csm-docs/docs/deployment/csmoperator/drivers/powermax/#install-driver

Expected Behavior

Variables defined should match its usage and give clarity to the customer without misleading

CSM Driver(s)

PowerMax

Installation Type

No response

Container Storage Modules Enabled

No response

Container Orchestrator

NA

Operating System

NA

csmbot commented 2 hours ago

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