Fix GlitchTip error-reporting noise + import-smoke CI#1004
Closed
abrichr wants to merge 2 commits into
Closed
Conversation
- error_reporting: filter unsupported-environment errors (headless/no-display, wrong platform, broken native deps) in before_send_event so CI/automated environments stop flooding the dashboard; only show the user alert when a GUI is actually running (never headless, never for filtered noise). - config: point ERROR_REPORTING_DSN at the live project openadaptai/openadapt (3798); 8771 was unreachable (not in the org), so its events were never visible. - CI: add import-smoke step so core modules must import cleanly (the dominant GlitchTip error class). TODO: Windows matrix for the pywinauto path. Validated the filter against the 17 live issues: drops 53/65 events of environmental noise, keeps the 12-event actionable long tail. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Member
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
GlitchTip (project
openadaptai/openadapt, 3798) was dominated by unactionable import-time crashes from automated/headless environments (bot serversadmin123-nexus-*, CI, no-display Linux) running OpenAdapt where it structurally can't run. These flood the dashboard and bury real issues. Triaging all 17 unresolved issues (65 events): ~80% of events are environmental noise, several "top bugs" are already fixed on this branch (pywinauto guarded, nodeclarative_base(bind=)) and recur only from old-release bots.What
error_reporting.before_send_eventnow drops unsupported-environment errors (no display /$DISPLAY/ X connection, wrong platform,No module named 'pywinauto', numpy ABI_ARRAY_API, headlesscrop_active_window). It also only shows the user alert when a GUI is actually running — previously it popped aQMessageBoxon every event inside the Sentry hook (broken/blocking headless).config.ERROR_REPORTING_DSN→ live project 3798. Previously 8771, which isn't in the org and is unreachable, so its events were never visible in the dashboard.main.yml): core modules must import cleanly — the dominant GlitchTip error class (same class as openadapt-ml 0.14.1: multiple broken internal imports prevent serve, train, and demo commands #999). TODO left for a Windows matrix (pywinauto path).gitignore *.pem— there was an untracked private key (deploy/omniparser-key.pem) at risk ofgit add -A.Validation
Ran the new filter against the 17 live issues: it drops 53/65 events (all the high-volume environmental flood) and keeps the 12-event actionable long tail. At runtime it catches even more (the live hook sees full exception messages).
Not done (deliberate)
try/exceptaroundposthog/dictalchemy3/click— they're declared deps (partial-install artifacts ≠ code bugs;posthogis subclassed at module scope). The import-smoke CI is the correct guard.numpy<2pin (harmful in 2026; the one stale ABI error is filtered).Dashboard
Separately muted (status=ignored) the 7 high-volume noise issues (53 events) via the API —
ignorednotresolved, since old-release bots would reopenresolved.🤖 Generated with Claude Code