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.

Sign up for GitHub

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

Jump to bottom

Add docker-friendly remote service discovery providers #72

Closed
2 tasks done
tcalmant opened this issue Dec 1, 2016 · 3 comments
Closed
2 tasks done

Add docker-friendly remote service discovery providers #72

tcalmant opened this issue Dec 1, 2016 · 3 comments
Milestone
0.6.5

Comments

@tcalmant
Copy link
Owner

tcalmant commented Dec 1, 2016 •
edited
Loading

Multicast is not available on Docker networks, which means that most of built-in discovery services can't work on that kind of architecture.

The following providers could be interesting to handle this situation:

  • Broadcast discovery (this might work on Docker)
  • Key-Value servers (ZooKeeper, Redis, ...) discoveries
    • based on hostnames
    • based on all available IPs, with a Framework UUID check on each IPs
  • File-based discovery (requires static host names or IPs and a static binding port)
  • Redis discovery
  • Zookeeper discovery
@debbabi
Copy link

debbabi commented Dec 1, 2016

+1

tcalmant added a commit that referenced this issue Dec 9, 2016
@tcalmant
First implementation of the Redis-based discovery, along with Imports…
7ed7913 
…Registry update

Works with all cases: service registration, update and removal.
Related to issue #72
tcalmant added a commit that referenced this issue Dec 9, 2016
@tcalmant
First implementation of the Redis-based discovery, along with Imports…
f98423a 
…Registry update

Works with all cases: service registration, update and removal.
Related to issue #72
tcalmant added a commit that referenced this issue Dec 9, 2016
@tcalmant
Redis discovery: corrected hostname handling
cd1fd69 
Now ensuring that "str" hostnames are stored instead of "bytes"
Related to issue #72
@tcalmant tcalmant modified the milestone: 0.6.5 Apr 28, 2017
@tcalmant
Copy link
Owner Author

tcalmant commented Aug 7, 2017

Added ZooKeeper discovery in commit d31c415
All feedback is welcome.

@tcalmant
Copy link
Owner Author

Tests added this week, leading to some fixes for compatibility with Python 2.7.
Considering this task done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.6.5
Development

No branches or pull requests
2 participants
@debbabi @tcalmant