search

aledbf / kube-keepalived-vip

Kubernetes Virtual IP address/es using keepalived
Apache License 2.0
189 stars 75 forks source link

conflict custom notify script and healthcheck #95

Open unteem opened 5 years ago

unteem commented 5 years ago

Hi. There is an issue if you use a notify script with env variable KEEPALIVED_NOTIFY as there is already a notify script for the health check. It seems that keepalived only accept one notify script.

Workaround is to add the health check script in the custom notify script but it might be best for the health check to use a check script (like its done for haproxy) instead of notify script? What do you think?

steven-sheehy commented 5 years ago

It can't use a script since it specifically needs the MASTER/BACKUP state that the notify script provides it as parameters.