Skip to content

Commit

Permalink
only add --certificate-authority if either k8s_ca_uri or k8s_ca_pem i…
Browse files Browse the repository at this point in the history 
…s provided.
  • Loading branch information
@nrobert13
nrobert13 committed Mar 8, 2019
1 parent d9b7934 commit be16853
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions templates/linux-mac-common.html
View file
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,9 @@ <h3>Run configuration commands</h3>
<img class="clippy" width="13" src="{{ .Web_Path_Prefix }}static/clippy.svg" alt="">
</button>
<pre><code>kubectl config set-cluster {{ .ClusterName }} \
{{- if or .K8sCaPem .K8sCaURI }}
--certificate-authority=${HOME}/.kube/certs/{{ .ClusterName}}/k8s-ca.crt \
{{- end }}
--server={{ .K8sMasterURI }}</code></pre>
</div>

Expand Down

2 comments on commit be16853

@bjornryden
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spotted this as it broke our setup, but this may be due to faulty configuration. However, shouldn't this feature at least be consistent with the Windows configuration too?

@nrobert13
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bjornryden, I guess you're right. Thanks for pointing it out.
#90

Please sign in to comment.