kingjaypee12/MKM
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

ci: improve deployment script verbosity and error handling

Browse Source 
- 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
This commit is contained in:
Jp
2026-02-26 11:06:21 +08:00
parent baf68078a7
commit 618f826e1e
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
Makefile
View File

@@ -6,13 +6,13 @@ build:
deploy: build deploy: build
@if [ ! -f .env ]; then cp .env.example .env; fi @if [ ! -f .env ]; then cp .env.example .env; fi
@echo "Cleaning up existing deployment if any..." @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..." @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 @echo "---" >> k8s/merged_deployment.yaml
@cat k8s/deployment.yaml >> k8s/merged_deployment.yaml @cat k8s/deployment.yaml >> k8s/merged_deployment.yaml
@echo "Deploying via podman kube play..." @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 @rm k8s/merged_deployment.yaml
stop: stop: