keepalived 1.2.17 does not call vrrp_instance notify script about keepalived HOT 11 CLOSED

I believe this patch is causing the problem:

5c41773

from keepalived.

Try with following configuration

notify {
/etc/keepalived_notify.sh
}

But I am considering reverting original patch causing this issue.

from keepalived.

Do you see the same issue if you also revert 89f9102?

from keepalived.

I also reverted that patch and it works as expected.

from keepalived.

Changing to the altered syntax as suggested does work.

If the syntax is changed to require curly braces, shouldn't that be documented somewhere? The man page has the old syntax.

from keepalived.

Also, I noticed that with the changed syntax, the notify script is actually called twice at startup. In my test, I'm simply logging the invocations to a log file, and I see this:

Thu Jun 18 10:46:51 EDT 2015 Called with args: INSTANCE pcr-ny4-mktdata-relay MASTER 200
Thu Jun 18 10:46:51 EDT 2015 Called with args: INSTANCE pcr-ny4-mktdata-relay MASTER 200

Is that intended? That seems like another bug...

from keepalived.

Unless two state transitions have occurred, then no, the the notification script should only get invoked once.

from keepalived.

Well yes, that's my point. I simply started up keepalived, and it immediately made those 2 calls to the notify script indicating MASTER state. Should I open a separate bug for this?

from keepalived.

Yes, open a separate bug. Could you show me your configuration please? I'll repro this and try and figure out where it's going wrong.

from keepalived.

Ajschorr, I note that you already posted your config in your first post.

from keepalived.

Ah, sorry, I would have posted it again, but I opened a new issue #157 and assumed that you would see it there. It's pretty similar to what I posted above. But please take a look at #157 to see a bunch of debug info. I didn't dig deeper since I'm not familiar with the code base...

from keepalived.