From 0f510c93aab20ed3480e74e621edf2a1a5a850b9 Mon Sep 17 00:00:00 2001 From: acamilo Date: Wed, 23 Sep 2026 13:13:59 +0000 Subject: [PATCH] infra(05-deploy): refuse a bad FLY_FEED_VIA before the release is flipped, not half way through --- infra/05-deploy.sh | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/infra/05-deploy.sh b/infra/05-deploy.sh index bb38302..b80557a 100644 --- a/infra/05-deploy.sh +++ b/infra/05-deploy.sh @@ -112,6 +112,11 @@ fi require_pve_host need pct +# Who serves the feed (docs/design/flybus.md): refused here, before section 1 flips +# /opt/fly/current, rather than half way through the deploy or at flysim's boot. +FLY_FEED_VIA_EFFECTIVE="$(feed_via_normalize "${FLY_FEED_VIA:-}")" \ + || die "05-deploy: FLY_FEED_VIA must be 'direct' or 'bus', got '${FLY_FEED_VIA}'" + # CHROMIUM_PROFILE is validated here, not left to the launcher: a typo or a # `vgl` on a container that never had VirtualGL installed would only show up as # flystage refusing to start, i.e. a black stream, minutes after the deploy @@ -413,9 +418,6 @@ log "05-deploy: non-secret env files" # never drift apart (see cpuset_partition's own header comment). They are # assigned in section 0b, which needs them earlier than this for the # deploy-time cpu pinning; nothing between here and there changes them. -# Who serves the feed (docs/design/flybus.md): refused here rather than at flysim's boot. -FLY_FEED_VIA_EFFECTIVE="$(feed_via_normalize "${FLY_FEED_VIA:-}")" \ - || die "05-deploy: FLY_FEED_VIA must be 'direct' or 'bus', got '${FLY_FEED_VIA}'" tmp_fly_env="$(mktemp)" tmp_flypush_env="$(mktemp)"