connect sidecar add healthcheck in annotation

[Veres72]

Now, consul sidecar check services through ip:port. If service listening port, but don't have success probe, consul direct traffic in the dead pod.
Add some annotations such as:
"consul.hashicorp.com/service-check": "http://127.0.0.1:8080/ping/serice/alive':10s"

May be here add checks

consul-k8s/connect-inject/container_init.go

Lines 245 to 261 in 8b83fd9

	services {
	id = "${POD_NAME}-{{ .ServiceName }}"
	name = "{{ .ServiceName }}"
	address = "${POD_IP}"
	port = {{ .ServicePort }}
	{{- if .Tags}}
	tags = {{.Tags}}
	{{- end}}
	{{- if .Meta}}
	meta = {
	{{- range $key, $value := .Meta }}
	{{$key}} = "{{$value}}"
	{{- end }}
	}
	{{- end}}
	}
	EOF

example:

  checks {
    name = "{{ .ServiceName }} check"
    http = "{{ .checkPath }}"
    interval = "{{ .checkIntervas }}"
  }

[lkysow]

Hi all, with the release of consul-helm 0.26.0 we support syncing kube readiness status to Consul. So instead of setting checks via annotations in the PR can you try creating kube readiness probes? That way Consul and Kube will agree on the readiness status.

If that works we will close this issue, however if there's a reason that that won't work please let us know.

[Veres72]

Hi.
Unfortunately I don't have opportunity to check it. I think you can close a issue.