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

update inputs.influxdb readme #8569

Merged
merged 2 commits into from
Dec 17, 2020
Merged

update inputs.influxdb readme #8569

merged 2 commits into from
Dec 17, 2020

Conversation

sjwang90
Copy link
Contributor

update influxdb metrics - include missing enterprise metrics.

Closes #4017

@sjwang90 sjwang90 added docs Issues related to Telegraf documentation and configuration descriptions area/influxdb labels Dec 15, 2020
Comment on lines 88 to 89
- influxdb_tsm1_engine: The TSM storage engine that metrics are collected from.
- influxdb_tsm1_engine: The TSM storage engine that metrics are collected from.
Copy link
Member

Choose a reason for hiding this comment

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

Those are duplicates. :-)

@srebhan srebhan self-assigned this Dec 16, 2020
Copy link
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.

One comment in the document.

Copy link
Contributor

@timhallinflux timhallinflux left a comment

Choose a reason for hiding this comment

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

address the dupe..then gtg.

@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 Dec 17, 2020
@sjwang90 sjwang90 merged commit ee91b48 into master Dec 17, 2020
@sjwang90 sjwang90 deleted the readme-influxdb branch December 17, 2020 16:21
ssoroka pushed a commit that referenced this pull request Dec 18, 2020
* update inputs.influxdb readme

* remove duplicate

(cherry picked from commit ee91b48)
arstercz pushed a commit to arstercz/telegraf that referenced this pull request Mar 5, 2023
* update inputs.influxdb readme

* remove duplicate
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/influxdb docs Issues related to Telegraf documentation and configuration descriptions 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
Development

Successfully merging this pull request may close these issues.

Update InfluxDB Input Plugin to capture AE stats being exposed with InfluxDB 1.6
3 participants