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(vmbda): fix hotplug api call #147

Merged
merged 2 commits into from
Jul 5, 2024
Merged

Conversation

Isteb4k
Copy link
Contributor

@Isteb4k Isteb4k commented Jun 20, 2024

Description

  1. Patched the virt-controller: it now communicates with virt-api via the virtualization-api.

Previously, during the hot plugging process, virt-controller communicated with virt-api through the KubeVirt API service. However, since this API service was removed, the hot plugging process stopped working.

  1. The Protection service does not attempt to add a finalizer if the resource is in the process of being deleted (to avoid the error No new finalizers can be added if the object is being deleted).

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 force-pushed the fix/vmbda/hotplug-api-call branch 2 times, most recently from 8932bf9 to 7773720 Compare June 20, 2024 15:13
@Isteb4k Isteb4k force-pushed the feat/vd/apply-new-design branch from 8265e04 to 995b937 Compare June 20, 2024 15:38
@Isteb4k Isteb4k force-pushed the fix/vmbda/hotplug-api-call branch 4 times, most recently from 0464fde to 15192fb Compare June 20, 2024 17:44
@Isteb4k Isteb4k force-pushed the feat/vd/apply-new-design branch from 995b937 to 1a36f22 Compare June 21, 2024 07:52
@Isteb4k Isteb4k force-pushed the fix/vmbda/hotplug-api-call branch 5 times, most recently from c6b03cc to 76ffe0d Compare June 21, 2024 12:25
@Isteb4k Isteb4k force-pushed the feat/vd/apply-new-design branch from bc83836 to 035882b Compare June 21, 2024 13:12
@Isteb4k Isteb4k force-pushed the fix/vmbda/hotplug-api-call branch 9 times, most recently from 7193d5f to 3929e96 Compare June 24, 2024 18:40
@Isteb4k Isteb4k force-pushed the feat/vd/apply-new-design branch 3 times, most recently from f55189e to 93938af Compare June 25, 2024 12:05
Base automatically changed from feat/vd/apply-new-design to main June 25, 2024 12:37
@Isteb4k Isteb4k force-pushed the fix/vmbda/hotplug-api-call branch 2 times, most recently from fd7f0c3 to ffef406 Compare June 25, 2024 15:58
@Isteb4k Isteb4k force-pushed the fix/vmbda/hotplug-api-call branch 6 times, most recently from 3198fc0 to efeed43 Compare July 2, 2024 14:42
@Isteb4k Isteb4k force-pushed the fix/vmbda/hotplug-api-call branch from efeed43 to e488fe8 Compare July 3, 2024 13:18
@Isteb4k Isteb4k requested review from diafour and yaroslavborbat July 3, 2024 13:28
@Isteb4k Isteb4k marked this pull request as ready for review July 3, 2024 13:28
@Isteb4k Isteb4k requested a review from yaroslavborbat July 4, 2024 11:54
@Isteb4k Isteb4k force-pushed the fix/vmbda/hotplug-api-call branch 4 times, most recently from e109e1b to 0a6d145 Compare July 5, 2024 09:54
@Isteb4k Isteb4k force-pushed the fix/vmbda/hotplug-api-call branch from 0a6d145 to c302c5c Compare July 5, 2024 11:00
@Isteb4k Isteb4k merged commit 0cce992 into main Jul 5, 2024
1 check passed
@Isteb4k Isteb4k deleted the fix/vmbda/hotplug-api-call branch July 5, 2024 11:10
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