Skip to content

add OpenTelemetry entry#9464

Merged
sspaink merged 1 commit intoinfluxdata:masterfrom
mindw:patch-1
Jul 1, 2021
Merged

add OpenTelemetry entry#9464
sspaink merged 1 commit intoinfluxdata:masterfrom
mindw:patch-1

Conversation

@mindw
Copy link
Copy Markdown
Contributor

@mindw mindw commented Jul 1, 2021

Required for all PRs:

  • Updated associated README.md.
  • Wrote appropriate unit tests.

@telegraf-tiger
Copy link
Copy Markdown
Contributor

telegraf-tiger bot commented Jul 1, 2021

Thanks so much for the pull request!
🤝 ✒️ Just a reminder that the CLA has not yet been signed, and we'll need it before merging. Please sign the CLA when you get a chance, then post a comment here saying !signed-cla

@telegraf-tiger telegraf-tiger bot added the feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin label Jul 1, 2021
@mindw
Copy link
Copy Markdown
Contributor Author

mindw commented Jul 1, 2021

!signed-cla

srebhan
srebhan approved these changes Jul 1, 2021
View reviewed changes
Copy link
Copy Markdown
Member

@srebhan srebhan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks @mindw.

@srebhan srebhan added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label Jul 1, 2021
@srebhan srebhan self-assigned this Jul 1, 2021
@srebhan srebhan added the docs Issues related to Telegraf documentation and configuration descriptions label Jul 1, 2021
@sspaink sspaink merged commit 138c204 into influxdata:master Jul 1, 2021
reimda pushed a commit that referenced this pull request Jul 7, 2021
@mindw @reimda
add OpenTelemetry entry (#9464)
c0365e6 
(cherry picked from commit 138c204)
bhsu-ms pushed a commit to bhsu-ms/telegraf that referenced this pull request Jul 12, 2021
@mindw @bhsu-ms
add OpenTelemetry entry (influxdata#9464)
667af1c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@srebhan srebhan srebhan approved these changes

+1 more reviewer

@sspaink sspaink sspaink approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@srebhan srebhan

Labels

docs Issues related to Telegraf documentation and configuration descriptions feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mindw @sspaink @srebhan