Bug 1544873

Summary: Introspection fails because LLDP discovery assumes the Chassis ID to be a MAC address (OSP-12)
Product: Red Hat OpenStack Reporter: Bob Fournier <bfournie>
Component: openstack-ironic-inspectorAssignee: Bob Fournier <bfournie>
Status: CLOSED ERRATA QA Contact: mlammon
Severity: high Docs Contact:
Priority: high    
Version: 12.0 (Pike)CC: bfournie, jamsmith, j.thadden, mburns, mlammon, slinaber
Target Milestone: z3Keywords: Triaged, ZStream
Target Release: 12.0 (Pike)   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: openstack-ironic-inspector-6.0.2-1.el7ost Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1535685 Environment:
Last Closed: 2018-08-20 12:46:38 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1535685    
Bug Blocks:    

Comment 3 mlammon 2018-07-18 19:15:01 UTC
Install osp 12 latest puddle: 2018-07-13.1

This cannot be verified using specific hardware (not available), however we can verify that the new code is in place and RPM which we do here:

Env:
openstack-ironic-inspector-6.0.2-1.el7ost.noarch

It appears all changes are in place
sed -n '21p' /usr/lib/python2.7/site-packages/ironic_inspector/plugins/local_link_connection.py
from oslo_utils import netutils

sed -n '91,92p' /usr/lib/python2.7/site-packages/ironic_inspector/plugins/local_link_connection.py
    def _get_lldp_processed_patch(self, name, item, lldp_proc_data, port,
                                  node_info):

sed -n '100,107p' /usr/lib/python2.7/site-packages/ironic_inspector/plugins/local_link_connection.py

            # Only accept mac address for chassis ID
            if (item == SWITCH_ID_ITEM_NAME and
                    not netutils.is_valid_mac(value)):
                LOG.info("Skipping switch_id since it's not a MAC: %s", value,
                         node_info=node_info)
                return

ed -n '147,148p' /usr/lib/python2.7/site-packages/ironic_inspector/plugins/local_link_connection.py
                                                       proc_data, port,
                                                       node_info)

Comment 5 errata-xmlrpc 2018-08-20 12:46:38 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2018:2517