Page MenuHomeElementl

Heartbeat right away in the daemon on startup
ClosedPublic

Authored by dgibson on Apr 26 2021, 4:59 PM.

Details

Summary

Adding 30 seconds before any heartbeats are logged in D7494 feels like a mistake - particularly now that errors are not considered to be equated with 'not running'. Better to get immediate feedback that the daemon is running, then 30 seconds later get any errors that came in during the first iteration.

Test Plan

Integration

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

johann added inline comments.
integration_tests/test_suites/daemon-test-suite/tox.ini
18

Why did this come up?

This revision is now accepted and ready to land.Apr 26 2021, 11:43 PM