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

[FEATURE]: Driver should allow usage of valid arrays if secret file has combination of valid and invalid arrays. #871

Open
sakshi-garg1 opened this issue Jun 21, 2023 · 1 comment
Assignees
Labels
needs-triage Issue requires triage. type/feature-request New feature request. This is the default label associated with a feature request issue.

Comments

@sakshi-garg1
Copy link
Contributor

sakshi-garg1 commented Jun 21, 2023

Feature Description

Driver should allow provisioning on valid arrays if secret files has combination of valid and invalid arrays.

When deploying the csi-powerstore driver with some invalid array in the secret file, provisioning fails with 'no available topology found' error, even on valid storage arrays. The 'node' pods for csi-powerstore will enter a crash loop shortly after PVC is created.

Removing the bad entry and redeploying removes the issue. Replacing the bad entry and redeploying brings the issue back, making it highly reproducible. This issue was discovered because an old PowerStore array entry was left in a secret file:

  • endpoint: "https://XX.XXX.XX/api/rest"
    globalID: "XXXXXX"
    username:
    password:
    skipCertificateValidation: true
    blockProtocol: "ISCSI"
    nasName: "XXXX"

This behavior might be common across all the drivers e.g. In csi-powerscale when invalid arrays were preset in secret.
Multiple restarts in isilion-controlloer pod
image

PVC remains in pending state with below events
image
image

@sakshi-garg1 sakshi-garg1 added needs-triage Issue requires triage. type/bug Something isn't working. This is the default label associated with a bug issue. labels Jun 21, 2023
@csmbot
Copy link
Collaborator

csmbot commented Jun 21, 2023

@sakshi-garg1: 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 assign an appropriate priority label.


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.

@AkshaySainiDell AkshaySainiDell changed the title [BUG]: Bad Entries in Secret File Cause Failed Provisioning [FEATURE]: Driver should allow usage of valid arrays if secret files has combination of valid and invalid arrays. Jun 22, 2023
@AkshaySainiDell AkshaySainiDell added type/feature-request New feature request. This is the default label associated with a feature request issue. and removed type/bug Something isn't working. This is the default label associated with a bug issue. labels Jun 22, 2023
@AkshaySainiDell AkshaySainiDell changed the title [FEATURE]: Driver should allow usage of valid arrays if secret files has combination of valid and invalid arrays. [FEATURE]: Driver should allow usage of valid arrays if secret has combination of valid and invalid arrays. Jun 22, 2023
@AkshaySainiDell AkshaySainiDell changed the title [FEATURE]: Driver should allow usage of valid arrays if secret has combination of valid and invalid arrays. [FEATURE]: Driver should allow usage of valid arrays if secret file has combination of valid and invalid arrays. Jun 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-triage Issue requires triage. type/feature-request New feature request. This is the default label associated with a feature request issue.
Projects
None yet
Development

No branches or pull requests

3 participants