Add shared cache node helm chart by KevinGruber2001 · Pull Request #16 · EduIDE/EduIDE-deployment

KevinGruber2001 · 2025-11-16T12:54:56Z

No description provided.

coderabbitai · 2025-11-16T12:55:02Z

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: bf1df7d9-ebc4-4f39-b9db-acb5a8d2e436

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch feat/15-integrate-shared-cache-node-into-theia-cloud-combined

📝 Coding Plan

Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Tip

You can customize the tone of the review comments and chat replies.

Configure the tone_instructions setting to customize the tone of the review comments and chat replies. For example, you can set the tone to Act like a strict teacher, Act like a pirate and more.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

This PR adds a shared Gradle build cache node using a new theia-shared-cache Helm subchart. It configures the cache for the test2 test environment and adds the necessary TLS infrastructure (internal CA, cache certificates, trust bundle), Grafana dashboards, and Prometheus service monitors. A reference to a Reposilite Maven proxy monitoring setup is also added.

Changes:

Adds theia-shared-cache as a Helm subchart dependency to theia-cloud-combined, with cache-related TLS certificates and trust bundles in theia-certificates
Adds Prometheus ServiceMonitor and Grafana dashboards for both the shared cache and Reposilite
Enables caching (enableCaching, cacheUrl) in the operator config for test2 and in the reference values file, and adds a cache host entry across all deployment environments

Reviewed changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
`charts/theia-cloud-combined/Chart.yaml`	Adds `theia-shared-cache` as a subchart dependency
`charts/theia-cloud-combined/values.yaml`	Adds `cache` host and `theia-shared-cache` disabled-by-default config
`charts/theia-certificates/templates/cache-certificate.yml`	New public TLS certificate for external cache access
`charts/theia-certificates/templates/cache-internal-certificate.yml`	New internal TLS certificate for in-cluster cache comms
`charts/theia-certificates/templates/internal-ca.yml`	New internal CA + ClusterIssuer for self-signed internal certs
`charts/theia-certificates/templates/trust-bundle.yml`	New trust-manager Bundle to distribute the internal CA to target namespaces
`charts/theia-certificates/values.yaml`	Adds `cache` host config to values
`charts/theia-monitoring/values.yaml`	Adds `sharedCacheNamespaces` and `reposiliteNamespaces` values
`charts/theia-monitoring/templates/servicemonitor-cache.yaml`	New Prometheus ServiceMonitor for the cache
`charts/theia-monitoring/templates/servicemonitor-reposilite.yaml`	New Prometheus ServiceMonitor for Reposilite
`charts/theia-monitoring/templates/dashboard-cache.yaml`	New Grafana dashboard for cache metrics
`charts/theia-monitoring/templates/dashboard-reposilite.yaml`	New Grafana dashboard for Reposilite metrics
`deployments/test2.theia-test.artemis.cit.tum.de/values.yaml`	Enables the shared cache and configures caching for test2
`deployments/test1.theia-test.artemis.cit.tum.de/values.yaml`	Adds `cache` host entry
`deployments/test3.theia-test.artemis.cit.tum.de/values.yaml`	Not updated — missing `cache` host entry (unchanged, not in PR)
`deployments/theia.artemis.cit.tum.de/values.yaml`	Adds `cache` host entry
`deployments/theia-staging.artemis.cit.tum.de/values.yaml`	Adds `cache` host entry
`value-reference-files/theia-cloud-helm-values.yml`	Adds `enableCaching` and `cacheUrl` to reference config

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-07T10:49:06Z

charts/theia-monitoring/templates/dashboard-cache.yaml

+            "current": { "selected": false, "text": "test2", "value": "test2" },
+            "hide": 0,
+            "includeAll": false,
+            "label": "Namespace",
+            "multi": false,
+            "name": "namespace",
+            "options": [
+              { "selected": true, "text": "test2", "value": "test2" }
+            ],
+            "query": "test2",
+            "skipUrlSync": false,
+            "type": "custom"


The Grafana dashboard templates for both dashboard-cache.yaml and dashboard-reposilite.yaml hardcode the namespace dropdown variable to only the test2 namespace (see the templating.list section near the end of each file). This means when the dashboards are deployed to production environments (e.g., theia, theia-staging), the namespace selector will only offer test2 as an option. Unlike the existing dashboard-theiacloud.yaml, which uses Helm templating for the query and options fields of the namespace variable, these new dashboards embed the namespace list directly as static JSON. Consider using Prometheus label_values() queries (as done in dashboard-session-startup.yaml) or dynamically templating the namespace options so the dashboards are useful across all environments.

Suggested change

"current": { "selected": false, "text": "test2", "value": "test2" },

"hide": 0,

"includeAll": false,

"label": "Namespace",

"multi": false,

"name": "namespace",

"options": [

{ "selected": true, "text": "test2", "value": "test2" }

],

"query": "test2",

"skipUrlSync": false,

"type": "custom"

"current": {},

"datasource": null,

"definition": "label_values(kube_pod_info, \"namespace\")",

"hide": 0,

"includeAll": false,

"label": "Namespace",

"multi": false,

"name": "namespace",

"options": [],

"query": "label_values(kube_pod_info, \"namespace\")",

"refresh": 2,

"skipUrlSync": false,

"sort": 1,

"type": "query"

Copilot · 2026-03-07T10:49:06Z

charts/theia-certificates/templates/cache-internal-certificate.yml

+    - theia-cloud-combined-cache
+    - theia-cloud-combined-cache.theia-cloud.svc.cluster.local


The cache-internal-certificate.yml is deployed as a cluster-scoped resource but contains no namespace-aware templating. Its dnsNames hardcode theia-cloud-combined-cache and theia-cloud-combined-cache.theia-cloud.svc.cluster.local. However, when the theia-shared-cache subchart is installed as part of theia-cloud-combined, the actual Kubernetes service name is generated from the parent chart name and subchart name (i.e., tum-theia-cloud-theia-shared-cache or similar), not theia-cloud-combined-cache. Additionally, the deployment namespace may not be theia-cloud. This certificate will likely not match the actual service DNS names, causing TLS verification failures. Verify the exact service name generated by the subchart and update the dnsNames accordingly.

Suggested change

- theia-cloud-combined-cache

- theia-cloud-combined-cache.theia-cloud.svc.cluster.local

- {{ printf "%s-%s" .Release.Name "theia-shared-cache" | quote }}

- {{ printf "%s-%s.%s.svc.cluster.local" .Release.Name "theia-shared-cache" .Release.Namespace | quote }}

Copilot · 2026-03-07T10:49:07Z

value-reference-files/theia-cloud-helm-values.yml

  sessionsPerUser: 10
  storageClassName: csi-rbd-sc
+  enableCaching: true
+  cacheUrl: "http://theia-shared-cache:5071"


The cacheUrl in the value reference file uses http://theia-shared-cache:5071 (HTTP, port 5071, no /cache/ path suffix), while the actual test2 deployment uses https://theia-cloud-combined-cache:8080/cache/ (HTTPS, port 8080, with /cache/ path suffix). These two values are inconsistent — they differ in protocol, hostname, port, and path. Since the reference file is used as documentation for how to configure caching, this inconsistency could mislead users. The reference file should match the pattern used in the working deployment, or add a comment explaining that the URL format depends on whether TLS is enabled and on the service name specific to the deployment.

Suggested change

cacheUrl: "http://theia-shared-cache:5071"

# URL of the cache service; adjust protocol/host/port/path for your environment.

cacheUrl: "https://theia-cloud-combined-cache:8080/cache/"

Copilot · 2026-03-07T10:49:07Z

value-reference-files/theia-cloud-helm-values.yml


 operator:
-  image: theiacloud/theia-cloud-operator:1.1.0-next
+  image: ghcr.io/ls1intum/theia/operator:pr-41


The operator image is set to a PR-specific tag ghcr.io/ls1intum/theia/operator:pr-41 in the reference file. Reference/example files should not contain PR-specific image tags, as these are ephemeral and will become unavailable once the PR is merged or closed. This should be updated to a stable tag such as latest or a pinned release version.

Suggested change

image: ghcr.io/ls1intum/theia/operator:pr-41

image: ghcr.io/ls1intum/theia/operator:latest

Copilot · 2026-03-07T10:49:07Z

charts/theia-certificates/templates/trust-bundle.yml

+            - test2
+            - theia
+            - theia-staging
+            - test1
+            - test3


The trust-bundle.yml hardcodes the target namespace list and does not use Helm templating, so it cannot adapt to different deployment environments when the chart is installed. If this chart is installed for the theia.artemis.cit.tum.de deployment (which doesn't have its own trust-bundle.yml override), the same hardcoded list including test2, theia, theia-staging, test1, and test3 will be applied. While this may be intentional since the Bundle is cluster-scoped and serves all environments, it means any new environment added to the cluster requires a manual update to this file. More critically, if this chart is managed per-deployment, the single hardcoded list could cause conflicts (duplicate Bundle names). Consider whether the namespace list should be driven by a templated .Values reference like other similar resources in the codebase.

Suggested change

- test2

- theia

- theia-staging

- test1

- test3

{{- toYaml .Values.trustBundle.targetNamespaces | nindent 12 }}

Copilot · 2026-03-07T10:49:07Z

charts/theia-monitoring/templates/dashboard-reposilite.yaml

+      "templating": {
+        "list": [
+          {
+            "current": { "selected": false, "text": "test2", "value": "test2" },
+            "hide": 0,
+            "includeAll": false,
+            "label": "Namespace",
+            "multi": false,
+            "name": "namespace",
+            "options": [
+              { "selected": true, "text": "test2", "value": "test2" }
+            ],
+            "query": "test2",
+            "skipUrlSync": false,
+            "type": "custom"
+          }
+        ]


Same issue as in dashboard-cache.yaml: the namespace variable in the templating.list section hardcodes only test2 as both the static option and the default query value. This makes the dashboard non-functional in other environments. Consider using a dynamic label_values() query or Helm template-driven namespace options to support all environments.

Copilot · 2026-03-07T10:49:07Z

charts/theia-cloud-combined/Chart.yaml

+  - name: theia-shared-cache
+    version: "0.2.3-pr.24"
+    repository: "oci://ghcr.io/ls1intum/charts"


The Chart.lock file has not been updated to include the new theia-shared-cache dependency. After adding a new dependency to Chart.yaml, helm dependency update must be run to regenerate Chart.lock so that the new dependency (theia-shared-cache version 0.2.3-pr.24) is recorded. Without this, helm dependency build will fail or may use an incorrect/stale lock file.

Copilot · 2026-03-07T10:49:08Z

charts/theia-cloud-combined/Chart.yaml

    repository: "https://ls1intum.github.io/theia-workspace-garbage-collector/"
+
+  - name: theia-shared-cache
+    version: "0.2.3-pr.24"


The theia-shared-cache subchart version is 0.2.3-pr.24, which is a PR-specific pre-release version. Using a PR-specific version in the Chart.yaml means the chart depends on a transient artifact that will eventually become unavailable. This should be updated to a stable, released version once the upstream PR is merged, before merging this PR into the production branch.

Suggested change

version: "0.2.3-pr.24"

version: "0.2.3"

Copilot

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 5 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-09T14:19:39Z

charts/theia-monitoring/templates/servicemonitor-cache.yaml

+      scheme: https
+      tlsConfig:
+        insecureSkipVerify: true


The servicemonitor-cache.yaml uses scheme: https with insecureSkipVerify: true for scraping the cache server's metrics endpoint. While TLS is enabled on the cache server, skipping TLS verification defeats the security purpose of using TLS. Prometheus should be configured to use the internal CA trust bundle (the theia-internal-trust bundle distributed via trust-manager) to properly verify the server certificate instead of disabling verification entirely.

Copilot · 2026-03-09T14:19:40Z

charts/theia-monitoring/templates/dashboard-cache.yaml

+        "list": [
+          {
+            "current": { "selected": false, "text": "test2", "value": "test2" },
+            "hide": 0,
+            "includeAll": false,
+            "label": "Namespace",
+            "multi": false,
+            "name": "namespace",
+            "options": [
+              { "selected": true, "text": "test2", "value": "test2" }
+            ],
+            "query": "test2",
+            "skipUrlSync": false,
+            "type": "custom"
+          }
+        ]


The Grafana dashboard definitions for both shared-cache and Reposilite have the namespace template variable hardcoded to "test2" as its only option and default. If the theia-monitoring chart is deployed cluster-wide for multiple environments (as stated by the comment "This chart is installed ONCE per cluster"), these dashboards will only show data for test2 unless the namespace variable is updated. Consider using a Prometheus-based label-values query (using the label_values() query type in Grafana) to dynamically populate the namespace dropdown, so the dashboard works across all monitored namespaces without manual updates.

Copilot · 2026-03-09T14:19:41Z

charts/theia-monitoring/templates/dashboard-reposilite.yaml

+          {
+            "current": { "selected": false, "text": "test2", "value": "test2" },
+            "hide": 0,
+            "includeAll": false,
+            "label": "Namespace",
+            "multi": false,
+            "name": "namespace",
+            "options": [
+              { "selected": true, "text": "test2", "value": "test2" }
+            ],
+            "query": "test2",
+            "skipUrlSync": false,
+            "type": "custom"
+          }
+        ]
+      },


Same hardcoded namespace issue as in dashboard-cache.yaml: the namespace template variable has only "test2" as its only option and default value. This means the Reposilite dashboard will only show metrics for test2. The namespace list should be made dynamic or driven from a template variable populated via a Prometheus query.

Copilot · 2026-03-09T14:19:41Z

charts/theia-cloud-combined/Chart.yaml

+  - name: theia-shared-cache
+    version: 0.3.1
+    repository: "oci://ghcr.io/eduide/charts"


The theia-shared-cache dependency has been added to Chart.yaml, but the Chart.lock file has not been updated to include this new dependency. The Chart.lock still only lists the original four dependencies without theia-shared-cache. This means helm dependency update must be run to regenerate the lock file; otherwise, Helm will fail to correctly resolve and download the new sub-chart during deployment. Please run helm dependency update charts/theia-cloud-combined to regenerate Chart.lock.

Copilot · 2026-03-09T14:19:41Z

charts/theia-certificates/templates/cache-internal-certificate.yml

+apiVersion: cert-manager.io/v1
+kind: Certificate
+metadata:
+  name: cache-internal-cert
+spec:
+  secretName: cache-internal-cert-secret
+  issuerRef:
+    name: theia-internal-ca-issuer
+    kind: ClusterIssuer
+  commonName: "theia-shared-cache"
+  dnsNames:
+    - theia-cloud-combined-cache
+    - theia-cloud-combined-cache.{{ .Release.Namespace }}.svc.cluster.local
+  privateKey:
+    rotationPolicy: Never


The cache-internal-certificate.yml in the theia-certificates chart is not conditional (no if/enabled guard), meaning the certificate and its dependency on theia-internal-ca-issuer will always be applied for every environment that uses theia-certificates. The theia-internal-ca-issuer ClusterIssuer is likewise always created. However, the cache is only enabled in the test2 deployment (theia-shared-cache.enabled: true). For environments where the cache is not deployed, this certificate will be generated but unused, and any environments that don't have the theia-internal-ca-issuer ClusterIssuer pre-existing (i.e., because this is also a new addition) will now have it created for all environments regardless of cache being used.

Copilot

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-09T16:56:38Z

charts/theia-cloud-combined/Chart.yaml

  - name: theia-cloud
    version: 1.4.0-next.0
-    repository: "https://ls1intum.github.io/theia-cloud-helm"
+    repository: "https://eduide.github.io/EduIDE-helm"


The theia-cloud chart repository in Chart.yaml was changed from https://ls1intum.github.io/theia-cloud-helm to https://eduide.github.io/EduIDE-helm. This is a significant upstream repository change that affects where the theia-cloud dependency is fetched from. The Chart.lock still references the old repository URL, which further confirms the lock file is out of date (see related comment). Additionally, the workflow in deploy-theia.yml at line 140-142 still patches the Chart.yaml looking for the old URL patterns (e.g., https://ls1intum.github.io/theia-cloud-helm and https://eclipse-theia.github.io/theia-cloud-helm/), but won't find the new URL https://eduide.github.io/EduIDE-helm, meaning the branch-override logic in the CI workflow will no longer work correctly when helm_chart_branch is specified.

Copilot · 2026-03-09T16:56:39Z

charts/theia-monitoring/templates/dashboard-reposilite.yaml

+      "templating": {
+        "list": [
+          {
+            "current": { "selected": false, "text": "test2", "value": "test2" },
+            "hide": 0,
+            "includeAll": false,
+            "label": "Namespace",
+            "multi": false,
+            "name": "namespace",
+            "options": [
+              { "selected": true, "text": "test2", "value": "test2" }
+            ],
+            "query": "test2",
+            "skipUrlSync": false,
+            "type": "custom"
+          }
+        ]


The Grafana dashboard namespace variable for dashboard-reposilite.yaml uses a hardcoded "type": "custom" with a fixed "query": "test2" and a single option "test2". This is inconsistent with the existing patterns in the codebase (see dashboard-session-startup.yaml which uses a dynamic label_values() query, and dashboard-theiacloud.yaml which uses a static comma-separated list of all namespaces). This makes the dashboard usable only in the test2 namespace without modification.

Copilot · 2026-03-09T16:56:39Z

.github/workflows/deploy-theia.yml

    permissions:
      contents: read
-    runs-on: arc-runner-set-stateless
+    runs-on: ubuntu-latest


The runner was changed from arc-runner-set-stateless (a custom self-hosted runner) to ubuntu-latest (a GitHub-hosted runner). The deployment workflow uses a KUBECONFIG secret to connect to the Kubernetes cluster. Running on GitHub-hosted runners means that Kubernetes cluster access must be routed through GitHub's infrastructure rather than from a trusted internal runner. Verify that this is intentional and that the network routing from GitHub-hosted runners to your cluster is properly secured and allowed.

Copilot

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-09T17:56:14Z

charts/theia-monitoring/templates/servicemonitor-reposilite.yaml

+stringData:
+  username: "prometheus"
+  password: "prometheus"


The servicemonitor-reposilite.yaml embeds the Reposilite Prometheus credentials (username: "prometheus", password: "prometheus") as plaintext in the YAML file. Even though the comment states the metrics endpoint doesn't need to be secure, having literal credentials committed to the repository (and rendered as Kubernetes Secrets) is a security anti-pattern. If the credentials ever change, they would be in source control history. Consider using a Helm value with a secret (e.g., {{ .Values.reposilite.metricsPassword }}) that can be supplied at deploy time.

Copilot · 2026-03-09T17:56:14Z

.github/workflows/deploy-theia.yml

    name: Install Theia Cloud Helm Chart
    permissions:
      contents: read
-    runs-on: arc-runner-set-stateless
+    runs-on: ubuntu-latest
    # Link to GitHub Environment for secrets and protection rules
    environment: ${{ inputs.environment }}



The Patch Chart.yaml step in this workflow uses sed patterns to replace the theia-cloud chart repository URL with a local path when helm_chart_branch is specified. Those patterns match ls1intum.github.io/theia-cloud-helm and eclipse-theia.github.io/theia-cloud-helm. However, Chart.yaml now sets the repository for theia-cloud to "https://eduide.github.io/EduIDE-Helm" — none of the existing sed patterns match this new URL. As a result, the Patch Chart.yaml step will silently do nothing when helm_chart_branch is provided, and the deployment will use the remote EduIDE chart instead of the local branch. The sed patterns should be updated to include the new URL.

Copilot · 2026-03-09T17:56:15Z

charts/theia-certificates/templates/internal-ca.yml

+apiVersion: cert-manager.io/v1
+kind: Certificate
+metadata:
+  name: theia-internal-ca
+spec:
+  isCA: true
+  commonName: theia-internal-ca
+  secretName: theia-internal-ca-secret
+  issuerRef:
+    name: theia-cloud-selfsigned-issuer
+    kind: ClusterIssuer
+  privateKey:
+    algorithm: ECDSA
+    size: 256
+  duration: 87600h    # 10 years
+  renewBefore: 8760h  # renew 1 year before expiry
+---
+apiVersion: cert-manager.io/v1
+kind: ClusterIssuer
+metadata:
+  name: theia-internal-ca-issuer
+spec:
+  ca:
+    secretName: theia-internal-ca-secret


The internal-ca.yml creates a theia-internal-ca Certificate and theia-internal-ca-issuer ClusterIssuer without any Helm conditional ({{- if ... }}). These are cluster-scoped/global resources. If the theia-certificates chart is deployed for multiple environments (test1, test2, test3, theia, theia-staging), re-deploying it in a second namespace will attempt to re-create the same ClusterIssuer and Certificate (by the same name), which will either conflict or silently overwrite. Consider wrapping these with a Helm conditional (e.g., {{- if .Values.internalCA.enabled }}) or documenting that this chart must only be installed once per cluster.

…branch

Aligns with EduIDE-Helm PR #11 which adds v1beta11 support. With helm_chart_tag=pr-11, CI will now resolve theia-cloud-crds:1.2.0-next.1.pr-11 from GHCR.

Points theia-cloud-crds at the eduide-fork conversion-webhook image (conversion-webhook:pr-70) which includes the v1beta11 AppDefinition mapper required for the sidecar redesign.

theia-no-ls:pr-46 and langserver-java:pr-46 no longer exist in the registry. The no-ls AppDefinitions reference images from ls1intum/theia directly, so these preloading entries served no purpose.

…ollector

- preserve explicit mountWorkspace=false in Helm template (avoid default overriding false) - support sidecar imageTag/defaultImageTag fallback when sidecar image has no explicit tag - set landing default app to a defined sidecar app and remove stale additionalApps entries - remove hardcoded latest tags in sidecar appdefs to honor defaultImageTag overrides - preload no-ls IDE + sidecar langserver images used by test3

- replace ls1intum/theia image references with ghcr.io/eduide/eduide for no-ls IDE and sidecar images - align test3 appdefinitions and preloading with PR-tagged image pipeline

- add explicit workflow overrides for preloading image indices 7..10 - ensure java-17-no-ls/rust-no-ls and langserver-java/langserver-rust track IDE tag input

…15-integrate-shared-cache-node-into-theia-cloud-combined

KevinGruber2001 linked an issue Nov 16, 2025 that may be closed by this pull request

Integrate Shared cache node into theia-cloud-combined #15

Open

KevinGruber2001 temporarily deployed to test1 November 16, 2025 12:55 — with GitHub Actions Inactive

KevinGruber2001 temporarily deployed to test1 November 16, 2025 16:26 — with GitHub Actions Inactive

KevinGruber2001 temporarily deployed to test1 November 16, 2025 17:15 — with GitHub Actions Inactive

add environment

f120cbd

KevinGruber2001 had a problem deploying to test1 November 18, 2025 14:00 — with GitHub Actions Failure

KevinGruber2001 had a problem deploying to test1 November 18, 2025 14:02 — with GitHub Actions Failure

KevinGruber2001 temporarily deployed to test1 November 18, 2025 14:12 — with GitHub Actions Inactive

KevinGruber2001 and others added 8 commits November 18, 2025 15:28

Update deployments/test2.theia-test.artemis.cit.tum.de/values.yaml

7df870e

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update deployments/test2.theia-test.artemis.cit.tum.de/values.yaml

3921349

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

add test2 to workflow

d2ac8e7

Add shared cache node helm chart

ca5e4f5

Change storage class

62febb9

Increase Cache Version number

f4e5a6f

set opertator

6a42e81

update shared cache version number

d459ac9

KevinGruber2001 force-pushed the feat/15-integrate-shared-cache-node-into-theia-cloud-combined branch from f3474a5 to d459ac9 Compare November 18, 2025 16:13

KevinGruber2001 temporarily deployed to test2 November 18, 2025 16:13 — with GitHub Actions Inactive

KevinGruber2001 had a problem deploying to test2 November 18, 2025 16:15 — with GitHub Actions Failure

KevinGruber2001 had a problem deploying to test1 November 18, 2025 16:15 — with GitHub Actions Failure

update shared cache chart and add ui values

54a63cf

KevinGruber2001 temporarily deployed to test2 November 25, 2025 17:14 — with GitHub Actions Inactive

KevinGruber2001 added 2 commits November 25, 2025 18:31

create certificate for cache

e6eb229

add missing cache reference

84149d2

KevinGruber2001 had a problem deploying to test2 November 25, 2025 17:34 — with GitHub Actions Failure

update shared cache version

f7ce461

KevinGruber2001 temporarily deployed to test2 November 25, 2025 17:54 — with GitHub Actions Inactive

add test2 configureation for cache

e7518e0

KevinGruber2001 temporarily deployed to test2 November 25, 2025 18:08 — with GitHub Actions Inactive

minor grafana fix

1830714

KevinGruber2001 temporarily deployed to test2 February 9, 2026 14:23 — with GitHub Actions Inactive

merge main

9ffb433

Copilot AI review requested due to automatic review settings March 7, 2026 10:44

KevinGruber2001 requested a deployment to test3 March 7, 2026 10:44 — with GitHub Actions Waiting

Copilot AI reviewed Mar 7, 2026

View reviewed changes

KevinGruber2001 added 3 commits March 7, 2026 12:02

minor fixes

1acb48d

update org

36610c9

minor syntax fix

4567c8d

Copilot AI reviewed Mar 9, 2026

View reviewed changes

KevinGruber2001 added 2 commits March 9, 2026 17:42

update eduide url

80c5f00

temporarly use ubutu runners

e197807

Copilot AI reviewed Mar 9, 2026

View reviewed changes

KevinGruber2001 added 2 commits March 9, 2026 17:59

minor error

ae18595

add reposilite metrics secret

8a84d02

Copilot AI reviewed Mar 9, 2026

View reviewed changes

KevinGruber2001 and others added 14 commits March 9, 2026 21:36

add operator arguments

3f16f1e

Migrate test3 to AppDefinition v1beta11 sidecars on clean main-based …

a56f738

…branch

ci: bump theia-cloud-crds base version to 1.2.0-next.1

ddd6a0e

Aligns with EduIDE-Helm PR #11 which adds v1beta11 support. With helm_chart_tag=pr-11, CI will now resolve theia-cloud-crds:1.2.0-next.1.pr-11 from GHCR.

ci(test3): override conversion-webhook image to PR build

36e2400

Points theia-cloud-crds at the eduide-fork conversion-webhook image (conversion-webhook:pr-70) which includes the v1beta11 AppDefinition mapper required for the sidecar redesign.

chore(test3): remove stale pr-46 preloading images

20d3c61

theia-no-ls:pr-46 and langserver-java:pr-46 no longer exist in the registry. The no-ls AppDefinitions reference images from ls1intum/theia directly, so these preloading entries served no purpose.

fix: correct garbage-collector image path to ghcr.io/eduide/garbage-c…

d316777

…ollector

Merge branch 'main' into feat/test3-pr70-sidecar-clean

edd722b

fix(test3): switch no-ls and langserver images to eduide registry

6665fb5

- replace ls1intum/theia image references with ghcr.io/eduide/eduide for no-ls IDE and sidecar images - align test3 appdefinitions and preloading with PR-tagged image pipeline

ci(deploy): tag no-ls and sidecar preloading images with ide_images_tag

606c591

- add explicit workflow overrides for preloading image indices 7..10 - ensure java-17-no-ls/rust-no-ls and langserver-java/langserver-rust track IDE tag input

merge main

d898086

resolve conflicts

6b188a7

merge v1beta11Merge branch 'feat/test3-pr70-sidecar-clean' into feat/…

ed4fda4

…15-integrate-shared-cache-node-into-theia-cloud-combined

add conversion webhook to test2

9ea35a1

		- theia-cloud-combined-cache
		- theia-cloud-combined-cache.theia-cloud.svc.cluster.local

	cacheUrl: "http://theia-shared-cache:5071"
	# URL of the cache service; adjust protocol/host/port/path for your environment.
	cacheUrl: "https://theia-cloud-combined-cache:8080/cache/"

	image: ghcr.io/ls1intum/theia/operator:pr-41
	image: ghcr.io/ls1intum/theia/operator:latest

Conversation

KevinGruber2001 commented Nov 16, 2025

Uh oh!

coderabbitai bot commented Nov 16, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Mar 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 7, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

coderabbitai bot commented Nov 16, 2025 •

edited

Loading