search

airbnb / smartstack-cookbook

The chef recipes for running and testing Airbnb's SmartStack
MIT License
245 stars 45 forks source link

lack of support & proper testing #18

Closed francoisnicholas closed 8 years ago

francoisnicholas commented 8 years ago

The lack of support from the author of this cookbook is just sad. Doc is poorly written. Poor testing. It would be a good idea to simplify the testing of this cookbook.

e.g: node 1 - synapse node 2 - nerve node 3 - zookeeper

then apply your test "helloworld" service & then check haproxy for status & processed request.

Comparing the current work/testing here, Consul by HashiCorp is looking attractive. Why?

Good documentation. Good support. Good testing.

Hopefully the author of smartstack cookbook will do something about this.

jolynch commented 8 years ago

@fnicholas I'm not sure what you hope to achieve from this issue. Pull requests or forking are always options for you if you're unhappy with the code.

I don't work on the smartstack cookbook, but I do help maintain synapse/nerve and we've received similar feedback there. I think the reality is that SmartStack is a system that Airbnb and Yelp help maintain because we use it, but open source doesn't mean free lunch.

I know that I do most of my work on SmartStack outside working hours, on my own free time, and my understanding is that Igor does this as well. Contrast that with HashiCorp where service discovery is their core competency and they make money from it...

igor47 commented 8 years ago

this cookbook has drifted pretty far from our internal version of it at airbnb. backporting changes between a private and a public repo is kind of a pain. i'm tempted to delete this project off github. i might just clearly mark the README as deprecated...

francoisnicholas commented 8 years ago

sounds good. i suggest it should be deleted.