Commit 19e1619
fix(devserver): exclude .artifacts from granian reload watcher (#117609)
Fixes getsentry/self-hosted#4379
Granian's watchfiles-based reloader watches the entire repo directory by
default. The TeeStream introduced in #117486 writes all
honcho output to .artifacts/dev.log inside the repo root, which
triggered an infinite reload cycle: server logs →-> dev.log updated ->
watchfiles detects change -> granian reloads -> repeat.
Fix by passing reload_ignore_dirs=[".artifacts"] to Granian so
watchfiles skips that directory. The option is surfaced through the
existing options dict so it can be overridden via SENTRY_WEB_OPTIONS if
needed.
<!-- Describe your PR here. -->
<!--
Sentry employees and contractors can delete or ignore the following.
-->
### Legal Boilerplate
Look, I get it. The entity doing business as "Sentry" was incorporated
in the State of Delaware in 2015 as Functional Software, Inc. and is
gonna need some rights from me in order to utilize my contributions in
this here PR. So here's the deal: I retain all rights, title and
interest in and to my contributions, and by keeping this boilerplate
intact I confirm that Sentry can use, modify, copy, and redistribute my
contributions, under Sentry's choice of terms.
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>1 parent 4c69f4f commit 19e1619
1 file changed
Lines changed: 2 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
| 24 | + | |
24 | 25 | | |
25 | 26 | | |
26 | 27 | | |
| |||
67 | 68 | | |
68 | 69 | | |
69 | 70 | | |
| 71 | + | |
70 | 72 | | |
71 | 73 | | |
72 | 74 | | |
| |||
0 commit comments