Bug #35797: nic network/bridge comparison can be nil - Foreman

Bug #35797

nic network/bridge comparison can be nil

Added by Ben Magistro 4 months ago. Updated about 1 month ago.

Status:

Closed

Priority:

Normal

Assignee:

Category:

Compute resources - libvirt

Target version:

Difficulty:

Triaged:

Yes

Bugzilla link:

Pull request:

https://github.com/theforeman/foreman/pull/9526

Fixed in Releases:

3.6.0

Found in Releases:

3.3.0

Red Hat JIRA:

Description

When provisioning a VM with a bridge and sriov passthrough, the nic iteration may compare nil to nil resulting in random mac address being associated with the interface and boot/discovery not working as expected.

Nov 19 14:26:32 provisioning foreman[22162]: Comparing networks --   <Fog::Libvirt::Compute::Nic
Nov 19 14:26:32 provisioning foreman[22162]: mac="52:54:00:f8:65:09",
Nov 19 14:26:32 provisioning foreman[22162]: id=nil,
Nov 19 14:26:32 provisioning foreman[22162]: type="bridge",
Nov 19 14:26:32 provisioning foreman[22162]: network=nil,
Nov 19 14:26:32 provisioning foreman[22162]: bridge="br-vm-mgmt",
Nov 19 14:26:32 provisioning foreman[22162]: model="virtio" 
Nov 19 14:26:32 provisioning foreman[22162]: > to {"type"=>"bridge", "bridge"=>"br-vm-mgmt", "model"=>"virtio"}
Nov 19 14:26:32 provisioning foreman[22162]: Comparing networks --  to
Nov 19 14:26:32 provisioning foreman[22162]: Found

Nov 19 14:26:32 provisioning foreman[22162]: Comparing networks --   <Fog::Libvirt::Compute::Nic
Nov 19 14:26:32 provisioning foreman[22162]: mac="52:54:00:cd:74:06",
Nov 19 14:26:32 provisioning foreman[22162]: id=nil,
Nov 19 14:26:32 provisioning foreman[22162]: type="network",
Nov 19 14:26:32 provisioning foreman[22162]: network="passthrough-p3p2",
Nov 19 14:26:32 provisioning foreman[22162]: bridge=nil,
Nov 19 14:26:32 provisioning foreman[22162]: model="virtio" 
Nov 19 14:26:32 provisioning foreman[22162]: > to {"type"=>"network", "network"=>"passthrough-p3p2", "bridge"=>"br-vm-mgmt", "model"=>"virtio"}
Nov 19 14:26:32 provisioning foreman[22162]: Comparing networks -- passthrough-p3p2 to passthrough-p3p2
Nov 19 14:26:32 provisioning foreman[22162]: Found

Nov 19 14:26:32 provisioning foreman[22162]: Comparing networks --   <Fog::Libvirt::Compute::Nic
Nov 19 14:26:32 provisioning foreman[22162]: mac="52:54:00:45:28:cf",
Nov 19 14:26:32 provisioning foreman[22162]: id=nil,
Nov 19 14:26:32 provisioning foreman[22162]: type="network",
Nov 19 14:26:32 provisioning foreman[22162]: network="passthrough-p3p2",
Nov 19 14:26:32 provisioning foreman[22162]: bridge=nil,
Nov 19 14:26:32 provisioning foreman[22162]: model="virtio" 
Nov 19 14:26:32 provisioning foreman[22162]: > to {"type"=>"network", "network"=>"passthrough-p3p2", "bridge"=>"br-vm-mgmt", "model"=>"virtio"}
Nov 19 14:26:32 provisioning foreman[22162]: Comparing networks -- passthrough-p3p2 to passthrough-p3p2
Nov 19 14:26:32 provisioning foreman[22162]: Found

tfm-nics.png	View tfm-nics.png	66.8 KB	Ben Magistro, 11/30/2022 08:12 PM
virtmgr-nic1.png	View virtmgr-nic1.png	73.7 KB	Ben Magistro, 11/30/2022 08:12 PM
virtmgr-nic3.png	View virtmgr-nic3.png	59 KB	Ben Magistro, 11/30/2022 08:12 PM
virtmgr-nic2.png	View virtmgr-nic2.png	58.3 KB	Ben Magistro, 11/30/2022 08:12 PM

Associated revisions

Revision d459e62d (diff)
Added by Ben Magistro 3 months ago

Fixes #35797 - libvirt nil network comparison

When libvirt compares networks, it is possible that `network` will be
`nil` resulting in a match instead of continuing on to the bridge
comparison.

Signed-off-by: Ben Magistro <koncept1@gmail.com>

History

#1 Updated by The Foreman Bot 4 months ago

Status changed from New to Ready For Testing
Pull request https://github.com/theforeman/foreman/pull/9526 added

#2 Updated by Ben Magistro 4 months ago

File virtmgr-nic3.png virtmgr-nic3.png added
File virtmgr-nic2.png virtmgr-nic2.png added
File virtmgr-nic1.png virtmgr-nic1.png added
File tfm-nics.png tfm-nics.png added

While testing/updating a revised patch, captured some screen shots from tfm and virt-manager showing the nics (macs) not mapping correctly so attaching them here for reference.