Wednesday, 2024-08-14

rpittaugood morning ironic! o/07:34
masgharGood morning!08:06
*** jcosmao is now known as Guest42508:13
opendevreviewMerged openstack/ironic master: Follow up to the runbooks change (#922142)  https://review.opendev.org/c/openstack/ironic/+/92591710:27
iurygregorygood morning ironic10:33
* TheJulia caffinates14:20
opendevreviewDoug Goldstein proposed openstack/ironic master: idrac: inherit driver interface from redfish  https://review.opendev.org/c/openstack/ironic/+/92622714:59
JayFI just asked a couple of folks privately, but https://review.opendev.org/q/topic:ironic-guest-metadata really needs reviews if people have a sec15:06
cardoedtantsur: I noticed that my https://review.opendev.org/c/openstack/ironic/+/926227 is a different approach to your https://review.opendev.org/c/openstack/ironic/+/902107 I'm happy to drop mine or take any feedback on mine to make it complete with what you wanted.15:14
dtantsurcardoe: I don't mind taking your approach, I don't have time to work on mine.15:18
dtantsurwe'll need to do my change one day, it's not mutually exclusive with yours15:18
dtantsurand we probably need to remove the no longer accessible wsman interfaces15:18
dtantsurand add a release note15:18
* dtantsur is in a meeting, thus the chaotic c omments15:19
cardoeWell the wsman interfaces are gone already.15:20
dtantsurahhh hmmm. I'm terribly out of date then.15:20
cardoeSo the idrac driver is now a superset of the redfish. Unfortunately since it implements a bunch of the supported fields itself as things have gotten added to the redfish, they haven't been added to the Dell. So just as a first swag I made the idrac inherit from the redfish.15:21
dtantsuryeah, it's a fair call15:21
TheJuliaJayF: I commented on your ironic patch and left a -1, sorry. On a plus side, you were aware it was coming :)15:21
dtantsurnote though that the order of classes in all these lists actually matter for the calculation of the default interface15:22
dtantsurI'll put your patch on my queue and try to provide a more sanely sounding comment when I get some rest (probably tomorrow)15:22
cardoeI appreciate it.15:22
rpittaubye everyone, see you on friday o/15:54
JayFthe -1 always existed in your head, now I know about it :P 15:55
opendevreviewElod Illes proposed openstack/metalsmith master: DNM: gate health test  https://review.opendev.org/c/openstack/metalsmith/+/92629915:59
JayFThat's not a great sign ^ 16:04
JayFHas anyone given metalsmith the needful care and feeding this cycle 16:04
* JayF asked Elod if there was something specific he was concerned about16:10
JayF> only those libs that haven't merged any patch recently (in the last 45 days) < this is why, so not a big deal 16:15
JayFhttps://review.opendev.org/c/openstack/ironic/+/925756 is a very easy review and something that improves our docs a lot if someone wants to land it18:38
JayFtrying to get it in before it conflicts with anything18:38
JayFhttps://review.opendev.org/c/openstack/python-ironicclient/+/924101 also would be nice to land before client freeze (~1wk)19:00
opendevreviewcid proposed openstack/ironic master: [WIP] Add inspection rules  https://review.opendev.org/c/openstack/ironic/+/91830319:24
cid o/19:24
opendevreviewMerged openstack/ironic master: Link to configuration options  https://review.opendev.org/c/openstack/ironic/+/92575619:38
iurygregoryforum session submitted \o/20:08
opendevreviewMerged openstack/python-ironicclient master: Add CLI support for runbooks  https://review.opendev.org/c/openstack/python-ironicclient/+/92410120:10
JayFThanks for landing those, and g/l Iury20:57
cardoeWell I'll rebase https://review.opendev.org/c/openstack/python-ironicclient/+/924895 and I was hoping to get it landed before the client freeze as well.21:45
JayFlink it in here when you rebase and I'll reissue my vote21:46
opendevreviewDoug Goldstein proposed openstack/python-ironicclient master: support passing disable_ramdisk for clean and service state  https://review.opendev.org/c/openstack/python-ironicclient/+/92489521:56
opendevreviewDoug Goldstein proposed openstack/python-ironicclient master: support passing disable_ramdisk for clean and service state  https://review.opendev.org/c/openstack/python-ironicclient/+/92489522:02
opendevreviewDoug Goldstein proposed openstack/python-ironicclient master: support passing disable_ramdisk for clean and service state  https://review.opendev.org/c/openstack/python-ironicclient/+/92489522:04
cardoeThere no white space changes in cid's part. /me shake fist at editor.22:05

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