Thursday, 2023-06-15

rpittaugood morning ironic! o/06:27
Nisha_Agarwalmorning ironic!!!08:14
adam-metal3Hello Ironic!08:43
dtantsurhi adam-metal3 08:59
JayFo/15:49
JayFI'm coming to you LIVE from table 14 at the Summit/PTG lol15:49
dtantsurhey, how is it going there? :)15:53
TheJuliaHeading down there at 10!16:02
TheJuliahttps://usercontent.irccloud-cdn.com/file/aIt8P9kP/1686844998.JPG16:03
rpittauhappy PTG :)16:10
rpittauand good night o/16:10
*** dking is now known as Guest318016:19
*** Guest3180 is now known as dking-summit16:33
JayFdking-summit: https://specs.openstack.org/openstack/ironic-inspector-specs/specs/lldp-reporting.html16:33
* dtantsur is curious about the context16:49
JayFdtantsur: dking-summit was having some trouble understanding the lldp output16:56
JayFhe's parsing it in a custom collector that runs after the get_lldp_info16:56
JayFthe data he gets back looks extremely raw, so I think there's a step being missed (not in code; likely in config) to parse it16:56
JayFhis data looks like the unprocessed piece here -> https://specs.openstack.org/openstack/ironic-inspector-specs/specs/lldp-reporting.html#data-model-impact16:57
JayFhttps://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-py39&project=openstack%2Fironic&skip=0 did our unit tests ~fix themselves?16:58
JayFor my temp fix ended up holding up?16:58
opendevreviewJay Faulkner proposed openstack/ironic master: Revert "Disabling test_upgrade_twice temporarily for CI fix"  https://review.opendev.org/c/openstack/ironic/+/88619617:13
JayFfyi we are taking notes from the in-person conversations here: https://etherpad.opendev.org/p/ironic-openinfra-202317:48
martindahlAre there any open issues / areas that I as the n00b can attack to add any value?18:30
hjensasJayF: regarding LLDP there are a couple of processing hooks https://opendev.org/openstack/ironic-inspector/src/branch/master/ironic_inspector/plugins - lldp_basic and local_link_connection that may need to be enabled in config.18:36
JayFhjensas: ack, I'll pass that on to dking in person, to check the logs18:52
JayFthat's basically what I thought might be going on, just didn't know the right words/location to point him at :D 18:52
*** dking is now known as Guest320021:35
*** dking is now known as Guest320622:59
*** Guest3206 is now known as dking-summit23:02
JayFdking-summit: https://docs.openstack.org/ironic/latest/contributor/vision.html this was the one we did back in Rocky, pointing at Zed23:02

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!