temporarily keep existing behaviour

Author: esemeniuc

validator/src/commands/block_engine/mod.rs

@@ -15,10 +15,10 @@ pub fn command(_default_args: &DefaultArgs) -> App<'_, '_> {
                 .required(true)
         )
         .arg(
-            Arg::with_name("disable_block_engine_autoconfig")
-                .long("disable-block-engine-autoconfig")
+            Arg::with_name("enable_block_engine_autoconfig")
+                .long("enable-block-engine-autoconfig")
                 .takes_value(false)
-                .help("Disables Block Engine auto-configuration. This stops the validator client from using the most performant Block Engine region. Values provided to `--block-engine-url` will be used as-is."),
+                .help("Enables Block Engine auto-configuration. This stops the validator client from using the most performant Block Engine region. Values provided to `--block-engine-url` will be used as-is."),
         )
         .arg(
             Arg::with_name("trust_block_engine_packets")
@@ -30,8 +30,10 @@ pub fn command(_default_args: &DefaultArgs) -> App<'_, '_> {
 
 pub fn execute(subcommand_matches: &ArgMatches, ledger_path: &Path) -> Result<()> {
     let block_engine_url = value_t_or_exit!(subcommand_matches, "block_engine_url", String);
+    // Temporary default change: disable autoconfig by default until users are comfortable
+    // with it; explicit flag remains supported
     let disable_block_engine_autoconfig =
-        subcommand_matches.is_present("disable_block_engine_autoconfig");
+        !subcommand_matches.is_present("enable_block_engine_autoconfig");
     let trust_packets = subcommand_matches.is_present("trust_block_engine_packets");
     let admin_client = admin_rpc_service::connect(ledger_path);
 

validator/src/commands/run/args.rs

@@ -1694,11 +1694,11 @@ pub fn add_args<'a>(app: App<'a, 'a>, default_args: &'a DefaultArgs) -> App<'a,
             .help("Number of CUs to allocate for bundles at beginning of slot.")
     )
     .arg(
-        Arg::with_name("disable_block_engine_autoconfig")
-            .long("disable-block-engine-autoconfig")
-            .value_name("DISABLE_BLOCK_ENGINE_AUTOCONFIG")
+        Arg::with_name("enable_block_engine_autoconfig")
+            .long("enable-block-engine-autoconfig")
+            .value_name("ENABLE_BLOCK_ENGINE_AUTOCONFIG")
             .takes_value(false)
-            .help("Disables Block Engine auto-configuration. This stops the validator client from using the most performant Block Engine region. Values provided to `--block-engine-url` will be used as-is."),
+            .help("Enables Block Engine auto-configuration. This stops the validator client from using the most performant Block Engine region. Values provided to `--block-engine-url` will be used as-is."),
     )
     .arg(
         Arg::with_name("shred_receiver_address")

validator/src/commands/run/execute.rs

@@ -640,7 +640,9 @@ pub fn execute(
         } else {
             String::default()
         },
-        disable_block_engine_autoconfig: matches.is_present("disable_block_engine_autoconfig"),
+        // Temporary default change: disable autoconfig by default until users are comfortable
+        // with it; explicit flag remains supported
+        disable_block_engine_autoconfig: !matches.is_present("disable_block_engine_autoconfig"),
         trust_packets: matches.is_present("trust_block_engine_packets"),
     }));