fix makefile

Author: diamonwiggins

applications/mlflow/Makefile

@@ -30,7 +30,7 @@ endef
 # Target to package charts and update versions
 .PHONY: package-and-update
 package-and-update: add-helm-repositories
-	@for chart in $(HELM_CHARTS_DIR)/*; do \
+	for chart in $(HELM_CHARTS_DIR)/*; do \
 	    echo "Packaging $$chart"; \
 	    helm package $$chart -u -d $(KOTS_DIR); \
 	    version=$$(eval $(call get_helm_chart_version,$$chart)); \
@@ -42,44 +42,44 @@ package-and-update: add-helm-repositories
 # Target to add Helm repositories
 .PHONY: add-helm-repositories
 add-helm-repositories:
-	@helm repo add cnpg https://cloudnative-pg.github.io/charts
-	@helm repo add minio https://operator.min.io/
-	@helm repo update
+	helm repo add cnpg https://cloudnative-pg.github.io/charts
+	helm repo add minio https://operator.min.io/
+	helm repo update
 
 # Target to lint Helm charts using chart-testing
 .PHONY: lint
 lint: add-helm-repositories
-	@echo "Linting Helm charts..."
-	@if [ -f ./ct.yaml ]; then \
+	echo "Linting Helm charts..."
+	if [ -f ./ct.yaml ]; then \
 		ct lint --config ./ct.yaml --chart-dirs $(HELM_CHARTS_DIR) --charts $(HELM_CHARTS_DIR)/mlflow $(HELM_CHARTS_DIR)/infra; \
 	else \
 		echo "Warning: ct.yaml not found. Using default configuration."; \
 		ct lint --chart-dirs $(HELM_CHARTS_DIR) --charts $(HELM_CHARTS_DIR)/mlflow $(HELM_CHARTS_DIR)/infra; \
 	fi
-	@echo "Linting completed successfully."
+	echo "Linting completed successfully."
 
 # Target to template Helm charts (render templates without installing)
 # Explicitly disables the Replicated SDK for testing non-SDK scenarios
 .PHONY: template
 template: add-helm-repositories
-	@echo "Templating Helm charts with Replicated SDK disabled..."
-	@mkdir -p ./rendered-templates
-	@for chart in $(CHARTS); do \
+	echo "Templating Helm charts with Replicated SDK disabled..."
+	mkdir -p ./rendered-templates
+	for chart in $(CHARTS); do \
 		echo "Rendering templates for $$chart..."; \
 		if [ "$$chart" = "mlflow" ]; then \
 			helm template $(HELM_CHARTS_DIR)/$$chart --output-dir ./rendered-templates/$$chart --set replicated.enabled=false --debug; \
 		else \
 			helm template $(HELM_CHARTS_DIR)/$$chart --output-dir ./rendered-templates/$$chart --debug; \
 		fi \
 	done
-	@echo "Templates rendered in ./rendered-templates directory."
+	echo "Templates rendered in ./rendered-templates directory."
 
 # Target to run a full install test (requires a Kubernetes cluster)
 .PHONY: test
 test: lint template
-	@echo "Running full chart installation test..."
-	@echo "Note: This requires a connected Kubernetes cluster."
-	@if [ -f ./ct.yaml ]; then \
+	echo "Running full chart installation test..."
+	echo "Note: This requires a connected Kubernetes cluster."
+	if [ -f ./ct.yaml ]; then \
 		ct install --config ./ct.yaml --chart-dirs $(HELM_CHARTS_DIR) \
 			--charts $(HELM_CHARTS_DIR)/infra && \
 		ct install --config ./ct.yaml --chart-dirs $(HELM_CHARTS_DIR) \
@@ -91,31 +91,31 @@ test: lint template
 		ct install --chart-dirs $(HELM_CHARTS_DIR) \
 			--charts $(HELM_CHARTS_DIR)/mlflow --set replicated.enabled=false; \
 	fi
-	@echo "Full installation test completed successfully."
+	echo "Full installation test completed successfully."
 
 # Target to login to the Replicated registry
 .PHONY: registry-login
 registry-login:
-	@echo "Authenticating with Replicated registry..."
-	@if [ -z "$$REPLICATED_LICENSE_ID" ]; then \
+	echo "Authenticating with Replicated registry..."
+	if [ -z "$$REPLICATED_LICENSE_ID" ]; then \
 		echo "Error: REPLICATED_LICENSE_ID environment variable must be set"; \
 		exit 1; \
 	fi
-	@helm registry login registry.replicated.com \
+	helm registry login registry.replicated.com \
 		--username="$$REPLICATED_LICENSE_ID" \
 		--password="$$REPLICATED_LICENSE_ID"
-	@echo "Registry login successful."
+	echo "Registry login successful."
 
 # Target to test Helm installation with charts from Replicated registry
 .PHONY: test-replicated-helm
 test-replicated-helm: registry-login
-	@echo "Running Helm installation test with charts from Replicated registry..."
-	@echo "Note: This requires REPLICATED_APP and REPLICATED_CHANNEL env vars."
-	@if [ -z "$$REPLICATED_APP" ] || [ -z "$$REPLICATED_CHANNEL" ]; then \
+	echo "Running Helm installation test with charts from Replicated registry..."
+	echo "Note: This requires REPLICATED_APP and REPLICATED_CHANNEL env vars."
+	if [ -z "$$REPLICATED_APP" ] || [ -z "$$REPLICATED_CHANNEL" ]; then \
 		echo "Error: REPLICATED_APP and REPLICATED_CHANNEL must be set"; \
 		exit 1; \
 	fi
-	@if [ -f ./ct.yaml ]; then \
+	if [ -f ./ct.yaml ]; then \
 		ct install --config ./ct.yaml \
 			--chart-repos "replicated=oci://registry.replicated.com/$$REPLICATED_APP/$$REPLICATED_CHANNEL" \
 			--charts "replicated/infra" && \
@@ -131,12 +131,12 @@ test-replicated-helm: registry-login
 			--chart-repos "replicated=oci://registry.replicated.com/$$REPLICATED_APP/$$REPLICATED_CHANNEL" \
 			--charts "replicated/mlflow"; \
 	fi
-	@echo "Replicated Helm installation test completed successfully."
+	echo "Replicated Helm installation test completed successfully."
 
 # Example target to check versions (optional)
 .PHONY: check-versions
 check-versions:
-	@for chart_dir in $(HELM_CHARTS_DIR)/*; do \
+	for chart_dir in $(HELM_CHARTS_DIR)/*; do \
 	    if [ -d $$chart_dir ]; then \
 	        version=$$(eval $(call get_helm_chart_version,$$chart_dir)); \
 	        echo "$$chart_dir version: $$version"; \