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

Mark the collectd exporter as deprecated #96

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

juliusv
Copy link
Member

@juliusv juliusv commented May 11, 2020

Signed-off-by: Julius Volz julius.volz@gmail.com

Context: #95 (comment)

@juliusv
Copy link
Member Author

juliusv commented May 11, 2020

@SuperQ

Signed-off-by: Julius Volz <julius.volz@gmail.com>
@kamilpoleszczuk
Copy link

Hi. Is it confirmed that repository will be removed? As per discussion here https support can be easily added to collectd_exporter (even by patching locally). Unfortunately TLS is not supported by write_prometheus plugin.

@juliusv
Copy link
Member Author

juliusv commented May 15, 2020

@octo How is TLS usually addressed on the collectd side?

@obitech
Copy link

obitech commented May 15, 2020

Hey, any background on why it's being deprecated?

@juliusv
Copy link
Member Author

juliusv commented May 15, 2020

Ah sorry, I should have included more context in here as well, see #95 (comment)

@octo
Copy link
Contributor

octo commented May 29, 2020

@juliusv collectd does not currently accept metrics via HTTP[S] POST requests. collectd/go-plugins#1 proposes such a plugin.

@hasso
Copy link
Contributor

hasso commented May 30, 2020

Collectd_exporter had much (several times) better performance than collectd when we investigated about year ago or so. I might revisit it at some point, but I don't think that anything is really changed in this regard.

@SuperQ
Copy link
Member

SuperQ commented Jun 11, 2020

@hasso If you could provide some benchmarks and proof of this, it would make for a very good use case to keep this project going.

@matthiasr
Copy link

matthiasr commented Jun 26, 2020 via email

@stale stale bot added the stale label Aug 25, 2020
@juliusv juliusv mentioned this pull request Jan 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants