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]: CSM Operator does not calculate status correctly when module is deployed with driver #1122

Closed
JacobGros opened this issue Jan 30, 2024 · 0 comments
Labels
area/csm-operator type/bug Something isn't working. This is the default label associated with a bug issue.
Milestone

Comments

@JacobGros
Copy link

Bug Description

Operator is not correctly setting status when module is deployed with driver.

This statement: https://github.com/dell/csm-operator/blob/e135454e5df50402c4080ed94e5303228b869167/pkg/utils/status.go#L350

Will not work, resulting in only driver state being calculated.
This lead to a case where AM controller failed to deploy, but since driver deploy succeeded, we saw overall CSM status set to succeeded.

Logs

See JRIA

Screenshots

No response

Additional Environment Information

No response

Steps to Reproduce

  1. Set module to bad image
  2. Install driver and module as one CSM
  3. CSM status is set to "Succeeded", but module is not deployed

Expected Behavior

CSM status should be "failed"

CSM Driver(s)

PowerFlex, but issue is probably with all drivers

Installation Type

CSM Operator

Container Storage Modules Enabled

No response

Container Orchestrator

k8s 1.23

Operating System

rhel 8.6

@JacobGros JacobGros 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 and removed needs-triage Issue requires triage. labels Jan 30, 2024
@JacobGros JacobGros added this to the v1.9.1 milestone Jan 30, 2024
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

2 participants