Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Builds: check celery and redis before marking a build as stale #8269

Closed
wants to merge 4 commits into from
Closed

Builds: check celery and redis before marking a build as stale #8269

wants to merge 4 commits into from
+118 −99

Commits on Mar 3, 2022

  1. Builds: check celery and redis before marking a build as stale 

    Previously, we were checking only if the build has been running for a long-long
time (2hs) before marking it as stale.

This new approach tries to reduce this time to 15m and adds extra checks on top
of that:

- the build has been running for more than 15 minutes
- the build is not being executed by Celery (celery.inspect.active)
- the build is not queued in Redis queues

When all of these are `True` we mark the build as stale.
    @humitos
    humitos committed Mar 3, 2022
    Configuration menu
    Copy the full SHA
    087e7c3 View commit details
    Browse the repository at this point in the history

  2. Delete non-working test

    @humitos
    humitos committed Mar 3, 2022
    Configuration menu
    Copy the full SHA
    85f529a View commit details
    Browse the repository at this point in the history

  3. Minor fixes

    @humitos
    humitos committed Mar 3, 2022
    Configuration menu
    Copy the full SHA
    fefbe62 View commit details
    Browse the repository at this point in the history

  4. Lint

    @humitos
    humitos committed Mar 3, 2022
    Configuration menu
    Copy the full SHA
    97aee88 View commit details
    Browse the repository at this point in the history