From b3c55772090d5167e69a41f8858da4af04f345dc Mon Sep 17 00:00:00 2001 From: Neil Massey Date: Wed, 9 Oct 2024 15:46:18 +0100 Subject: [PATCH] Only connect to S3 if deleting from object storage --- nlds_utils/reset_object_status.py | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/nlds_utils/reset_object_status.py b/nlds_utils/reset_object_status.py index ed9c8025..2cd9867a 100755 --- a/nlds_utils/reset_object_status.py +++ b/nlds_utils/reset_object_status.py @@ -108,12 +108,14 @@ def reset_object_status( raise click.UsageError("Error - group not specified") if holding_id is None: raise click.UsageError("Error - holding id not specified") - if access_key is None: - raise click.UsageError("Error - access key not specified") - if secret_key is None: - raise click.UsageError("Error - secret key not specified") - s3_client = _connect_to_s3(access_key, secret_key) + # only need to contact S3 if deleting from object storage + if delete: + s3_client = _connect_to_s3(access_key, secret_key) + if access_key is None: + raise click.UsageError("Error - access key not specified") + if secret_key is None: + raise click.UsageError("Error - secret key not specified") nlds_cat = _connect_to_catalog() nlds_cat.start_session()