FlexMeasures / flexmeasures / 29753287553

88%

Build Type

push

github

Committed by web-flow

Commit Message

feat: add custom job timeouts, also configurable per queue (#2318)

* feat: add custom job timeouts, also configurable per queue

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* insert PR id

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* config: preserve forecasting timeout default

Keep the built-in forecasting queue timeout at one hour while the global default remains 180 seconds.

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* feat: report unknown timeout queues

Log an error when job timeout configuration names a queue that FlexMeasures does not create.

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* test: cover timeout defaults

Cover the preserved forecasting default, global default overrides, and unknown timeout queue logging.

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* docs: describe job timeout queue defaults

Document the supported timeout queue names and the default forecasting queue timeout.

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* docs: update changelog entry

Mention that the new timeout settings preserve the one-hour forecasting default.

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* config: use regular forecasting timeout default

Let forecasting use the global RQ job timeout unless hosts configure a queue override.

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* feat: log forecasting timeout guidance

Record and log an actionable hint when a forecasting job fails due to an RQ timeout.

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* test: update timeout default expectation

Expect forecasting queues to inherit the global timeout by default.

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* test: cover forecasting timeout guidance

Assert timeout failures log and persist a configuration hint on forecasting jobs.

Signed-... (continued)

Coverage Stats

59 of 65 new or added lines in 4 files covered. (90.77%)

36 existing lines in 3 files now uncovered.

15433 of 18737 relevant lines covered (82.37%)

0.82 hits per line