Skip to content

csi: update csi external sidecars to latest#357

Closed
iPraveenParihar wants to merge 2 commits into
ceph:mainfrom
iPraveenParihar:update-image-defaults
Closed

csi: update csi external sidecars to latest#357
iPraveenParihar wants to merge 2 commits into
ceph:mainfrom
iPraveenParihar:update-image-defaults

Conversation

@iPraveenParihar
Copy link
Copy Markdown
Contributor

Describe what this PR does

Below sidecars are updated with latest available versions

Sidecar Component Version
csi-attacher v4.10.0
csi-snapshotter v8.4.0
csi-resizer v2.0.0
csi-provisioner v6.0.0
csi-node-driver-registrar v2.15.0
csi-snapshot-metadata v0.2.0

Same updated in Ceph-CSI repo in PR ceph/ceph-csi#5421

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
  • Reviewed the developer guide on Submitting a Pull Request
  • Pending release notes updated with breaking and/or notable changes for the next major release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Integration tests have been added, if necessary.

Below sidecars are updated with latest available versions

csi-attacher: v4.10.0
csi-snapshotter: v8.4.0
csi-resizer: v2.0.0
csi-provisioner: v6.0.0
csi-node-driver-registrar: v2.15.0
csi-snapshot-metadata: v0.2.0

Signed-off-by: Praveen M <m.praveen@ibm.com>
@iPraveenParihar iPraveenParihar marked this pull request as ready for review November 13, 2025 09:11
@Madhu-1
Copy link
Copy Markdown
Collaborator

Madhu-1 commented Nov 13, 2025

@iPraveenParihar the second commit looks we got performance degradation in sidecar. Do we have any idea what is failing and why its failing?

@iPraveenParihar
Copy link
Copy Markdown
Contributor Author

@Madhu-1, with csi-provisioner v6.0.0 if the CreateVolume fails with InvalidArgument it will retry only after 5m interval (kubernetes-csi/external-provisioner#1369)

When csi driver is started/restarted it takes sometime for ceph-csi-config ConfigMap to be available in container, when not available at CreateVolume call ceph-csi will return InvalidArgument gRPC error.

Volume provisioning failed with infeasible error. Retries will be delayed. rpc error: code = InvalidArgument desc = failed to fetch monitor list using clusterID (a55f31b6-9dc5-496c-bc9f-f5e62b96bba5): missing configuration for cluster ID: "a55f31b6-9dc5-496c-bc9f-f5e62b96bba5"

@Madhu-1
Copy link
Copy Markdown
Collaborator

Madhu-1 commented Nov 13, 2025

@Madhu-1, with csi-provisioner v6.0.0 if the CreateVolume fails with InvalidArgument it will retry only after 5m interval (kubernetes-csi/external-provisioner#1369)

When csi driver is started/restarted it takes sometime for ceph-csi-config ConfigMap to be available in container, when not available at CreateVolume call ceph-csi will return InvalidArgument gRPC error.

Volume provisioning failed with infeasible error. Retries will be delayed. rpc error: code = InvalidArgument desc = failed to fetch monitor list using clusterID (a55f31b6-9dc5-496c-bc9f-f5e62b96bba5): missing configuration for cluster ID: "a55f31b6-9dc5-496c-bc9f-f5e62b96bba5"

Thanks for the link, can we set the retryIntervalMax to 30sec by default in the code?

@iPraveenParihar
Copy link
Copy Markdown
Contributor Author

yes, we can provide--retry-interval-max=30s to csi-provisioner.

@iPraveenParihar
Copy link
Copy Markdown
Contributor Author

iPraveenParihar commented Nov 13, 2025

Also, we need to add --automaxprocs=true as ceph-csi adds it by default now.

@Madhu-1
Copy link
Copy Markdown
Collaborator

Madhu-1 commented Nov 13, 2025

Also, we need to add --automaxprocs=true as ceph-csi adds it by default now.

This will cause problem if someoneuses old sidecars, we need dynamic functionality for this one (which we dont have today)

Signed-off-by: Praveen M <m.praveen@ibm.com>
@github-actions
Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.

@github-actions github-actions Bot added the stale label Dec 14, 2025
@github-actions
Copy link
Copy Markdown

This pull request has been automatically closed due to inactivity. Please re-open if these changes are still required.

@github-actions github-actions Bot closed this Dec 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants