Skip to content
This repository has been archived by the owner on Aug 25, 2021. It is now read-only.

removing line 47: -consul-image="{{… #63

Closed
wants to merge 1 commit into from
Closed

removing line 47: -consul-image="{{… #63

wants to merge 1 commit into from

Conversation

TheMacStack
Copy link

due to error on pod

callum@minta:~/code/demo/consul-helm$ kubectl logs consul-connect-injector-webhook-deployment-5cfb588495-q845p -n default
flag provided but not defined: -consul-image
Usage:
  -default-inject
        Inject by default. (default true)
  -listen string
        Address to bind listener to. (default ":8080")
  -tls-auto string
        MutatingWebhookConfiguration name. If specified, will auto generate cert bundle.
  -tls-auto-hosts string
        Comma-separated hosts for auto-generated TLS cert. If specified, will auto generate cert bundle.
  -tls-cert-file string
        PEM-encoded TLS certificate to serve. If blank, will generate random cert.
  -tls-key-file string
        PEM-encoded TLS private key to serve. If blank, will generate random cert.

After removing that line the pod launches cleanly

…ues.connectInject.imageConsul }}" \

due to error on pod

```
callum@minta:~/code/demo/consul-helm$ kubectl logs consul-connect-injector-webhook-deployment-5cfb588495-q845p -n monitoring
flag provided but not defined: -consul-image
Usage:
  -default-inject
        Inject by default. (default true)
  -listen string
        Address to bind listener to. (default ":8080")
  -tls-auto string
        MutatingWebhookConfiguration name. If specified, will auto generate cert bundle.
  -tls-auto-hosts string
        Comma-separated hosts for auto-generated TLS cert. If specified, will auto generate cert bundle.
  -tls-cert-file string
        PEM-encoded TLS certificate to serve. If blank, will generate random cert.
  -tls-key-file string
        PEM-encoded TLS private key to serve. If blank, will generate random cert.
```
@adilyse
Copy link
Contributor

adilyse commented Dec 4, 2018

As discussed in #64 , this is not the preferred solution to the issue you've encountered, since it removes the ability to configure the Consul version. This will be very important moving forward, as we will continue to release updated Consul versions.

@adilyse adilyse closed this Dec 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants