Skip to content

EAI-7854 Pin aiwb-infra-cnpg to one instance on small profile - #801

Merged
pre merged 1 commit into
mainfrom
EAI-7854-small-cnpg-instances
Aug 7, 2026
Merged

EAI-7854 Pin aiwb-infra-cnpg to one instance on small profile#801
pre merged 1 commit into
mainfrom
EAI-7854-small-cnpg-instances

Conversation

@pre

@pre pre commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Related:

The small profile did not override instances for aiwb-infra-cnpg, so the
chart default (multiple replicas) was used on single-node clusters where the
extra CNPG replicas cannot be scheduled.

Set instances: 1, matching what root/values_medium.yaml already does.

Testing

Renders on the small profile with a single CNPG instance for aiwb-infra-cnpg.

The small profile had no override for aiwb-infra-cnpg, so it inherited
instances: 3 from the published aiwb-cnpg chart, while the medium
profile explicitly set instances: 1. A small cluster therefore ran
three Postgres replicas silently, consuming more resources than a
medium cluster.

airm-infra-cnpg already defaults to a single instance upstream, so it
needs no override. The large profile keeps three instances.
@pre
pre requested a review from a team as a code owner August 7, 2026 07:27
@pre
pre enabled auto-merge August 7, 2026 07:56

@silokimmo silokimmo 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.

LGTM

@pre
pre merged commit 29aeba0 into main Aug 7, 2026
7 checks passed
@pre
pre deleted the EAI-7854-small-cnpg-instances branch August 7, 2026 08:39
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