search

logstash-plugins / logstash-input-rabbitmq

Apache License 2.0
30 stars 43 forks source link

Explicitly declare exchanges. #72

Closed andrewvc closed 8 years ago

andrewvc commented 8 years ago

Refactor integration tests to use more complex exchange topology

Fixes #48

elasticsearch-release commented 8 years ago

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then say 'jenkins, test it'.

andrewvc commented 8 years ago

@michaelklishin do you have a minute to review this PR to make sure it all looks good. I'd really appreciate your blessing here in light of #48.

untergeek commented 8 years ago

LGTM™

elasticsearch-bot commented 8 years ago

Andrew Cholakian merged this into the following branches!

Branch Commits
master 57f1e11526ec06621a3b88f372fb7180a5073a06, d277b394be98e45c9c89a2b75926815d26d3420c, 2f997e972197e3a417789fb06114b23638ad1e8f, 7b6f9b8216f48bf478b62ada24b04b5477642aaa, 439cf44da45fd164cd26b007a338d3cc3b3da797, d9066ee3ef4476022ff8c43d66bed0b499692c5f, 0157e27fbbbcb6f4f8ac820502461cc4f7a7b170