Skip to content

Update 50-remote.md#4383

Closed
sambuc wants to merge 8 commits intomasterfrom
feat/remote-clusters
Closed

Update 50-remote.md#4383
sambuc wants to merge 8 commits intomasterfrom
feat/remote-clusters

Conversation

@sambuc
Copy link
Copy Markdown
Contributor

@sambuc sambuc commented Mar 3, 2026

Document Remote clusters configuration

Comment thread docs/docs/20-admins/30-operation/50-remote.md Outdated
Co-authored-by: Alessandro Degano <40891147+aledegano@users.noreply.github.com>

_If you already have automatic volume provisioning setup in your cluster, you may skip this._

For example, to install cinder-csi:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Remove the example as it is:

  1. Very specific
  2. Probably not complete, as it is missing authentication necessary for the node plugin to cinder

Instead, I would suggest to simply list a few, example, storageclasses commonly used (cinder-csi, azure etc...)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

As discussed yesterday, I just removed the example, as this is too specific to the cloud provider, and it makes no sense to document multiple ways of achieving the same things.

The cloud provider documentation should take care of that, and we won't be able to keep up to date with these anyway.

- [ ] Dedicated **Namespace** for the user sessions (for example `renku-user-sessions`)
- [ ] Dedicated **ServiceAccount** (for example `renku-session-manager`) with the following rights

```yaml
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This role, along with a SA and the RoleBinding should go in the chart, no reason for the admins to have to create it out-of-band.

Copy link
Copy Markdown
Contributor Author

@sambuc sambuc Mar 13, 2026

Choose a reason for hiding this comment

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

Applied in SwissDataScienceCenter/amalthea#1088, and adapted the documentation here.

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.

The problem is that this has very little to do with either Amalthea or the full Renku deployment. So it can go in a separate helm chart. But for now the easiest is to manage it out of band.

Here as an example on how to create one, adapt as required:

- ```bash
kubectl create priorityclass renku-user-sessions-priority --value=1000 --description="default priority for renku sessions"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should go in the helm chart, possibly with opt-out, no need to have admins manage this out-of-band.

Copy link
Copy Markdown
Contributor Author

@sambuc sambuc Mar 13, 2026

Choose a reason for hiding this comment

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

Applied in SwissDataScienceCenter/amalthea#1088, and updated the documentation

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.

Lets just start with managing this out of band. We can adapt it later. With the elimination of CSI rclone things will be easier. And if we add this to the amalthea helm chart then we need to document really well what it is for.

sambuc added 2 commits March 13, 2026 10:47
Address comment by removing the example, as this is too specific to the cloud provider.
comments about priority class and the service account, tied to SwissDataScienceCenter/amalthea#1088
@sambuc sambuc marked this pull request as ready for review March 13, 2026 16:25
@sambuc sambuc requested a review from a team as a code owner March 13, 2026 16:25
@sambuc sambuc closed this Mar 30, 2026
@sambuc sambuc deleted the feat/remote-clusters branch May 1, 2026 06:10
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