Skip to content

fix: Update csi-rclone to remove compromised certificates. - #4706

Merged
leafty merged 2 commits into
release-2.21.2from
release-2.21.2-update-csi-rclone
Oct 22, 2026
Merged

leafty merged 2 commits into
release-2.21.2from
release-2.21.2-update-csi-rclone

Conversation

@aledegano

@aledegano aledegano commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Fixes https://emudhra.we-have-a-problem.com/

/deploy #notest extra-values=global.csi-rclone.install=true,notebooks.cloudstorage.enabled=true,notebooks.cloudstorage.storageClass=csi-rclone-emudhra,csi-rclone.storageClassName=csi-rclone-emudhra,global.rcloneStorageClass=csi-rclone-emudhra,amalthea-sessions.rcloneStorageClass=csi-rclone-emudhra-secret-annotation,csi-rclone.csiNodepluginRclone.tolerations[0].effect=NoSchedule,csi-rclone.csiNodepluginRclone.tolerations[0].operator=Exists

@aledegano
aledegano requested a review from a team as a code owner October 21, 2026 14:01
@leafty

leafty commented Oct 21, 2026

Copy link
Copy Markdown
Member

Merge to release-2.21.2?

@RenkuBot

Copy link
Copy Markdown
Collaborator

You can access the deployment of this PR at https://ci-renku-4706.dev.renku.ch

@leafty leafty left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

csi-rclone helm chart uses renku/certificates:0.0.2 ❌

Note: confirmed on deployed deamonset pods:

root@ci-renku-4706-csi-rclone-nodeplugin-7mfgl:/# ls -al /etc/ssl/certs/ | grep emSi
lrwxrwxrwx 1 root root     35 Oct 21 15:02 14bc7599.0 -> ca-cert-emSign_ECC_Root_CA_-_G3.pem
lrwxrwxrwx 1 root root     31 Oct 21 15:02 2923b3f9.0 -> ca-cert-emSign_Root_CA_-_G1.pem
lrwxrwxrwx 1 root root     31 Oct 21 15:02 406c9bb1.0 -> ca-cert-emSign_Root_CA_-_C1.pem
lrwxrwxrwx 1 root root     35 Oct 21 15:02 4b718d9b.0 -> ca-cert-emSign_ECC_Root_CA_-_C3.pem
-rw-r--r-- 1 root root    814 Oct 21 15:02 ca-cert-emSign_ECC_Root_CA_-_C3.pem
-rw-r--r-- 1 root root    859 Oct 21 15:02 ca-cert-emSign_ECC_Root_CA_-_G3.pem
-rw-r--r-- 1 root root   1257 Oct 21 15:02 ca-cert-emSign_Root_CA_-_C1.pem
-rw-r--r-- 1 root root   1302 Oct 21 15:02 ca-cert-emSign_Root_CA_-_G1.pem

Only the G1 cert should be present, the other ones are parents of the compromised ones (same command on pods initialized with renku/certificates:0.0.3 have only G1).

@aledegano
aledegano changed the base branch from master to release-2.21.2 October 21, 2026 15:17
@leafty
leafty force-pushed the release-2.21.2-update-csi-rclone branch from ac5655c to 43e163b Compare October 22, 2026 09:11

@leafty leafty left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fixed now!

@leafty
leafty enabled auto-merge (squash) October 22, 2026 09:35
@aledegano

Copy link
Copy Markdown
Contributor Author

Fixed now!

Thank you! 🙏

@leafty
leafty merged commit 548ad66 into release-2.21.2 Oct 22, 2026
33 checks passed
@leafty
leafty deleted the release-2.21.2-update-csi-rclone branch October 22, 2026 09:43
@RenkuBot

Copy link
Copy Markdown
Collaborator

Tearing down the temporary RenkuLab deployment for this PR.

leafty added a commit that referenced this pull request Oct 22, 2026
* chore: create release 2.21.2

* feat: update renku/certificates to v0.0.3 (#4704)

* build: upgrade csi-rclone to v0.7.2 (#4706)

Fixes https://emudhra.we-have-a-problem.com/

---------

Co-authored-by: Flora Thiebaut <flora.thiebaut@sdsc.ethz.ch>

* build: upgrade amalthea to v0.31.1 (#4708)

* build: upgrade renku-ui to v4.32.1 (#4707)

* build: upgrade renku-frontend-buildpacks to v0.12.1 (#4709)

* fix: Update remaining init images. (#4705)

Fixes https://emudhra.we-have-a-problem.com/ for
 - init-realm
 - init-db
 - platform-init

---------

Co-authored-by: Flora Thiebaut <flora.thiebaut@sdsc.ethz.ch>
Co-authored-by: Alessandro Degano <40891147+aledegano@users.noreply.github.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.

3 participants