Skip to content

test(8.10): enable hub mixed upgrade scenario#6306

Open
eamonnmoloney wants to merge 2 commits into
mainfrom
ci/enable-hub-mixed-upgrade
Open

test(8.10): enable hub mixed upgrade scenario#6306
eamonnmoloney wants to merge 2 commits into
mainfrom
ci/enable-hub-mixed-upgrade

Conversation

@eamonnmoloney

Copy link
Copy Markdown
Contributor

Summary

  • Enables the 8.10 hub-mixed upgrade-minor scenario (hubmu).
  • Adds the migrator feature to the upgrade flow.
  • Keeps the existing CloudNativePG pre-install fixture and companion Keycloak/Elasticsearch dependencies.
  • Updates the WebModeler init container wait target to the CloudNativePG service used by the scenario (postgresql-cluster-rw).

Validation

  • make go.test chartPath=charts/camunda-platform-8.10
  • deploy-camunda matrix list --repo-root . --versions 8.10 | grep -E 'hub|hubl|hubm|hubw'

Copilot AI review requested due to automatic review settings June 3, 2026 04:55
@eamonnmoloney eamonnmoloney requested a review from a team as a code owner June 3, 2026 04:55
@eamonnmoloney eamonnmoloney requested review from hisImminence and removed request for a team June 3, 2026 04:55
@github-actions github-actions Bot added the version/8.10 Camunda applications/cycle version label Jun 3, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Enables the previously-disabled 8.10 hub-mixed upgrade-minor integration scenario (hubmu) and aligns the scenario’s WebModeler PostgreSQL readiness wait with the CloudNativePG fixture service name used in 8.10 integration scenarios.

Changes:

  • Enables the hub-mixed upgrade-minor scenario in charts/camunda-platform-8.10/test/ci-test-config.yaml and adds migrator to its features list.
  • Updates the hub-mixed feature values to wait for postgresql-cluster-rw:5432 (CloudNativePG) instead of the old WebModeler Postgres service.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
charts/camunda-platform-8.10/test/integration/scenarios/chart-full-setup/values/features/hub-mixed.yaml Updates WebModeler initContainer wait target to the CNPG service (postgresql-cluster-rw).
charts/camunda-platform-8.10/test/ci-test-config.yaml Enables the hub-mixed upgrade-minor scenario in CI and updates its feature list.

Comment on lines +706 to +709
flow: upgrade-minor
identity: keycloak
persistence: elasticsearch
features: [hub-mixed]
features: [hub-mixed,migrator]

@hisImminence hisImminence left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small, focused PR — mechanically correct. Two notes before merge:

Stale PR summary: description says "Adds the migrator feature" but commit 2 removed it. Final state is features: [hub-mixed] with no migrator. That's correct for 8.10 (per the comment at line 208: "upgrade migrators are only for 8.7 → 8.8"), but the summary should be updated to avoid confusing future readers.

postgresql-cluster-rw hardcode (hub-mixed.yaml:40-41): correct — the cnpg dependency profile provisions a CNPG Cluster via postgresql-cluster.yaml, which creates the postgresql-cluster-rw R/W service before install. Consider a short inline comment so the next reader knows why the name is static rather than release-scoped.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

version/8.10 Camunda applications/cycle version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants