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

[inputs.vsphere] Exclude resources by inventory path #6859

Merged
merged 11 commits into from
Jan 16, 2020
Merged

[inputs.vsphere] Exclude resources by inventory path #6859

merged 11 commits into from
Jan 16, 2020

Conversation

prydin
Copy link
Contributor

@prydin prydin commented Jan 6, 2020

Required for all PRs:

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

This PR addresses issue #5639 (omit metrics by name).

Description

In addition to host_include, datastore_include, datacenter_include, cluster_include and vm_include configuration parameter, we are adding their *_exclude counterparts. They allow you to exclude any resource using an inventory path. For example vm_exclude = [/host/**/host1/**] would exclude any VM on the host host1.

Caveat

If the exclude inventory path contains a reference to an inventory node to which the service account does nto have access, the inventory discovery will fail. Rewrite such paths using wildcards instead so that you avoid referencing any nodes you don't have access to.

closes #5639

@prydin prydin changed the title Exclude resources by inventory path [inputs.vsphere] Exclude resources by inventory path Jan 6, 2020
@danielnelson danielnelson added this to the 1.14.0 milestone Jan 15, 2020
@danielnelson
Copy link
Contributor

@prydin Could you take a look at the conflict in endpoint.go?

@prydin
Copy link
Contributor Author

prydin commented Jan 15, 2020

@danielnelson Done!

@danielnelson danielnelson merged commit c7b7336 into influxdata:master Jan 16, 2020
athoune pushed a commit to bearstech/telegraf that referenced this pull request Apr 17, 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

vSphere: omit objects by name
2 participants