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

fix(core, kubevirt): add ability to configure burst for virt-api rate limiter #103

Merged
merged 2 commits into from
May 27, 2024

Conversation

Isteb4k
Copy link
Contributor

@Isteb4k Isteb4k commented May 17, 2024

Description

When migrating a big number of virtual machines (>50), we encounter a bottleneck in virt-api in the form of a rate limiter that prevents migration completion.

A patch has been updated to virt-api, enabling the configuration of Burst for the rate limiter via an environment variable VIRT_API_RATE_LIMITER_BURST.

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

@Isteb4k Isteb4k changed the title fix(core, kubevirt): add ability to configure burst for virt-api rate… fix(core, kubevirt): add ability to configure qps for virt-api rate limiter May 17, 2024
@Isteb4k Isteb4k changed the base branch from main to refactor/crd/api-add-name-suffix May 17, 2024 08:58
@Isteb4k Isteb4k force-pushed the refactor/crd/api-add-name-suffix branch from 9457918 to 35f2479 Compare May 17, 2024 08:59
@Isteb4k Isteb4k force-pushed the fix/kubevirt/vms-migration branch from 13a57b1 to d74862b Compare May 17, 2024 08:59
@Isteb4k Isteb4k changed the title fix(core, kubevirt): add ability to configure qps for virt-api rate limiter fix(core, kubevirt): add ability to configure burst for virt-api rate limiter May 17, 2024
@Isteb4k Isteb4k force-pushed the fix/kubevirt/vms-migration branch from d74862b to c84163b Compare May 17, 2024 10:08
@Isteb4k Isteb4k marked this pull request as ready for review May 20, 2024 11:04
@Isteb4k Isteb4k requested review from diafour and yaroslavborbat May 20, 2024 11:04
templates/kubevirt/kubevirt.yaml Outdated Show resolved Hide resolved
@fl64 fl64 added the validation/skip/doc_changes Skip doc changes validation label May 20, 2024
@Isteb4k Isteb4k force-pushed the fix/kubevirt/vms-migration branch 2 times, most recently from c7b7534 to d476bba Compare May 20, 2024 11:45
@Isteb4k Isteb4k requested a review from fl64 May 20, 2024 11:50
@fl64 fl64 force-pushed the refactor/crd/api-add-name-suffix branch from 1249071 to 847eaff Compare May 20, 2024 15:25
@Isteb4k Isteb4k force-pushed the fix/kubevirt/vms-migration branch from d476bba to b971fe5 Compare May 21, 2024 11:28
@Isteb4k Isteb4k force-pushed the fix/kubevirt/vms-migration branch from 6ef464f to f486c10 Compare May 21, 2024 12:38
Base automatically changed from feat/api/add-name-suffix to main May 24, 2024 09:39
@Isteb4k Isteb4k force-pushed the fix/kubevirt/vms-migration branch from f486c10 to 522ecd9 Compare May 24, 2024 09:47
@Isteb4k Isteb4k requested a review from diafour May 24, 2024 11:59
@Isteb4k Isteb4k merged commit e5c4605 into main May 27, 2024
1 check passed
@Isteb4k Isteb4k deleted the fix/kubevirt/vms-migration branch May 27, 2024 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
validation/skip/doc_changes Skip doc changes validation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants