[CLI] Increase max_connections for postgres in local testnet (aptos-labs#12473)

Author: banool

crates/aptos/CHANGELOG.md

@@ -3,6 +3,7 @@
 All notable changes to the Aptos CLI will be captured in this file. This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html) and the format set out by [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 ## Unreleased
+- Increased `max_connections` for postgres container created as part of local testnet to address occasional startup failures due to overloaded DB.
 
 ## [3.0.1] - 2024/03/05
 - Fix bug in `aptos update revela` if default install directory doesn't exist.

crates/aptos/src/node/local_testnet/postgres.rs

@@ -25,7 +25,7 @@ use tracing::{info, warn};
 
 pub const POSTGRES_CONTAINER_NAME: &str = "local-testnet-postgres";
 const POSTGRES_VOLUME_NAME: &str = "local-testnet-postgres-data";
-const POSTGRES_IMAGE: &str = "postgres:14.9";
+const POSTGRES_IMAGE: &str = "postgres:14.11";
 const DATA_PATH_IN_CONTAINER: &str = "/var/lib/mydata";
 const POSTGRES_DEFAULT_PORT: u16 = 5432;
 
@@ -271,6 +271,23 @@ impl ServiceManager for PostgresManager {
                 // directory inside the container that is mounted from the host system.
                 format!("PGDATA={}", DATA_PATH_IN_CONTAINER),
             ]),
+            cmd: Some(
+                vec![
+                    "postgres",
+                    "-c",
+                    // The default is 100 as of Postgres 14.11. Given the local testnet
+                    // can be composed of many different processors all with their own
+                    // connection pools, 100 is insufficient.
+                    "max_connections=200",
+                    "-c",
+                    // The default is 128MB as of Postgres 14.11. We 2x that value to
+                    // match the fact that we 2x'd max_connections.
+                    "shared_buffers=256MB",
+                ]
+                .into_iter()
+                .map(|s| s.to_string())
+                .collect(),
+            ),
             ..Default::default()
         };
 

crates/aptos/src/node/local_testnet/processors.rs

@@ -98,7 +98,9 @@ impl ProcessorManager {
             ending_version: None,
             number_concurrent_processing_tasks: None,
             enable_verbose_logging: None,
-            db_pool_size: None,
+            // The default at the time of writing is 30 but we don't need that
+            // many in a local testnet environment.
+            db_pool_size: Some(8),
             gap_detection_batch_size: 50,
         };
         let manager = Self {