Skip to content

DUX-5108 Write error log on startup failures#442

Merged
9999years merged 1 commit into
mainfrom
wiggles/dux-5108/wrxn
Apr 28, 2026
Merged

DUX-5108 Write error log on startup failures#442
9999years merged 1 commit into
mainfrom
wiggles/dux-5108/wrxn

Conversation

@9999years
Copy link
Copy Markdown
Member

Instead of immediately propagating a startup error, hold onto the result until we can write a compilation log.

The initialize job usually gets canceled before this can complete; this is necessary-but-not-sufficient.

@linear
Copy link
Copy Markdown

linear Bot commented Apr 28, 2026

@9999years 9999years marked this pull request as ready for review April 28, 2026 20:05
@9999years 9999years requested a review from a team as a code owner April 28, 2026 20:05
@9999years 9999years requested review from dtpowl and lf- April 28, 2026 20:05
@9999years 9999years enabled auto-merge (squash) April 28, 2026 20:05
@9999years 9999years marked this pull request as draft April 28, 2026 20:13
auto-merge was automatically disabled April 28, 2026 20:13

Pull request was converted to draft

Instead of immediately propagating a startup error, hold onto the result
until we can write a compilation log.

The `initialize` job usually gets canceled before this can complete;
this is necessary-but-not-sufficient.
@9999years 9999years force-pushed the wiggles/dux-5108/wrxn branch from 3af83b4 to 8bc41c3 Compare April 28, 2026 21:36
@9999years 9999years marked this pull request as ready for review April 28, 2026 21:48
@9999years 9999years enabled auto-merge (squash) April 28, 2026 22:46
@9999years 9999years merged commit f6c9f90 into main Apr 28, 2026
39 checks passed
@9999years 9999years deleted the wiggles/dux-5108/wrxn branch April 28, 2026 23:15
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.

2 participants