Skip to content

Fix Traefik restart and ACME reliability - #300

Merged
arjunkomath merged 3 commits into
mainfrom
fix/traefik-restart-acme-reliability
Aug 22, 2026
Merged

Fix Traefik restart and ACME reliability#300
arjunkomath merged 3 commits into
mainfrom
fix/traefik-restart-acme-reliability

Conversation

@arjunkomath

Copy link
Copy Markdown
Member

Summary

  • keep Prometheus bucket values stable across YAML round trips so dynamic reconciliation does not trigger false static-config restarts
  • wait for Traefik metrics readiness after required restarts instead of sleeping for a fixed two seconds
  • retry rollout certificate issuance three times with durable 10-second and 20-second Inngest backoff

Testing

  • cd agent && go test ./internal/traefik -run 'TestEnsurePrometheusMetricsConfig|TestWaitForMetricsReady|TestEnsureDynamicConfigReloaded|TestWaitForSuccessfulReloadAfter'\n- cd agent && go test ./internal/agent\n- cd agent && TMPDIR="$(cd "$TMPDIR" && pwd -P)" go test ./...\n- cd agent && go build ./...\n- cd agent && test -z "$(gofmt -l .)"\n- cd web && pnpm test (69 files, 505 tests)\n- cd web && ./node_modules/.bin/tsc --noEmit\n- cd web && pnpm lint (0 errors; 7 unrelated existing warnings)\n\n## Local macOS note\n\nThe default logical /var/... temp path exposes an existing registryauth path-canonicalization bug in TestManagerInstallDirtyAndPermissions. agent/internal/registryauth is unchanged in this PR; the complete Go suite passes when TMPDIR uses its equivalent physical /private/var/... path.

@github-actions

github-actionsBot commented Aug 21, 2026

Copy link
Copy Markdown

React Doctor found no new issues. 🎉

Reviewed by React Doctor for commit dd89d31.

@arjunkomath
arjunkomath added this pull request to the merge queueAug 22, 2026
Merged via the queue into main with commit 509e391Aug 22, 2026
21 checks passed
@arjunkomath
arjunkomath deleted the fix/traefik-restart-acme-reliability branch August 22, 2026 01:42
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@arjunkomath