Skip to content

remove the usage of the splitPostgresInstances flag from the SV apps deployment as it no longer changes the number of instances#6431

Open
mblaze-da wants to merge 1 commit into
mainfrom
mblaze-da/sv-split-postgres-flag-removal
Open

remove the usage of the splitPostgresInstances flag from the SV apps deployment as it no longer changes the number of instances#6431
mblaze-da wants to merge 1 commit into
mainfrom
mblaze-da/sv-split-postgres-flag-removal

Conversation

@mblaze-da

@mblaze-da mblaze-da commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

…deployment as it no longer changes the number of instances

[static]

Signed-off-by: Mateusz Błażejewski <mateusz.blazejewski@digitalasset.com>
@mblaze-da

Copy link
Copy Markdown
Contributor Author

/cluster_test

@github-actions

Copy link
Copy Markdown

Deploy cluster test triggered for Commit 1cd250a93b4bb9e9cbd1eae48ca8dbd0c1e07394 in , please contact a Contributor to approve it in CircleCI: https://app.circleci.com/pipelines/github/DACH-NY/canton-network-internal/74394

@mblaze-da
mblaze-da requested review from isegall-da and nicu-da July 15, 2026 15:52

@isegall-da isegall-da 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.

There seem to be quite a few other places that we have the same flag, can we remove it consistently across the board?

@mblaze-da

Copy link
Copy Markdown
Contributor Author

There seem to be quite a few other places that we have the same flag, can we remove it consistently across the board?

@isegall-da in splitwell and validator1 it actually does what it is supposed to (reduces the number of DB instances). So I'd be hesitant about removing it there, unless we don't care anymore.

@nicu-da

nicu-da commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

There seem to be quite a few other places that we have the same flag, can we remove it consistently across the board?

@isegall-da in splitwell and validator1 it actually does what it is supposed to (reduces the number of DB instances). So I'd be hesitant about removing it there, unless we don't care anymore.

Can we check if it's actually enabled on any cluster?

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.

3 participants