From 67d602e7a6aa936a541b6610bfa63c8fb0c0f8b5 Mon Sep 17 00:00:00 2001 From: Kai Fricke Date: Tue, 10 May 2022 09:24:10 +0200 Subject: [PATCH] [ci] Fix automatic buildkite token fetching in fetch_release_logs.py (#24606) The script expected a return string, not an environment variable. --- release/release_logs/fetch_release_logs.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/release/release_logs/fetch_release_logs.py b/release/release_logs/fetch_release_logs.py index a4c596920..fe955e97f 100644 --- a/release/release_logs/fetch_release_logs.py +++ b/release/release_logs/fetch_release_logs.py @@ -95,7 +95,7 @@ def maybe_fetch_buildkite_token() -> str: return buildkite_token print("Missing BUILDKITE_TOKEN, retrieving from AWS secrets store") - os.environ["BUILDKITE_TOKEN"] = boto3.client( + buildkite_token = boto3.client( "secretsmanager", region_name="us-west-2" ).get_secret_value( SecretId="arn:aws:secretsmanager:us-west-2:029272617770:secret:" @@ -103,6 +103,8 @@ def maybe_fetch_buildkite_token() -> str: )[ "SecretString" ] + os.environ["BUILDKITE_TOKEN"] = buildkite_token + return buildkite_token def get_results_from_build_collection(