search

aweber / rabbitmq-autocluster

This project is now maintained by the RabbitMQ Team, visit the official repo @
https://github.com/rabbitmq/rabbitmq-autocluster
BSD 3-Clause "New" or "Revised" License
336 stars 120 forks source link

Use separate plugin for AWS #105

Closed binarin closed 8 years ago

binarin commented 8 years ago

WDYT? Fixes #99 Probably fixes #63 Requires https://github.com/rabbitmq/rabbitmq-common/pull/141

Enabling AWS backend now requires enabling just autocluster_aws plugin (rabbitmq_aws and autocluster will be loaded by rabbitmq automatically).

The same approach will work for any other backend with external dependences.

codecov-io commented 8 years ago

Current coverage is 41.71% (diff: 0.00%)

No coverage report found for master at a2e2aa0.

Powered by Codecov. Last update a2e2aa0...7379895

gmr commented 8 years ago

I missed the do not merge, I got excited by this. What did you want to do in addition?

binarin commented 8 years ago

I wanted some early feedback on whether you like this approach - and looks like this question is answered =)

It was DO NOT MERGE because it depends on https://github.com/rabbitmq/rabbitmq-common/pull/141 and I haven't thoroughly tested it (as my free Amazon credits expired long ago).