Skip to content

Commit 097779f

Browse files
committed
fix kms_setup
1 parent 9f7ec22 commit 097779f

File tree

3 files changed

+12
-12
lines changed

3 files changed

+12
-12
lines changed

.evergreen/scripts/cleanup.sh

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,4 @@
11
#!/bin/bash
22

3-
if [ -f "$DRIVERS_TOOLS"/.evergreen/csfle/secrets-export.sh ]; then
4-
bash .evergreen/teardown-encryption.sh
5-
fi
63
rm -rf "${DRIVERS_TOOLS}" || true
74
rm -f ./secrets-export.sh || true

.evergreen/scripts/kms_tester.py

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -40,12 +40,19 @@ def _setup_gcp_vm(base_env: dict[str, str]) -> None:
4040
LOGGER.info("Setting up GCP VM...")
4141

4242

43-
def _create_archive():
43+
def _create_archive() -> None:
4444
run_command("git add .", cwd=ROOT)
4545
run_command('git commit -m "add files"', check=False, cwd=ROOT)
4646
run_command(f"git archive -o {TMP_DRIVER_FILE} HEAD", cwd=ROOT)
4747

4848

49+
def _load_kms_config(sub_test_target: str) -> None:
50+
config = read_env(f"{CSFLE_FOLDER}/{sub_test_target}kms/secrets-export.sh")
51+
base_env = os.environ.copy()
52+
for key, value in config.items():
53+
base_env[key] = str(value)
54+
55+
4956
def setup_kms(sub_test_name: str) -> None:
5057
if "-" in sub_test_name:
5158
sub_test_target, sub_test_type = sub_test_name.split("-")
@@ -73,10 +80,7 @@ def setup_kms(sub_test_name: str) -> None:
7380
if sub_test_target == "azure":
7481
os.environ["AZUREKMS_VMNAME_PREFIX"] = "PYTHON_DRIVER"
7582
run_command(f"{CSFLE_FOLDER}/{sub_test_target}kms/setup.sh")
76-
config = read_env(f"{CSFLE_FOLDER}/{sub_test_target}kms/secrets-export.sh")
77-
base_env = os.environ.copy()
78-
for key, value in config.items():
79-
base_env[key] = str(value)
83+
base_env = _load_kms_config(sub_test_target)
8084

8185
if sub_test_target == "azure":
8286
_setup_azure_vm(base_env)
@@ -91,17 +95,16 @@ def setup_kms(sub_test_name: str) -> None:
9195

9296

9397
def test_kms_vm(sub_test_name: str) -> None:
94-
env = os.environ.copy()
98+
env = _load_kms_config(sub_test_name)
9599
if sub_test_name == "azure":
96100
key_name = os.environ["KEY_NAME"]
97101
key_vault_endpoint = os.environ["KEY_VAULT_ENDPOINT"]
98102
env[
99103
"AZUREKMS_CMD"
100104
] = f'KEY_NAME="{key_name}" KEY_VAULT_ENDPOINT="{key_vault_endpoint}" bash ./.evergreen/just.sh run-tests'
101-
cmd = f"{CSFLE_FOLDER}/azurekms/run-command.sh"
102105
else:
103106
env["GCPKMS_CMD"] = "./.evergreen/just.sh run-tests"
104-
cmd = f"{CSFLE_FOLDER}/gcpkms/run-command.sh"
107+
cmd = f"{CSFLE_FOLDER}/{sub_test_name}kms/run-command.sh"
105108
run_command(cmd, env=env)
106109

107110

.evergreen/scripts/setup-dev-env.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
#!/bin/bash
22

3-
set -eux
3+
set -eu
44

55
HERE=$(dirname ${BASH_SOURCE:-$0})
66
HERE="$( cd -- "$HERE" > /dev/null 2>&1 && pwd )"

0 commit comments

Comments
 (0)