Skip to content

Add option to amqp output to publish persistent messages#3528

Merged
danielnelson merged 2 commits intomasterfrom
amqp-delivery-mode
Dec 1, 2017
Merged

Add option to amqp output to publish persistent messages#3528
danielnelson merged 2 commits intomasterfrom
amqp-delivery-mode

Conversation

@danielnelson
Copy link
Contributor

closes #3526

Required for all PRs:

  • Signed CLA.
  • Associated README.md updated.
  • Has appropriate unit tests.

@danielnelson danielnelson added area/rabbitmq feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin labels Nov 29, 2017
@danielnelson danielnelson added this to the 1.5.0 milestone Nov 29, 2017
@danielnelson
Copy link
Contributor Author

@hanej Would you be able to test this patch? I can you link you to a build from CircleCI if you tell me which package you prefer.

@hanej
Copy link

hanej commented Nov 30, 2017

Absolutely. Thank you. We're running on RHEL 6/7 x86_64.

@danielnelson
Copy link
Contributor Author

danielnelson commented Nov 30, 2017

@danielnelson
Copy link
Contributor Author

@hanej I'm hoping to tag 1.5.0-rc1 tomorrow morning, if you can test before then we can squeeze this in.

@hanej
Copy link

hanej commented Dec 1, 2017

It works perfectly. Thank you very much. The messages get persisted after broker restart.

@danielnelson danielnelson merged commit 44320a5 into master Dec 1, 2017
@danielnelson danielnelson deleted the amqp-delivery-mode branch December 1, 2017 02:40
aromeyer pushed a commit to aromeyer/telegraf that referenced this pull request May 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/rabbitmq feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Telegraf AMQP Output - Persistent messages

2 participants