Skip to content

deploy: update CSI sidecars to latest versions available#5421

Merged
mergify[bot] merged 5 commits intoceph:develfrom
iPraveenParihar:update-csi-sidecars-to-latest
Nov 13, 2025
Merged

deploy: update CSI sidecars to latest versions available#5421
mergify[bot] merged 5 commits intoceph:develfrom
iPraveenParihar:update-csi-sidecars-to-latest

Conversation

@iPraveenParihar
Copy link
Copy Markdown
Contributor

@iPraveenParihar iPraveenParihar commented Jul 10, 2025 •
edited
Loading

Describe what this PR does

  1. 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
  1. deploy: include '--automaxprocs' to csi sidecars

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.
Show available bot commands

These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

  • /retest ci/centos/<job-name>: retest the <job-name> after unrelated
    failure (please report the failure too!)

@iPraveenParihar iPraveenParihar self-assigned this Jul 10, 2025
@iPraveenParihar iPraveenParihar marked this pull request as ready for review July 10, 2025 06:45
Rakshith-R
Rakshith-R reviewed Jul 10, 2025
View reviewed changes
Comment thread charts/ceph-csi-rbd/README.md Outdated
| `nodeplugin.profiling.enabled` | Specifies whether profiling should be enabled | `false` |
| `nodeplugin.registrar.image.repository` | Node Registrar image repository URL | `registry.k8s.io/sig-storage/csi-node-driver-registrar` |
| `nodeplugin.registrar.image.tag` | Image tag | `v2.13.0` |
| `nodeplugin.registrar.image.tag` | Image tag | `v2.14.0` |
Copy link
Copy Markdown
Contributor

@Rakshith-R Rakshith-R Jul 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

attacher image needs to be updated in this file

Rakshith-R
Rakshith-R reviewed Jul 10, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@Rakshith-R Rakshith-R left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update image versions in ceph-csi-op repo too.

@iPraveenParihar iPraveenParihar force-pushed the update-csi-sidecars-to-latest branch from 527b9e5 to 14f1272 Compare July 10, 2025 07:19
@iPraveenParihar
Copy link
Copy Markdown
Contributor Author

iPraveenParihar commented Jul 10, 2025 •
edited
Loading

Please update image versions in ceph-csi-op repo too.

They were already updated here ceph/ceph-csi-operator#252.
Will have to update in Rook :)

@nixpanic
Copy link
Copy Markdown
Member

nixpanic commented Jul 10, 2025 •
edited
Loading

They were already updated here ceph/ceph-csi-operator#252. Will have to update in Rook :)

https://github.com/ceph/ceph-csi-operator/blob/90402565977568471446eb4992b08d033b6ce9eb/internal/controller/defaults.go#L34

csi-snapshotter still needs to be updated there.

Rakshith-R
Rakshith-R previously approved these changes Jul 10, 2025
View reviewed changes
@Rakshith-R Rakshith-R requested a review from a team July 11, 2025 07:00
Madhu-1
Madhu-1 previously approved these changes Jul 11, 2025
View reviewed changes
@Madhu-1
Copy link
Copy Markdown
Collaborator

Madhu-1 commented Jul 11, 2025

@Mergifyio queue

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Jul 11, 2025 •
edited
Loading

queue

🛑 The pull request has been removed from the queue default

Details

The merge conditions cannot be satisfied due to failing checks.

You can take a look at Queue: Embarked in merge queue check runs for more details about the failure.

mergify bot added a commit that referenced this pull request Jul 11, 2025
@mergify
Merge of #5421
dd79603
@mergify mergify bot mentioned this pull request Jul 11, 2025
Closed
78 tasks
@mergify mergify bot added the ok-to-test Label to trigger E2E tests label Jul 11, 2025
@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Jul 11, 2025

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Jul 11, 2025

/test ci/centos/k8s-e2e-external-storage/1.32

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Jul 11, 2025

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Jul 11, 2025

/test ci/centos/mini-e2e-helm/k8s-1.32

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Jul 11, 2025

/test ci/centos/k8s-e2e-external-storage/1.33

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Jul 11, 2025

/test ci/centos/mini-e2e/k8s-1.32

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Jul 11, 2025

/test ci/centos/mini-e2e-helm/k8s-1.33

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Jul 11, 2025

/test ci/centos/mini-e2e/k8s-1.33

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Jul 11, 2025

/test ci/centos/k8s-e2e-external-storage/1.31

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Jul 11, 2025

/test ci/centos/mini-e2e-helm/k8s-1.31

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Jul 11, 2025

/test ci/centos/mini-e2e/k8s-1.31

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Jul 11, 2025
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Jul 11, 2025

This pull request has been removed from the queue for the following reason: checks failed.

The merge conditions cannot be satisfied due to failing checks:

You can check the last failing draft PR here: #5424.

You may have to fix your CI before adding the pull request to the queue again.
If you update this pull request, to fix the CI, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio requeue comment.

@Madhu-1
Copy link
Copy Markdown
Collaborator

Madhu-1 commented Jul 11, 2025

@iPraveenParihar can you please check , looks like some real issue as all test failed.

@iPraveenParihar
deploy: update groupsnapshot.storage.k8s.io/v1beta1 to v1beta2
858d546 
With external-snapshotter v8.4.0 the v1beta1 VolumeGroupSnapshot API
is now marked as deprecated. and v1beta2 is introduced.

Signed-off-by: Praveen M <m.praveen@ibm.com>
@iPraveenParihar
e2e: update external-snapshotter client to v8.4.0
6c77928 
Signed-off-by: Praveen M <m.praveen@ibm.com>
@iPraveenParihar iPraveenParihar force-pushed the update-csi-sidecars-to-latest branch from bc8fc12 to 6c77928 Compare November 12, 2025 07:40
@iPraveenParihar
ci: add claimProvision timeout
8d29021 
Signed-off-by: Praveen M <m.praveen@ibm.com>
@iPraveenParihar
Copy link
Copy Markdown
Contributor Author

iPraveenParihar commented Nov 12, 2025

/test ci/centos/k8s-e2e-external-storage/1.34

@iPraveenParihar
Copy link
Copy Markdown
Contributor Author

iPraveenParihar commented Nov 12, 2025

@Madhu-1, the failing CI has passed. PTAL again

@Madhu-1
Copy link
Copy Markdown
Collaborator

Madhu-1 commented Nov 12, 2025

@iPraveenParihar lets rerun all the test again to ensure there is no flaky tests to avoid causing more problems after merging this PR?

@iPraveenParihar iPraveenParihar added the ok-to-test Label to trigger E2E tests label Nov 12, 2025
@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Nov 12, 2025

/test ci/centos/k8s-e2e-external-storage/1.34

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Nov 12, 2025

/test ci/centos/k8s-e2e-external-storage/1.32

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Nov 12, 2025

/test ci/centos/mini-e2e-helm/k8s-1.34

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Nov 12, 2025

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Nov 12, 2025

/test ci/centos/mini-e2e-helm/k8s-1.32

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Nov 12, 2025

/test ci/centos/k8s-e2e-external-storage/1.33

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Nov 12, 2025

/test ci/centos/mini-e2e/k8s-1.32

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Nov 12, 2025

/test ci/centos/mini-e2e/k8s-1.34

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Nov 12, 2025

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Nov 12, 2025

/test ci/centos/mini-e2e-helm/k8s-1.33

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Nov 12, 2025

/test ci/centos/mini-e2e/k8s-1.33

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Nov 12, 2025
@iPraveenParihar
Copy link
Copy Markdown
Contributor Author

iPraveenParihar commented Nov 12, 2025

@Madhu-1, all CI passed 🎉 .
PTAL thanks!

@mergify mergify bot added the queued label Nov 13, 2025
mergify bot added a commit that referenced this pull request Nov 13, 2025
@mergify
Merge of #5421
3de01b1
@mergify mergify bot mentioned this pull request Nov 13, 2025
Closed
78 tasks
@mergify mergify bot merged commit 75cb245 into ceph:devel Nov 13, 2025
40 checks passed
@mergify mergify bot removed the queued label Nov 13, 2025
@iPraveenParihar iPraveenParihar mentioned this pull request Nov 13, 2025
Closed
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Madhu-1 Madhu-1 Madhu-1 approved these changes

@Rakshith-R Rakshith-R Rakshith-R approved these changes

@black-dragon74 black-dragon74 black-dragon74 left review comments

Assignees

@iPraveenParihar iPraveenParihar

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@iPraveenParihar @nixpanic @Madhu-1 @ceph-csi-bot @Rakshith-R @black-dragon74