canonical-ubuntu-qa team mailing list archive

[Tim Andersson <mp+467708@xxxxxxxxxxxxxxxxxx>]

Tim Andersson has proposed merging ~andersson123/autopkgtest-cloud:web-reactive-status-fix into autopkgtest-cloud:master.

Requested reviews:
  Canonical's Ubuntu QA (canonical-ubuntu-qa)

For more details, see:
https://code.launchpad.net/~andersson123/autopkgtest-cloud/+git/autopkgtest-cloud/+merge/467708
-- 
Your team Canonical's Ubuntu QA is requested to review the proposed merge of ~andersson123/autopkgtest-cloud:web-reactive-status-fix into autopkgtest-cloud:master.

diff --git a/charms/focal/autopkgtest-web/reactive/autopkgtest_web.py b/charms/focal/autopkgtest-web/reactive/autopkgtest_web.py
index 14587e6..3e97373 100644
--- a/charms/focal/autopkgtest-web/reactive/autopkgtest_web.py
+++ b/charms/focal/autopkgtest-web/reactive/autopkgtest_web.py
@@ -172,9 +172,9 @@ def set_up_systemd_units():
         if "[Install]" in pathlib.Path(unit).read_text():
             subprocess.check_call(["systemctl", "enable", base])
 
-    status.maintenance("systemd units installed")
     if any_changed:
         set_flag("autopkgtest-web.autopkgtest-web-target-needs-restart")
+    status.active("systemd units installed")
 
 
 @when_all(