Friday, 2024-04-26

rpittaugood morning ironic, happy friday! o/07:47
mohammed_Hi! Can reviewers check if one of the unresolved suggestions here is blocking the merge https://review.opendev.org/c/openstack/sushy-tools/+/87536608:11
mohammed_eager to move this forward and follow up with small improvement cycles ! 08:11
opendevreviewcid proposed openstack/ironic master: wip: Provision aarch64 fake-bare-metal-vms  https://review.opendev.org/c/openstack/ironic/+/91544108:58
opendevreviewcid proposed openstack/ironic master: wip: Provision aarch64 fake-bare-metal-vms  https://review.opendev.org/c/openstack/ironic/+/91544109:01
opendevreviewcid proposed openstack/ironic master: wip: Provision aarch64 fake-bare-metal-vms  https://review.opendev.org/c/openstack/ironic/+/91544109:05
opendevreviewFabian Wiesel proposed openstack/sushy-oem-idrac master: Added method for getting kvm session  https://review.opendev.org/c/openstack/sushy-oem-idrac/+/91702011:07
iurygregorygood morning Ironic11:16
opendevreviewFabian Wiesel proposed openstack/sushy-oem-idrac master: Added method for getting kvm session  https://review.opendev.org/c/openstack/sushy-oem-idrac/+/91702012:12
opendevreviewFabian Wiesel proposed openstack/sushy-oem-idrac master: Added method for getting kvm session  https://review.opendev.org/c/openstack/sushy-oem-idrac/+/91702012:14
opendevreviewFabian Wiesel proposed openstack/sushy-oem-idrac master: Added method for getting kvm session  https://review.opendev.org/c/openstack/sushy-oem-idrac/+/91702012:16
TheJuliagood morning12:58
dtantsurTGIF!12:58
TheJuliaIndeed13:05
TheJuliaA quick review would be appreciated: https://review.opendev.org/c/openstack/ironic/+/91693213:51
dtantsurWow, that's radical13:57
TheJuliadevstack is toasty there. :(13:59
TheJulianow to see what neutron folks say about crazy networking idea14:01
dtantsur\o/14:04
dtantsurTheJulia: first tell them we're about to rewrite neutron in go. When they freak out, say "okay, this is plan B then: <your pitch>" :D14:04
TheJuliadtantsur: that is evil14:04
dtantsurI envision enormous success!14:04
rpittaudtantsur, janders, when you have a moment please there's a question for you in the 2024.2 workitems patch https://review.opendev.org/c/openstack/ironic-specs/+/916295 14:19
TheJuliadtantsur: ... I think so, I mean the ask was sort of limited to "please don't break ml2" and "don't send hate mail"14:48
TheJulia:)14:48
dtantsurthat's something we can try :)15:10
dtantsurTheJulia: answered15:12
TheJuliahuh?15:12
dtantsurah, sorry, I mixed up the lines in IRC. it was for rpittau 15:13
TheJuliaAhh, no worries15:13
rpittauthanks dtantsur :)15:13
dtantsurIt *is* your question in the spec, hence my confusion :D15:13
TheJuliaI just commented on it as well :)15:16
dtantsurTheJulia: it is one way to put that. But let's also wait for janders to wake up on Monday.15:17
TheJulia++15:17
dtantsurLeaving a bit earlier today, have a nice weekend folks!15:17
TheJuliais the nc-si stuff related to specific vendors?15:17
TheJuliaor is it a configuration on existing vendors we know about?15:18
dtantsurmmmm.. there was one vendor that decided to bring down the NICs when the host is powered off15:18
dtantsurstarts with D, ends with ll15:18
TheJuliaI ask because I have a vendor related power bug filed and it woudl be sort of similar15:18
TheJuliaffs15:18
TheJuliaOkie Dokie! Have a great weekend!15:18
dtantsurFeel free to share with me here or downstream, I'll check on Monday15:18
TheJuliait is mellenox15:18
TheJuliathey have no actual power off state15:19
TheJuliaat least, in some of their cards15:19
dtantsur\o/15:19
TheJuliaso only "reboot" is an option15:19
opendevreviewMerged openstack/ironic master: Ensure [molds] password is not logged  https://review.opendev.org/c/openstack/ironic/+/91660715:19
dtantsurwell, one more reason to do this work, I guess15:19
dtantsurcards = as in DPU?15:19
TheJuliayup15:19
dtantsursigh, I see. well, one more reason to pitch this to our PM :D15:19
TheJuliaI'm unsure if this differs with the DPUs with external power15:19
dtantsurI'd expect it to be different, but who knows15:20
TheJuliayeah15:20
TheJuliawithout all the variations in a lab I can go poke and prod... yeah15:20
dtantsuroh yeah, that's the worst part of it15:20
dtantsurwe can only do so much by just reading specs (see also: redfish)15:21
TheJuliaI can only take a comment or two at face value15:21
TheJuliaas well15:21
TheJuliayup15:21
TheJuliaThe other side of the same problem15:21
dtantsurhave to go for real, chat with you later o/15:21
TheJuliao/15:22
rpittauI think I'll go too, have a great weekend everyone! o/15:23
opendevreviewVerification of a change to openstack/ironic master failed: Fix iscsi url generate method for ipxe  https://review.opendev.org/c/openstack/ironic/+/91030015:50
opendevreviewVerification of a change to openstack/ironic master failed: Validate url options at config input layer  https://review.opendev.org/c/openstack/ironic/+/91660616:00
*** awb_ is now known as awb`16:35
*** awb` is now known as awb16:35
opendevreviewcid proposed openstack/ironic master: wip: Provision aarch64 fake-bare-metal-vms  https://review.opendev.org/c/openstack/ironic/+/91544117:53
opendevreviewcid proposed openstack/ironic master: wip: Flexible IPMI credential persistence method configuration  https://review.opendev.org/c/openstack/ironic/+/91722918:03
opendevreviewcid proposed openstack/ironic master: wip: Flexible IPMI credential persistence method configuration  https://review.opendev.org/c/openstack/ironic/+/91722918:13
cidgood night and happy weekend everyone \o18:16
opendevreviewMerged openstack/ironic master: Trivial: use canonical name of microversion-parse  https://review.opendev.org/c/openstack/ironic/+/91686323:19
opendevreviewJulia Kreger proposed openstack/ironic master: docs: document stance on partition image use  https://review.opendev.org/c/openstack/ironic/+/91538823:38
opendevreviewJulia Kreger proposed openstack/ironic master: docs: update ilo docs regarding status -> use redfish  https://review.opendev.org/c/openstack/ironic/+/91538723:41
opendevreviewJulia Kreger proposed openstack/ironic master: Inject a randomized publisher id  https://review.opendev.org/c/openstack/ironic/+/91502223:46
opendevreviewJulia Kreger proposed openstack/ironic master: ci: re-enable grenade test job  https://review.opendev.org/c/openstack/ironic/+/91051623:53

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