From 469ff7a086da874b71abd01078d12cd5a74eec6d Mon Sep 17 00:00:00 2001 From: Thomas E Lackey Date: Mon, 15 Apr 2024 20:27:35 +0000 Subject: [PATCH] Skip checks on requests we've already seen (#805) Reviewed-on: https://git.vdb.to/cerc-io/stack-orchestrator/pulls/805 Co-authored-by: Thomas E Lackey Co-committed-by: Thomas E Lackey --- .../deploy/webapp/deploy_webapp_from_registry.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/stack_orchestrator/deploy/webapp/deploy_webapp_from_registry.py b/stack_orchestrator/deploy/webapp/deploy_webapp_from_registry.py index f030b735..7c21b2a8 100644 --- a/stack_orchestrator/deploy/webapp/deploy_webapp_from_registry.py +++ b/stack_orchestrator/deploy/webapp/deploy_webapp_from_registry.py @@ -256,7 +256,10 @@ def command(ctx, kube_config, laconic_config, image_registry, deployment_parent_ requests_by_name = {} skipped_by_name = {} for r in requests: - # TODO: Do this _after_ filtering deployments and cancellations to minimize round trips. + if r.id in previous_requests and previous_requests[r.id].get("status", "") != "RETRY": + print(f"Skipping request {r.id}, we've already seen it.") + continue + app = laconic.get_record(r.attributes.application) if not app: print("Skipping request %s, cannot locate app." % r.id)