Commit 421cea1
committed
fix: [#2831] remove --lazy-apps to unbreak OpenTelemetry setup in web
maykin_common.otel.setup_otel() detects uwsgi and defers its real
initialization to a uwsgidecorators.postfork hook -- correct for uWSGI's
default prefork model, but --lazy-apps loads the app in each worker
*after* it's already forked, so that hook has no future fork event to
attach to and never fires. OTel silently never initializes for this
process, dropping every metric/trace it would otherwise record, while
Celery (no uwsgi involved) exports fine -- which is why only
Celery-attributed data ever showed up.
--lazy-apps was added in 781af34 to fix Python 3.13 segfaults at the
fork boundary. Removing it trades that fix for working observability;
if the segfaults resurface, the better fix is teaching maykin_common to
skip the postfork deferral when uwsgi.opt indicates lazy-apps, rather
than reintroducing this.
Closes: #28311 parent 7d4a1b4 commit 421cea1
1 file changed
Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
73 | 73 | | |
74 | 74 | | |
75 | 75 | | |
| 76 | + | |
| 77 | + | |
76 | 78 | | |
77 | 79 | | |
78 | 80 | | |
79 | 81 | | |
80 | | - | |
81 | 82 | | |
82 | 83 | | |
0 commit comments