Fixes #26775: Use empty default for OIDC_CUSTOM_PARAMS to fix Compose brace-parsing

[RajdeepKushwaha5]

Describe your changes:

Fixes #26775

Docker Compose rejects ${OIDC_CUSTOM_PARAMS:-{}} because the bare braces {} conflict with Compose's variable substitution syntax, causing a parse error on startup.

Changed the default from ${OIDC_CUSTOM_PARAMS:-{}} to ${OIDC_CUSTOM_PARAMS:-} (empty default) across all 8 Docker Compose files. This is correct because:

1. conf/openmetadata.yaml already uses customParams: ${OIDC_CUSTOM_PARAMS:-} (empty default)
2. The server null-checks getCustomParams() != null before using the value
3. An empty/unset value is handled gracefully — no behavioral change for users who don't set OIDC_CUSTOM_PARAMS

Files changed:

• docker/development/docker-compose.yml
• docker/development/docker-compose-postgres.yml
• docker/development/docker-compose-gcp.yml
• docker/development/docker-compose-postgres-fuseki.yml
• docker/docker-compose-openmetadata/docker-compose-openmetadata.yml
• docker/docker-compose-quickstart/docker-compose.yml
• docker/docker-compose-quickstart/docker-compose-postgres.yml
• docker/docker-compose-quickstart/docker-compose-rdf.yml

Validation: All compose files pass docker compose config --quiet (GCP file fails due to a pre-existing unrelated GOOGLE_APPLICATION_CREDENTIALS issue).

Type of change:

• Bug fix

Checklist:

• I have read the CONTRIBUTING document.
• My PR title is Fixes <issue-number>: <short explanation>
• I have commented on my code, particularly in hard-to-understand areas.
• For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

• I have added a test that covers the exact scenario we are fixing. Validated via docker compose config --quiet on all 8 files.

---

Summary by Gitar

• Bug fixes included in merge:
  • Resolved 409 entity already exist errors in DataObservabilityGovernanceTab.spec.ts
  • Fixed UI display issue for custom properties containing dots in property names
  • Corrected pod and label naming length constraints for omjob deployments

_{This will update automatically on new commits.}

[github-actions]

Hi there 👋 Thanks for your contribution!

The OpenMetadata team will review the PR shortly! Once it has been labeled as safe to test, the CI workflows
will start executing and we'll be able to make sure everything is working as expected.

Let us know if you need any help!

[Copilot]

Pull request overview

This PR fixes a Docker Compose interpolation edge case that could generate malformed YAML for customParams in OpenMetadata OIDC config when OIDC_CUSTOM_PARAMS is unset (notably in some Compose-compatible runtimes like Podman Compose).

Changes:

• Quote the default value for OIDC_CUSTOM_PARAMS in Compose interpolation from ${OIDC_CUSTOM_PARAMS:-{}} to ${OIDC_CUSTOM_PARAMS:-"{}"}.
• Apply the same fix consistently across quickstart, production, and development Compose files.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
docker/docker-compose-quickstart/docker-compose.yml	Quotes OIDC_CUSTOM_PARAMS default to avoid brace-parsing ambiguity in Compose interpolation.
docker/docker-compose-quickstart/docker-compose-postgres.yml	Same quoting fix for the Postgres quickstart Compose.
docker/docker-compose-quickstart/docker-compose-rdf.yml	Same quoting fix for the RDF quickstart Compose.
docker/docker-compose-openmetadata/docker-compose-openmetadata.yml	Same quoting fix for the OpenMetadata Compose bundle.
docker/development/docker-compose.yml	Same quoting fix for the development Compose.
docker/development/docker-compose-postgres.yml	Same quoting fix for the Postgres development Compose.
docker/development/docker-compose-gcp.yml	Same quoting fix for the GCP development Compose.
docker/development/docker-compose-postgres-fuseki.yml	Same quoting fix for the Postgres+Fuseki development Compose.

[Copilot]

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.

[gitar-bot]

Code Review 👍 Approved with suggestions 0 resolved / 1 findings

Fixes Compose brace-parsing for OIDC_CUSTOM_PARAMS by adjusting the default value. Note that the actual change differs from the PR description—verify the empty default is the intended fix rather than quoted braces.

💡 Quality: PR description does not match actual change

📄 docker/development/docker-compose-gcp.yml:122 📄 docker/development/docker-compose-gcp.yml:336

The PR description states the fix is to change ${OIDC_CUSTOM_PARAMS:-{}} to ${OIDC_CUSTOM_PARAMS:-"{}"} (quoted braces), but the actual diff changes it to ${OIDC_CUSTOM_PARAMS:-} (empty default). The empty default is actually the correct approach — it aligns with conf/openmetadata.yaml which already uses customParams: ${OIDC_CUSTOM_PARAMS:-}, and the downstream code null-checks customParams before use. However, the description should be updated to accurately reflect the change to avoid confusing reviewers and future maintainers.

🤖 Prompt for agents

Code Review: Fixes Compose brace-parsing for OIDC_CUSTOM_PARAMS by adjusting the default value. Note that the actual change differs from the PR description—verify the empty default is the intended fix rather than quoted braces.

1. 💡 Quality: PR description does not match actual change
   Files: docker/development/docker-compose-gcp.yml:122, docker/development/docker-compose-gcp.yml:336

   The PR description states the fix is to change `${OIDC_CUSTOM_PARAMS:-{}}` to `${OIDC_CUSTOM_PARAMS:-"{}"}` (quoted braces), but the actual diff changes it to `${OIDC_CUSTOM_PARAMS:-}` (empty default). The empty default is actually the correct approach — it aligns with `conf/openmetadata.yaml` which already uses `customParams: ${OIDC_CUSTOM_PARAMS:-}`, and the downstream code null-checks `customParams` before use. However, the description should be updated to accurately reflect the change to avoid confusing reviewers and future maintainers.

Options

Display: compact → Showing less information.

Comment with these commands to change:

Compact

gitar display:verbose

_{Was this helpful? React with 👍 / 👎 | Gitar}

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.