Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG]: Driver should not be expecting a secret which is not used at all for PowerMax when authorization is enabled #1584

Open
shanmydell opened this issue Nov 13, 2024 · 4 comments
Labels
area/csi-powermax Issue pertains to the CSI Driver for Dell EMC PowerMax type/bug Something isn't working. This is the default label associated with a bug issue.
Milestone

Comments

@shanmydell
Copy link
Collaborator

Bug Description

Driver should not be expecting a secret which is not used at all for PowerMax when authorization is enabled

We are requesting the customer to create a secret without a username and password, a stale entry that will not be validated and used by the driver and should be skipped. We should validate if authorization is enabled and make this secret optional

Logs

https://github.com/dell/csi-powermax/blob/main/samples/secret/secret.yaml

apiVersion: v1
kind: Secret
metadata:
name: powermax-creds
namespace: powermax
type: Opaque
data:

set username to the base64 encoded username

if authorization is enabled, username will be ignored

username: bm90X3RoZV91c2VybmFtZQ==

set password to the base64 encoded password

if authorization is enabled, password will be ignored

password: bm90X3RoZV9wYXNzd29yZA==

Uncomment the following key if you wish to use ISCSI CHAP authentication (v1.3.0 onwards)

chapsecret:

Screenshots

No response

Additional Environment Information

No response

Steps to Reproduce

Install the driver without secret when authorization is enabled, it will be crashing

Expected Behavior

Install the driver with this secret being optional/not required when authorization is enabled

CSM Driver(s)

PowerMax

Installation Type

No response

Container Storage Modules Enabled

No response

Container Orchestrator

NA

Operating System

NA

@shanmydell shanmydell added needs-triage Issue requires triage. type/bug Something isn't working. This is the default label associated with a bug issue. labels Nov 13, 2024
@shanmydell shanmydell added this to the v1.13.0 milestone Nov 13, 2024
@csmbot
Copy link
Collaborator

csmbot commented Nov 13, 2024

@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 [email protected].

@anandrajak1 anandrajak1 removed the needs-triage Issue requires triage. label Nov 13, 2024
@anandrajak1
Copy link
Contributor

/sync

@csmbot
Copy link
Collaborator

csmbot commented Nov 13, 2024

link: 30420

@prablr79 prablr79 added the area/csi-powermax Issue pertains to the CSI Driver for Dell EMC PowerMax label Nov 19, 2024
@thikade
Copy link

thikade commented Nov 27, 2024

I agree, we simply put user: "dummy" and password: "dummy" in there, to satisfy the operator.
.... bit weird though, that you can then see user: "dummy" in the controller logs ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/csi-powermax Issue pertains to the CSI Driver for Dell EMC PowerMax type/bug Something isn't working. This is the default label associated with a bug issue.
Projects
None yet
Development

No branches or pull requests

5 participants