Skip to content
Snippets Groups Projects
Commit 8459bd6c authored by Ian's avatar Ian
Browse files

fix: update redo tag validation and key prefix to use redo_of field

parent 4bc30b7e
No related branches found
No related tags found
No related merge requests found
...@@ -27,7 +27,7 @@ def get_existing_keys(config: RunConfig) -> Set[str]: ...@@ -27,7 +27,7 @@ def get_existing_keys(config: RunConfig) -> Set[str]:
Bucket="maap-ops-workspace", Bucket="maap-ops-workspace",
Prefix=( Prefix=(
f"{config.username}/dps_output/{config.algo_id}/" f"{config.username}/dps_output/{config.algo_id}/"
f"{config.algo_version}/{config.redo_tag}/" f"{config.algo_version}/{config.redo_of}/"
), ),
): ):
for obj in page.get("Contents", []): for obj in page.get("Contents", []):
...@@ -41,7 +41,7 @@ def get_existing_keys(config: RunConfig) -> Set[str]: ...@@ -41,7 +41,7 @@ def get_existing_keys(config: RunConfig) -> Set[str]:
def validate_redo_tag(config: RunConfig) -> None: def validate_redo_tag(config: RunConfig) -> None:
"""Validate redo tag parameters and check for existing outputs""" """Validate redo tag parameters and check for existing outputs"""
if not config.force_redo and config.redo_tag == config.tag: if not config.force_redo and config.redo_of == config.tag:
raise ValueError( raise ValueError(
f"Cannot redo with same tag '{config.tag}' " f"Cannot redo with same tag '{config.tag}' "
"- use --force-redo to override" "- use --force-redo to override"
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment