Skip to content

Add GitHub workflow for integration tests on multiple Java versions - #1381

Open
ZPascal wants to merge 1 commit into
cloudfoundry:mainfrom
ZPascal:gh-integration-test-matrix
Open

ZPascal wants to merge 1 commit into
cloudfoundry:mainfrom
ZPascal:gh-integration-test-matrix

Conversation

@ZPascal

@ZPascal ZPascal commented Oct 3, 2026 •

Copy link
Copy Markdown

Runs the integration tests against a Cloud Foundry deployed on KinD (kind-deployment) in a Java 8/11/17/21 matrix. Every job gets its own Cloud Foundry.

Functionality

  • integration-test.yml deploys Cloud Foundry with the setup-cf action of kind-deployment and runs ./mvnw -Pintegration-test test against it. It runs on a schedule (nightly, latest versions on Java 21), manually (workflow_dispatch, optionally with preview) and from ci-java.yml after the build job (Java 21).
  • Validation runs deploy the capi-release that matches SUPPORTED_API_VERSION, resolved by .github/scripts/resolve-capi-version.sh and passed to kind-deployment as a chart version override. Preview runs (schedule, or manual with preview) use the latest versions.
  • permissions: contents: read, a concurrency group per ref, and a guard so the nightly run only fires in cloudfoundry/cf-java-client.

PostgreSQL connections

All Cloud Foundry components share one PostgreSQL. Its default of 100 connections runs out during a full run (too many clients already), after which UAA and so every test context fails. The workflow asks kind-deployment for postgres-max-connections: '500', which is set when PostgreSQL is installed. That needs cloudfoundry/kind-deployment#516.

Temporary: until #516 is merged, setup-cf and ref point at a commit of the ZPascal/kind-deployment fork that contains it. Once it is merged, repin both to the upstream commit (marked with a TEMPORARY comment in the workflow).

Dependencies

@ZPascal
ZPascal force-pushed the gh-integration-test-matrix branch 7 times, most recently from 6b1c866 to fffcc32 Compare October 5, 2026 11:32
@Lokowandtg

Copy link
Copy Markdown
Contributor

The failures because of "suspended" where addressed with #1371

@Lokowandtg

Copy link
Copy Markdown
Contributor

The failures because of "missing :resources" might be a problem in kind? As a workaround I added the following line to https://github.com/cloudfoundry/cf-k8s-releases/blob/4500c9dec5007c5c86c630bb86c3bb596e98259c/capi/helm/files/nginx/nginx_external_endpoints.conf#L47
upload_set_form_field "resources" [];
resulting in running tests.

@ZPascal
ZPascal force-pushed the gh-integration-test-matrix branch from 127d129 to cc8c779 Compare October 8, 2026 04:53
@ZPascal
ZPascal marked this pull request as ready for review October 8, 2026 04:55
@ZPascal
ZPascal changed the base branch from main to 5.x.x October 8, 2026 05:50
@ZPascal
ZPascal changed the base branch from 5.x.x to main October 8, 2026 05:50

This branch has not been deployed

No deployments
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