From 1841e89ed06c432c7158c98aa4963062469807d6 Mon Sep 17 00:00:00 2001 From: Thomas E Lackey Date: Mon, 5 Aug 2024 23:26:16 -0500 Subject: [PATCH] Add option to recreate deployments rather than update them. --- .../webapp/deploy_webapp_from_registry.py | 6 ++++- stack_orchestrator/deploy/webapp/util.py | 27 ++++++++++++------- 2 files changed, 22 insertions(+), 11 deletions(-) diff --git a/stack_orchestrator/deploy/webapp/deploy_webapp_from_registry.py b/stack_orchestrator/deploy/webapp/deploy_webapp_from_registry.py index ba01c9e5..e29eb6d5 100644 --- a/stack_orchestrator/deploy/webapp/deploy_webapp_from_registry.py +++ b/stack_orchestrator/deploy/webapp/deploy_webapp_from_registry.py @@ -45,6 +45,7 @@ def process_app_deployment_request( image_registry, force_rebuild, fqdn_policy, + recreate_on_deploy, logger ): logger.log("BEGIN - process_app_deployment_request") @@ -165,6 +166,7 @@ def process_app_deployment_request( deploy_to_k8s( deployment_record, deployment_dir, + recreate_on_deploy, logger ) @@ -220,12 +222,13 @@ def dump_known_requests(filename, requests, status="SEEN"): @click.option("--include-tags", help="Only include requests with matching tags (comma-separated).", default="") @click.option("--exclude-tags", help="Exclude requests with matching tags (comma-separated).", default="") @click.option("--force-rebuild", help="Rebuild even if the image already exists.", is_flag=True) +@click.option("--recreate-on-deploy", help="Remove and recreate deployments instead of updating them.", is_flag=True) @click.option("--log-dir", help="Output build/deployment logs to directory.", default=None) @click.pass_context def command(ctx, kube_config, laconic_config, image_registry, deployment_parent_dir, # noqa: C901 request_id, discover, state_file, only_update_state, dns_suffix, fqdn_policy, record_namespace_dns, record_namespace_deployments, dry_run, - include_tags, exclude_tags, force_rebuild, log_dir): + include_tags, exclude_tags, force_rebuild, recreate_on_deploy, log_dir): if request_id and discover: print("Cannot specify both --request-id and --discover", file=sys.stderr) sys.exit(2) @@ -365,6 +368,7 @@ def command(ctx, kube_config, laconic_config, image_registry, deployment_parent_ image_registry, force_rebuild, fqdn_policy, + recreate_on_deploy, logger ) status = "DEPLOYED" diff --git a/stack_orchestrator/deploy/webapp/util.py b/stack_orchestrator/deploy/webapp/util.py index 97e756d9..5c5d014d 100644 --- a/stack_orchestrator/deploy/webapp/util.py +++ b/stack_orchestrator/deploy/webapp/util.py @@ -309,17 +309,24 @@ def push_container_image(deployment_dir, logger): logger.log("Finished pushing images.") -def deploy_to_k8s(deploy_record, deployment_dir, logger): - if not deploy_record: - command = "start" - else: - command = "update" - +def deploy_to_k8s(deploy_record, deployment_dir, recreate, logger): logger.log("Deploying to k8s ...") - logger.log(f"Running {command} command on deployment dir: {deployment_dir}") - result = subprocess.run([sys.argv[0], "deployment", "--dir", deployment_dir, command], - stdout=logger.file, stderr=logger.file) - result.check_returncode() + + if recreate: + commands_to_run = ["stop", "start"] + else: + if not deploy_record: + commands_to_run = ["start"] + else: + commands_to_run = ["update"] + + for command in commands_to_run: + logger.log(f"Running {command} command on deployment dir: {deployment_dir}") + result = subprocess.run([sys.argv[0], "deployment", "--dir", deployment_dir, command], + stdout=logger.file, stderr=logger.file) + result.check_returncode() + logger.log(f"Finished {command} command on deployment dir: {deployment_dir}") + logger.log("Finished deploying to k8s.")