Plugins » Katello

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1528193

Description of problem:
When subscription-manager (re)registers it will update the hosts interfaces from the subscription-manager facts. When the host has a bond with physical slaves the subscription-manger facts contain in the field mac_address the 'virtual' mac address of the bond (being the mac of the primary interface). The real physical mac address of the interface is in field permanent_mac_address.

The puppet facts always the permanent mac address in the mac_address field.

The rhsm fact parser only uses the field mac_address and creates the interface with this virtual mac. This results either of the following problems:
- duplicate interface created for the slave with the new mac address of the primary. Where puppet has created the slave with the correct permanent address
- omits one of the interfaces, because it renames the existing interface that has the same mac address

This is all independent if the host is already pre-created in Sat6. It also happens on re-registration using e.g. 'subscription-manager register --force'

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

How reproducible:

Steps to Reproduce:
1. Create Client with a bond0 interface
2. (Optional) Create host in Sat6
3. Client: Register using subscription-manager register --org=.. --actkey=..
4. Sat6: Dump interfaces: sudo -u hoici hammer host interface list --host=...
5. Client: Run Puppet to upload facts: puppet agent -t -v --tags=dummy
6. Sat6: Dump interfaces: sudo -u hoici hammer host interface list --host=...
7. Client: Re-Register using subscription-manager register --org=.. --actkey=.. --force
8. Sat6: Dump interfaces: sudo -u hoici hammer host interface list --host=...

Actual results:
In Step 4 not all slaves are included
In Step 5 a duplicate identifier of a slave might exist, one with the mac of the bond0 (added by s-m facts) and the other with the permanent mac (added by
puppet facts)
Step 7 might fail with 'invalid interfaces error'
In Step 8 a duplicate identifier of a slave might exist, one with the mac of the bond0 (added by s-m facts) and the other with the permanent mac (added by
puppet facts)

Expected results:
Subscription-manager parsed facts correctly adds slave interfaces with physical address

Additional info: