Bug 2222243 - Kubemacpool does not handle properly interfaces update on running VMs
Summary: Kubemacpool does not handle properly interfaces update on running VMs
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Container Native Virtualization (CNV)
Classification: Red Hat
Component: Networking
Version: 4.14.0
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
: ---
Assignee: Petr Horáček
QA Contact: Nir Rozen
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-07-12 11:51 UTC by Ram Lavi
Modified: 2023-07-24 08:42 UTC (History)
0 users

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2023-07-24 08:42:20 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker CNV-30888 0 None None None 2023-07-12 11:53:12 UTC

Description Ram Lavi 2023-07-12 11:51:47 UTC
Description of problem:
When a running VM changes (adds/removes/updates) an interface, the changes will occur on the VMI only after reboot, but kubemacpool performs the changes immediately, hence creating an inherent diff between the macMap and what is actually consumed.

For example, If a running VM removes an interface, the MAC will be removed from the macMap, while the VMI is still using it, effectively exposing the MAC for duplication.


Version-Release number of selected component (if applicable):


How reproducible:
100%

Steps to Reproduce:
1. Create a running VM (=VM1) with 1 secondary interface, remember the MAC assigned.
2. Remove the secondary interface
3. Create another VM (=VM2), using the MAC assigned to VM-1.

Actual results:
Kubemacpool should reject VM2

Expected results:
Kubemacpool allows creation of VM2.

Additional info:
Since Kubemacpool assigned MACs consecutively, the issue should not manifest unless one of the 2 occurs:
1. Kubemacpool assigned all the free MACs in the range, and circles back to this MAC.
2. The user specifically chooses the MAC, overriding the consecutive MAC assignment.

Comment 1 Petr Horáček 2023-07-24 08:42:20 UTC
Considering the conditions under which this issue may turn into a real problem, how much effort fixing this would require, and that ideas about alternatives for MAC allocation have been circulating lately, I'm marking this as WONTFIX. If anybody would disagree about the low severity, please reopen this BZ.


Note You need to log in before you can comment on or make changes to this bug.