airbnb / nerve

A service registration daemon that performs health checks; companion to airbnb/synapse
MIT License
942 stars 151 forks source link

Refactor for multiple backend support #41

Closed bobtfish closed 10 years ago

bobtfish commented 10 years ago

This is a prequel to adding etcd support (https://github.com/airbnb/synapse/issues/35) - just adds dynamic backend support (defaulting to the zookeeper backend), giving a place to plug in etcd.

bobtfish commented 10 years ago

N.B. etcd support is coming: https://github.com/bobtfish/nerve/tree/etcd_suppport

I just didn't want to drop an epic sized patch, so I've pull requested this refactoring first, so that we can bike-shed about how it should look if needed ;_

I'll follow up with a 2nd PR for the etcd support when I've properly finished it, got it working and we've got the refactoring to support it merged.