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

[v7r3] Introduce job status 'Scouting' in WMS #7118

Merged

Conversation

michmx
Copy link

@michmx michmx commented Jul 25, 2023

Scout jobs are a small subset of the primary jobs that run first, and the rest are executed only when scouting is done. Main jobs get the status ‘Scouting’ while waiting for the execution of the subset. This pull request defines the state “Scouting” at JOB_STATES and enables the transitions. Closes #7083

BEGINRELEASENOTES

*WorkloadManagement
NEW: Introducing 'Scouting' status in job state transitions

ENDRELEASENOTES

@DIRACGridBot DIRACGridBot added the alsoTargeting:rel-v8r0 Cherry pick this PR to rel-v8r0 after merge label Jul 25, 2023
@fstagni
Copy link
Contributor

fstagni commented Jul 25, 2023

I am not really in vein of fixing the tests in v7r3 which is close to be deprecated. Merging as the PR is simple enough.

@fstagni fstagni merged commit 980fe29 into DIRACGrid:rel-v7r3 Jul 25, 2023
21 of 24 checks passed
@DIRACGridBot DIRACGridBot added the sweep:done All sweeping actions have been done for this PR label Jul 25, 2023
DIRACGridBot pushed a commit to DIRACGridBot/DIRAC that referenced this pull request Jul 25, 2023
@DIRACGridBot
Copy link

Sweep summary

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

Successful:

  • rel-v8r0

STAGING: State(3, [CHECKING, WAITING, FAILED, KILLED], defState=STAGING),
CHECKING: State(2, [STAGING, WAITING, RESCHEDULED, FAILED, DELETED], defState=CHECKING),
RECEIVED: State(1, [CHECKING, WAITING, FAILED, DELETED], defState=RECEIVED),
STAGING: State(3, [WAITING, FAILED, KILLED], defState=STAGING),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fstagni at least give people that still care about v7r3 a chance to review!
The CHECKING must not be dropped here
4d3e477

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops I missed that one!

@michmx can you create a new PR amending this (and correct #7121 ?)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for that, I copy-pasted from our own patch in the server! I will fix it ASAP.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alsoTargeting:rel-v8r0 Cherry pick this PR to rel-v8r0 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.

4 participants