Skip to content

`SQLDataStore`: print connection timeout in warning, remove calls to deprecated `logger.warn()`

Santiago Gil requested to merge santigl/print-connection-timeout-in-warning into master

Description

When a worker specifies a timeout that is shorter than the database timeout, the SQLDataStore class prints a warning.

This adds the value of the DB timeout to that WARN message so that the user gets an idea of what value to set in the worker to ensure that it gets assigned work.

Changes proposed in this merge request:

buildgrid/server/persistence/sql/impl.py:

  • Print self.connection_timeout in the warning issued when the worker's timeout is too low
  • Change the usages of logger.warn(), which is deprecated, for logger.warning().
Edited by Santiago Gil

Merge request reports