Skip to content

WIP dependency updates - #488

Draft
jfaltermeier wants to merge 6 commits into
mainfrom
jf/dependencies
Draft

jfaltermeier wants to merge 6 commits into
mainfrom
jf/dependencies

Conversation

@jfaltermeier

@jfaltermeier jfaltermeier commented Apr 16, 2026 •

Copy link
Copy Markdown
Contributor
  • move Java components and images to Java 25, Quarkus 3.39.5, Fabric8 7.8.0, and current Maven, logging, testing, webhook, CORS, and packaging configuration
  • update Node and Theia workspaces to Theia 1.74.1, TypeScript 5.9, Vite 8, landing-page React 19, Express 5, Kubernetes client 2, Playwright 1.63, Lerna 10, VSCE 4, webpack, Axios, and UUID 11
  • update Kubernetes, Minikube, Terraform, cert-manager, Keycloak, ingress controllers, providers, demo images, and lockfiles while making HAProxy the default ingress controller
  • pin GitHub Actions to immutable SHAs, refresh Docker actions, use Java 21 for license checks, and refresh package indexes before native dependency installation
  • adapt generated Axios typing, monitor routes, React JSX types, and ESLint configuration for the upgraded stack

https://github.com/eclipse-theia/theia-cloud/actions/runs/36545466534
https://github.com/eclipse-theia/theia-cloud/actions/runs/36545491646

@jfaltermeier
jfaltermeier force-pushed the jf/dependencies branch 2 times, most recently from dfa5346 to 6b4db23 Compare April 16, 2026 12:59
* move Java components and images to Java 25, Quarkus 3.39.5, Fabric8
7.8.0, and current Maven, logging, testing, webhook, CORS, and packaging
configuration
* update Node and Theia workspaces to Theia 1.74.1, TypeScript 5.9, Vite
8, landing-page React 19, Express 5, Kubernetes client 2, Playwright
1.63, Lerna 10, VSCE 4, webpack, Axios, and UUID 11
* update Kubernetes, Minikube, Terraform, cert-manager, Keycloak,
ingress controllers, providers, demo images, and lockfiles while making
HAProxy the default ingress controller
* pin GitHub Actions to immutable SHAs, refresh Docker actions, use Java
21 for license checks, and refresh package indexes before native
dependency installation
* adapt generated Axios typing, monitor routes, React JSX types, and
ESLint configuration for the upgraded stack
jfaltermeier and others added 4 commits September 25, 2026 19:13
* upgrade testing-page to React 19 to match the landing page
* align JUnit Jupiter with the Quarkus BOM (6.1.3)
* correct the generated Axios patch comment and document it in OpenAPI.md
* update the Java version in the project prompt template
* remove an unused step id from the OpenShift e2e workflow
Pod cleanup in beforeEach takes about 30s on OpenShift,
so the default 60s test timeout was too tight for the tests that log in twice.
Raise their timeout to 120s and cap the loading animation wait at 60s so it fits in the test budget.
@lucas-koehler

Copy link
Copy Markdown
Contributor

The OpenShift E2E tests work after increasing the timeout from 1 to 2 minutes. It seems the one test's run time was right aroung the previous timeout of 1 minute.
E2E Run: https://github.com/eclipse-theia/theia-cloud/actions/runs/36584636067

@lucas-koehler

lucas-koehler commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

I also started an E2E Run against Microshift 4.22.0-rc.5 (latest 4.x). All tests failed for this: https://github.com/eclipse-theia/theia-cloud/actions/runs/36590723726/
So updating this seems to require additional updates. I don't think we have to do this in this PR but should probably upgrade MicroShift eventually because the currently used version 4.18 uses deprecated K8S version 1.31 (see here)

@lucas-koehler

Copy link
Copy Markdown
Contributor

The Quarkus scaffold Dockerfile.jvm and Dockerfile.legacy-jar still use openjdk-11 as base image. The PR edits a comment in the legacy one but leaves that line.
Looking at all the Dockerfiles in java/service/org.eclipse.theia.cloud.service/src/main/docker, I wonder whether we still use them at all because the Dockerfile to build the service is located in dockerfiles/service.

If we don't use any of these, we might want to remove them.

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.

2 participants