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

[8.0] fix: JobAgent rescheduling wrong jobs #7397

Merged
merged 2 commits into from
Jan 17, 2024

Conversation

aldbr
Copy link
Contributor

@aldbr aldbr commented Jan 16, 2024

When an inner computing element (InProcess/Singularity) processing multiple jobs gets an issue, the wrong job is rescheduled, basically the 1st one.

Done jobs are rescheduled and run twice, which can waste CPU resources.
I will try to simplify the logic in v9.0 by making computing elements (normal and inner) more homogeneous (#7398).

BEGINRELEASENOTES
*WorkloadManagement
FIX: JobAgent rescheduling wrong jobs
ENDRELEASENOTES

@aldbr aldbr requested a review from fstagni January 16, 2024 17:17
@DIRACGridBot DIRACGridBot added the alsoTargeting:integration Cherry pick this PR to integration after merge label Jan 16, 2024
@aldbr aldbr marked this pull request as ready for review January 17, 2024 07:40
@aldbr aldbr force-pushed the v8.0_FIX_JobAgent-ce-failures branch from a5bedbf to c1c49c4 Compare January 17, 2024 11:56
@aldbr aldbr requested a review from fstagni January 17, 2024 11:56
@fstagni fstagni merged commit c11eb31 into DIRACGrid:rel-v8r0 Jan 17, 2024
25 checks passed
@DIRACGridBot DIRACGridBot added the sweep:done All sweeping actions have been done for this PR label Jan 17, 2024
DIRACGridBot pushed a commit to DIRACGridBot/DIRAC that referenced this pull request Jan 17, 2024
@DIRACGridBot
Copy link

Sweep summary

Sweep ran in https://github.com/DIRACGrid/DIRAC/actions/runs/7556373977

Successful:

  • integration

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alsoTargeting:integration Cherry pick this PR to integration after merge sweep:done All sweeping actions have been done for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants