Skip to content

Fix tracker Sentry noise and transient sandbox health #patch#324

Closed
tthuwng wants to merge 1 commit intoprodfrom
codex/sentry-tracker-fixes-20260504
Closed

Fix tracker Sentry noise and transient sandbox health #patch#324
tthuwng wants to merge 1 commit intoprodfrom
codex/sentry-tracker-fixes-20260504

Conversation

@tthuwng
Copy link
Copy Markdown
Contributor

@tthuwng tthuwng commented May 5, 2026

Summary\n- Return an empty task breakdown for runs that have not discovered tasks yet\n- Retry transient Daytona sandbox refresh failures before surfacing health errors\n\n## Tests\n- uv run pytest services/tracker/tests/unit/test_fastapi_server.py::TestFastapiServer::test_fetch_benchmark_with_no_discovered_tasks services/tracker/tests/unit/test_sandbox.py::TestSandboxHealth::test_refresh_sandbox_data_retries_transient_failure\n- uv run ruff check services/tracker/src/tracker/utils.py services/tracker/src/tracker/sandbox.py services/tracker/tests/unit/test_fastapi_server.py services/tracker/tests/unit/test_sandbox.py


Open in Devin Review

@assert-app
Copy link
Copy Markdown

assert-app Bot commented May 5, 2026

Review on Assert →

@tthuwng tthuwng changed the title Fix tracker Sentry noise and transient sandbox health Fix tracker Sentry noise and transient sandbox health #patch May 5, 2026
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@tthuwng
Copy link
Copy Markdown
Contributor Author

tthuwng commented May 5, 2026

Closing per deployment decision: keep this round scoped out of Valkyrie and avoid tracker code changes.

@tthuwng tthuwng closed this May 5, 2026
@tthuwng tthuwng deleted the codex/sentry-tracker-fixes-20260504 branch May 5, 2026 02:07
Sign up for free to 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