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]: Minimal CR for Powerflex is failing in Csm-operator #1671

Closed
mgandharva opened this issue Jan 10, 2025 · 2 comments
Closed

[BUG]: Minimal CR for Powerflex is failing in Csm-operator #1671

mgandharva opened this issue Jan 10, 2025 · 2 comments
Assignees
Labels
area/csm-operator type/bug Something isn't working. This is the default label associated with a bug issue.
Milestone

Comments

@mgandharva
Copy link
Contributor

Bug Description

Minimal doesn't have common but we are checking the image causing panic error in conmmonconfig.go
Line 320 : if string(cr.Spec.Driver.Common.Image) != "" {

Add driver as init container to daemonset by JacobGros · Pull Request #828 · dell/csm-operator
https://github.com/dell/csm-operator/blob/3b06f1c528a798575efe958c677e8987b27817ce/pkg/drivers/commonconfig.go#L319
image-2025-01-09-16-42-13-650 (1)

Logs

image-2025-01-09-16-41-38-959 (1)
image-2025-01-09-22-52-05-139

Screenshots

No response

Additional Environment Information

No response

Steps to Reproduce

Apply the powerflex minimal sample
kubectl create -f powerflex.yaml

Expected Behavior

After applying the manifest file powerflex driver should be installed without any error

CSM Driver(s)

Powerflex

Installation Type

No response

Container Storage Modules Enabled

No response

Container Orchestrator

NA

Operating System

NA

@mgandharva mgandharva added needs-triage Issue requires triage. type/bug Something isn't working. This is the default label associated with a bug issue. area/csm-operator labels Jan 10, 2025
@mgandharva mgandharva added this to the v1.13.0 milestone Jan 10, 2025
@csmbot
Copy link
Collaborator

csmbot commented Jan 10, 2025

@mgandharva: 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].

@gallacher gallacher removed the needs-triage Issue requires triage. label Jan 10, 2025
@mgandharva
Copy link
Contributor Author

link: ECS01E-29

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/csm-operator 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

4 participants