dell / csm

Dell Container Storage Modules (CSM)
Apache License 2.0
71 stars 16 forks source link

[BUG]: Comment out duplicate entries in the sample secret.yaml file #1030

Closed dtonnesen closed 1 year ago

dtonnesen commented 1 year ago

The sample secret.yaml file is below. As as user I find it very confusing to include this extra stanza at the bottom:

nasName: "nas-server"

The problem is that all these entries are already present, and uncommented, in the top of the file. If you do not notice that, your CSI install will fail because it adds a non-existent PowerFlex setup. I suggest commenting all entries at the top, which explain what each variable is, and just leaving the bottom stanza. In addition, a comment can be added that if you have multiple systems you can add another stanza.

Username for accessing PowerFlex system.

If authorization is enabled, username will be ignored.

csmbot commented 1 year ago

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

sharmilarama commented 1 year ago

/sync

hoppea2 commented 1 year ago

/sync

csmbot commented 1 year ago

link: 18930

coulof commented 1 year ago

Thanks for your feedback @dtonnesen, you will see the changes in the next release