Skip to content

Conversation

@HansVRP
Copy link
Contributor

@HansVRP HansVRP commented Dec 17, 2025

No description provided.

@HansVRP HansVRP requested review from pvbouwel and soxofaan December 17, 2025 09:56
@HansVRP HansVRP requested a review from soxofaan December 17, 2025 15:30
@soxofaan
Copy link
Member

FYI: I pushed a commit to simplify the run/queuing logic a bit (and merge master)

@soxofaan
Copy link
Member

I also noticed that some tests are getting more flaky with these changes,
not sure yet what is going on

@HansVRP
Copy link
Contributor Author

HansVRP commented Dec 17, 2025

checking if it can be related to omitting this 'created' state

@soxofaan
Copy link
Member

soxofaan commented Dec 17, 2025

checking if it can be related to omitting this 'created' state

I was also looking at this, and it is indeed a subtle interaction between

  • not counting "created" as "running"
  • mixing openeo job status (queued, running) and internal status (e.g. queued_for_start) in the same "status" column"
  • unpredictability of threading

Apparently, it can happen that the "status" of a job goes from "queued_for_start" to "created". While this looks wrong, it's actually possible now that status polling and job starting happens in different threads.
If this happens on backend "bar", the job manager (wrongly) thinks that "bar" has capacity, and launches the last job there, while it used to launch it on "foo" (which was set up to actually finish faster than "bar")

@soxofaan
Copy link
Member

I tried to put "created" back (also on master),
Let's see what this gives

@soxofaan
Copy link
Member

flakiness seems to be solved again by adding "created" to "running" count, oh well

I'll add a changelog entry and merge then I think

@soxofaan soxofaan merged commit c6fe140 into master Dec 17, 2025
7 checks passed
@soxofaan soxofaan deleted the issue838_enforece_limited_queue branch December 17, 2025 17:11
HansVRP pushed a commit that referenced this pull request Dec 19, 2025
HansVRP added a commit that referenced this pull request Dec 19, 2025
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.

3 participants