Skip to content

remove the default replicas from validation#538

Open
Madhu-1 wants to merge 1 commit into
ceph:mainfrom
Madhu-1:fix-532
Open

remove the default replicas from validation#538
Madhu-1 wants to merge 1 commit into
ceph:mainfrom
Madhu-1:fix-532

Conversation

@Madhu-1

@Madhu-1 Madhu-1 commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator

Removing the default replica count from the CRD to ensure that we can still allow the user to deploy single controller plugin deployment on the single node cluster.

fixes: #532

Scenario Expected Replicas Actual Replicas Result
Driver CR without replicas, no OperatorConfig 1 (node-count capped) 1 Pass
Driver CR without replicas, OperatorConfig replicas: 2 2 (propagated from OperatorConfig) 2 Pass
Driver CR with explicit replicas: 3 3 (user override wins) 3 Pass

@Madhu-1

Madhu-1 commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator Author

@travisn PTAL all the cases should work , the only issue was with default value in validation

Removing the default replica count from the
CRD to ensure that we can still allow the user
to deploy single controller plugin deployment
on the single node cluster.

fixes: ceph#532

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

due to default replica count, on single node the csi deployment is not created with replica 1

1 participant