search

dell / csm

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

[BUG]: Document update : PowerFlex expecting secret CR as <csm-cr-name>-config in operator #1350

Closed shanmydell closed 2 months ago

shanmydell commented 2 months ago

Bug Description

Document update : PowerFlex expecting secret CR as -config in operator

Logs

https://github.com/dell/csm-operator/blob/aeda49e08b3c333b9e51f3fb0a34ac17ca3a0db3/pkg/drivers/powerflex.go#L112

Screenshots

No response

Additional Environment Information

No response

Steps to Reproduce

https://github.com/dell/csm-operator/blob/aeda49e08b3c333b9e51f3fb0a34ac17ca3a0db3/pkg/drivers/powerflex.go#L112

Expected Behavior

Proper documentation for customer clarity

CSM Driver(s)

PowerFlex, Please validate it across other drivers and update the documentation accordingly

Installation Type

No response

Container Storage Modules Enabled

No response

Container Orchestrator

NA

Operating System

NA

shanmydell commented 2 months ago

@gallacher : Please sync the issue

suryagupta4 commented 2 months ago

powermax: secret is encoded as cr.Name + "certs" Ref: https://github.com/dell/csm-operator/blob/main/pkg/drivers/powermax.go#L267

In the docs, its clearly mentioned the secret name to be used as powermax-certs, no discrepancy in docs here. Ref: https://dell.github.io/csm-docs/docs/deployment/csmoperator/drivers/powermax/#installation

powerstore: secret is encoded as cr.Name + "-config" Ref: https://github.com/dell/csm-operator/blob/main/pkg/drivers/powerstore.go#L65

In the docs, the example which is given for secret has powerstore-config and it is not mentioned anywhere that the name can be changed. Ref: https://dell.github.io/csm-docs/docs/deployment/csmoperator/drivers/powerstore/#user-privileges image

hoppea2 commented 2 months ago

/sync

csmbot commented 2 months ago

link: 26078