sts-sponsors team mailing list archive

[Alberto Donato <mp+435208@xxxxxxxxxxxxxxxxxx>]

Alberto Donato has proposed merging ~ack/maas-ci/+git/system-tests:vault-log into ~maas-committers/maas-ci/+git/system-tests:master.

Commit message:
configure Vault for logging, better systemd unit config



Requested reviews:
  MAAS Committers (maas-committers)

For more details, see:
https://code.launchpad.net/~ack/maas-ci/+git/system-tests/+merge/435208
-- 
Your team MAAS Committers is requested to review the proposed merge of ~ack/maas-ci/+git/system-tests:vault-log into ~maas-committers/maas-ci/+git/system-tests:master.

diff --git a/systemtests/fixtures.py b/systemtests/fixtures.py
index a7512dd..0768396 100644
--- a/systemtests/fixtures.py
+++ b/systemtests/fixtures.py
@@ -308,6 +308,8 @@ def vault(maas_container: str, config: dict[str, Any]) -> Optional[Vault]:
         """\
         disable_mlock = true
         ui = true
+        log_level = "debug"
+        log_requests_level = "debug"
 
         storage "file" {
             path = "/var/snap/vault/common/data"
@@ -327,11 +329,13 @@ def vault(maas_container: str, config: dict[str, Any]) -> Optional[Vault]:
         f"""\
         [Unit]
         Description=Vault service for MAAS testing
-        Wants=network.target
+        After=syslog.target
+        ConditionFileNotEmpty={vault_config_file}
 
         [Service]
         ExecStart=/snap/bin/vault server -config {vault_config_file}
-        Restart=on-failure
+        ExecReload=/bin/kill --signal HUP $MAINPID
+        Restart=always
         Type=simple
 
         [Install]