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

HAProxy configuration is not being updated #99

Closed
geowa4 opened this issue Nov 14, 2014 · 4 comments
Closed

HAProxy configuration is not being updated #99

geowa4 opened this issue Nov 14, 2014 · 4 comments

Comments

@geowa4
Copy link

geowa4 commented Nov 14, 2014

I've spent part of today trying to learn Synapse as I think it could be useful in my infrastructure. I set up a Vagrant box, where I was able to get it to successfully query the AWS API. The issue though was that it never updated the HAProxy configuration. I have committed this learning exercise to GitHub so it can be reproduced easily. I've also copied a portion of the README here for coherence.

Reproduction

vagrant up
vagrant ssh
cd /vagrant
java -jar synapse.jar --config bug.json
# quit
cat /etc/haproxy/haproxy.cfg

Synapse clearly indicates via standard out that it has identified the two EC2 instances.
It does not modify /etc/haproxy/haproxy.cfg as configured though it reports that it has regenerated the config.

synapse.jar was built with the source at e012cd3 in airbnb/synapse.

@Jaykah
Copy link
Contributor

Jaykah commented Nov 16, 2014

Looks like the issue that I'm also having - check out #94 for details.

jolynch pushed a commit that referenced this issue Sep 3, 2015
Addresses incompatibility of per_callback threading in ZK, the way that
synapse calls unsubscribe within the callback, and ruby 2.X really not
liking joining on the current running thread.

Should fix #137 #99 and possibly #94
@jolynch
Copy link
Collaborator

jolynch commented Sep 7, 2015

@geowa4 I think you may have run into an issue with the documentation saying options were default to true (do_writes, do_reloads, do_socket) when they were not at the time. On latest master do you still see the issue? Can you paste your config?

I believe I have a fix for MRI installs in #138, but it looks like you're using JRuby?

@geowa4
Copy link
Author

geowa4 commented Sep 10, 2015

@jolynch Since this didn't work for me almost a year ago, I decided to use something else. If I have time, I might verify this is fixed, but, honestly, I can't put this too high up on my priority list.

@jolynch
Copy link
Collaborator

jolynch commented Sep 10, 2015

OK, I'm going to close the issue since I think the documentation and code have been fixed. If you come across the problem again please let me know.

@jolynch jolynch closed this as completed Sep 10, 2015
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

3 participants