Skip to content

feat(jobs): display per-job out-of-memory evidence - #649

Merged
ImLunaHey merged 2 commits into
mainfrom
luna/dep-7915-job-oom
Oct 26, 2026
Merged

ImLunaHey merged 2 commits into
mainfrom
luna/dep-7915-job-oom

Conversation

@ImLunaHey

@ImLunaHey ImLunaHey commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor
  • Display OOM detection, unavailable reasons, and timestamped process kills in depot jobs <id> and depot jobs metrics <id>.
  • Preserve event details and optional detection values in JSON. Sanitize process names in terminal output; show unavailable when an older server omits OOM evidence.
  • Pin published protobuf and Connect modules from depot/proto#53.

Validation: go test ./..., golangci-lint, module verification, CLI build, and command-help smoke check with published dependencies.

API: depot/api#6241.

Part of DEP-7915.


Note

Low Risk
Read-only CLI presentation of new API fields with safe fallbacks when OOM data is missing; no auth or data-mutation paths.

Overview
Adds out-of-memory (OOM) evidence to depot jobs <id> and depot jobs metrics <id> after the existing job/metrics table output.

Table mode shows detection status (unavailable, none observed, or detected), optional unavailable reasons, timestamped kernel-kill events (PID and sanitized process name), and truncation when more than 100 events exist. Help text now calls out OOM alongside CPU/memory metrics. JSON responses pass through the API’s oom payload unchanged.

Bumps the generated Depot API Connect/protobuf modules to the revision that exposes GithubActionsJobOOM. Shared rendering lives in new writeOOM; tests cover job and metrics commands in JSON and table modes, including older servers with no OOM data and terminal-safe process names.

Reviewed by Cursor Bugbot for commit fd0afdb. Bugbot is set up for automated code reviews on this repo. Configure here.

@linear-code

linear-code Bot commented Oct 26, 2026

Copy link
Copy Markdown

DEP-7915

@ImLunaHey
ImLunaHey marked this pull request as ready for review October 26, 2026 16:54

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Risk: medium. Approved. Cursor Bugbot completed successfully with no findings that need human review.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Router and Approver

@ImLunaHey
ImLunaHey merged commit 86e8cc3 into main Oct 26, 2026
12 checks passed
@ImLunaHey
ImLunaHey deleted the luna/dep-7915-job-oom branch October 26, 2026 16:56
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