dell/csm

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

shanmydell opened this issue · 4 comments

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

@gallacher : Please sync the issue

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

/sync

link: 26078