From e9e276464a1394cb4c6d42ae4a11eb307cb9944a Mon Sep 17 00:00:00 2001 From: Andreas Putzo Date: Fri, 16 Mar 2018 11:52:47 +0100 Subject: [PATCH] Do not suggest to potentially use bultin echo From man(1) echo on MacOS: Some shells may provide a builtin echo command which is similar or identical to this utility. Most notably, the builtin echo in sh(1) does not accept the -n option. Following the setuo instructions of the chart it's quite hard to spot if builtin echo is used which is why it might be better to use absolute path. --- templates/NOTES.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/templates/NOTES.txt b/templates/NOTES.txt index 96e7076..95dc23e 100644 --- a/templates/NOTES.txt +++ b/templates/NOTES.txt @@ -1,7 +1,7 @@ 1. Please run the following commands now to configure the link between Kubernetes and StorageOS(*): ClusterIP=$(kubectl get svc/{{ .Values.service.name }} --namespace {{ .Values.service.namespace }} -o custom-columns=IP:spec.clusterIP --no-headers=true) -ApiAddress=$(echo -n "tcp://$ClusterIP:{{ .Values.service.externalPort }}" | base64) +ApiAddress=$(/bin/echo -n "tcp://$ClusterIP:{{ .Values.service.externalPort }}" | base64) kubectl patch secret/{{ .Values.api.secretName }} --namespace {{ .Values.api.secretNamespace }} --patch "{\"data\": {\"apiAddress\": \"$ApiAddress\"}}" *) Unfortunately this is needed until the Kubernetes apiserver can use kube-dns to resolve service