Skip to content

Commit

Permalink
Extend operator and service K8S permissions for eager start handling (#…
Browse files Browse the repository at this point in the history
…86)

Allow operator to edit pods to add annotations for eager start.
This is necessary to update the OAuth proxy config map in time.

Allow service to read Deployments and ReplicaSets to trace owner refs.
Read access to Deployments and ReplicaSets allows the service to trace owner references from Pods over ReplicaSets and Deployments to the Pod's Session.
This is required to find a Pod's corresponding Session (and vice-versa) in eager start mode. This is necessary because the Pod's ENV cannot contain the session name in eager start mode.
  • Loading branch information
lucas-koehler authored Jan 14, 2025
1 parent 8ff00bc commit 3903368
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 2 deletions.
2 changes: 1 addition & 1 deletion charts/theia-cloud-base/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 1.1.0-next.0
version: 1.1.0-next.1

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
Expand Down
3 changes: 2 additions & 1 deletion charts/theia-cloud-base/templates/operator-role.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,4 +25,5 @@ rules:
- configmaps
- deployments
- leases
verbs: ["list", "create", "watch", "get", "patch", "delete", "update"]
- pods
verbs: ["list", "create", "watch", "get", "edit", "patch", "delete", "update"]
3 changes: 3 additions & 0 deletions charts/theia-cloud-base/templates/service-role.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,10 @@ rules:
verbs: ["list", "create", "watch", "get", "patch", "delete"]
- apiGroups:
- ""
- apps
- metrics.k8s.io
resources:
- deployments
- replicasets
- pods
verbs: ["list", "get", "watch"]

0 comments on commit 3903368

Please sign in to comment.