Skip to content

ateapi: carry the golden actor's crash message into the template status - #2644

Merged
Dmitry Berkovich (dberkov) merged 1 commit into
agent-substrate:mainfrom
dims:template-crash-reason
Oct 28, 2026
Merged

Dmitry Berkovich (dberkov) merged 1 commit into
agent-substrate:mainfrom
dims:template-crash-reason

Conversation

@dims

@dims Davanum Srinivas (dims) commented Oct 14, 2026 •

Copy link
Copy Markdown
Collaborator

When the actor used to build a template crashes, the template status now includes the actor's crash message. This makes failures such as a bad image volume or failed restore visible directly on the template.

Overlaps with #2453, which also adds crash details to template failures.

  • Tests pass: go test -race ./cmd/ateapi/internal/controlapi/; go vet.
  • Appropriate changes to documentation are included in the PR: none needed.

@dims Davanum Srinivas (dims) changed the title [WIP] ateapi: carry the golden actor's crash message into the template status ateapi: carry the golden actor's crash message into the template status Oct 14, 2026
@dims
Davanum Srinivas (dims) marked this pull request as ready for review October 14, 2026 14:13
@dims
Davanum Srinivas (dims) force-pushed the template-crash-reason branch 4 times, most recently from 9f9dbef to e358902 Compare October 18, 2026 21:48
@bowei Bowei Du (bowei) added area/api-machinery kind/feature An enhancement / feature request or implementation labels Oct 19, 2026
@dims
Davanum Srinivas (dims) force-pushed the template-crash-reason branch 2 times, most recently from 1618dd1 to 1fbc8ee Compare October 21, 2026 11:01
@dims
Davanum Srinivas (dims) force-pushed the template-crash-reason branch 7 times, most recently from 4d26e7c to be52ed3 Compare October 27, 2026 12:43
A crashed golden left the template with "golden actor crashed before its
snapshot was taken" and nothing else; the reason sat in the actor's crash
status. Append it, so a bad image volume or a failed restore is readable
from the template.

Signed-off-by: Davanum Srinivas <[email protected]>
@laoj2

Copy link
Copy Markdown
Collaborator

@dberkov
Dmitry Berkovich (dberkov) added this pull request to the merge queue Oct 28, 2026
Merged via the queue into agent-substrate:main with commit 475f49b Oct 28, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/api-machinery kind/feature An enhancement / feature request or implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants