Skip to content

pvf priority#2305

Merged
turuslan merged 5 commits intomasterfrom
feature/pvf-priority
Dec 16, 2024
Merged

pvf priority#2305
turuslan merged 5 commits intomasterfrom
feature/pvf-priority

Conversation

@turuslan
Copy link
Copy Markdown
Contributor

@turuslan turuslan commented Dec 4, 2024

Referenced issues

Description of the Change

  • metric for pvf queue size
  • split pvf queue by job kind
  • process queued approval jobs before queued backing jobs

Possible Drawbacks

  • gives all time to approval jobs (rust gives 94% to approval and 6% to backing)

Signed-off-by: turuslan <turuslan.devbox@gmail.com>
Signed-off-by: turuslan <turuslan.devbox@gmail.com>
@turuslan turuslan requested review from ErakhtinB and kamilsa December 4, 2024 17:17
Signed-off-by: turuslan <turuslan.devbox@gmail.com>
@turuslan turuslan enabled auto-merge (squash) December 16, 2024 02:37
@turuslan turuslan merged commit d056f23 into master Dec 16, 2024
@turuslan turuslan deleted the feature/pvf-priority branch December 16, 2024 02:58
turuslan added a commit that referenced this pull request May 5, 2025
Signed-off-by: turuslan <turuslan.devbox@gmail.com>

#2305 added loop over multiple queues in `deque`, but didn't add free workers check
@turuslan turuslan mentioned this pull request May 5, 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