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

[Feature Request] multiple varnish instance #2285

Closed
btbars opened this issue Jan 18, 2017 · 6 comments
Closed

[Feature Request] multiple varnish instance #2285

btbars opened this issue Jan 18, 2017 · 6 comments
Labels
feature request Requests for new plugin and for new features to existing plugins
Milestone

Comments

@btbars
Copy link

btbars commented Jan 18, 2017

Feature Request

hello,
please, allow us to configure multiple instances of varnish as it is possible for haproxy plugin
we should be able to add an instance name to the config file:

varnishstat -n /var/lib/varnish/static/ ( path to _.vsm )
by default we cannot change the default path ( /var/lib/varnish/$(uname -n )

thank you

@sparrc
Copy link
Contributor

sparrc commented Jan 21, 2017

Not sure I understand the request, but I believe you can do this by setting up multiple instances of [[inputs.varnish]]

@sparrc sparrc closed this as completed Jan 21, 2017
@btbars
Copy link
Author

btbars commented Jan 23, 2017

on a console, if we have multiple instances, we should call
varnishstat -n /var/lib/varnish/instanceX/ ( path to _.vsm )
varnishstat -n /var/lib/varnish/InstanceY/ ( path to _.vsm )
....
but you cannot specified the instance ( -n ) for the varnishstats on the config file

@sparrc
Copy link
Contributor

sparrc commented Jan 23, 2017

OK, so it's just a feature request to add the -n option to the varnish plugin?

@sparrc sparrc reopened this Jan 23, 2017
@btbars
Copy link
Author

btbars commented Jan 26, 2017

yes

@sparrc sparrc added this to the Future Milestone milestone Jan 27, 2017
@danielnelson danielnelson removed this from the Future Milestone milestone Jun 14, 2017
@danielmotaleite
Copy link

danielmotaleite commented Mar 22, 2019

The documentation now reports this:

   # instance_name = instanceName

So i think this is already fixed and could be closed

@danielnelson
Copy link
Contributor

Thanks @danielmotaleite, looks like it was added in Telegraf 1.5.0 (#3398)

@danielnelson danielnelson added this to the 1.5.0 milestone Mar 22, 2019
@danielnelson danielnelson added the feature request Requests for new plugin and for new features to existing plugins label Mar 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Requests for new plugin and for new features to existing plugins
Projects
None yet
Development

No branches or pull requests

4 participants