-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
virt-controller, hot plug: Avoid race between status update and annotation update #11921
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/cc @AlonaKaplan |
…ation update Determining if an annotation update is required is done according to the interfaces in the VMI status. Currently, updating the VMI status with multus infoSource happens after the first annotation update attempt. This causes an unrequired attempt to update annotations, because the status is completely missing at this stage. Fix it by updating annotations only after the VMI is running, eliminating the transient state false hotplug detection. Signed-off-by: Or Shoval <oshoval@redhat.com>
…eratePodAnnotations Both are related to network, make them next to each other. Signed-off-by: Or Shoval <oshoval@redhat.com>
@oshoval: The following tests failed, say
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
@AlonaKaplan @EdDev |
i am closing it, if someone will see it is important we can reopen the 2nd commits is fine, but it is just a nit |
What this PR does
Determining if an annotation update is required is done according to the
interfaces in the VMI status.
Currently, updating the VMI status with multus infoSource happens
after the first annotation update attempt.
This causes an unrequired attempt to update annotations,
because the status is completely missing at this stage.
Fix it by updating annotations only after the VMI is running,
eliminating the transient state false hotplug detection.
Beside that:
Move
updateMultusAnnotation
next toGeneratePodAnnotations
.Both are related to network, make them next to each other.
Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.
Release note