Skip to content

ci: fix the scheduled tests on the current runner image - #285

Merged
mogita merged 3 commits into
mainfrom
ci/scheduled-test-java-11
Nov 14, 2026
Merged

mogita merged 3 commits into
mainfrom
ci/scheduled-test-java-11

Conversation

@mogita

@mogita mogita commented Nov 14, 2026 •

Copy link
Copy Markdown
Contributor

Problem

The scheduled tests fail before and during the test run. They use the JDK preinstalled on the runner, and ubuntu-latest now ships Java 25, which Gradle 8.4 cannot run on. They also do not pass the multi-tenant app credentials, so TeamUsageStatsIntegrationTest fails at initialization.

Solution

The workflow sets up Java 11 with the same step the Build workflow uses, and passes STREAM_MULTI_TENANT_KEY and STREAM_MULTI_TENANT_SECRET the same way.

How to verify

  1. Run the "Scheduled tests" workflow on this branch. The tests run and pass.

Note

Low Risk
CI workflow-only changes with no application code or production runtime impact.

Overview
Aligns the Scheduled tests workflow with the main CI job so Gradle can run on current ubuntu-latest runners and integration tests get the required Stream multi-tenant secrets.

Adds an Set up Java 11 step (actions/setup-java@v4, Amazon Corretto) instead of relying on the runner’s default JDK (now Java 25, incompatible with Gradle 8.4). The Run tests step now exports STREAM_MULTI_TENANT_KEY and STREAM_MULTI_TENANT_SECRET using the same vars/secrets fallback pattern as the build workflow, so tests like TeamUsageStatsIntegrationTest can initialize.

Reviewed by Cursor Bugbot for commit 601fcb0. Bugbot is set up for automated code reviews on this repo. Configure here.

@mogita mogita changed the title ci: set up Java 11 in the scheduled tests ci: fix the scheduled tests on the current runner image Nov 14, 2026
@mogita
mogita merged commit 635d7b9 into main Nov 14, 2026
6 checks passed
@mogita
mogita deleted the ci/scheduled-test-java-11 branch November 14, 2026 03:03
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.

1 participant