Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Azure IoT Hub / Azure Event Hub input plugin #5370

Closed
wants to merge 274 commits into from

Conversation

R290
Copy link
Contributor

@R290 R290 commented Feb 3, 2019

As discussed in issue #5231.
Feedback on code quality or anything else is greatly appreciated.

Required for all PRs:

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

@eaidland
Copy link

eaidland commented Jun 6, 2019

Hi!

What's the status for this merge/release ?

@R290
Copy link
Contributor Author

R290 commented Jun 7, 2019

Hi!

What's the status for this merge/release ?

I've been using this plugin continuously since February without any issues.
No upstream updates have been tested (latest commits are 52bd698 and Azure/azure-event-hubs-go@aca3e9c).

To fully enjoy this plugin the following issue needs to be fixed: Azure/azure-event-hubs-go#95, I've not yet been able to submit a PR.

@eaidland
Copy link

eaidland commented Jun 7, 2019

We'll have to wait until it's part of the default telegraf plugins.

@R290
Copy link
Contributor Author

R290 commented Sep 24, 2019

Closed until dep has minimal support for Go modules. Until then the required package github.com/Azure/azure-event-hubs-go/v2 can not be successfully handled by dep ensure. Alternatively Telegraf support for Go modules might happen sooner.

danielnelson and others added 25 commits January 10, 2020 12:43
@R290
Copy link
Contributor Author

R290 commented Jan 20, 2020

This pull request is reopened since #6912 is merged.

@R290 R290 reopened this Jan 20, 2020
@R290
Copy link
Contributor Author

R290 commented Jan 20, 2020

I don't exactly understand what happened here, I expected one commit to be included instead of 274. I'll open a new pull request and reference this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.