Page MenuHomeElementl

Add an error boundary around check_add_heartbeat in daemon loop
ClosedPublic

Authored by dgibson on Jul 22 2021, 6:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 16, 12:16 PM
Unknown Object (File)
Sun, May 14, 7:37 AM
Unknown Object (File)
Tue, May 9, 4:59 AM
Unknown Object (File)
Apr 25 2023, 3:07 PM
Unknown Object (File)
Apr 12 2023, 11:29 PM
Unknown Object (File)
Apr 9 2023, 12:36 PM
Unknown Object (File)
Apr 8 2023, 11:04 AM
Unknown Object (File)
Mar 31 2023, 7:42 AM
Subscribers
None

Details

Summary

There are two heartbeat additions here in this file - we wrapped one of them in a guard but not the other one. This diff adds it to the other one.

Test Plan

BK, run a daemon that fails this check (e.g. with bad run storage credentials), daemon logs instead of crashing. Heartbeat failures may eventually cause it to crash, but not right away.

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable