The GitHub Actions job "Tests" on 
airflow.git/fix/dbt-cloud-trigger-timeout-message has failed.
Run started by GitHub user eran-moses-human (triggered by guan404ming).

Head commit for run:
44ae5c7b72b45665c92856b6bec054bc24c0ddc8 / eran-moses-human 
<[email protected]>
Fix DbtCloudRunJobTrigger timeout error message and add final status check

The timeout error message in DbtCloudRunJobTrigger.run() printed
self.end_time (an absolute epoch timestamp) labelled as "seconds",
producing nonsensical output like "after 1771200015.8 seconds" instead
of a meaningful duration.

Additionally, the timeout check fired before sleeping, without a final
status poll. A job completing during asyncio.sleep() could be
incorrectly reported as timed out.

Changes:
- Move asyncio.sleep() before the timeout check so the trigger sleeps
  first, then evaluates the deadline.
- Add a final is_still_running() call when the timeout fires so that
  jobs completing at the boundary are handled correctly.
- Replace the misleading epoch-as-duration message with a clear
  "within the configured timeout" message.
- Update existing timeout test and add a new test for the edge case
  where a job completes at the timeout boundary.

Closes: #61979
Co-authored-by: Cursor <[email protected]>

Report URL: https://github.com/apache/airflow/actions/runs/22053345348

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to