-
-
Notifications
You must be signed in to change notification settings - Fork 218
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
Only monitor running containers. #109
base: main
Are you sure you want to change the base?
Only monitor running containers. #109
Conversation
docker-entrypoint
Outdated
then | ||
running_filter="" | ||
else | ||
running_filter=",\"status\":\[\"runing\"\]" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be "running"
?
running_filter=",\"status\":\[\"runing\"\]" | |
running_filter=",\"status\":\[\"running\"\]" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I spotted this and changed it in my main branch (which the Dockerhub container is based off), but I've made some other changes in there which won't want merging upstream. I thought I'd edited it in this branch too, but missed it. Should all be good now.
Please see the PR #115 |
This change means that containers that are not running (stopping, starting, created, exited, dead and paused) will be filtered out.
It prevents paused containers from being automatically restarted.