Add log timers around all communication over Celery.

Review Request #14373 — Created March 18, 2025 and updated — Latest diff uploaded

Information

ReviewBot
release-4.x

Reviewers

In order to help debug networking-related problems, we now include log
timers around all outgoing requests to Celery. This includes task runs
and tool refreshes. These are DEBUG-logged unless timing exceeds a
threshold, helping to catch delayed requests.

As this utilizes new functionality in Djblets 5.3 / Review Board 7.0.1,
we're including a backport of the modern log_timed(). This will be
removed when we can depend on newer releases.

Tested worker status refreshes, tool refreshes, automatic tool runs,
and manual tool runs. All provided the correct logging information.

Unit tests pass.

Changes between revision 2 and 3

orig
1
2
3

Commits

Summary ID Author
Add log timers around all communication over Celery.
In order to help debug networking-related problems, we now include log timers around all outgoing requests to Celery. This includes task runs and tool refreshes. These are DEBUG-logged unless timing exceeds a threshold, helping to catch delayed requests. As this utilizes new functionality in Djblets 5.3 / Review Board 7.0.1, we're including a backport of the modern `log_timed()`. This will be removed when we can depend on newer releases.
954127f1816b68be0ea84eae612db4f8edbed2c5 Christian Hammond
Add log timers around all communication over Celery.
In order to help debug networking-related problems, we now include log timers around all outgoing requests to Celery. This includes task runs and tool refreshes. These are DEBUG-logged unless timing exceeds a threshold, helping to catch delayed requests. As this utilizes new functionality in Djblets 5.3 / Review Board 7.0.1, we're including a backport of the modern `log_timed()`. This will be removed when we can depend on newer releases.
839f20a186152d9427d7e714af409a3fa7e921cf Christian Hammond
extension/reviewbotext/compat/__init__.py
extension/reviewbotext/compat/logs.py
Loading...