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

Fea/specify container to be collected #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

bugre
Copy link

@bugre bugre commented Sep 13, 2023

Add option to filter containers by 'name' attribute that will get stats collected.

Add option to filter by 'name' what containers get it's stats collected.
Reflect addition on README.md
@wywywywy
Copy link
Owner

Hi @bugre

Thank you for the PR, but I think I'm going to archive this repo as it's gotten quite out of date. Feel free to fork it if it fits your needs.

Thanks.

@bugre
Copy link
Author

bugre commented Sep 13, 2023

Hi @wywywywy , not a problem.
I've already forked and made the changes locally, but thought of sharing if it could help someone.
Thanks for letting me know.
If you don't mind asking, is there a better way of getting the docker stats of a container into prometheus as of today? (not in a k8s or swarm or other orchestration environment, but running a few containers manually? (aside from use/try directly the cgroups metrics)

@wywywywy
Copy link
Owner

If you don't mind asking, is there a better way of getting the docker stats of a container into prometheus as of today? (not in a k8s or swarm or other orchestration environment, but running a few containers manually? (aside from use/try directly the cgroups metrics)

I had a look a while ago, and to be honest I didn't find any lightweight exporter similar to this one unfortunately. :(

So this is still useful if someone takes the time to maintain it properly.

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.

2 participants