airbnb / nerve

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

bin/nerve: Update YAML exception handling to Psych semantics #56

Closed bploetz closed 4 years ago

bploetz commented 10 years ago

As noted here https://github.com/airbnb/synapse/issues/52#issuecomment-45909582 updating the bin/nerve script to follow the new Psych exception semantics in Ruby 1.9.3+, matching what synapse is now doing.

Note that this now means nerve will only run in Ruby 1.9.3+. If you want a more generic solution that works across all Ruby versions, let me know, but I just followed the lead of what was done in synapse.

bploetz commented 10 years ago

Looks like there's no .travis.yml file in the pull_requests branch (looks out of date compared to master), which is why the build is failing.

jolynch commented 8 years ago

Hi @bploetz sorry for not getting to this sooner. I fixed this in #80 while I was trying to get dynamic config working. When I merge that PR I'll close this one too :-)