dell / csm

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

[BUG]: Steps for Upgrading Drivers with Dell CSM Operator incorrect or confusing #1588

Open gashof opened 4 hours ago

gashof commented 4 hours ago

Bug Description

Is this correct here for upgrading the driver with CSM Operator?

https://dell.github.io/csm-docs/docs/deployment/csmoperator/#upgrading-drivers-with-dell-csm-operator

Use the object name in the kubectl edit command: kubectl edit csm <driver-object>/<object-name> -n <driver-namespace>

To me it looks like is csm and we are putting it in twice.

These work and allow me to upgrade the drive versions: oc edit csm -n powermax oc edit csm powermax -n powermax

But my understanding of what the doc is asking to use is this and it fails: [root@ocpcsi-bastion-10-241-171-47 ~]# oc edit csm csm/powermax -n powermax error: there is no need to specify a resource type as a separate argument when passing arguments in resource/name form (e.g. 'oc get resource/' instead of 'oc get resource resource/'

We do in the very next line show how to edit the file correctly, but the above causes some confusion. If we can explain it a little more clearly that would be great.

Logs

N/A

Screenshots

No response

Additional Environment Information

No response

Steps to Reproduce

N/A

Expected Behavior

N/A

CSM Driver(s)

N/A

Installation Type

No response

Container Storage Modules Enabled

No response

Container Orchestrator

N/A

Operating System

N/A

csmbot commented 4 hours ago

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