diff --git a/cookbooks/lego/Makefile b/cookbooks/lego/Makefile index 54052a7..1c85a71 100644 --- a/cookbooks/lego/Makefile +++ b/cookbooks/lego/Makefile @@ -6,6 +6,10 @@ PROJECT_UID = 10023 PROJECT_GID = 10000 +# .path units are not picked up by common.mk's wildcard (only .service/.target/ +# .timer/.mount), so install it explicitly. It is started by lego.target. +TARGET_FILES += $(TARGET_CHROOT)/etc/systemd/system/lego-hooks.path + # Include common Makefile include ../../scripts/common.mk diff --git a/cookbooks/lego/README.md b/cookbooks/lego/README.md index a922376..e4e3182 100644 --- a/cookbooks/lego/README.md +++ b/cookbooks/lego/README.md @@ -6,16 +6,54 @@ Lego is a Let's Encrypt/ACME client started as a Podman Quadlet. It handles auto This cookbook: -- Runs an initial certificate fetch via **lego-run.service** when no certificates exist. -- Schedules automatic certificate renewal via **lego-renew.timer**. -- Stores certificates with secure permissions (umask 0077). -- Supports renewal hooks to reload dependent services when certificates are renewed. +- Runs Lego v5 with a configuration file (`/etc/quadlets/lego/lego.yaml`) describing the accounts, challenges and certificates. +- Obtains the missing certificates and renews the expiring ones via **lego.service**, started at boot and daily by **lego.timer**. +- Stores certificates with secure permissions (0600, owned by the `lego` user). +- Runs renewal hooks (`/etc/quadlets/lego/renew-hooks.d/*.sh`) to reload dependent services when certificates are obtained or renewed. ## Prerequisites -- Configuration file `/etc/quadlets/lego/config.env` must exist with ACME configuration. +- Configuration file `/etc/quadlets/lego/lego.yaml` must exist. See [config/examples/lego.yaml](config/examples/lego.yaml) and the [Lego configuration file reference](https://go-acme.github.io/lego/references/ref-file/). - DNS or HTTP challenge must be properly configured. +## Certificate naming convention + +Lego v5 names the certificate files after the certificate ID in the configuration file: `/var/lib/quadlets/lego/certificates/.{crt,key,issuer.crt,pem,json}`. + +The other cookbooks (vsftpd, smtprelay, etc.) expect a certificate named **default**: + +```yaml +certificates: + default: + domains: + - ftp.example.tld +``` + +## How hooks work + +1. Lego obtains or renews a certificate and calls its deploy hook (`/etc/quadlets/lego/deploy-hook.sh`, inside the container). +2. The deploy hook creates a `.renewed` file next to the certificate. +3. **lego-hooks.path** notices the `.renewed` file and starts **lego-hooks.service**. +4. **lego-hooks.service** runs every script of `/etc/quadlets/lego/renew-hooks.d/` (on the host, as root) and removes the `.renewed` files. + +Other cookbooks drop their hook in `other/lego/.sh`: it is installed in `/etc/quadlets/lego/renew-hooks.d/` automatically. + +## Using a custom ACME server + +To use an ACME server whose TLS certificate is signed by a private CA (Pebble, Step CA, etc.), set the `server` of the account in `lego.yaml` and add a drop-in to trust the CA: + +```ini +# /etc/containers/systemd/lego.container.d/custom-ca.conf +[Container] +Environment=LEGO_CA_CERTIFICATES=/etc/lego/my-ca.pem +``` + +The CA certificate goes in `/etc/quadlets/lego/my-ca.pem` (`/etc/quadlets/lego` is mounted on `/etc/lego` in the container). + +## Development mode + +In development mode (`make fcos-vm` or `make install`, which install the examples), the **lego-dev.conf** drop-in replaces Lego with a script that generates a self-signed certificate for `localhost`, named `default`. + ## Usage In a separate terminal, follow the logs. @@ -30,7 +68,7 @@ Install the Podman Quadlets and fetch the initial certificate. sudo make clean install ``` -You should see the **lego-run.service** fetching a certificate from Let's Encrypt. +You should see the **lego.service** fetching a certificate from Let's Encrypt. The certificate will be stored in `/var/lib/quadlets/lego/certificates/`. Check the certificate: @@ -39,12 +77,12 @@ Check the certificate: sudo ls -la /var/lib/quadlets/lego/certificates/ ``` -The **lego-renew.timer** will periodically check and renew the certificate before expiration. +The **lego.timer** will periodically check and renew the certificate before expiration. To manually trigger a renewal check: ```sh -sudo systemctl start lego-renew.service +sudo systemctl start lego.service ``` Restart the **lego.target** unit. @@ -59,8 +97,29 @@ Finally, remove the quadlets, their configuration and their data. sudo make uninstall clean ``` +## Upgrading from Lego v4 + +Lego v5 uses a configuration file and a new storage layout. +The simplest upgrade path is to obtain new certificates. + +```sh +sudo systemctl stop lego.target lego-renew.timer +sudo rm -rf /var/lib/quadlets/lego/* +sudo rm -rf /etc/quadlets/lego/config.env /etc/quadlets/lego/renew-hook +sudo rm -rf /etc/containers/systemd/lego-run.container /etc/containers/systemd/lego-renew.container \ + /etc/containers/systemd/lego-run.container.d /etc/containers/systemd/lego-renew.container.d \ + /etc/systemd/system/lego-renew.timer /etc/systemd/system/lego-renew-hooks.service +sudo make install +``` + +Then, write `/etc/quadlets/lego/lego.yaml` (with a certificate named `default`) and start Lego. + +```sh +sudo systemctl start lego.target +``` + ## Integration tests ```sh -sudo make test +sudo make pytest ``` diff --git a/cookbooks/lego/config/deploy-hook.sh b/cookbooks/lego/config/deploy-hook.sh new file mode 100755 index 0000000..504065a --- /dev/null +++ b/cookbooks/lego/config/deploy-hook.sh @@ -0,0 +1,7 @@ +#!/bin/sh + +# This hook runs inside the Lego container (busybox sh) after a certificate +# has been obtained or renewed. It flags the certificate as renewed so that +# lego-hooks.path triggers the renewal hooks on the host. +set -eu +touch "${LEGO_HOOK_CERT_PATH%.*}.renewed" diff --git a/cookbooks/lego/config/examples/config.env b/cookbooks/lego/config/examples/config.env deleted file mode 100644 index 59d0915..0000000 --- a/cookbooks/lego/config/examples/config.env +++ /dev/null @@ -1,3 +0,0 @@ -LEGO_GLOBAL_ARGS=-a -m nicolas.masse@itix.fr -d changeme.example.tld --http -LEGO_RUN_ARGS= -LEGO_RENEW_ARGS=--days 30 --renew-hook=/etc/lego/renew-hook/flag-as-renewed.sh diff --git a/cookbooks/lego/config/examples/lego-dev.sh b/cookbooks/lego/config/examples/lego-dev.sh index 8a5620f..63a7e74 100755 --- a/cookbooks/lego/config/examples/lego-dev.sh +++ b/cookbooks/lego/config/examples/lego-dev.sh @@ -2,18 +2,24 @@ set -Eeuo pipefail -# In development mode, it is not possible to get a certificate from Let's Encrypt, so we just create a self-signed certificate for localhost, so that other services can still use it. -mkdir -p /var/lib/quadlets/lego/certificates -if [ -f /var/lib/quadlets/lego/certificates/localhost.crt ] && [ -f /var/lib/quadlets/lego/certificates/localhost.key ]; then - renewal="yes" -else - renewal="no" +# In development mode, it is not possible to get a certificate from Let's Encrypt, +# so we just create a self-signed certificate for localhost, so that other services can still use it. +# The certificate is named "default", like the certificate declared in the example lego.yaml. +CERT_DIR=/var/lib/quadlets/lego/certificates +CERT_NAME=default + +if [ -f "$CERT_DIR/$CERT_NAME.crt" ] && [ -f "$CERT_DIR/$CERT_NAME.key" ] && openssl x509 -in "$CERT_DIR/$CERT_NAME.crt" -noout -checkend $((30 * 86400)) &>/dev/null; then + echo "Self-signed certificate is still valid for more than 30 days." + exit 0 fi echo "Generating self-signed certificate for localhost..." -openssl req -x509 -nodes -days 3650 -newkey rsa:2048 -keyout /var/lib/quadlets/lego/certificates/localhost.key -out /var/lib/quadlets/lego/certificates/localhost.crt -subj "/CN=localhost" -addext "subjectAltName=DNS:localhost" +install -d -m 0700 -o 10023 -g 10000 "$CERT_DIR" +( + umask 0077 + openssl req -x509 -nodes -days 365 -newkey rsa:2048 -keyout "$CERT_DIR/$CERT_NAME.key" -out "$CERT_DIR/$CERT_NAME.crt" -subj "/CN=localhost" -addext "subjectAltName=DNS:localhost" +) +chown 10023:10000 "$CERT_DIR/$CERT_NAME.key" "$CERT_DIR/$CERT_NAME.crt" -if [[ "$renewal" == "yes" ]]; then - echo "Flagging certificate as renewed..." - touch /var/lib/quadlets/lego/certificates/localhost.renewed -fi +echo "Flagging certificate as renewed..." +install -m 0600 -o 10023 -g 10000 /dev/null "$CERT_DIR/$CERT_NAME.renewed" diff --git a/cookbooks/lego/config/examples/lego.yaml b/cookbooks/lego/config/examples/lego.yaml new file mode 100644 index 0000000..1a52ac9 --- /dev/null +++ b/cookbooks/lego/config/examples/lego.yaml @@ -0,0 +1,51 @@ +# Lego v5 configuration file. +# Reference: https://go-acme.github.io/lego/references/ref-file/ +# +# Heads up! This file is readable by the lego user (uid 10023). +# If you use a DNS-01 challenge, put the DNS provider credentials in a +# separate file (see "envFile" below) with restrictive permissions. + +# Where accounts and certificates are stored (inside the container). +# Mapped to /var/lib/quadlets/lego on the host. +storage: /var/lib/lego + +# Logs go to the systemd journal: no colors. +log: + format: text + +accounts: + default: + # Let's Encrypt staging: https://acme-staging-v02.api.letsencrypt.org/directory + server: https://acme-v02.api.letsencrypt.org/directory + email: changeme@example.tld + acceptsTermsOfService: true + +challenges: + http: + http: + address: ":80" + # Example of a DNS-01 challenge: + # + # dns: + # dns: + # provider: gandiv5 + # envFile: /etc/lego/dns.env + +certificates: + # The certificate ID ("default") is the name of the files in /var/lib/quadlets/lego/certificates. + # Other cookbooks (vsftpd, smtprelay, etc.) expect a certificate named "default". + default: + account: default + challenge: http + domains: + - changeme.example.tld + renew: + # The lego.timer unit already adds a random delay. + disableRandomSleep: true + +hooks: + # Flag certificates as renewed so that the renewal hooks of other cookbooks are triggered. + # The script is interpreted by /bin/sh because SELinux forbids the container + # to execute files from /etc/quadlets/lego (mounted read-only, not relabeled). + deploy: + command: /bin/sh /etc/lego/deploy-hook.sh diff --git a/cookbooks/lego/config/renew-hook/flag-as-renewed.sh b/cookbooks/lego/config/renew-hook/flag-as-renewed.sh deleted file mode 100755 index 2def757..0000000 --- a/cookbooks/lego/config/renew-hook/flag-as-renewed.sh +++ /dev/null @@ -1,3 +0,0 @@ -#!/bin/sh -set -Eeuo pipefail -touch ${LEGO_CERT_PATH%.*}.renewed diff --git a/cookbooks/lego/dropins/lego-renew.container.d/lego-dev.conf b/cookbooks/lego/dropins/lego-renew.container.d/lego-dev.conf deleted file mode 100644 index b29053f..0000000 --- a/cookbooks/lego/dropins/lego-renew.container.d/lego-dev.conf +++ /dev/null @@ -1,7 +0,0 @@ -[Container] -# When in development mode, it is not possible to renew a certificate from Let's Encrypt, so we just skip the entrypoint. -Entrypoint=/bin/true - -[Service] -# And we update the self-signed certificate and flag it as renewed. -ExecStartPost=/etc/quadlets/lego/lego-dev.sh diff --git a/cookbooks/lego/dropins/lego-run.container.d/lego-dev.conf b/cookbooks/lego/dropins/lego.container.d/lego-dev.conf similarity index 100% rename from cookbooks/lego/dropins/lego-run.container.d/lego-dev.conf rename to cookbooks/lego/dropins/lego.container.d/lego-dev.conf diff --git a/cookbooks/lego/lego-hooks.path b/cookbooks/lego/lego-hooks.path new file mode 100644 index 0000000..ad7f94d --- /dev/null +++ b/cookbooks/lego/lego-hooks.path @@ -0,0 +1,13 @@ +[Unit] +Description=Watch for certificates obtained or renewed by Lego +Documentation=https://go-acme.github.io/lego/ +PartOf=lego.target + +[Path] +# The Lego deploy hook touches a .renewed file next to each certificate +# it obtains or renews. +PathExistsGlob=/var/lib/quadlets/lego/certificates/*.renewed +Unit=lego-hooks.service + +[Install] +WantedBy=lego.target diff --git a/cookbooks/lego/lego-renew-hooks.service b/cookbooks/lego/lego-hooks.service similarity index 59% rename from cookbooks/lego/lego-renew-hooks.service rename to cookbooks/lego/lego-hooks.service index cd9048a..a2bba5d 100644 --- a/cookbooks/lego/lego-renew-hooks.service +++ b/cookbooks/lego/lego-hooks.service @@ -1,12 +1,10 @@ [Unit] Description=Process Lego certificate renewal hooks -# Lego touches .renewed files when renewed certificates are available +Documentation=https://go-acme.github.io/lego/ +# Lego touches .renewed files when new certificates are available ConditionPathExistsGlob=/var/lib/quadlets/lego/certificates/*.renewed -After=lego-renew.service +After=lego.service [Service] Type=oneshot ExecStart=/etc/quadlets/lego/process-hooks.sh - -[Install] -WantedBy=lego-renew.service diff --git a/cookbooks/lego/lego-renew.container b/cookbooks/lego/lego-renew.container deleted file mode 100644 index 4cae08a..0000000 --- a/cookbooks/lego/lego-renew.container +++ /dev/null @@ -1,53 +0,0 @@ -[Unit] -Description=Lego Let's Encrypt client - Renew a certificate -Documentation=https://go-acme.github.io/lego/ - -# Only start if there are certificates to renew -ConditionPathExistsGlob=/var/lib/quadlets/lego/certificates/*.crt -# and if Lego has been configured ! -ConditionPathExists=/etc/quadlets/lego/config.env - -[Container] -ContainerName=lego-renew - -# No need for root privileges -User=10023 -Group=10000 -AddCapability=CAP_NET_BIND_SERVICE - -# Image -Image=lego.image -AutoUpdate=registry - -# Network configuration -Network=host - -# Environment variables from config -EnvironmentFile=/etc/quadlets/lego/config.env - -# Volume mounts -Volume=/var/lib/quadlets/lego:/.lego:z -Volume=/etc/quadlets/lego/renew-hook:/etc/lego/renew-hook:ro - -# Be safe, set the umask to 0077 so that private keys are not world-readable -PodmanArgs=--umask=0077 - -# Command to run -Exec=$LEGO_GLOBAL_ARGS renew $LEGO_RENEW_ARGS - -[Service] -# If the command fails, don't try to restart it. -# Otherwise, we would hammer the Let's Encrypt servers and possibly get banned. -Restart=no - -# Getting a certificate may take some time -TimeoutStartSec=600 - -# This container is a job - run once to completion -Type=oneshot - -# These environment variables are sourced to be used by systemd in the Exec* commands -EnvironmentFile=/etc/quadlets/lego/config.env - -[Install] -WantedBy=lego.target diff --git a/cookbooks/lego/lego-run.container b/cookbooks/lego/lego.container similarity index 52% rename from cookbooks/lego/lego-run.container rename to cookbooks/lego/lego.container index fddf339..c7b904c 100644 --- a/cookbooks/lego/lego-run.container +++ b/cookbooks/lego/lego.container @@ -1,15 +1,15 @@ [Unit] -Description=Lego Let's Encrypt client - Fetch a certificate +Description=Lego Let's Encrypt client - Obtain or renew the certificates Documentation=https://go-acme.github.io/lego/ +After=network-online.target +Wants=network-online.target PartOf=lego.target -# Only start if no certificates exist yet -ConditionPathExistsGlob=!/var/lib/quadlets/lego/certificates/*.crt -# and if Lego has been configured ! -ConditionPathExists=/etc/quadlets/lego/config.env +# Only start if Lego has been configured ! +ConditionPathExists=/etc/quadlets/lego/lego.yaml [Container] -ContainerName=lego-run +ContainerName=lego # No need for root privileges User=10023 @@ -23,17 +23,16 @@ AutoUpdate=registry # Network configuration Network=host -# Environment variables from config -EnvironmentFile=/etc/quadlets/lego/config.env - # Volume mounts -Volume=/var/lib/quadlets/lego:/.lego:z +Volume=/var/lib/quadlets/lego:/var/lib/lego:z +Volume=/etc/quadlets/lego:/etc/lego:ro # Be safe, set the umask to 0077 so that private keys are not world-readable PodmanArgs=--umask=0077 -# Command to run -Exec=$LEGO_GLOBAL_ARGS run $LEGO_RUN_ARGS +# Lego v5 obtains the missing certificates and renews the expiring ones +# described in its configuration file. It does nothing otherwise. +Exec=--config /etc/lego/lego.yaml [Service] # If the command fails, don't try to restart it. @@ -41,13 +40,10 @@ Exec=$LEGO_GLOBAL_ARGS run $LEGO_RUN_ARGS Restart=no # Getting a certificate may take some time -TimeoutStartSec=600 +TimeoutStartSec=900 # This container is a job - run once to completion Type=oneshot -# These environment variables are sourced to be used by systemd in the Exec* commands -EnvironmentFile=/etc/quadlets/lego/config.env - [Install] WantedBy=lego.target diff --git a/cookbooks/lego/lego.image b/cookbooks/lego/lego.image index c84e1c1..1111b03 100644 --- a/cookbooks/lego/lego.image +++ b/cookbooks/lego/lego.image @@ -3,5 +3,4 @@ Description=podman pull docker.io/goacme/lego Documentation=https://go-acme.github.io/lego/ [Image] -# TODO: Migrate this Quadlet to Lego v5 (https://go-acme.github.io/lego/migration/index.html) -Image=docker.io/goacme/lego:v4 +Image=docker.io/goacme/lego:v5 diff --git a/cookbooks/lego/lego.target b/cookbooks/lego/lego.target index a8ac1d0..90d479a 100644 --- a/cookbooks/lego/lego.target +++ b/cookbooks/lego/lego.target @@ -1,8 +1,11 @@ [Unit] Description=Lego Let's Encrypt client Documentation=https://go-acme.github.io/lego/ -Requires=lego-run.service lego-renew.timer -After=lego-run.service lego-renew.timer +Requires=lego.timer lego-hooks.path +# A failure to obtain or renew a certificate (network issue, ACME server down, etc.) +# must not stop the target. lego.timer will try again later. +Wants=lego.service +After=lego.service lego.timer lego-hooks.path # Allow isolation - can stop/start this target independently AllowIsolate=yes diff --git a/cookbooks/lego/lego-renew.timer b/cookbooks/lego/lego.timer similarity index 67% rename from cookbooks/lego/lego-renew.timer rename to cookbooks/lego/lego.timer index 20efcea..b234efb 100644 --- a/cookbooks/lego/lego-renew.timer +++ b/cookbooks/lego/lego.timer @@ -1,5 +1,5 @@ [Unit] -Description=Lego Let's Encrypt client - Renew a certificate +Description=Lego Let's Encrypt client - Renew the certificates Documentation=https://go-acme.github.io/lego/ PartOf=lego.target @@ -7,6 +7,7 @@ PartOf=lego.target OnCalendar=daily RandomizedDelaySec=1h Persistent=true +Unit=lego.service [Install] WantedBy=lego.target diff --git a/cookbooks/lego/tests/test_01_install.py b/cookbooks/lego/tests/test_01_install.py new file mode 100644 index 0000000..93354e0 --- /dev/null +++ b/cookbooks/lego/tests/test_01_install.py @@ -0,0 +1,281 @@ +import subprocess +import textwrap +import time +import test_quadlet # noqa: F401 +import pytest +import testinfra + +CERT_DIR = "/var/lib/quadlets/lego/certificates" +CERT_PATH = f"{CERT_DIR}/default.crt" +KEY_PATH = f"{CERT_DIR}/default.key" +LEGO_CONFIG = "/etc/quadlets/lego/lego.yaml" +HOOK_MARKER = "/tmp/lego-hook-executed" + + +def prime_dns_cache(dns_server_ip: str, names: list[str]) -> None: + """Workaround for flaky ACME validations in the test environment. + + Pebble resolves the domains over TCP. When the libvirt dnsmasq has to forward the AAAA + query of a test domain upstream over TCP, the query may time out. A UDP query puts the + (negative) answer in the dnsmasq cache, so that the subsequent TCP query of Pebble succeeds. + """ + for name in names: + for qtype in ("A", "AAAA"): + subprocess.run(["dig", "+time=2", "+tries=2", qtype, f"@{dns_server_ip}", name], capture_output=True) + + +@pytest.fixture(scope="module") +def dns_names() -> list[str]: + """lego domain must be DNS-resolvable so Pebble can reach the VM on port 80 for HTTP-01 validation.""" + return ["lego"] + + +@pytest.fixture(scope="module") +def fcos_extra_files(pebble_acme_server, top_level_domain) -> dict: + """Extra files to inject into the FCOS VM for lego tests.""" + return { + # Lego configuration pointing at the Pebble ACME test server. + LEGO_CONFIG: ( + textwrap.dedent(f"""\ + storage: /var/lib/lego + log: + format: text + accounts: + default: + server: {pebble_acme_server['directory_url']} + email: test@{top_level_domain} + acceptsTermsOfService: true + challenges: + http: + http: + address: ":80" + certificates: + default: + account: default + challenge: http + domains: + - lego.{top_level_domain} + renew: + disableRandomSleep: true + hooks: + deploy: + command: /bin/sh /etc/lego/deploy-hook.sh + """), + 10023, # owner: lego + 10000, # group: itix-svc + 0o644, + ), + # Pebble root CA certificate, needed by lego to authenticate the Pebble ACME server. + "/etc/quadlets/lego/pebble.pem": ( + pebble_acme_server["ca_cert"], + 0, # owner: root + 0, # group: root + 0o644, + ), + # Drop-in to trust the Pebble CA. + "/etc/containers/systemd/lego.container.d/pebble.conf": ( + textwrap.dedent("""\ + [Container] + Environment=LEGO_CA_CERTIFICATES=/etc/lego/pebble.pem + """), + 0, # owner: root + 0, # group: root + 0o644, + ), + # Test hook: creates a marker file to verify that process-hooks.sh calls it. + "/etc/quadlets/lego/renew-hooks.d/test-hook.sh": ( + textwrap.dedent(f"""\ + #!/bin/sh + set -eu + touch {HOOK_MARKER} + """), + 0, # owner: root + 0, # group: root + 0o700, + ), + } + + +""" +Verify that the Lego Quadlet is correctly installed and configured on a fresh VM boot. +""" +class TestLegoQuadlet(test_quadlet.TestQuadlet): + expected_services = [ + {"name": "lego.target", "state": "active", "exists": True}, + {"name": "lego.timer", "state": "active", "exists": True}, + {"name": "lego-hooks.path", "state": "active", "exists": True}, + ] + + expected_files = [ + {"path": "/var/lib/quadlets/lego", "type": "directory", "owner": "lego", "group": "itix-svc", "mode": 0o755}, + {"path": "/etc/quadlets/lego", "type": "directory"}, + {"path": "/etc/quadlets/lego/deploy-hook.sh", "type": "file", "owner": "root", "mode": 0o755}, + {"path": "/etc/quadlets/lego/process-hooks.sh", "type": "file", "owner": "root", "mode": 0o755}, + {"path": "/etc/quadlets/lego/renew-hooks.d", "type": "directory"}, + ] + + expected_podman_images = [ + {"name": "docker.io/goacme/lego", "tag": "v5", "state": "present"}, + ] + + expected_main_service = "lego.target" + expected_main_service_timeout = 600 # Allow time for image pull + ACME negotiation + + def get_serial(self, host) -> str: + result = host.run(f"openssl x509 -in {CERT_PATH} -noout -serial") + assert result.rc == 0, f"Failed to read certificate serial: {result.stderr}" + return result.stdout.strip() + + def wait_for_hooks(self, host, timeout: int = 120) -> None: + """Block until the hook marker exists and all .renewed files have been processed.""" + deadline = time.monotonic() + timeout + while time.monotonic() < deadline: + marker = host.file(HOOK_MARKER).exists + renewed = host.run(f"ls {CERT_DIR}/*.renewed 2>/dev/null | wc -l").stdout.strip() + if marker and renewed == "0": + return + time.sleep(2) + raise TimeoutError( + f"Renewal hooks were not processed within {timeout}s:\n" + + host.run("journalctl -u lego-hooks.service -u lego-hooks.path --no-pager -n 50").stdout + ) + + def run_lego(self, host) -> None: + result = host.run("systemctl start lego.service") + assert result.rc == 0, ( + f"lego.service failed: {result.stderr}\n" + + host.run("journalctl -u lego.service --no-pager -n 50").stdout + ) + + # ------------------------------------------------------------------ + # Test 1 — TLS certificate retrieval + # ------------------------------------------------------------------ + + @pytest.mark.flaky(reruns=6, reruns_delay=10) + def test_lego_certificate_obtained(self, fcos_host, top_level_domain, dns_server_ip): + """lego.service must successfully fetch a TLS certificate from the Pebble ACME server.""" + state = self.wait_for_unit_done(fcos_host, "lego.service", timeout=600) + if state == "failed": + # The first attempt may fail if the DNS record of the VM was not + # resolvable by Pebble when lego started: try again. + prime_dns_cache(dns_server_ip, [f"lego.{top_level_domain}"]) + self.run_lego(fcos_host) + + for path in (CERT_PATH, KEY_PATH): + f = fcos_host.file(path) + assert f.is_file, f"{path} not found" + assert f.user == "lego", f"{path} must be owned by lego, got {f.user}" + assert f.mode == 0o600, f"{path} must not be world-readable: got {oct(f.mode)}" + + domain = f"lego.{top_level_domain}" + result = fcos_host.run(f"openssl x509 -in {CERT_PATH} -noout -ext subjectAltName") + assert result.rc == 0, f"openssl failed to parse certificate: {result.stderr}" + assert f"DNS:{domain}" in result.stdout, ( + f"Expected {domain!r} in the certificate SAN:\n{result.stdout}" + ) + + # ------------------------------------------------------------------ + # Test 2 — Hooks are executed after the first issuance + # ------------------------------------------------------------------ + + def test_lego_hooks_executed_on_issuance(self, fcos_host): + """lego-hooks.path must trigger process-hooks.sh, which runs renew-hooks.d/ and cleans up .renewed files.""" + self.wait_for_hooks(fcos_host) + + # ------------------------------------------------------------------ + # Test 3 — Lego is idempotent + # ------------------------------------------------------------------ + + def test_lego_skips_renewal(self, fcos_host): + """A second run of lego.service must not renew a fresh certificate.""" + serial_before = self.get_serial(fcos_host) + fcos_host.run(f"rm -f {HOOK_MARKER}") + + self.run_lego(fcos_host) + + assert self.get_serial(fcos_host) == serial_before, "The certificate has been renewed but should not" + result = fcos_host.run("journalctl -u lego.service --no-pager -n 20") + assert "Skip renewal" in result.stdout, f"Expected 'Skip renewal' in lego logs:\n{result.stdout}" + time.sleep(5) + assert not fcos_host.file(HOOK_MARKER).exists, "Hooks have been run but no certificate was renewed" + + # ------------------------------------------------------------------ + # Test 4 — Certificate renewal + # ------------------------------------------------------------------ + + @pytest.mark.flaky(reruns=3, reruns_delay=5) + def test_lego_certificate_renewal(self, fcos_host, top_level_domain, dns_server_ip): + """lego.service must obtain a new certificate when forced to (renew.days: -1) and run the hooks.""" + prime_dns_cache(dns_server_ip, [f"lego.{top_level_domain}"]) + serial_before = self.get_serial(fcos_host) + fcos_host.run(f"rm -f {HOOK_MARKER}") + + # Force the renewal: a negative number of days always renews the certificate. + result = fcos_host.run( + f"cp -a {LEGO_CONFIG} {LEGO_CONFIG}.orig && " + f"sed -i 's/disableRandomSleep: true/disableRandomSleep: true\\n days: -1/' {LEGO_CONFIG}" + ) + assert result.rc == 0, f"Failed to edit lego.yaml: {result.stderr}" + try: + self.run_lego(fcos_host) + finally: + fcos_host.run(f"mv {LEGO_CONFIG}.orig {LEGO_CONFIG}") + + assert self.get_serial(fcos_host) != serial_before, ( + f"Certificate serial did not change after renewal (still {serial_before})" + ) + self.wait_for_hooks(fcos_host) + + # ------------------------------------------------------------------ + # Test 5 — Timer configuration + # ------------------------------------------------------------------ + + def test_lego_timer_configured(self, fcos_host): + """lego.timer must be active and scheduled for daily execution.""" + result = fcos_host.run("systemctl show lego.timer --property=ActiveState,TimersCalendar,Unit") + assert result.rc == 0 + assert "ActiveState=active" in result.stdout, f"lego.timer is not active:\n{result.stdout}" + # OnCalendar=daily is normalized as "*-*-* 00:00:00" + assert "OnCalendar=*-*-* 00:00:00" in result.stdout, ( + f"Expected a daily schedule in timer properties:\n{result.stdout}" + ) + assert "Unit=lego.service" in result.stdout, f"lego.timer must start lego.service:\n{result.stdout}" + + # ------------------------------------------------------------------ + # Test 6 — Reboot survival (must run last) + # ------------------------------------------------------------------ + + def test_lego_survives_reboot(self, fcos_vm, fcos_host, test_ssh_key): + """After a reboot, lego.target must be active and the certificate must be intact.""" + serial_before = self.get_serial(fcos_host) + + # Schedule the reboot so that the SSH command returns before the connection drops. + result = fcos_host.run("systemd-run --on-active=3 systemctl reboot") + assert result.rc == 0, f"Failed to schedule the reboot: {result.stderr}" + time.sleep(15) # let the reboot begin before we start polling SSH + + fcos_vm.wait_ssh(ssh_key=test_ssh_key, timeout=300) + + # The module-scoped fcos_host fixture is stale after reboot; create a fresh connection. + new_host = testinfra.get_host( + f"ssh://root@{fcos_vm.ip}", + ssh_extra_args=( + f"-i {test_ssh_key}" + " -o StrictHostKeyChecking=no" + " -o UserKnownHostsFile=/dev/null" + ), + ) + + self.wait_for_service(new_host, "lego.target", timeout=300) + self.check_expected_services(new_host, self.expected_services) + + # lego.service runs at each boot but must not renew a fresh certificate. + state = self.wait_for_unit_done(new_host, "lego.service", timeout=300) + assert state == "inactive", ( + f"lego.service ended in unexpected state: {state!r}\n" + + new_host.run("journalctl -b -u lego.service --no-pager -n 50").stdout + ) + result = new_host.run("systemctl show lego.service --property=Result") + assert "Result=success" in result.stdout, f"lego.service did not succeed after reboot:\n{result.stdout}" + + assert self.get_serial(new_host) == serial_before, "Certificate changed after reboot"