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

Cleanup duplicates in loaded plugins list #7852

Merged
merged 2 commits into from
Aug 19, 2020
Merged

Cleanup duplicates in loaded plugins list #7852

merged 2 commits into from
Aug 19, 2020

Conversation

Hipska
Copy link
Contributor

@Hipska Hipska commented Jul 17, 2020

Fixes #7839

This can also be done by directly generating the map and passing that to the PluginNameCounts method, let me know if this is more appropriate.

Required for all PRs:

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

@danielnelson
Copy link
Contributor

@Hipska looks like the tests are failing due to a formatting issue, can you run make fmt and add the changes?

@Hipska
Copy link
Contributor Author

Hipska commented Aug 11, 2020

Done!

@Hipska
Copy link
Contributor Author

Hipska commented Aug 19, 2020

@danielnelson Something else that needs to be done?

@ssoroka ssoroka merged commit 073a9be into influxdata:master Aug 19, 2020
@Hipska Hipska deleted the plugin-counts branch August 19, 2020 22:33
pmalek-sumo pushed a commit to SumoLogic/telegraf that referenced this pull request Aug 20, 2020
pmalek-sumo pushed a commit to SumoLogic/telegraf that referenced this pull request Aug 26, 2020
idohalevi pushed a commit to idohalevi/telegraf that referenced this pull request Sep 29, 2020
arstercz pushed a commit to arstercz/telegraf that referenced this pull request Mar 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Command outputs duplicated loaded plugin names
3 participants