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

bin/doozer_init talks about "my_cal" (based on two digits at end of hostname) but /ctl/cal seems to be first come first serve #58

Open
tv42 opened this issue Apr 4, 2013 · 1 comment

Comments

@tv42
Copy link

tv42 commented Apr 4, 2013

I don't see anything that would make it reliably "my" cal.

doozer_init happily deletes the cal entry, thus making whatever node had that CAL entry hang (it doesn't even realize it got kicked out; other nodes remove it from /ctl/node).

Even if this was enough to kick "others" out from using "my_cal" (which isn't, there's a huge risk of more and more nodes hanging), nothing guarantees the new doozerd started by doozer_init gets "my_cal". It's racy.

@mreiferson
Copy link
Contributor

yea, the script relies on the sequential nature of how we deployed things at bitly, see #46 for some updates that I haven't finished (the script currently in master was prematurely merged)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants