Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • M metaseq
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 95
    • Issues 95
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 41
    • Merge requests 41
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Administrator
  • metaseq
  • Merge requests
  • !241

[api] Fix worker queue logic

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge waitforever into main Jul 22, 2022
  • Overview 2
  • Commits 1
  • Pipelines 0
  • Changes 1

Created by: stephenroller

Patch Description In #227, we fixed a bug causing unlike-workloads to be batched together, but introduced another. Without resetting the queue outside the if statement, whichever queue was last served would continue to be held onto until work was submitted to it. As a result, no work would be accomplished until someone happened to ask for a generation of that same form.

This fixes our idle loop.

Testing steps Deployed a couple days now.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: waitforever