-
Notifications
You must be signed in to change notification settings - Fork 5.6k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
feat(inputs.amqp_consumer): Allow specification of queue arguments #16141
feat(inputs.amqp_consumer): Allow specification of queue arguments #16141
Conversation
Thanks so much for the pull request! |
… forward it to QueueDeclare*()
61a724f
to
6aaf3b6
Compare
@n1b0r Thanks for the PR! Could you sign the CLA, run |
!signed-cla |
Hello, I signed the CLA, make lint does not change anything, and I commented the PR about AI. I am not sure why some circleci tests are failing. Should I do something about it ? |
Could you add this field to the sample config and readme as well? Thanks! |
Download PR build artifacts for linux_amd64.tar.gz, darwin_arm64.tar.gz, and windows_amd64.zip. 📦 Click here to get additional PR build artifactsArtifact URLs |
@n1b0r please restore the original PR description with the checkbox! It's there for a reason... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for your contribution @n1b0r!
Summary
Allow specification of queue arguments
Checklist
Related issues
resolves #16140