Skip to content

[codex] Add worker pod termination status logs#812

Merged
bill-ph merged 2 commits into
mainfrom
codex/worker-termination-summary
Jun 25, 2026
Merged

[codex] Add worker pod termination status logs#812
bill-ph merged 2 commits into
mainfrom
codex/worker-termination-summary

Conversation

@bill-ph

@bill-ph bill-ph commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add a reusable control-plane helper that summarizes worker pod phase/reason/message and selected container state, reason, exit code, signal, timestamps, and restart count
  • include that summary on Worker pod terminated. logs from pod reconciliation
  • snapshot and log pod/container status before the health-check path deletes an unresponsive worker pod

Validation

  • just test-controlplane-k8s
  • just lint

@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown

Test Impact Plan

Deterministic summary of how this PR changes tests, CI runners, and coverage-risk signals.

Summary

Area Added Changed Deleted
Test files 0 1 0
E2E/journey files 0 0 0
Workflow files 0 0 0

Signals

  • Test cases: +0 / -0
  • Assertions: +12 / -0
  • Skips or known failures added: 0
  • Workflow continue-on-error added: 0
  • Workflow path filters added: 0
  • Test commands removed from justfile: 0
  • E2E/journey retry lines added: 0

Coverage risk: neutral or increased

No coverage-reduction warnings detected.

@bill-ph bill-ph marked this pull request as ready for review June 25, 2026 18:46
@bill-ph bill-ph merged commit 6bc1519 into main Jun 25, 2026
30 of 31 checks passed
@bill-ph bill-ph deleted the codex/worker-termination-summary branch June 25, 2026 21:26
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