Thursday, 2025-06-26

opendevreviewTakashi Kajinami proposed openstack/networking-generic-switch master: Update project URL  https://review.opendev.org/c/openstack/networking-generic-switch/+/95335901:07
opendevreviewTakashi Kajinami proposed openstack/ironic master: Remove Python 3.9 support  https://review.opendev.org/c/openstack/ironic/+/95336001:12
opendevreviewTakashi Kajinami proposed openstack/ironic-inspector master: Remove Python 3.9 support  https://review.opendev.org/c/openstack/ironic-inspector/+/95336101:12
opendevreviewTakashi Kajinami proposed openstack/ironic-ui master: Remove Python 3.9 support  https://review.opendev.org/c/openstack/ironic-ui/+/95336201:13
opendevreviewTakashi Kajinami proposed openstack/python-ironicclient master: Remove Python 3.9 support  https://review.opendev.org/c/openstack/python-ironicclient/+/95336301:13
opendevreviewTakashi Kajinami proposed openstack/python-ironic-inspector-client master: Remove Python 3.9 support  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/95336401:14
opendevreviewTakashi Kajinami proposed openstack/networking-generic-switch master: Remove Python 3.9 support  https://review.opendev.org/c/openstack/networking-generic-switch/+/95336501:14
opendevreviewTakashi Kajinami proposed openstack/networking-baremetal master: Remove Python 3.9 support  https://review.opendev.org/c/openstack/networking-baremetal/+/95336601:15
opendevreviewTakashi Kajinami proposed openstack/ironic master: Remove Python 3.9 support  https://review.opendev.org/c/openstack/ironic/+/95336002:12
opendevreviewTakashi Kajinami proposed openstack/ironic-inspector master: Remove Python 3.9 support  https://review.opendev.org/c/openstack/ironic-inspector/+/95336102:13
opendevreviewTakashi Kajinami proposed openstack/networking-generic-switch master: Remove Python 3.9 support  https://review.opendev.org/c/openstack/networking-generic-switch/+/95336502:15
opendevreviewTakashi Kajinami proposed openstack/python-ironic-inspector-client master: Remove Python 3.9 support  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/95336402:15
opendevreviewTakashi Kajinami proposed openstack/python-ironicclient master: Remove Python 3.9 support  https://review.opendev.org/c/openstack/python-ironicclient/+/95336302:15
opendevreviewTakashi Kajinami proposed openstack/networking-baremetal master: Remove Python 3.9 support  https://review.opendev.org/c/openstack/networking-baremetal/+/95336602:15
opendevreviewTakashi Kajinami proposed openstack/ironic-ui master: Remove Python 3.9 support  https://review.opendev.org/c/openstack/ironic-ui/+/95336202:24
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic master: Redfish Firmware Interface - NIC Support  https://review.opendev.org/c/openstack/ironic/+/95339403:06
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic master: Fix Sensor Data values  https://review.opendev.org/c/openstack/ironic/+/95339504:20
rpittaugood morning ironic! o/06:43
rpittaucongrats dtantsur, iurygregory and janders :)06:44
rpittautkajinam: thanks for them, but all the py3.9 removal patches will have to wait :)06:45
queensly[m]Good morning06:48
opendevreviewRiccardo Pittau proposed openstack/ironic-python-agent-builder master: Build ironic-python-agent DIB image with CentOS Stream 10  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/95096507:08
tkajinamrpittau, ok. in case you want me to abandon these now then lmk07:33
opendevreviewTakashi Kajinami proposed openstack/ironic master: Remove rootwrap  https://review.opendev.org/c/openstack/ironic/+/95340407:50
rpittautkajinam: no need to abandon them, we'll just have to wait a little :)07:52
tkajinamrpittau, ack. ok !07:52
ContinuityTheJulia: yes koala08:17
opendevreviewAbhishek Bongale proposed openstack/ironic master: fix: handle unexpected 'loop' in actions field  https://review.opendev.org/c/openstack/ironic/+/95332308:43
opendevreviewSyed Haseeb Ahmed proposed openstack/ironic master: Add interface name to inventory for inspection hooks  https://review.opendev.org/c/openstack/ironic/+/95341409:30
opendevreviewTakashi Kajinami proposed openstack/ironic master: Remove rootwrap  https://review.opendev.org/c/openstack/ironic/+/95340410:20
opendevreviewTakashi Kajinami proposed openstack/ironic master: Remove rootwrap  https://review.opendev.org/c/openstack/ironic/+/95340410:21
opendevreviewTakashi Kajinami proposed openstack/ironic master: Use specific util methods in vfat image creation  https://review.opendev.org/c/openstack/ironic/+/95341710:34
dtantsurcardoe: (without looking at your patch just yet): does the inventory zoo situation affect the core inventory as well or only the plugin_data part?10:36
iurygregorygood morning ironic11:16
opendevreviewSyed Haseeb Ahmed proposed openstack/ironic master: Add interface name to inventory for inspection hooks  https://review.opendev.org/c/openstack/ironic/+/95341411:18
clifgm ironic o/12:18
rpittaupatch to build IPA with centos stream 10 passes CI \o/12:54
rpittauhttps://review.opendev.org/c/openstack/ironic-python-agent-builder/+/95096512:54
iurygregory\o/12:57
cardoedtantsur: I’m only concerning myself with the core inventory and not the core.12:59
cardoeErrr and not plugin data.12:59
dtantsurAnd even that has variations? omg13:00
* dtantsur takes full blame for this mess13:01
dtantsurcardoe: https://review.opendev.org/c/openstack/ironic/+/944868 is not looking bad, but I don't recognize some of the fields and it misses some that IPA does send13:14
cardoeI forget what I started with but it wasn’t IPA. I’m gonna switch to that.13:16
cardoeI think I used the ironic mock data as the initial template.13:17
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent master: Provide usable __repr__ for serializable objects  https://review.opendev.org/c/openstack/ironic-python-agent/+/95345413:59
*** gdelnc is now known as Mufasa14:24
*** Mufasa is now known as gdelnc14:24
opendevreviewcid proposed openstack/ironic master: Fix inspection rules validation  https://review.opendev.org/c/openstack/ironic/+/95346414:45
opendevreviewAbhishek Bongale proposed openstack/ironic stable/2025.1: fix: handle unexpected 'loop' in actions field  https://review.opendev.org/c/openstack/ironic/+/95346514:56
opendevreviewTakashi Kajinami proposed openstack/virtualpdu master: Fix home_page url  https://review.opendev.org/c/openstack/virtualpdu/+/95347115:30
opendevreviewMerged openstack/virtualpdu master: Fix home_page url  https://review.opendev.org/c/openstack/virtualpdu/+/95347116:04
opendevreviewQueensly Kyerewaa Acheampongmaa proposed openstack/ironic master: Add manual clean step to set BMC clock via Redfish Manager  https://review.opendev.org/c/openstack/ironic/+/95347716:55
opendevreviewcid proposed openstack/ironic master: Fix inspection rules validation  https://review.opendev.org/c/openstack/ironic/+/95346417:01
TheJuliaContinuity: Ahh, hmm... I guess it occurs to me maybe we're doing something we shouldn't be doing in general. I'll try to discuss with Harald and see what his thoughts are17:02
opendevreviewVerification of a change to openstack/ironic master failed: Change the novncproxy service code to handle eventlet-lessness  https://review.opendev.org/c/openstack/ironic/+/95335117:46
cardoedtantsur: can we just throw away the serializable thing and just use dataclasses?18:29
cardoeSo I'll use BlockDevice as an example... https://github.com/openstack/ironic-python-agent/blob/883e3cf057aa1e3f7ed5a6fd485e7ab62a8cfd55/ironic_python_agent/hardware.py#L79318:32
cardoeThe serializable_fields are what is sent because we use the base class's method to create the object to submit.18:32
cardoeBut https://github.com/openstack/ironic-python-agent/blob/883e3cf057aa1e3f7ed5a6fd485e7ab62a8cfd55/ironic_python_agent/hardware.py#L1101 is one way stuff is gathered up which isn't going through the serialize18:40
cardoehttps://github.com/openstack/ironic-python-agent/blob/883e3cf057aa1e3f7ed5a6fd485e7ab62a8cfd55/ironic_python_agent/inspector.py#L261 is the call18:41
cardoebecause the inspector creates the payload like... https://github.com/openstack/ironic-python-agent/blob/883e3cf057aa1e3f7ed5a6fd485e7ab62a8cfd55/ironic_python_agent/encoding.py#L67-L7218:42
cardoeExcept that the child functions in list_hardware_info() aren't creating objects that are inheriting from Serializable in most cases. They're creating regular dicts. So what's executing is line 72.18:43
cardoeSo the shape of the data that IPA seems to imply isn't actually 100% correct because there's no enforcement.18:43
cardoeSo here's what I'm doing with my patch, I've just dumped via an updated IPA 6 different hardware models. I'll update the fields to match the intersection between them for the best compatibility. I'm going to make the behavior treat the inventory as if its empty if what's stored in swift or the DB doesn't match that shape.18:48
opendevreviewVerification of a change to openstack/ironic master failed: Change the novncproxy service code to handle eventlet-lessness  https://review.opendev.org/c/openstack/ironic/+/95335118:49
cardoeToday that means the redfish inspector won't be valid. Haseeb on my team is working on making it conform. But it also means that some older IPAs and some ironic-inspector migrated data won't be valid either.18:50
cardoeI'm going to add a "version" field to the inspection data. I'm only focused on the inspection data and not the plugin data. I feel that plugin_data should be owned by the different hooks and it's their choice how to shape their field.18:51
cardoeIt'll just be an integer and we start with 0.18:51
cardoeYou know what. I'll take this out of IRC and make a bug.18:51
* TheJulia blinks and realizes she is only now looking at IRC for the second time all day18:59
opendevreviewSyed Haseeb Ahmed proposed openstack/ironic master: Add interface name to inventory for inspection hooks  https://review.opendev.org/c/openstack/ironic/+/95341419:04
opendevreviewMerged openstack/ironic master: Change the novncproxy service code to handle eventlet-lessness  https://review.opendev.org/c/openstack/ironic/+/95335121:19

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