Ubuntu Mirror Charm

Merge ~pjdc/ubuntu-mirror-charm/+git/ubuntu-mirror-charm:rsync-process-check-socket-activation into ubuntu-mirror-charm:master

  1. Git
  2. lp:~pjdc/ubuntu-mirror-charm/+git/ubuntu-mirror-charm
  3. rsync-process-check-socket-activation
  4. Merge into master
Proposed by Paul Collins
Status: Merged
Approved by: Paul Collins
Approved revision: 153bec437ebdc5eceed5bb6e7fda7ef79f95fb58
Merged at revision: 4959511def71b101dce1cffacd03646bc9c74ee6
Proposed branch: ~pjdc/ubuntu-mirror-charm/+git/ubuntu-mirror-charm:rsync-process-check-socket-activation
Merge into: ubuntu-mirror-charm:master
Diff against target: 7 lines (+1/-1)
1 file modified
templates/nagios-check_rsync.tmpl (+1/-1)
Reviewer Review Type Date Requested Status
Haw Loeung +1 Approve
Canonical IS Reviewers Pending
Review via email: mp+390780@code.launchpad.net

Commit message

now that rsync is managed via socket activation, there can be 0 processes

To post a comment you must log in.
Revision history for this message
Haw Loeung (hloeung) wrote :

LGTM

review: Approve (+1)
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision 4959511def71b101dce1cffacd03646bc9c74ee6

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/templates/nagios-check_rsync.tmpl b/templates/nagios-check_rsync.tmpl
2index d108cf1..f0038f6 100644
3--- a/templates/nagios-check_rsync.tmpl
4+++ b/templates/nagios-check_rsync.tmpl
5@@ -1 +1 @@
6-command[check_rsync_${name}]=/usr/lib/nagios/plugins/check_procs -c 1:${rsync_crit} -w 1:${rsync_warn} -C rsync -a daemon
7+command[check_rsync_${name}]=/usr/lib/nagios/plugins/check_procs -c 0:${rsync_crit} -w 0:${rsync_warn} -C rsync -a daemon

Subscribers

People subscribed via source and target branches