launchpad-reviewers team mailing list archive

[Colin Watson <mp+429653@xxxxxxxxxxxxxxxxxx>]

Colin Watson has proposed merging ~cjwatson/launchpad:charm-appserver-nagios into launchpad:master.

Commit message:
charm: Add a Nagios appserver check

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/429653

This is based on what we have on production.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad:charm-appserver-nagios into launchpad:master.

diff --git a/charm/launchpad-appserver/reactive/launchpad-appserver.py b/charm/launchpad-appserver/reactive/launchpad-appserver.py
index c5fac0e..a7d59f6 100644
--- a/charm/launchpad-appserver/reactive/launchpad-appserver.py
+++ b/charm/launchpad-appserver/reactive/launchpad-appserver.py
@@ -11,7 +11,14 @@ from charms.launchpad.base import (
     get_service_config,
     lazr_config_files,
 )
-from charms.reactive import helpers, set_state, when, when_not
+from charms.reactive import (
+    clear_flag,
+    helpers,
+    set_flag,
+    set_state,
+    when,
+    when_not,
+)
 from ols import base
 
 
@@ -106,3 +113,33 @@ def configure():
 @when("service.configured")
 def check_is_running():
     hookenv.status_set("active", "Ready")
+
+
+@when("nrpe-external-master.available", "service.configured")
+@when_not("launchpad.appserver.nrpe-external-master.published")
+def nrpe_available(nrpe):
+    config = hookenv.config()
+    healthy_regex = (
+        r"(\/\+icing\/rev[0-9a-f]+\/).*(Is your project registered yet\?)"
+    )
+    nrpe.add_check(
+        [
+            "/usr/lib/nagios/plugins/check_http",
+            "-H",
+            "localhost",
+            "-p",
+            str(config["port_main"]),
+            "-l",
+            "--regex=%s" % healthy_regex,
+        ],
+        name="check_launchpad_appserver",
+        description="Launchpad appserver",
+        context=config["nagios_context"],
+    )
+    set_flag("launchpad.appserver.nrpe-external-master.published")
+
+
+@when("launchpad.appserver.nrpe-external-master.published")
+@when_not("nrpe-external-master.available")
+def nrpe_unavailable():
+    clear_flag("launchpad.appserver.nrpe-external-master.published")