Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set max Postgres DB Connections #1878

Merged
merged 6 commits into from
Apr 12, 2024
Merged

Set max Postgres DB Connections #1878

merged 6 commits into from
Apr 12, 2024

Conversation

badgersrus
Copy link
Contributor

Why this change is needed

The dev host node errored late last night:

CRIT [04-11|21:36:08.096] failed to add batch to L2 repo           node_id=0xd6bCf7694fc4Ed7c7C61B3fC5757919E237D0bf5 component=host enclave_id=0x281330F7a477e1e525687b092089480cC1041BF7 batch=0x8ee70adf010b2dee3cee5378d96464a964f6fea10fac715d248d3f4eaf498651 err="could not add batch: failed to begin host db transaction. Cause: pq: remaining connection slots are reserved for roles with privileges of the \"pg_use_reserved_connections\" role"

What changes were made as part of this PR

  • Set max connections to 100 same as edgelessDB
  • (unrelated) Add check to in-mem test for batchByTx to catch any flakeyness

PR checks pre-merging

Please indicate below by ticking the checkbox that you have read and performed the required
PR checks

  • PR checks reviewed and performed

@badgersrus badgersrus requested a review from tudor-malene April 12, 2024 09:56
@badgersrus
Copy link
Contributor Author

I'll deploy this branch to dev first to make sure it fixes the issue

Copy link
Collaborator

@tudor-malene tudor-malene left a comment

Choose a reason for hiding this comment

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

lgtm

@badgersrus badgersrus merged commit 261253b into main Apr 12, 2024
11 checks passed
@badgersrus badgersrus deleted the will/tenscan_test branch April 12, 2024 10:41
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