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

Add the ability to select processes to monitor by regex pattern match #1375

Closed
wants to merge 2 commits into from
Closed

Add the ability to select processes to monitor by regex pattern match #1375

wants to merge 2 commits into from

Conversation

ckrough
Copy link
Contributor

@ckrough ckrough commented Feb 17, 2015

Adds the ability to select processes to monitor by regex pattern match.

  • Check will monitor processes whose names match the list of regex patterns provided in 'proc_regex'
  • Check will monitor processes whose names exactly match the strings provided in 'proc_names'
  • All processes matching proc_names OR proc_regex filters are monitored.
  • All processes are monitored when no proc_names or proc_regex filters are defined.
  • Sample configuration provides proc_regex examples

isaacdd and others added 2 commits January 13, 2015 19:39
This adds a new agent check for supervisord. The check reports whether
supervisord is running or not and  whether one or more processes are
runnnig or not. It also reports prcesses' uptime. The check works over
http inet server and works with sockets. See supervisord.yaml.example
for more details.
- Check will monitor processes whose names match the list of regex patterns provided in 'proc_regex'
- Check will monitor processes whose names exactly match the strings provided in 'proc_names'
- All processes matching proc_names OR proc_regex filters are monitored.
- All processes are monitored when no proc_names or proc_regex filters are defined.
- Sample configuration provides proc_regex examples
@ckrough
Copy link
Contributor Author

ckrough commented Feb 17, 2015

Tests are failing from #1373, this change doesn't break them any further.

@remh
Copy link
Contributor

remh commented Feb 17, 2015

Thanks a lot @ckrough . We'll review and we aim at releasing this for our 5.3.0 release.

@remh remh added this to the 5.3.0 milestone Feb 17, 2015
@isaacdd isaacdd force-pushed the isaac/supervisord_check branch 2 times, most recently from e9f7f70 to 86f35c2 Compare March 5, 2015 20:20
@yannmh yannmh removed the 1 - Ready label Mar 13, 2015
@remh remh modified the milestones: To be determined, 5.3.0 Mar 16, 2015
@isaacdd isaacdd force-pushed the isaac/supervisord_check branch 3 times, most recently from c130c99 to 46db43d Compare March 18, 2015 15:55
@yannmh
Copy link
Member

yannmh commented Jul 1, 2015

You PR looks great. Thanks a lot @ckrough
I rebased it and opened a new one here #1747. The plan is to merge it for the 5.5.0 release (which should be out by the end of the month).

@yannmh yannmh closed this Jul 1, 2015
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.

4 participants