Skip to content

feat: Allow to rotate all nodeplugins at once.#66

Open
aledegano wants to merge 2 commits intomasterfrom
max_surge_100%
Open

feat: Allow to rotate all nodeplugins at once.#66
aledegano wants to merge 2 commits intomasterfrom
max_surge_100%

Conversation

@aledegano
Copy link
Copy Markdown

When we roll-out an update to csi-rclone, currently,
the nodeplugins are deleted and updated one by one:
this makes the roll-out slow for no reason.

The nodeplugins can instead be all deleted and recreated
at the same time for a faster roll-out.

When we roll-out an update to csi-rclone, currently,
the nodeplugins are deleted and updated one by one:
this makes the roll-out slow for no reason.

The nodeplugins can instead be all deleted and recreated
at the same time for a faster roll-out.
@aledegano aledegano requested a review from a team as a code owner June 20, 2025 09:54
@leafty
Copy link
Copy Markdown
Member

leafty commented Jun 20, 2025

Let me see if we can test this 👍

Comment thread deploy/csi-rclone/templates/csi-nodeplugin-rclone.yaml
Copy link
Copy Markdown
Member

@olevski olevski left a comment

Choose a reason for hiding this comment

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

Alessandro this is good. If you rebase it, it will be good to go. We can also do "OnDelete" if you want and then you can roll out an update but manually delete all the pods when there is no usage. Or when they get restarted due to oom kills.

@olevski
Copy link
Copy Markdown
Member

olevski commented Feb 6, 2026

Also can you please target release 2.15.0 instead of going straight to main.

@aledegano
Copy link
Copy Markdown
Author

aledegano commented Feb 6, 2026

Also can you please target release 2.15.0 instead of going straight to main.

uhm... are you thinking about the Renku helm chart?...

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.

4 participants