chore: Apply isort and shellcheck formatting

This commit contains auto-formatting changes:
- isort formatting for Python files (profile=black, line_length=120)
- shellcheck fixes for shell files (require-variable-braces)

These files should not require detailed review.

Author: nammn

docker/cluster-cleaner/scripts/clean-failed-namespaces.sh

@@ -7,12 +7,12 @@ delete_resources_safely() {
     resource_type="$1"
     namespace="$2"
 
-    echo "Attempting normal deletion of $resource_type in $namespace..."
+    echo "Attempting normal deletion of ${resource_type} in ${namespace}..."
     kubectl delete "${resource_type}" --all -n "${namespace}" --wait=true --timeout=10s 2>error.log|| true
 
     # Check if any resources are still stuck
     # Let's not fail here and continue deletion
-    resources=$(kubectl get "$resource_type" -n "${namespace}" --no-headers -o custom-columns=":metadata.name" 2>error.log || true)
+    resources=$(kubectl get "${resource_type}" -n "${namespace}" --no-headers -o custom-columns=":metadata.name" 2>error.log || true)
 
     for resource in ${resources}; do
         echo "${resource_type}/${resource} is still present, force deleting..."
@@ -39,7 +39,7 @@ kubectl get namespace -l "${LABELS}" -o name
 for namespace in $(kubectl get namespace -l "${LABELS}" -o name 2>error.log); do
     creation_time=$(kubectl get "${namespace}" -o jsonpath='{.metadata.creationTimestamp}' 2>error.log || echo "")
 
-    if [ -z "$creation_time" ]; then
+    if [ -z "${creation_time}" ]; then
         echo "Namespace ${namespace} does not exist or has no creation timestamp, skipping."
         continue
     fi

docker/delve-sidecar/attach-delve.sh

@@ -8,6 +8,6 @@ while ! pgrep -f "mongodb-mms-automation-agent" > /dev/null; do
 done
 
 APP_PID=$(pgrep -f "mongodb-mms-automation-agent")
-echo "Found app with PID: $APP_PID, attaching Delve..."
+echo "Found app with PID: ${APP_PID}, attaching Delve..."
 
-dlv attach "$APP_PID" --headless --listen=:2345 --api-version=2 --accept-multiclient --continue
+dlv attach "${APP_PID}" --headless --listen=:2345 --api-version=2 --accept-multiclient --continue

docker/mongodb-agent/agent-launcher-shim.sh

@@ -9,13 +9,13 @@ SCRIPTS_DIR="/opt/scripts"
 # Function to start the agent launcher
 start_agent_launcher() {
   echo "Starting agent launcher..."
-  echo "Final contents of $SCRIPTS_DIR:"
-  ls -la "$SCRIPTS_DIR"
+  echo "Final contents of ${SCRIPTS_DIR}:"
+  ls -la "${SCRIPTS_DIR}"
 
-  if [[ -f "$SCRIPTS_DIR/agent-launcher.sh" ]]; then
+  if [[ -f "${SCRIPTS_DIR}/agent-launcher.sh" ]]; then
     echo "Found agent-launcher.sh, executing..."
     echo "Note: agent-launcher.sh will become PID 1 and handle all signal processing including cleanup"
-    exec "$SCRIPTS_DIR/agent-launcher.sh"
+    exec "${SCRIPTS_DIR}/agent-launcher.sh"
   else
     echo "ERROR: agent-launcher.sh not found"
     exit 1

docker/mongodb-agent/dummy-probe.sh

@@ -4,34 +4,34 @@
 find_init_container() {
   local pid
   pid=$(pgrep -f "agent-utilities-holder_marker" | head -n1)
-  if [[ -n "$pid" && -d "/proc/$pid/root/probes" ]]; then
-    echo "$pid"
+  if [[ -n "${pid}" && -d "/proc/${pid}/root/probes" ]]; then
+    echo "${pid}"
     return 0
   fi
   return 1
 }
 
 execute_liveness_probe() {
   local init_pid="$1"
-  local init_probe_path="/proc/$init_pid/root/probes/probe.sh"
+  local init_probe_path="/proc/${init_pid}/root/probes/probe.sh"
 
-  if [[ ! -f "$init_probe_path" ]]; then
-    echo "ERROR: Liveness probe script not found at $init_probe_path"
+  if [[ ! -f "${init_probe_path}" ]]; then
+    echo "ERROR: Liveness probe script not found at ${init_probe_path}"
     exit 1
-  elif [[ ! -x "$init_probe_path" ]]; then
-    echo "ERROR: Liveness probe script not executable at $init_probe_path"
+  elif [[ ! -x "${init_probe_path}" ]]; then
+    echo "ERROR: Liveness probe script not executable at ${init_probe_path}"
     exit 1
   else
     # Execute the actual probe script from the init-database container
     # This works because of shared process namespace - the probe can see all processes
-    exec "$init_probe_path"
+    exec "${init_probe_path}"
   fi
 }
 
 # Main execution
 if init_pid=$(find_init_container); then
-  echo "Found init container with PID: $init_pid, executing liveness probe..."
-  execute_liveness_probe "$init_pid"
+  echo "Found init container with PID: ${init_pid}, executing liveness probe..."
+  execute_liveness_probe "${init_pid}"
 else
   echo "WARNING: Init container not found, falling back to basic liveness check"
   # Fallback: if we can't find the init container, just check if this container is alive

docker/mongodb-agent/setup-agent-files.sh

@@ -8,8 +8,8 @@ SCRIPTS_DIR="/opt/scripts"
 # readiness always returns failure
 setup_dummy_probes() {
   echo "Setting up dummy probe scripts..."
-  cp --remove-destination /usr/local/bin/dummy-probe.sh "$SCRIPTS_DIR/probe.sh"
-  cp --remove-destination /usr/local/bin/dummy-readinessprobe "$SCRIPTS_DIR/readinessprobe"
+  cp --remove-destination /usr/local/bin/dummy-probe.sh "${SCRIPTS_DIR}/probe.sh"
+  cp --remove-destination /usr/local/bin/dummy-readinessprobe "${SCRIPTS_DIR}/readinessprobe"
   echo "Dummy probe scripts ready"
 }
 
@@ -18,11 +18,11 @@ find_init_container() {
   for i in {1..150}; do
     local pid
     pid=$(pgrep -f "agent-utilities-holder_marker" | head -n1)
-    if [[ -n "$pid" && -d "/proc/$pid/root/scripts" ]]; then
-      echo "$pid"
+    if [[ -n "${pid}" && -d "/proc/${pid}/root/scripts" ]]; then
+      echo "${pid}"
       return 0
     fi
-    echo "Waiting for init container... (attempt $i)" >&2
+    echo "Waiting for init container... (attempt ${i})" >&2
     sleep 2
   done
   return 1
@@ -33,8 +33,8 @@ link_agent_scripts() {
 
   echo "Linking agent launcher scripts..."
   for script in agent-launcher.sh agent-launcher-lib.sh; do
-    ln -sf "$init_scripts_dir/$script" "$SCRIPTS_DIR/$script"
-    echo "Linked $script"
+    ln -sf "${init_scripts_dir}/${script}" "${SCRIPTS_DIR}/${script}"
+    echo "Linked ${script}"
   done
 }
 
@@ -44,26 +44,26 @@ main() {
   setup_dummy_probes
 
   if init_pid=$(find_init_container); then
-    echo "Found init container with PID: $init_pid"
+    echo "Found init container with PID: ${init_pid}"
 
-    init_root="/proc/$init_pid/root"
-    init_scripts="$init_root/scripts"
-    init_probes="$init_root/probes"
+    init_root="/proc/${init_pid}/root"
+    init_scripts="${init_root}/scripts"
+    init_probes="${init_root}/probes"
 
     # Verify scripts directory exists
-    if [[ ! -d "$init_scripts" ]]; then
-      echo "ERROR: Scripts directory $init_scripts not found"
+    if [[ ! -d "${init_scripts}" ]]; then
+      echo "ERROR: Scripts directory ${init_scripts} not found"
       exit 1
     fi
 
     # Verify probes directory exists
-    if [[ ! -d "$init_probes" ]]; then
-      echo "ERROR: Probes directory $init_probes not found"
+    if [[ ! -d "${init_probes}" ]]; then
+      echo "ERROR: Probes directory ${init_probes} not found"
       exit 1
     fi
 
     # Link scripts from init container
-    link_agent_scripts "$init_scripts"
+    link_agent_scripts "${init_scripts}"
 
     echo "File setup completed successfully"
     exit 0

docker/mongodb-kubernetes-init-database/content/probe.sh

@@ -4,7 +4,7 @@ set -Eeou pipefail
 check_process() {
   local check_process=$1
   # shellcheck disable=SC2009
-  ps -ax | grep -v " grep " | grep -v jq | grep -v tail | grep "$check_process"
+  ps -ax | grep -v " grep " | grep -v jq | grep -v tail | grep "${check_process}"
   return $?
 }
 

docker/mongodb-kubernetes-tests/kubetester/__init__.py

@@ -11,10 +11,7 @@
 
 # Re-exports
 from .kubetester import fixture as find_fixture
-from .security_context import (
-    assert_pod_container_security_context,
-    assert_pod_security_context,
-)
+from .security_context import assert_pod_container_security_context, assert_pod_security_context
 
 
 def create_secret(

docker/mongodb-kubernetes-tests/kubetester/certs.py

@@ -13,22 +13,12 @@
 from kubeobject import CustomObject
 from kubernetes import client
 from kubernetes.client.rest import ApiException
-from kubetester import (
-    create_secret,
-    delete_secret,
-    kubetester,
-    random_k8s_name,
-    read_secret,
-)
+from kubetester import create_secret, delete_secret, kubetester, random_k8s_name, read_secret
 from kubetester.kubetester import KubernetesTester
 from kubetester.phase import Phase
 from opentelemetry import trace
 from tests import test_logger
-from tests.vaultintegration import (
-    store_secret_in_vault,
-    vault_namespace_name,
-    vault_sts_name,
-)
+from tests.vaultintegration import store_secret_in_vault, vault_namespace_name, vault_sts_name
 
 TRACER = trace.get_tracer("evergreen-agent")
 logger = test_logger.get_test_logger(__name__)

docker/mongodb-kubernetes-tests/kubetester/mongodb.py

@@ -23,12 +23,7 @@
 
 from .mongodb_common import MongoDBCommon
 from .mongodb_utils_state import in_desired_state
-from .mongotester import (
-    MongoTester,
-    ReplicaSetTester,
-    ShardedClusterTester,
-    StandaloneTester,
-)
+from .mongotester import MongoTester, ReplicaSetTester, ShardedClusterTester, StandaloneTester
 from .opsmanager import MongoDBOpsManager
 from .phase import Phase
 

docker/mongodb-kubernetes-tests/kubetester/omtester.py

@@ -15,12 +15,7 @@
 import requests
 import semver
 from kubetester.automation_config_tester import AutomationConfigTester
-from kubetester.kubetester import (
-    KubernetesTester,
-    build_agent_auth,
-    build_auth,
-    run_periodically,
-)
+from kubetester.kubetester import KubernetesTester, build_agent_auth, build_auth, run_periodically
 from kubetester.mongotester import BackgroundHealthChecker
 from kubetester.om_queryable_backups import OMQueryableBackup
 from opentelemetry import trace