Skip to content

fix(template): standalone mode should skip externalClusters#760

Open
ArnaudTA wants to merge 1 commit intocloudnative-pg:mainfrom
ArnaudTA:fix/null-external-clusters
Open

fix(template): standalone mode should skip externalClusters#760
ArnaudTA wants to merge 1 commit intocloudnative-pg:mainfrom
ArnaudTA:fix/null-external-clusters

Conversation

@ArnaudTA
Copy link

@ArnaudTA ArnaudTA commented Dec 18, 2025

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. chart( cluster ) Related to the cluster chart labels Dec 18, 2025
@ArnaudTA ArnaudTA force-pushed the fix/null-external-clusters branch from c70eece to 5401c65 Compare December 18, 2025 23:39
ankitml added a commit to paradedb/charts that referenced this pull request Dec 29, 2025
Updated to match upstream PR cloudnative-pg#760 which fixes the same issue:
- Moved externalClusters field inside conditional in template
- Reverted cluster.yaml to unconditional include
- Preserved ParadeDB-specific custom externalClusters feature

This minimizes future rebase conflicts with upstream while
maintaining backward compatibility with ParadeDB's custom
external cluster configuration.

Related: cloudnative-pg#760
@ArnaudTA ArnaudTA force-pushed the fix/null-external-clusters branch from 66b3102 to b36317e Compare January 21, 2026 00:30
@ArnaudTA
Copy link
Author

Up plz

@silanosa
Copy link

@itay-grudev Could we please get this merged and released? This would fix the externalClusters bug we're facing with Helm version 4. It would be greatly appreciated.

Signed-off-by: ArnaudTa <33383276+ArnaudTA@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chart( cluster ) Related to the cluster chart size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants