From 0ecc7ae594df84082d574e54ac93a765086d995d Mon Sep 17 00:00:00 2001 From: Mike Reeves Date: Thu, 23 Apr 2026 11:25:44 -0400 Subject: [PATCH] soup: drop --local from postgres.telegraf_users reconcile The manager's /etc/salt/minion (written by so-functions:configure_minion) has no file_roots, so salt-call --local falls back to Salt's default /srv/salt and fails with "No matching sls found for 'postgres.telegraf_users' in env 'base'". || true was silently swallowing the error, which meant the DB roles for the pillar entries just populated by the so-telegraf-cred backfill loop never actually got created. Route through salt-master instead; its file_roots already points at the default/local salt trees. --- salt/manager/tools/sbin/soup | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/salt/manager/tools/sbin/soup b/salt/manager/tools/sbin/soup index d54af4a13..440ef47bc 100755 --- a/salt/manager/tools/sbin/soup +++ b/salt/manager/tools/sbin/soup @@ -536,7 +536,10 @@ post_to_3.1.0() { [[ -n "$mid" ]] || continue /usr/sbin/so-telegraf-cred add "$mid" || echo " warning: so-telegraf-cred add $mid failed" >&2 done - salt-call --local state.apply postgres.telegraf_users queue=True || true + # Run through the master (not --local) so state compilation uses the + # master's configured file_roots; the manager's /etc/salt/minion has no + # file_roots of its own and --local would fail with "No matching sls found". + salt-call state.apply postgres.telegraf_users queue=True || true POSTVERSION=3.1.0 }