From 618f826e1eec782cc8c8e9c887ecfc3ca0eb8bcb Mon Sep 17 00:00:00 2001 From: Jp Date: Thu, 26 Feb 2026 11:06:21 +0800 Subject: [PATCH] ci: improve deployment script verbosity and error handling - Remove stderr suppression from podman kube down to show potential errors - Remove @ prefix from kubectl command to show execution in output - Add comment clarifying the purpose of removing stderr redirection --- Makefile | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Makefile b/Makefile index 115d04d..d54854b 100644 --- a/Makefile +++ b/Makefile @@ -6,13 +6,13 @@ build: deploy: build @if [ ! -f .env ]; then cp .env.example .env; fi @echo "Cleaning up existing deployment if any..." - -podman kube down k8s/deployment.yaml >/dev/null 2>&1 + -podman kube down k8s/deployment.yaml @echo "Generating merged deployment with ConfigMap..." - @kubectl create configmap mkm-admin-config --from-env-file=.env --dry-run=client -o yaml > k8s/merged_deployment.yaml + kubectl create configmap mkm-admin-config --from-env-file=.env --dry-run=client -o yaml > k8s/merged_deployment.yaml @echo "---" >> k8s/merged_deployment.yaml @cat k8s/deployment.yaml >> k8s/merged_deployment.yaml @echo "Deploying via podman kube play..." - podman kube play k8s/merged_deployment.yaml + podman kube play k8s/merged_deployment.yaml # Remove the pipe to stderr to see full output @rm k8s/merged_deployment.yaml stop: