Skip to content

[DEV-15225] Fix postgres db name and recreation#555

Merged
ltellesfl merged 6 commits intomainfrom
dev-15225
Apr 13, 2026
Merged

[DEV-15225] Fix postgres db name and recreation#555
ltellesfl merged 6 commits intomainfrom
dev-15225

Conversation

@ltellesfl
Copy link
Copy Markdown
Contributor

@ltellesfl ltellesfl commented Apr 10, 2026

Note

Medium Risk
Changes Crunchy Postgres instance-set naming and anti-affinity selectors, which can affect pod scheduling and potentially trigger database pod replacement if labels no longer match existing resources.

Overview
Adjusts the generated Crunchy Postgres instances config to target the correct cluster label (postgres-data instead of postgres-insights) in pod anti-affinity rules.

Also parameterizes the instance set name via new crunchy_instance_name (default pgha1), updating the anti-affinity instance-set match and instance name to avoid hard-coded values and reduce unintended instance recreation across environments.

Reviewed by Cursor Bugbot for commit bf8397e. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 3738fb6. Configure here.

Copy link
Copy Markdown
Contributor

@NathanOkolita NathanOkolita left a comment

Choose a reason for hiding this comment

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

Looks good.

@ltellesfl ltellesfl merged commit 8b592a4 into main Apr 13, 2026
22 of 24 checks passed
@ltellesfl ltellesfl deleted the dev-15225 branch April 13, 2026 14:52
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