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.

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

master: add queue timeout #343

Merged
merged 1 commit into from
May 22, 2024
Merged

master: add queue timeout #343

merged 1 commit into from
May 22, 2024

Conversation

roxell
Copy link
Collaborator

@roxell roxell commented May 21, 2024

LAVA jobs can get stuck for weeks if the priority isn't set properly which makes it pointless running those jobs after a week/month when the artefacts isn't available anymore or when its to late (someone else already done it)to report a bug. Another aspect is that LAVA eat up a lot of time just parsing the queue to find the next job to schedule. By adding a LAVA queue timeout to the job so it gets canceled if not scheduled in time, makes the queue length manageable.

LAVA jobs can get stuck for weeks if the priority isn't set properly
which makes it pointless running those jobs after a week/month when the
artefacts isn't available anymore or when its to late (someone else
already done it)to report a bug. Another aspect is that LAVA eat up a
lot of time just parsing the queue to find the next job to schedule.
By adding a LAVA queue timeout to the job so it gets canceled if not
scheduled in time, makes the queue length manageable.

Signed-off-by: Anders Roxell <[email protected]>
@bhcopeland
Copy link
Member

Maybe I am being blind but where is queue: hours: defined in lava docs?

@roxell
Copy link
Collaborator Author

roxell commented May 22, 2024

Maybe I am being blind but where is queue: hours: defined in lava docs?

I'm not sure that there is, but it works if you add it under the overall "timeouts:". W should probably update the LAVA docs.
This is a working job: https://lkft.validation.linaro.org/scheduler/job/7579269

@bhcopeland
Copy link
Member

Agreed, we should get it added to LAVA docs.

Ack, LGTM

@roxell roxell merged commit 3c15b9f into Linaro:master May 22, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants