Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow the use of enable_failover #116

Open
philwinder opened this issue Feb 3, 2016 · 4 comments
Open

Allow the use of enable_failover #116

philwinder opened this issue Feb 3, 2016 · 4 comments

Comments

@philwinder
Copy link
Collaborator

AFAIK, enable_failover: true doesn't work. The framework should failover correctly.

@mwl mwl self-assigned this Feb 3, 2016
@mwl
Copy link
Collaborator

mwl commented Feb 3, 2016

I've just tested it. From what I can see it actually works.

@philwinder
Copy link
Collaborator Author

Can you remind me about the intended behaviour? Maybe I misunderstand the command.

@mwl
Copy link
Collaborator

mwl commented Feb 3, 2016

With failover enabled the framework will not deregister on Scheduler SIGTERM.
With failover disabled the framework will be deregistered on Scheduler SIGTERM.

What were your expectations?

@philwinder
Copy link
Collaborator Author

Ah ok. The problem I was having was when logstash wasn't setup correctly. Given that you set enable_failover: true. When an executor failed for some reason, it wasn't restarted. Eventually there would be no executors at all and the scheduler wouldn't start any. What's worse is that when I restarted the scheduler, it still didn't restart any executors, like it had some bad state.

The way I got it to work was to set enable_failover: false, correct my mistakes in configuration, and then everything has been working ever since.

@mwl mwl removed their assignment Jan 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants