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

Remove unmaintained example #842

Closed
wants to merge 1 commit into from
Closed

Conversation

SuperQ
Copy link
Member

@SuperQ SuperQ commented Feb 13, 2023

We don't generally maintain example service configs in our repos. This is more appropriately handled by prometheus-community/ansible.

Fixes: #832

Signed-off-by: SuperQ superq@gmail.com

@SuperQ SuperQ requested a review from RichiH February 13, 2023 07:58
@SuperQ SuperQ force-pushed the superq/cleanup_broken_config branch from 27f868a to 4894bbd Compare February 13, 2023 07:59
We don't generally maintain example service configs in our repos. This
is more appropriately handled by prometheus-community/ansible.

Signed-off-by: SuperQ <superq@gmail.com>
@SuperQ SuperQ force-pushed the superq/cleanup_broken_config branch from 4894bbd to 97a2a92 Compare February 26, 2023 09:20
Copy link
Member

@RichiH RichiH left a comment

Choose a reason for hiding this comment

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

Weird; my reply email went in, I got an update from GH with my response, then a bounce, and now my reply is gone.

I would prefer to keep (and by extension maintain) this for all exporters. More docs & handholding is always good. People won't know where the Ansible lives or how to copy from it in the common case, I fear.

@SuperQ
Copy link
Member Author

SuperQ commented Feb 27, 2023

I really don't want to maintain these examples, we don't have any consistency for this stuff on any of our repos.

Same reason why I don't like maintaining random init scripts, runit configs, supervisord configs, etc.

@RichiH
Copy link
Member

RichiH commented Feb 27, 2023

Semi-random idea: What if we put it into exporter-toolkit?

@dswarbrick
Copy link
Contributor

I agree that systemd service files, init scripts etc should really be provided by the distros which package the exporters, since they usually involve distro-specific paths, users / groups, etc.

@SuperQ SuperQ requested a review from RichiH May 2, 2023 09:16
@SuperQ SuperQ closed this Jul 7, 2023
@SuperQ
Copy link
Member Author

SuperQ commented Jul 7, 2023

Closing this for now.

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.

snmp_exporter.service sample file
3 participants