Skip to content

rebase: update Kubernetes CSI sidecars to current versions #5088

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 20, 2025
Merged

rebase: update Kubernetes CSI sidecars to current versions #5088

merged 1 commit into from
Jan 20, 2025

Conversation

@nixpanic
Copy link
Member

@nixpanic nixpanic commented Jan 17, 2025 •
edited
Loading

The Kubernetes CSI sidecars have had several releases that were not
included in deployments by Rook yet, update them to the versions that
are available today:

  • csi-node-driver-registrar:v2.13.0
  • csi-provisioner:v5.1.0
  • csi-attacher:v4.8.0

@mergify mergify bot added the rebase update the version of an external component label Jan 17, 2025
@Madhu-1 Madhu-1 requested a review from a team January 17, 2025 09:58
@iPraveenParihar
Copy link
Contributor

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

@iPraveenParihar
Copy link
Contributor

@nixpanic, new release just came in for external-provisioner 😄 https://github.com/kubernetes-csi/external-provisioner/releases/tag/v5.2.0

@iPraveenParihar
Copy link
Contributor

Also, lets update snapshotter version here as well?

ceph-csi/scripts/install-snapshot.sh

Lines 10 to 14 in 0cfb2b0

# shellcheck disable=SC1091
[ ! -e "${SCRIPT_DIR}"/utils.sh ] || source "${SCRIPT_DIR}"/utils.sh
SNAPSHOT_VERSION=${SNAPSHOT_VERSION:-"v5.0.1"}

@nixpanic
Copy link
Member Author

@nixpanic, new release just came in for external-provisioner 😄 https://github.com/kubernetes-csi/external-provisioner/releases/tag/v5.2.0

Haha, nice. The container image is not available yet though...

@nixpanic
Copy link
Member Author

Also, lets update snapshotter version here as well?

ceph-csi/scripts/install-snapshot.sh

Lines 10 to 14 in 0cfb2b0

# shellcheck disable=SC1091
[ ! -e "${SCRIPT_DIR}"/utils.sh ] || source "${SCRIPT_DIR}"/utils.sh
SNAPSHOT_VERSION=${SNAPSHOT_VERSION:-"v5.0.1"}

This PR does not update the snapshot sidecar. There is no need for changing the script, as SNAPSHOT_VERSION=v8.2.0 is set in build.env already, so it overloads the variable in the script.

@nixpanic
Copy link
Member Author

@Mergifyio queue

@mergify
Copy link
Contributor

mergify bot commented Jan 20, 2025 •
edited
Loading

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at bd962e5

@nixpanic
rebase: update Kubernetes CSI sidecars to current versions
5fb3db8 
The Kubernetes CSI sidecars have had several releases that were not
included in deployments by Rook yet, update them to the versions that
are available today:

- csi-node-driver-registrar:v2.13.0
- csi-provisioner:v5.1.0
- csi-attacher:v4.8.0

Signed-off-by: Niels de Vos <[email protected]>
@nixpanic nixpanic force-pushed the rebase/kubernetes-csi-sidecars/20250117 branch from 1999581 to 5fb3db8 Compare January 20, 2025 08:49
@mergify mergify bot added the ok-to-test Label to trigger E2E tests label Jan 20, 2025
@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Jan 20, 2025
@mergify mergify bot merged commit bd962e5 into ceph:devel Jan 20, 2025
37 checks passed
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

@iPraveenParihar iPraveenParihar iPraveenParihar approved these changes

Assignees

No one assigned

Labels

rebase update the version of an external component

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@nixpanic @iPraveenParihar @ceph-csi-bot @Madhu-1