SERVER-88970 Added yaml formatting to server repo

GitOrigin-RevId: 35db3811d8f749edd5b79ba910adcbc1ceb54cc4

.eslintrc.yml

@@ -17,13 +17,13 @@ rules:
   no-redeclare: 0
   no-constant-condition: 0
   no-loss-of-precision: 0
-  no-restricted-syntax: [
+  no-restricted-syntax:
-    'error',
+    [
+      "error",
       {
         message: "Invalid load call. Please convert your library to a module and import it instead.",
-      selector:
+        selector: 'CallExpression > Identifier[name="load"]',
-        'CallExpression > Identifier[name="load"]'
+      },
-    }
     ]
 
 globals:
@@ -237,4 +237,3 @@ globals:
   helloStatePrompt: true
   _validateMemberIndex: true
   help: true
-

.prettierignore

@@ -5,10 +5,12 @@
 *
 !*/
 
-# Do not ignore markdown
+# Do not ignore markdown and yaml
-# We are only going to do markdown file formatting for now
+# We are only going to do markdown and yaml file formatting for now
 # Hopefully we will use prettier for more file types in the future
 !*.md
+!*.yml
+!*.yaml
 
 # Ignore all template files
 # When we eventually enable prettier on javascript these files are invalid and should be ignored
@@ -17,5 +19,14 @@
 # Ignore .yy files since prettier seems to think these are json files
 **/*.yy
 
+# Ignored generated resmoke suites
+buildscripts/resmokeconfig/matrix_suites/generated_suites/*.yml
+
+# Ignore generated pnpm lock file
+pnpm-lock.yaml
+
+# Ignore version expansions that are created in evergreen
+version_expansions.yml
+
 # Ignore all formatting in third_party/*
 src/third_party

.prettierrc

@@ -1,4 +1,3 @@
 {
-    "tabWidth": 4,
     "bracketSpacing": false
 }

.vscode_defaults/linux-virtual-workstation.code-workspace

@@ -64,6 +64,9 @@
 			"editor.formatOnSave": true,
 			"editor.defaultFormatter": "eeyore.yapf"
 		},
+		"[starlark]": {
+			"editor.defaultFormatter": "BazelBuild.vscode-bazel"
+		},
 	},
 	"extensions": {
 		"recommendations": [

WORKSPACE.bazel

@@ -131,21 +131,3 @@ npm_translate_lock(
 load("@npm//:repositories.bzl", "npm_repositories")
 
 npm_repositories()
-
-# TODO: This comes with a lot of built in formatters
-# Since we are only using prettier for now we do not use the others
-# See https://github.com/aspect-build/rules_lint/releases/tag/v0.11.0 for all supported formatters
-http_archive(
-    name = "aspect_rules_lint",
-    sha256 = "41fad363f11ccab46a244f93f8ccb0f442bc235e606d2fad87801987ad0759b1",
-    strip_prefix = "rules_lint-0.12.0",
-    urls = [
-        # Implements retry by relisting each url multiple times to be used as a failover.
-        # TODO(SERVER-86719): Re-implement http_archive to allow sleeping between retries
-        "https://github.com/aspect-build/rules_lint/releases/download/v0.12.0/rules_lint-v0.12.0.tar.gz",
-        "https://github.com/aspect-build/rules_lint/releases/download/v0.12.0/rules_lint-v0.12.0.tar.gz",
-        "https://github.com/aspect-build/rules_lint/releases/download/v0.12.0/rules_lint-v0.12.0.tar.gz",
-        "https://github.com/aspect-build/rules_lint/releases/download/v0.12.0/rules_lint-v0.12.0.tar.gz",
-        "https://github.com/aspect-build/rules_lint/releases/download/v0.12.0/rules_lint-v0.12.0.tar.gz",
-    ],
-)

bazel/format/BUILD.bazel

@@ -1,4 +1,3 @@
-load("@aspect_rules_lint//format:defs.bzl", "multi_formatter_binary")
 load("@npm//:prettier/package_json.bzl", prettier = "bin")
 
 # TODO: SERVER-82329 eslint binary should almost exactly mirror prettier binary
@@ -12,8 +11,14 @@ prettier.prettier_binary(
     env = {"BAZEL_BINDIR": "."},
 )
 
-multi_formatter_binary(
+py_binary(
     name = "format",
-    markdown = ":prettier",
+    srcs = ["format.py"],
+    args = [
+        "--prettier",
+        "$(location :prettier)",
+    ],
+    data = [":prettier"],
+    main = "format.py",
     visibility = ["//visibility:public"],
 )

bazel/format/format.py

@@ -0,0 +1,50 @@
+import argparse
+import os
+import pathlib
+import subprocess
+
+
+def run_prettier(prettier: pathlib.Path, check: bool) -> int:
+    try:
+        command = [prettier, "."]
+        if check:
+            command.append("--check")
+        else:
+            command.append("--write")
+        print(f"Running command: '{command}'")
+        subprocess.run(command, check=True)
+    except subprocess.CalledProcessError:
+        print("Found formatting errors. Run 'bazel run //:format' to fix")
+        print(
+            "If bazel is not installed, visit https://github.com/bazelbuild/bazelisk/blob/master/README.md#installation for installation steps."
+        )
+        return 1
+
+    if check:
+        print("No formatting errors")
+    return 0
+
+
+def main() -> int:
+    # If we are running in bazel, default the directory to the workspace
+    default_dir = os.environ.get("BUILD_WORKSPACE_DIRECTORY")
+    if not default_dir:
+        print("This script must be run though bazel. Please run 'bazel run //:format' instead")
+        return 1
+
+    parser = argparse.ArgumentParser(prog='Format',
+                                     description='This script formats code in mongodb')
+
+    parser.add_argument("--check", help="Run in check mode", default=False, action="store_true")
+    parser.add_argument("--prettier", help="Set the path to prettier", required=True,
+                        type=pathlib.Path)
+
+    args = parser.parse_args()
+    prettier_path: pathlib.Path = args.prettier.resolve()
+
+    os.chdir(default_dir)
+    return run_prettier(prettier_path, args.check)
+
+
+if __name__ == "__main__":
+    exit(main())

buildscripts/antithesis/topologies/replica_set/docker-compose.yml

@@ -1,4 +1,4 @@
-version: '3.0'
+version: "3.0"
 
 services:
   database1:

buildscripts/antithesis/topologies/sharded_cluster/docker-compose.yml

@@ -1,4 +1,4 @@
-version: '3.0'
+version: "3.0"
 
 services:
   configsvr1:

buildscripts/iwyu/iwyu_config.yml

@@ -1,17 +1,17 @@
 # options passed to IWYU
 iwyu_options:
-  - '--mapping_file=etc/iwyu_mapping.imp'
+  - "--mapping_file=etc/iwyu_mapping.imp"
-  - '--no_fwd_decls'
+  - "--no_fwd_decls"
-  - '--prefix_header_includes=add'
+  - "--prefix_header_includes=add"
-  - '--transitive_includes_only'
+  - "--transitive_includes_only"
 
 # options passed to the fix script
 fix_options:
-  - '--blank_lines'
+  - "--blank_lines"
-  - '--nocomments'
+  - "--nocomments"
-  - '--noreorder'
+  - "--noreorder"
-  - '--separate_project_includes=mongo'
+  - "--separate_project_includes=mongo"
-  - '--safe_headers'
+  - "--safe_headers"
   - '--only_re=^src/mongo\/.*'
   # TODO SERVER-77051 we will eventually turn this on when our codebase is cleaned up with out.
   # - '--nosafe_headers'
@@ -21,17 +21,17 @@ fix_options:
 # since this is targeting IWYU added headers
 no_includes:
   # avoid boost crazyness
-  - 'boost/.+/detail/.+'
+  - "boost/.+/detail/.+"
-  - 'asio/impl/.+'
+  - "asio/impl/.+"
   - 'boost/.+\.ipp'
   # avoid stdlib detail headers
   - 'ext/alloc_traits\.h'
   - 'ext/type_traits\.h'
   - 'cxxabi\.h' # https://github.com/include-what-you-use/include-what-you-use/issues/909
-  - 'bits/.+'
+  - "bits/.+"
   - 'syscall\.h'
   # arch specific
-  - 'boost/predef/hardware/simd/x86.+'
+  - "boost/predef/hardware/simd/x86.+"
   - 'emmintrin\.h'
   # we use a third party format which confuses IWYU
   - 'format\.h'
@@ -42,24 +42,24 @@ no_includes:
 
 # path prefixes (non regex) to skip
 skip_files:
-  - 'src/third_party'
+  - "src/third_party"
-  - 'build/'
+  - "build/"
-  - 'src/mongo/tools/mongo_tidy_checks'
+  - "src/mongo/tools/mongo_tidy_checks"
-  - 'src/mongo/util/net'  # causes linkage issues
+  - "src/mongo/util/net" # causes linkage issues
-  - 'src/mongo/util/text.cpp'
+  - "src/mongo/util/text.cpp"
   # IWYU confused on forward declares
-  - 'src/mongo/db/exec/near.cpp'
+  - "src/mongo/db/exec/near.cpp"
-  - 'src/mongo/db/storage/wiredtiger/wiredtiger_index.cpp'
+  - "src/mongo/db/storage/wiredtiger/wiredtiger_index.cpp"
   # Asio is going to need some special treatment, the headers are very finicky
-  - 'src/mongo/transport/asio'
+  - "src/mongo/transport/asio"
   # causes IWYU to crash:
-  - 'src/mongo/db/update/update_internal_node.cpp'
+  - "src/mongo/db/update/update_internal_node.cpp"
-  - 'src/mongo/db/update/update_array_node.cpp'
+  - "src/mongo/db/update/update_array_node.cpp"
-  - 'src/mongo/db/update/update_object_node.cpp'
+  - "src/mongo/db/update/update_object_node.cpp"
-  - 'src/mongo/db/update/update_array_node_test.cpp'
+  - "src/mongo/db/update/update_array_node_test.cpp"
-  - 'src/mongo/db/update/update_object_node_test.cpp'
+  - "src/mongo/db/update/update_object_node_test.cpp"
-  - 'src/mongo/util/options_parser/environment.cpp'
+  - "src/mongo/util/options_parser/environment.cpp"
-  - 'src/mongo/util/options_parser/option_section.cpp'
+  - "src/mongo/util/options_parser/option_section.cpp"
 
 # regex file paths to add keep pragma
 # include quotes are angle brackets
@@ -68,11 +68,11 @@ keep_includes:
   - '<fmt/printf\.h>'
   - '<fmt/ranges\.h>'
   - '<fmt/chrono\.h>'
-  - '<yaml-cpp/yaml.h>'
+  - "<yaml-cpp/yaml.h>"
   - '<asio\.hpp>'
   - '<boost/utility/in_place_factory\.hpp>'
-  - '<libunwind.h>'
+  - "<libunwind.h>"
-  - '<fstream>'  # IWYU messes up template instantiation
+  - "<fstream>" # IWYU messes up template instantiation
   - '"mongo/rpc/object_check\.h"'
   - '"mongo/base/init\.h"'
   - '"mongo/scripting/mozjs/wrapconstrainedmethod\.h"'

buildscripts/iwyu/test/basic/test_config.yml

@@ -1,16 +1,16 @@
 # options passed to IWYU
 iwyu_options:
-  - '--max_line_length=100'
+  - "--max_line_length=100"
-  - '--no_fwd_decls'
+  - "--no_fwd_decls"
-  - '--prefix_header_includes=add'
+  - "--prefix_header_includes=add"
-  - '--transitive_includes_only'
+  - "--transitive_includes_only"
 
 # options passed to the fix script
 fix_options:
-  - '--blank_lines'
+  - "--blank_lines"
-  - '--nocomments'
+  - "--nocomments"
-  - '--noreorder'
+  - "--noreorder"
-  - '--safe_headers'
+  - "--safe_headers"
 
 # filename regex to swap no_include in place
 # quotes and brackets not included quotes are always assumed

buildscripts/iwyu/test/no_include/test_config.yml

@@ -1,22 +1,22 @@
 # options passed to IWYU
 iwyu_options:
-  - '--max_line_length=100'
+  - "--max_line_length=100"
-  - '--no_fwd_decls'
+  - "--no_fwd_decls"
-  - '--prefix_header_includes=add'
+  - "--prefix_header_includes=add"
-  - '--transitive_includes_only'
+  - "--transitive_includes_only"
 
 # options passed to the fix script
 fix_options:
-  - '--blank_lines'
+  - "--blank_lines"
-  - '--nocomments'
+  - "--nocomments"
-  - '--noreorder'
+  - "--noreorder"
-  - '--safe_headers'
+  - "--safe_headers"
 
 # filename regex to swap no_include in place
 # quotes and brackets not included quotes are always assumed
 # since this is targeting IWYU added headers
 no_includes:
- - 'b.h'
+  - "b.h"
 
 # prefixes (non regex) to skip
 skip_files:

buildscripts/packaging/msi/mongod.yaml

@@ -17,8 +17,6 @@ systemLog:
 net:
   port: 27017
   bindIp: 127.0.0.1
-
-
 #processManagement:
 
 #security:

buildscripts/resmokeconfig/loggers/buildlogger.yml

@@ -1,14 +1,14 @@
 logging:
   executor:
-    format: '[%(name)s] %(asctime)s %(message)s'
+    format: "[%(name)s] %(asctime)s %(message)s"
     handlers:
       - class: logging.StreamHandler
   fixture:
-    format: '[%(name)s] %(message)s'
+    format: "[%(name)s] %(message)s"
     handlers:
       - class: buildlogger
   tests:
-    format: '[%(name)s] %(message)s'
+    format: "[%(name)s] %(message)s"
     handlers:
       - class: buildlogger
 

buildscripts/resmokeconfig/loggers/console.yml

@@ -1,14 +1,14 @@
 logging:
   executor:
-    format: '[%(name)s] %(asctime)s %(message)s'
+    format: "[%(name)s] %(asctime)s %(message)s"
     handlers:
       - class: logging.StreamHandler
   fixture:
-    format: '[%(name)s] %(message)s'
+    format: "[%(name)s] %(message)s"
     handlers:
       - class: logging.StreamHandler
   tests:
-    format: '[%(name)s] %(message)s'
+    format: "[%(name)s] %(message)s"
     handlers:
       - class: logging.StreamHandler
 

buildscripts/resmokeconfig/loggers/evg.yml

@@ -1,14 +1,14 @@
 logging:
   executor:
-    format: '[%(name)s] %(asctime)s %(message)s'
+    format: "[%(name)s] %(asctime)s %(message)s"
     handlers:
       - class: logging.StreamHandler
   fixture:
-    format: '[%(name)s] %(message)s'
+    format: "[%(name)s] %(message)s"
     handlers:
       - class: evergreen
   tests:
-    format: '[%(name)s] %(message)s'
+    format: "[%(name)s] %(message)s"
     handlers:
       - class: evergreen
 

buildscripts/resmokeconfig/loggers/file.yml

@@ -1,18 +1,18 @@
 logging:
   executor:
-    format: '[%(name)s] %(asctime)s %(message)s'
+    format: "[%(name)s] %(asctime)s %(message)s"
     handlers:
       - class: logging.FileHandler
         filename: executor.log
         mode: w
   fixture:
-    format: '[%(name)s] %(message)s'
+    format: "[%(name)s] %(message)s"
     handlers:
       - class: logging.FileHandler
         filename: fixture.log
         mode: w
   tests:
-    format: '[%(name)s] %(message)s'
+    format: "[%(name)s] %(message)s"
     handlers:
       - class: logging.FileHandler
         filename: tests.log

buildscripts/resmokeconfig/matrix_suites/overrides/audit.yml

@@ -6,4 +6,4 @@
         shell_options:
           global_vars:
             TestData:
-              auditDestination: 'console'
+              auditDestination: "console"

buildscripts/resmokeconfig/matrix_suites/overrides/change_streams.yml

@@ -43,7 +43,7 @@
         # not delay changes to wait for notifications or a clock advancement from other shards.
         num_shards: 2
         mongos_options:
-          bind_ip_all: ''
+          bind_ip_all: ""
           set_parameters:
             enableTestCommands: 1
         mongod_options:

buildscripts/resmokeconfig/matrix_suites/overrides/encryption.yml

@@ -6,8 +6,8 @@
         shell_options:
           global_vars:
             TestData:
-              enableEncryption: ''
+              enableEncryption: ""
-              encryptionKeyFile: 'src/mongo/db/modules/enterprise/jstests/encryptdb/libs/ekf2'
+              encryptionKeyFile: "src/mongo/db/modules/enterprise/jstests/encryptdb/libs/ekf2"
 
 - name: testdata_gcm
   value:
@@ -16,22 +16,22 @@
         shell_options:
           global_vars:
             TestData:
-              encryptionCipherMode: 'AES256-GCM'
+              encryptionCipherMode: "AES256-GCM"
 
 - name: mongodfixture_ese
   value:
     executor:
       fixture:
         mongod_options:
-          enableEncryption: ''
+          enableEncryption: ""
-          encryptionKeyFile: 'src/mongo/db/modules/enterprise/jstests/encryptdb/libs/ekf2'
+          encryptionKeyFile: "src/mongo/db/modules/enterprise/jstests/encryptdb/libs/ekf2"
 
 - name: mongodfixture_gcm
   value:
     executor:
       fixture:
         mongod_options:
-          encryptionCipherMode: 'AES256-GCM'
+          encryptionCipherMode: "AES256-GCM"
 
 - name: excluded_files
   value:

buildscripts/resmokeconfig/matrix_suites/overrides/kill_primary.yml

@@ -77,7 +77,7 @@
       fixture:
         class: ReplicaSetFixture
         mongod_options:
-          enableMajorityReadConcern: ''
+          enableMajorityReadConcern: ""
           syncdelay: 5
           wiredTigerEngineConfigString: "debug_mode=(table_logging=true)"
           set_parameters:
@@ -101,7 +101,7 @@
       fixture:
         class: ReplicaSetFixture
         mongod_options:
-          enableMajorityReadConcern: ''
+          enableMajorityReadConcern: ""
           wiredTigerEngineConfigString: "debug_mode=(table_logging=true)"
           set_parameters:
             enableTestCommands: 1

buildscripts/resmokeconfig/matrix_suites/overrides/multiversion.yml

@@ -109,7 +109,6 @@
             TestData:
               nonClusteredConfigTransactions: true
 
-
 ### Suite-specific overrides ###
 - name: multiversion_future_git_tag_exclude_files
   value:
@@ -200,7 +199,7 @@
         shell_options:
           global_vars:
             TestData:
-              useRandomBinVersionsWithinReplicaSet: 'last-lts'
+              useRandomBinVersionsWithinReplicaSet: "last-lts"
               nonClusteredConfigTransactions: true
 
 - name: replica_sets_multiversion_testdata_last_continuous
@@ -210,7 +209,7 @@
         shell_options:
           global_vars:
             TestData:
-              useRandomBinVersionsWithinReplicaSet: 'last-continuous'
+              useRandomBinVersionsWithinReplicaSet: "last-continuous"
               nonClusteredConfigTransactions: true
 
 - name: rollback_multiversion_fuzzer_testdata_last_lts
@@ -229,7 +228,7 @@
                 storage:
                   verbosity: 1
               rollbackShutdowns: true
-              useRandomBinVersionsWithinReplicaSet: 'last-lts'
+              useRandomBinVersionsWithinReplicaSet: "last-lts"
               nonClusteredConfigTransactions: true
 
 - name: rollback_multiversion_fuzzer_testdata_last_continuous
@@ -248,7 +247,7 @@
                 storage:
                   verbosity: 1
               rollbackShutdowns: true
-              useRandomBinVersionsWithinReplicaSet: 'last-continuous'
+              useRandomBinVersionsWithinReplicaSet: "last-continuous"
               nonClusteredConfigTransactions: true
 
 - name: sharding_multiversion_selector
@@ -281,8 +280,8 @@
         shell_options:
           global_vars:
             TestData:
-              useRandomBinVersionsWithinReplicaSet: 'last-lts'
+              useRandomBinVersionsWithinReplicaSet: "last-lts"
-              mongosBinVersion: 'last-lts'
+              mongosBinVersion: "last-lts"
               nonClusteredConfigTransactions: true
 
 - name: sharding_multiversion_testdata_last_continuous
@@ -292,8 +291,8 @@
         shell_options:
           global_vars:
             TestData:
-              useRandomBinVersionsWithinReplicaSet: 'last-continuous'
+              useRandomBinVersionsWithinReplicaSet: "last-continuous"
-              mongosBinVersion: 'last-continuous'
+              mongosBinVersion: "last-continuous"
 
 - name: jstestfuzz_replication_multiversion_hooks
   value:

buildscripts/resmokeconfig/matrix_suites/overrides/replica_sets_stepdown_selector.yml

@@ -65,7 +65,6 @@
       exclude_files:
         - jstests/aggregation/**/*.js
 
-
 - name: reconfig_kill_primary_jscore_passthrough_exclude_files
   value:
     selector:
@@ -135,7 +134,6 @@
         - jstests/fle2/**/*.js
         - src/mongo/db/modules/*/jstests/fle2/**/*.js
 
-
 - name: kill_primary_jscore_passthrough_exclude_with_any_tags
   value:
     selector:

buildscripts/resmokeconfig/matrix_suites/overrides/serverless.yml

@@ -6,7 +6,7 @@
           global_vars:
             TestData:
               defaultReadConcernLevel: null
-              enableMajorityReadConcern: ''
+              enableMajorityReadConcern: ""
               hashTestNamesForMultitenancy: true
               tenantId: "636d957b2646ddfaf9b5e13f"
               useResponsePrefixChecking: true
@@ -17,9 +17,9 @@
         class: ReplicaSetFixture
         replset_name: "ChangeStreamMultitenantReplSet"
         mongod_options:
-          bind_ip_all: ''
+          bind_ip_all: ""
           serverless: true
-          noscripting: ''
+          noscripting: ""
           set_parameters:
             enableTestCommands: 1
             featureFlagServerlessChangeStreams: true

buildscripts/resmokeconfig/powercycle/powercycle_tasks.yml

@@ -13,7 +13,6 @@
 # - 'mongod_options'
 
 tasks:
-
   - name: powercycle
 
   - name: powercycle_smoke

buildscripts/resmokeconfig/suites/aggregation_auth.yml

@@ -33,7 +33,7 @@ executor:
   fixture:
     class: MongoDFixture
     mongod_options:
-      auth: ''
+      auth: ""
       keyFile: *keyFile
       set_parameters:
         enableTestCommands: 1

buildscripts/resmokeconfig/suites/aggregation_ese.yml

@@ -24,7 +24,7 @@ executor:
   fixture:
     class: MongoDFixture
     mongod_options:
-      enableEncryption: ''
+      enableEncryption: ""
       encryptionKeyFile: *keyFile
       set_parameters:
         enableTestCommands: 1

buildscripts/resmokeconfig/suites/aggregation_ese_gcm.yml

@@ -24,8 +24,8 @@ executor:
   fixture:
     class: MongoDFixture
     mongod_options:
-      enableEncryption: ''
+      enableEncryption: ""
       encryptionKeyFile: *keyFile
-      encryptionCipherMode: 'AES256-GCM'
+      encryptionCipherMode: "AES256-GCM"
       set_parameters:
         enableTestCommands: 1

buildscripts/resmokeconfig/suites/aggregation_read_concern_majority_passthrough.yml

@@ -35,7 +35,7 @@ executor:
       global_vars:
         TestData:
           defaultReadConcernLevel: majority
-          enableMajorityReadConcern: ''
+          enableMajorityReadConcern: ""
       eval: >-
         globalThis.testingReplication = true;
         await import('jstests/libs/override_methods/set_read_and_write_concerns.js');
@@ -49,7 +49,7 @@ executor:
   fixture:
     class: ReplicaSetFixture
     mongod_options:
-      enableMajorityReadConcern: ''
+      enableMajorityReadConcern: ""
       set_parameters:
         enableTestCommands: 1
     num_nodes: 2

buildscripts/resmokeconfig/suites/audit.yml

@@ -8,4 +8,4 @@ selector:
 executor:
   config:
     shell_options:
-      nodb: ''
+      nodb: ""

buildscripts/resmokeconfig/suites/auth.yml

@@ -14,4 +14,4 @@ executor:
       global_vars:
         TestData:
           roleGraphInvalidationIsFatal: true
-      nodb: ''
+      nodb: ""

buildscripts/resmokeconfig/suites/auth_audit.yml

@@ -11,8 +11,8 @@ selector:
 executor:
   config:
     shell_options:
-      nodb: ''
+      nodb: ""
       global_vars:
         TestData:
-          auditDestination: 'console'
+          auditDestination: "console"
           roleGraphInvalidationIsFatal: true

buildscripts/resmokeconfig/suites/benchmarks_first_half.yml

@@ -67,7 +67,6 @@ selector:
     # These benchmarks are being run as part of the benchmarks_replication.yml test suite.
     - build/install/bin/oplog_application_bm*
 
-
 executor:
   config: {}
   hooks:

buildscripts/resmokeconfig/suites/benchmarks_second_half.yml

@@ -70,7 +70,6 @@ selector:
     - build/install/bin/oplog_application_bm*
     - build/install/bin/oplog_applier_utils_bm*
 
-
 executor:
   config: {}
   hooks:

buildscripts/resmokeconfig/suites/buildscripts_test.yml

@@ -12,5 +12,4 @@ selector:
     - buildscripts/tests/resmoke_end2end/**/test_*.py # Requires compile task. Test run in resmoke_end2end_tests.yml instead.
     - buildscripts/tests/resmoke_validation/**/test_*.py # Ran in commit queue in resmoke_validation_tests
 
-
 executor: {}

buildscripts/resmokeconfig/suites/bulk_write_multi_op_causally_consistent_jscore_passthrough.yml

@@ -32,7 +32,6 @@ selector:
     # bulkWrite is consistent with the other CRUD operations here.
     - jstests/core/txns/multi_statement_transaction_write_error.js
 
-
     # Multiple cluster specific exclusions
 
     # Don't run transaction specific tests.

buildscripts/resmokeconfig/suites/bulk_write_targeted_override.yml

@@ -62,5 +62,5 @@ executor:
     tests: true
   config:
     shell_options:
-      nodb: ''
+      nodb: ""
       eval: await import('jstests/libs/override_methods/single_crud_op_as_bulk_write.js');

buildscripts/resmokeconfig/suites/causally_consistent_hedged_reads_jscore_passthrough.yml

@@ -85,7 +85,7 @@ executor:
             verbosity: 1
             asio: 2
     mongod_options:
-      enableMajorityReadConcern: ''
+      enableMajorityReadConcern: ""
       set_parameters:
         enableTestCommands: 1
         logComponentVerbosity:

buildscripts/resmokeconfig/suites/causally_consistent_jscore_passthrough.yml

@@ -86,7 +86,7 @@ executor:
             verbosity: 1
             asio: 2
     mongod_options:
-      enableMajorityReadConcern: ''
+      enableMajorityReadConcern: ""
       set_parameters:
         enableTestCommands: 1
         logComponentVerbosity:

buildscripts/resmokeconfig/suites/causally_consistent_jscore_passthrough_auth.yml

@@ -179,8 +179,8 @@ executor:
             verbosity: 1
             asio: 2
     mongod_options:
-      enableMajorityReadConcern: ''
+      enableMajorityReadConcern: ""
-      auth: ''
+      auth: ""
       keyFile: *keyFile
       set_parameters:
         enableTestCommands: 1

buildscripts/resmokeconfig/suites/causally_consistent_read_concern_snapshot_passthrough.yml

@@ -60,5 +60,5 @@ executor:
         enableTestCommands: 1
         writePeriodicNoops: 1
         minSnapshotHistoryWindowInSeconds: 3600
-      enableMajorityReadConcern: ''
+      enableMajorityReadConcern: ""
     num_nodes: 2

buildscripts/resmokeconfig/suites/change_streams.yml

@@ -24,7 +24,7 @@ executor:
       global_vars:
         TestData:
           defaultReadConcernLevel: null
-          enableMajorityReadConcern: ''
+          enableMajorityReadConcern: ""
       # Enable causal consistency for change streams suites using 1 node replica sets. Some tests
       # rely on the assumption that a w:majority write will be visible immediately in a subsequently
       # opened change stream. In 1 node replica sets, an operation that majority commits at
@@ -49,7 +49,7 @@ executor:
   fixture:
     class: ReplicaSetFixture
     mongod_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
     num_nodes: 2

buildscripts/resmokeconfig/suites/change_streams_mongos_sessions_passthrough.yml

@@ -32,7 +32,7 @@ executor:
       global_vars:
         TestData:
           defaultReadConcernLevel: null
-          enableMajorityReadConcern: ''
+          enableMajorityReadConcern: ""
       # Enable causal consistency for change streams suites using 1 node replica sets. See
       # change_streams.yml for detailed explanation.
       eval: >-
@@ -53,11 +53,11 @@ executor:
     # not delay changes to wait for notifications or a clock advancement from other shards.
     num_shards: 2
     mongos_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
     mongod_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
         periodicNoopIntervalSecs: 1

buildscripts/resmokeconfig/suites/change_streams_multi_stmt_txn_mongos_passthrough.yml

@@ -55,11 +55,11 @@ executor:
     # not delay changes to wait for notifications or a clock advancement from other shards.
     num_shards: 2
     mongos_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
     mongod_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
         periodicNoopIntervalSecs: 1

buildscripts/resmokeconfig/suites/change_streams_multi_stmt_txn_passthrough.yml

@@ -47,7 +47,7 @@ executor:
   fixture:
     class: ReplicaSetFixture
     mongod_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
     num_nodes: 1

buildscripts/resmokeconfig/suites/change_streams_multi_stmt_txn_passthrough_with_replica_set_endpoint.yml

@@ -58,7 +58,7 @@ executor:
   fixture:
     class: ReplicaSetFixture
     mongod_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
         featureFlagAllMongodsAreSharded: 1

buildscripts/resmokeconfig/suites/change_streams_multi_stmt_txn_sharded_collections_passthrough.yml

@@ -53,11 +53,11 @@ executor:
   fixture:
     class: ShardedClusterFixture
     mongos_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
     mongod_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
         writePeriodicNoops: 1

buildscripts/resmokeconfig/suites/change_streams_multitenant_passthrough.yml

@@ -69,7 +69,7 @@ executor:
       global_vars:
         TestData:
           defaultReadConcernLevel: null
-          enableMajorityReadConcern: ''
+          enableMajorityReadConcern: ""
           tenantId: "636d957b2646ddfaf9b5e13f"
           hashTestNamesForMultitenancy: true
       # Enable causal consistency for change streams suites using 1 node replica sets. See
@@ -93,9 +93,9 @@ executor:
     class: ReplicaSetFixture
     replset_name: "ChangeStreamMultitenantReplSet"
     mongod_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       serverless: true
-      noscripting: ''
+      noscripting: ""
       set_parameters:
         enableTestCommands: 1
         featureFlagServerlessChangeStreams: true

buildscripts/resmokeconfig/suites/change_streams_multitenant_sharded_collections_passthrough.yml

@@ -43,7 +43,7 @@ executor:
       global_vars:
         TestData:
           defaultReadConcernLevel: null
-          enableMajorityReadConcern: ''
+          enableMajorityReadConcern: ""
       # Enable causal consistency for change streams suites using 1 node replica sets. See
       # change_streams.yml for detailed explanation.
       eval: >-
@@ -64,11 +64,11 @@ executor:
     class: ShardedClusterFixture
     num_shards: 2
     mongos_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
     mongod_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
         periodicNoopIntervalSecs: 1

buildscripts/resmokeconfig/suites/change_streams_per_shard_cursor_passthrough.yml

@@ -33,7 +33,7 @@ executor:
       global_vars:
         TestData:
           defaultReadConcernLevel: null
-          enableMajorityReadConcern: ''
+          enableMajorityReadConcern: ""
       # Enable causal consistency for change streams suites using 1 node replica sets. See
       # change_streams.yml for detailed explanation.
       eval: >-
@@ -52,11 +52,11 @@ executor:
   fixture:
     class: ShardedClusterFixture
     mongos_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
     mongod_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
         writePeriodicNoops: 1

buildscripts/resmokeconfig/suites/change_streams_whole_cluster_passthrough.yml

@@ -26,7 +26,7 @@ executor:
       global_vars:
         TestData:
           defaultReadConcernLevel: null
-          enableMajorityReadConcern: ''
+          enableMajorityReadConcern: ""
       # Enable causal consistency for change streams suites using 1 node replica sets. See
       # change_streams.yml for detailed explanation.
       eval: >-
@@ -46,7 +46,7 @@ executor:
   fixture:
     class: ReplicaSetFixture
     mongod_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
     num_nodes: 1

buildscripts/resmokeconfig/suites/change_streams_whole_cluster_passthrough_with_replica_set_endpoint.yml

@@ -47,7 +47,7 @@ executor:
       global_vars:
         TestData:
           defaultReadConcernLevel: null
-          enableMajorityReadConcern: ''
+          enableMajorityReadConcern: ""
           testingReplicaSetEndpoint: true
       # Enable causal consistency for change streams suites using 1 node replica sets. See
       # change_streams_whole_cluster.yml for detailed explanation.
@@ -70,7 +70,7 @@ executor:
   fixture:
     class: ReplicaSetFixture
     mongod_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
         featureFlagAllMongodsAreSharded: 1

buildscripts/resmokeconfig/suites/change_streams_whole_db_passthrough.yml

@@ -27,7 +27,7 @@ executor:
       global_vars:
         TestData:
           defaultReadConcernLevel: null
-          enableMajorityReadConcern: ''
+          enableMajorityReadConcern: ""
       # Enable causal consistency for change streams suites using 1 node replica sets. See
       # change_streams.yml for detailed explanation.
       eval: >-
@@ -47,7 +47,7 @@ executor:
   fixture:
     class: ReplicaSetFixture
     mongod_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
     num_nodes: 1

buildscripts/resmokeconfig/suites/change_streams_whole_db_passthrough_with_replica_set_endpoint.yml

@@ -47,7 +47,7 @@ executor:
       global_vars:
         TestData:
           defaultReadConcernLevel: null
-          enableMajorityReadConcern: ''
+          enableMajorityReadConcern: ""
           testingReplicaSetEndpoint: true
       # Enable causal consistency for change streams suites using 1 node replica sets. See
       # change_streams.yml for detailed explanation.
@@ -70,7 +70,7 @@ executor:
   fixture:
     class: ReplicaSetFixture
     mongod_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
         featureFlagAllMongodsAreSharded: 1

buildscripts/resmokeconfig/suites/change_streams_with_replica_set_endpoint.yml

@@ -45,7 +45,7 @@ executor:
       global_vars:
         TestData:
           defaultReadConcernLevel: null
-          enableMajorityReadConcern: ''
+          enableMajorityReadConcern: ""
           testingReplicaSetEndpoint: true
       # Enable causal consistency for change streams suites using 1 node replica sets. Some tests
       # rely on the assumption that a w:majority write will be visible immediately in a subsequently
@@ -73,7 +73,7 @@ executor:
   fixture:
     class: ReplicaSetFixture
     mongod_options:
-      bind_ip_all: ''
+      bind_ip_all: ""
       set_parameters:
         enableTestCommands: 1
         featureFlagAllMongodsAreSharded: 1

buildscripts/resmokeconfig/suites/client_encrypt.yml

@@ -7,9 +7,9 @@ selector:
 executor:
   config:
     shell_options:
-      nodb: ''
+      nodb: ""
-      ssl: ''
+      ssl: ""
-      tlsAllowInvalidHostnames: ''
+      tlsAllowInvalidHostnames: ""
-      tlsAllowInvalidCertificates: ''
+      tlsAllowInvalidCertificates: ""
       tlsCAFile: jstests/libs/ca.pem
       tlsCertificateKeyFile: jstests/libs/client.pem

buildscripts/resmokeconfig/suites/clustered_collection_passthrough.yml

@@ -8,7 +8,6 @@ selector:
     - jstests/fle2/**/*.js
     - src/mongo/db/modules/*/jstests/fle2/**/*.js
   exclude_files:
-
     # Assumes the _id index is real.
     - jstests/core/**/collmod_convert_to_ttl.js
     - jstests/core/**/index_create_too_many.js

buildscripts/resmokeconfig/suites/concurrency_multitenancy_replication_with_atlas_proxy.yml

@@ -30,7 +30,6 @@ selector:
     - jstests/concurrency/fsm_workloads/auth_create_user.js
     - jstests/concurrency/fsm_workloads/auth_role_consistency.js
 
-
     # TODO SERVER-82260: FLE2 tests access an invalid VTS which has neither tenant nor user name.
     - src/mongo/db/modules/enterprise/jstests/concurrency/fsm_workloads/fle2_compact.js
     - src/mongo/db/modules/enterprise/jstests/concurrency/fsm_workloads/fle2_crud.js
@@ -94,5 +93,5 @@ executor:
         multitenancySupport: true
         logComponentVerbosity:
           command: 2
-      noscripting: ''
+      noscripting: ""
     num_nodes: 3

buildscripts/resmokeconfig/suites/concurrency_replication_bulk_write.yml

@@ -38,8 +38,7 @@ executor:
     shell_options:
       global_vars:
         TestData:
-          fsmPreOverridesLoadedCallback:
+          fsmPreOverridesLoadedCallback: import('jstests/libs/override_methods/single_crud_op_as_bulk_write.js');
-            import('jstests/libs/override_methods/single_crud_op_as_bulk_write.js');
   hooks:
     # The CheckReplDBHash hook waits until all operations have replicated to and have been applied
     # on the secondaries, so we run the ValidateCollections hook after it to ensure we're

buildscripts/resmokeconfig/suites/concurrency_replication_bulk_write_with_replica_set_endpoint.yml

@@ -57,8 +57,7 @@ executor:
     shell_options:
       global_vars:
         TestData:
-          fsmPreOverridesLoadedCallback:
+          fsmPreOverridesLoadedCallback: import('jstests/libs/override_methods/single_crud_op_as_bulk_write.js');
-            import('jstests/libs/override_methods/single_crud_op_as_bulk_write.js');
           testingReplicaSetEndpoint: true
   hooks:
     # The CheckReplDBHash hook waits until all operations have replicated to and have been applied

buildscripts/resmokeconfig/suites/concurrency_sharded_causal_consistency.yml

@@ -29,7 +29,6 @@ selector:
     # can cause OOM kills on test hosts
     - jstests/concurrency/fsm_workloads/findAndModify_update_grow.js
 
-
     # cannot createIndex after dropDatabase without sharding first
     - jstests/concurrency/fsm_workloads/plan_cache_drop_database.js
 

buildscripts/resmokeconfig/suites/concurrency_sharded_causal_consistency_and_balancer.yml

@@ -32,7 +32,6 @@ selector:
     # can cause OOM kills on test hosts
     - jstests/concurrency/fsm_workloads/findAndModify_update_grow.js
 
-
     # cannot createIndex after dropDatabase without sharding first
     - jstests/concurrency/fsm_workloads/plan_cache_drop_database.js
 
@@ -78,8 +77,7 @@ executor:
         TestData:
           runningWithCausalConsistency: true
           runningWithBalancer: true
-          fsmPreOverridesLoadedCallback:
+          fsmPreOverridesLoadedCallback: import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
-            import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
 
   hooks:
     - class: CheckShardFilteringMetadata

buildscripts/resmokeconfig/suites/concurrency_sharded_initial_sync.yml

@@ -196,15 +196,15 @@ executor:
         await import('jstests/libs/override_methods/enable_causal_consistency_without_read_pref.js');
       global_vars:
         TestData:
-          defaultReadPreference: {mode: "secondaryPreferred", tags: [{tag: "INIT_SYNC"}, {}]}
+          defaultReadPreference:
+            {mode: "secondaryPreferred", tags: [{tag: "INIT_SYNC"}, {}]}
           runningWithBalancer: true
           # Will prompt the FSM workers to use 'network_error_and_txn_override.js'
           # and generally retry operations.
           runningWithConfigStepdowns: true
           runningWithShardStepdowns: true
           useActionPermittedFile: true
-          fsmPreOverridesLoadedCallback:
+          fsmPreOverridesLoadedCallback: import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
-            import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
 
   hooks:
     - class: ContinuousInitialSync

buildscripts/resmokeconfig/suites/concurrency_sharded_kill_primary_with_balancer.yml

@@ -39,7 +39,6 @@ selector:
     # can cause OOM kills on test hosts
     - jstests/concurrency/fsm_workloads/findAndModify_update_grow.js
 
-
     # cannot createIndex after dropDatabase without sharding first
     - jstests/concurrency/fsm_workloads/plan_cache_drop_database.js
 
@@ -160,8 +159,7 @@ executor:
           runningWithShardStepdowns: true
           useActionPermittedFile: true
           runningWithBalancer: true
-          fsmPreOverridesLoadedCallback:
+          fsmPreOverridesLoadedCallback: import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
-            import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
   hooks:
     - class: ContinuousStepdown
       config_stepdown: true

buildscripts/resmokeconfig/suites/concurrency_sharded_local_read_write_multi_stmt_txn_with_balancer.yml

@@ -126,8 +126,7 @@ executor:
           runningWithSessions: true
           traceExceptions: false
           runningWithBalancer: true
-          fsmPreOverridesLoadedCallback:
+          fsmPreOverridesLoadedCallback: import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
-            import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
 
   hooks:
     - class: CheckShardFilteringMetadata

buildscripts/resmokeconfig/suites/concurrency_sharded_multi_stmt_txn_with_balancer.yml

@@ -117,8 +117,7 @@ executor:
           runningWithSessions: true
           traceExceptions: false
           runningWithBalancer: true
-          fsmPreOverridesLoadedCallback:
+          fsmPreOverridesLoadedCallback: import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
-            import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
 
   hooks:
     - class: CheckShardFilteringMetadata

buildscripts/resmokeconfig/suites/concurrency_sharded_replication.yml

@@ -26,7 +26,6 @@ selector:
     # can cause OOM kills on test hosts
     - jstests/concurrency/fsm_workloads/findAndModify_update_grow.js
 
-
     # cannot createIndex after dropDatabase without sharding first
     - jstests/concurrency/fsm_workloads/plan_cache_drop_database.js
 

buildscripts/resmokeconfig/suites/concurrency_sharded_replication_bulk_write.yml

@@ -26,7 +26,6 @@ selector:
     # can cause OOM kills on test hosts
     - jstests/concurrency/fsm_workloads/findAndModify_update_grow.js
 
-
     # cannot createIndex after dropDatabase without sharding first
     - jstests/concurrency/fsm_workloads/plan_cache_drop_database.js
 
@@ -79,8 +78,7 @@ executor:
       global_vars:
         TestData:
           runningWithBalancer: false
-          fsmPreOverridesLoadedCallback:
+          fsmPreOverridesLoadedCallback: import('jstests/libs/override_methods/single_crud_op_as_bulk_write.js');
-            import('jstests/libs/override_methods/single_crud_op_as_bulk_write.js');
   hooks:
     - class: CheckShardFilteringMetadata
     # TODO (SERVER-63855): remove 'RunDBCheckInBackground' or put it back.

buildscripts/resmokeconfig/suites/concurrency_sharded_replication_with_balancer.yml

@@ -29,7 +29,6 @@ selector:
     # can cause OOM kills on test hosts
     - jstests/concurrency/fsm_workloads/findAndModify_update_grow.js
 
-
     # cannot createIndex after dropDatabase without sharding first
     - jstests/concurrency/fsm_workloads/plan_cache_drop_database.js
 
@@ -83,8 +82,7 @@ executor:
       global_vars:
         TestData:
           runningWithBalancer: true
-          fsmPreOverridesLoadedCallback:
+          fsmPreOverridesLoadedCallback: import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
-            import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
   hooks:
     - class: CheckShardFilteringMetadata
     - class: CheckReplDBHashInBackground

buildscripts/resmokeconfig/suites/concurrency_sharded_replication_with_balancer_and_config_transitions.yml

@@ -31,7 +31,6 @@ selector:
     # can cause OOM kills on test hosts
     - jstests/concurrency/fsm_workloads/findAndModify_update_grow.js
 
-
     # cannot createIndex after dropDatabase without sharding first
     - jstests/concurrency/fsm_workloads/plan_cache_drop_database.js
 
@@ -88,8 +87,7 @@ executor:
       global_vars:
         TestData:
           runningWithBalancer: true
-          fsmPreOverridesLoadedCallback:
+          fsmPreOverridesLoadedCallback: import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
-            import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
   hooks:
     - class: ContinuousConfigShardTransition
     - class: CheckShardFilteringMetadata

buildscripts/resmokeconfig/suites/concurrency_sharded_terminate_primary_with_balancer.yml

@@ -39,7 +39,6 @@ selector:
     # can cause OOM kills on test hosts
     - jstests/concurrency/fsm_workloads/findAndModify_update_grow.js
 
-
     # cannot createIndex after dropDatabase without sharding first
     - jstests/concurrency/fsm_workloads/plan_cache_drop_database.js
 
@@ -160,8 +159,7 @@ executor:
           runningWithShardStepdowns: true
           useActionPermittedFile: true
           runningWithBalancer: true
-          fsmPreOverridesLoadedCallback:
+          fsmPreOverridesLoadedCallback: import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
-            import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
 
   hooks:
     - class: ContinuousStepdown

buildscripts/resmokeconfig/suites/concurrency_sharded_with_auto_bootstrap.yml

@@ -26,7 +26,6 @@ selector:
     # can cause OOM kills on test hosts
     - jstests/concurrency/fsm_workloads/findAndModify_update_grow.js
 
-
     # cannot createIndex after dropDatabase without sharding first
     - jstests/concurrency/fsm_workloads/plan_cache_drop_database.js
 

buildscripts/resmokeconfig/suites/concurrency_sharded_with_balancer_and_auto_bootstrap.yml

@@ -29,7 +29,6 @@ selector:
     # can cause OOM kills on test hosts
     - jstests/concurrency/fsm_workloads/findAndModify_update_grow.js
 
-
     # cannot createIndex after dropDatabase without sharding first
     - jstests/concurrency/fsm_workloads/plan_cache_drop_database.js
 
@@ -79,8 +78,7 @@ executor:
       global_vars:
         TestData:
           runningWithBalancer: true
-          fsmPreOverridesLoadedCallback:
+          fsmPreOverridesLoadedCallback: import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
-            import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
 
   hooks:
     - class: CheckShardFilteringMetadata

buildscripts/resmokeconfig/suites/concurrency_sharded_with_stepdowns.yml

@@ -35,7 +35,6 @@ selector:
     # can cause OOM kills on test hosts
     - jstests/concurrency/fsm_workloads/findAndModify_update_grow.js
 
-
     # cannot createIndex after dropDatabase without sharding first
     - jstests/concurrency/fsm_workloads/plan_cache_drop_database.js
 

buildscripts/resmokeconfig/suites/concurrency_sharded_with_stepdowns_and_balancer.yml

@@ -38,7 +38,6 @@ selector:
     # can cause OOM kills on test hosts
     - jstests/concurrency/fsm_workloads/findAndModify_update_grow.js
 
-
     # cannot createIndex after dropDatabase without sharding first
     - jstests/concurrency/fsm_workloads/plan_cache_drop_database.js
 
@@ -157,8 +156,7 @@ executor:
           runningWithShardStepdowns: true
           useActionPermittedFile: true
           runningWithBalancer: true
-          fsmPreOverridesLoadedCallback:
+          fsmPreOverridesLoadedCallback: import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
-            import("jstests/libs/override_methods/implicitly_retry_on_migration_in_progress.js");
 
   hooks:
     - class: ContinuousStepdown

buildscripts/resmokeconfig/suites/core_auth.yml

@@ -57,7 +57,7 @@ executor:
   fixture:
     class: MongoDFixture
     mongod_options:
-      auth: ''
+      auth: ""
       keyFile: *keyFile
       set_parameters:
         enableTestCommands: 1

buildscripts/resmokeconfig/suites/core_ese.yml

@@ -33,7 +33,7 @@ executor:
   fixture:
     class: MongoDFixture
     mongod_options:
-      enableEncryption: ''
+      enableEncryption: ""
       encryptionKeyFile: *keyFile
       set_parameters:
         enableTestCommands: 1

buildscripts/resmokeconfig/suites/cwrwc_rc_majority_passthrough.yml

@@ -52,7 +52,7 @@ executor:
       set_parameters:
         enableTestCommands: 1
         writePeriodicNoops: 1
-      enableMajorityReadConcern: ''
+      enableMajorityReadConcern: ""
     # This suite requires w="majority" writes to be applied on all secondaries. By using a 2-node
     # replica set and having secondaries vote, the majority of the replica set is all nodes.
     num_nodes: 2

buildscripts/resmokeconfig/suites/disk_wiredtiger.yml

@@ -7,7 +7,7 @@ selector:
 executor:
   config:
     shell_options:
-      nodb: ''
+      nodb: ""
       global_vars:
         TestData:
           storageEngine: wiredTiger

buildscripts/resmokeconfig/suites/ese.yml

@@ -8,4 +8,4 @@ selector:
 executor:
   config:
     shell_options:
-      nodb: ''
+      nodb: ""

buildscripts/resmokeconfig/suites/external_auth.yml

@@ -10,7 +10,7 @@ executor:
       global_vars:
         TestData:
           roleGraphInvalidationIsFatal: true
-      nodb: ''
+      nodb: ""
       gssapiServiceName: "mockservice"
       process_kwargs:
         env_vars:

buildscripts/resmokeconfig/suites/external_auth_aws.yml

@@ -7,4 +7,4 @@ selector:
 executor:
   config:
     shell_options:
-      nodb: ''
+      nodb: ""

buildscripts/resmokeconfig/suites/external_auth_oidc.yml

@@ -7,4 +7,4 @@ selector:
 executor:
   config:
     shell_options:
-      nodb: ''
+      nodb: ""

buildscripts/resmokeconfig/suites/external_auth_oidc_azure.yml

@@ -7,4 +7,4 @@ selector:
 executor:
   config:
     shell_options:
-      nodb: ''
+      nodb: ""

buildscripts/resmokeconfig/suites/external_auth_oidc_gcp.yml

@@ -7,4 +7,4 @@ selector:
 executor:
   config:
     shell_options:
-      nodb: ''
+      nodb: ""

buildscripts/resmokeconfig/suites/failpoints.yml

@@ -8,4 +8,4 @@ selector:
 executor:
   config:
     shell_options:
-      nodb: ''
+      nodb: ""

buildscripts/resmokeconfig/suites/failpoints_auth.yml

@@ -20,4 +20,4 @@ executor:
           keyFile: *keyFile
           keyFileData: *keyFileData
           roleGraphInvalidationIsFatal: true
-      nodb: ''
+      nodb: ""

buildscripts/resmokeconfig/suites/feature_flag_multiversion.yml

@@ -10,4 +10,4 @@ selector:
 executor:
   config:
     shell_options:
-      nodb: ''
+      nodb: ""

buildscripts/resmokeconfig/suites/fle.yml

@@ -8,10 +8,10 @@ selector:
 executor:
   config:
     shell_options:
-      nodb: ''
+      nodb: ""
-      ssl: ''
+      ssl: ""
-      tlsAllowInvalidHostnames: ''
+      tlsAllowInvalidHostnames: ""
-      tlsAllowInvalidCertificates: ''
+      tlsAllowInvalidCertificates: ""
       tlsCAFile: jstests/libs/ca.pem
       tlsCertificateKeyFile: jstests/libs/client.pem
       global_vars:

buildscripts/resmokeconfig/suites/fle2_query_analysis.yml

@@ -9,10 +9,10 @@ selector:
 executor:
   config:
     shell_options:
-      nodb: ''
+      nodb: ""
-      ssl: ''
+      ssl: ""
-      tlsAllowInvalidHostnames: ''
+      tlsAllowInvalidHostnames: ""
-      tlsAllowInvalidCertificates: ''
+      tlsAllowInvalidCertificates: ""
       tlsCAFile: jstests/libs/ca.pem
       tlsCertificateKeyFile: jstests/libs/client.pem
       global_vars:

buildscripts/resmokeconfig/suites/generational_fuzzer.yml

@@ -9,10 +9,10 @@ executor:
     tests: true
   config:
     shell_options:
-      nodb: ''
+      nodb: ""
       global_vars:
         TestData:
-          clusterType: 'standalone'
+          clusterType: "standalone"
           traceExceptions: false
           internalQueryAppendIdToSetWindowFieldsSort: true
           # Reduce the number of docs generated from $densify to avoid the 30min timeout.

buildscripts/resmokeconfig/suites/generational_fuzzer_replication.yml

@@ -9,7 +9,7 @@ executor:
     tests: true
   config:
     shell_options:
-      nodb: ''
+      nodb: ""
       global_vars:
         TestData:
-          clusterType: 'replset'
+          clusterType: "replset"

buildscripts/resmokeconfig/suites/initial_sync_fuzzer.yml

@@ -9,7 +9,7 @@ executor:
     tests: true
   config:
     shell_options:
-      nodb: ''
+      nodb: ""
       global_vars:
         TestData:
           logComponentVerbosity:

buildscripts/resmokeconfig/suites/integration_tests_replset_ssl_auth.yml

@@ -60,7 +60,7 @@ executor:
       tlsCertificateKeyFile: jstests/libs/server.pem
       keyFile: *keyFile
       clusterAuthMode: sendX509
-      auth: ''
+      auth: ""
     num_nodes: 2
     auth_options:
       <<: *authOptions

buildscripts/resmokeconfig/suites/jstestfuzz.yml

@@ -30,4 +30,4 @@ executor:
         enableTestCommands: 1
         internalQueryStatsRateLimit: -1
         internalQueryStatsErrorsAreCommandFatal: true
-      verbose: ''
+      verbose: ""

buildscripts/resmokeconfig/suites/jstestfuzz_interrupt.yml

@@ -30,4 +30,4 @@ executor:
       set_parameters:
         disableLogicalSessionCacheRefresh: false
         enableTestCommands: 1
-      verbose: ''
+      verbose: ""

buildscripts/resmokeconfig/suites/jstestfuzz_interrupt_replication.yml

@@ -42,5 +42,5 @@ executor:
         enableTestCommands: 1
         transactionLifetimeLimitSeconds: 1
         writePeriodicNoops: 1
-      verbose: ''
+      verbose: ""
     num_nodes: 2

buildscripts/resmokeconfig/suites/jstestfuzz_replication.yml

@@ -46,5 +46,5 @@ executor:
         writePeriodicNoops: 1
         internalQueryStatsRateLimit: -1
         internalQueryStatsErrorsAreCommandFatal: true
-      verbose: ''
+      verbose: ""
     num_nodes: 2

buildscripts/resmokeconfig/suites/jstestfuzz_replication_fcbis.yml

@@ -26,7 +26,7 @@ executor:
   fixture:
     class: ReplicaSetFixture
     mongod_options:
-      verbose: ''
+      verbose: ""
       set_parameters:
         disableLogicalSessionCacheRefresh: false
         enableTestCommands: 1

buildscripts/resmokeconfig/suites/jstestfuzz_replication_initsync.yml

@@ -26,7 +26,7 @@ executor:
   fixture:
     class: ReplicaSetFixture
     mongod_options:
-      verbose: ''
+      verbose: ""
       set_parameters:
         disableLogicalSessionCacheRefresh: false
         enableTestCommands: 1