Issues with locks not being released when the leader is down

[darkcrux]

Maybe we need to use LockDelay and TTL for sessions.

[thedebugger]

I think in the longer run, consul-skipper should add support for – TTL and Checks. The benefit of adding Checks to the session is that it takes the burden out of consul-skipper to renew the session; consul will invalidate the session if the checks fail. However, I think TTL provides a better abstraction to clients as they don't have to register a consul check. What do you say?

[jhaals]

I just discovered this project but quickly ran into the issue with stale leader when the application is killed without resigning. 👍 for adding support for TTL.

[euforia]

ran into this while testing a pre-prod cluster
+1