Skip to content

Show active sandbox count and observed memory - #64

Closed
sam2tom wants to merge 3 commits into
mainfrom
codex/runtime-observability-active-sandboxes
Closed

sam2tom wants to merge 3 commits into
mainfrom
codex/runtime-observability-active-sandboxes

Conversation

@sam2tom

@sam2tom sam2tom commented Sep 23, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • replace the Runtime Dashboard's Compute uptime trend with an Active Sandboxes time series
  • deduplicate active counts and observed memory by runtime_allocation_id
  • keep unavailable covered samples visible as zero while preserving true no-coverage gaps
  • preserve PostgreSQL durable Runtime history and the latest Dashboard/Session caching behavior from main

Semantics

  • one active managed allocation contributes one sandbox, regardless of duplicate Session observations
  • memory totals include only allocations with an observed usage/limit pair
  • allocation identity does not depend on started_at; uptime remains a current/detail value only
  • provider-neutral projection continues to support Docker and microsandbox observations

Validation

  • git diff --check
  • focused Dashboard Vitest: 48 passed
  • focused final zero-transition Vitest: 22 passed
  • Web Vitest: 70 files / 690 tests passed
  • TypeScript check passed
  • Vite production build passed
  • Playwright acceptance: 117 passed
  • Go, DB, Agents API, Runtime History, Runtime observation, Docker, microsandbox, Sandbox Node, and SDK stages of make check passed
  • independent blind review completed; one missing zero-transition test was fixed and re-review found no remaining blocker

Known platform gate

The final Rust executor stage of make check is blocked on macOS because current main compiles Linux-only rustix APIs (inotify, mkfifoat, and OFlags::PATH). This branch does not modify that executor implementation.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith with what you need. Autofix is disabled.

sam added 3 commits September 23, 2026 20:57
…bility-active-sandboxes

# Conflicts:
#	apps/web/src/features/dashboard/RuntimeTrendCharts.tsx
…bility-active-sandboxes

# Conflicts:
#	apps/web/src/features/dashboard/DashboardView.test.tsx
#	apps/web/src/features/dashboard/RuntimeTrendCharts.tsx
#	apps/web/src/features/sessions/SessionsView.tsx
sam2tom pushed a commit that referenced this pull request Sep 23, 2026
# Conflicts:
#	apps/web/e2e/agents-lifecycle.spec.ts
#	apps/web/src/features/dashboard/DashboardView.test.tsx
#	apps/web/src/features/dashboard/RuntimeTrendCharts.test.tsx
#	apps/web/src/features/dashboard/RuntimeTrendCharts.tsx
#	apps/web/src/features/dashboard/runtime-history.test.ts
#	apps/web/src/features/dashboard/runtime-trends.ts
#	contracts/agents-api/runtime-observability-design.md
@sam2tom

sam2tom commented Sep 23, 2026

Copy link
Copy Markdown
Collaborator Author

Superseded by #62. PR #62 preserves the lifecycle-state semantics, includes this PR via merge commit 25e20d7, and implements the Dashboard-wide summed active Sandbox series while keeping Session metrics binary.

@sam2tom sam2tom closed this Sep 23, 2026
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