launchpad-reviewers team mailing list archive

[Barry Price <barry.price@xxxxxxxxxxxxx>]

Barry Price has proposed merging ~barryprice/turnip:master into turnip:master.

Commit message:
Reformat our E2E check hint to avoid using parentheses, which Nagios considers illegal

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~barryprice/turnip/+git/turnip/+merge/399759
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~barryprice/turnip:master into turnip:master.

diff --git a/charm/layer/turnip-base/lib/charms/turnip/base.py b/charm/layer/turnip-base/lib/charms/turnip/base.py
index 4cd1f55..2f24293 100644
--- a/charm/layer/turnip-base/lib/charms/turnip/base.py
+++ b/charm/layer/turnip-base/lib/charms/turnip/base.py
@@ -320,7 +320,7 @@ def add_nagios_e2e_checks(nagios):
         nagios.add_check(
             [os.path.join(nrpe_dir(), 'check_git_refs'), url],
             name='check_turnip_git_refs_{}'.format(i),
-            description='Git E2E {} (hint: check nfs-ganesha status)'.format(
+            description='Git E2E {} - hint: check nfs-ganesha status'.format(url),
                 url),
             context=config['nagios_context'])