Skip to content

chart: support per-database URLs for v3 migrations - #67

Merged
marcopiraccini merged 1 commit into
mainfrom
v4-multiple-db-urls
Aug 31, 2026
Merged

marcopiraccini merged 1 commit into
mainfrom
v4-multiple-db-urls

Conversation

@marcopiraccini

@marcopiraccini marcopiraccini commented Aug 31, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • support exact per-service ICC database URLs while retaining the shared database_url fallback
  • preserve existing database names, credentials, and permissions during v3 to v4 migrations
  • keep chart 4.1 Secret aliases compatible and require a Workflow URL only when Workflow is enabled
  • update the installation and migration documentation for chart 4.2.0
  • add CI coverage for shared, exact, mixed, Traffic Inspector fallback, and Workflow database configurations

Why

Chart v3 accepted a complete connection URL for each ICC database. The v4 chart only accepted one base URL and derived fixed database names, which could not represent running v3 installations that use separate roles or custom database names.

This change lets those installations upgrade without renaming databases or changing credentials. ICC continues to apply schema migrations automatically when it starts.

Tests

  • bash test/database-urls.sh
  • helm lint with shared and per-database values
  • helm template with default and optional services
  • helm package chart --destination /tmp

Signed-off-by: marcopiraccini <marco.piraccini@gmail.com>
@marcopiraccini marcopiraccini changed the title Separated DB urls (v3 compatibility) chart: support per-database URLs for v3 migrations Aug 31, 2026
@marcopiraccini
marcopiraccini marked this pull request as ready for review August 31, 2026 10:38

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@marcopiraccini
marcopiraccini merged commit cf972e9 into main Aug 31, 2026
1 check passed
@marcopiraccini
marcopiraccini deleted the v4-multiple-db-urls branch August 31, 2026 12:48
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