| opendevreview | Doug Goldstein proposed openstack/ironic master: fix redfish processor inspection https://review.opendev.org/c/openstack/ironic/+/955537 | 00:59 |
|---|---|---|
| opendevreview | Doug Goldstein proposed openstack/ironic master: allow running inspection hooks on redfish interface https://review.opendev.org/c/openstack/ironic/+/933066 | 00:59 |
| cardoe | rpittau: I think the first two in that ^ series above should be good. It does make it now that cpu_arch is set to x86_64 for 64-bit machines correctly when inspecting with redfish. Since you were asking about what should be fixed. | 01:03 |
| opendevreview | Queensly Kyerewaa Acheampongmaa proposed openstack/ironic master: Add manual clean and automated verify steps to set BMC clock via Redfish Manager https://review.opendev.org/c/openstack/ironic/+/953477 | 05:27 |
| opendevreview | Jacob Anders proposed openstack/sushy master: Improve handling of empty-string eTags. https://review.opendev.org/c/openstack/sushy/+/956204 | 05:45 |
| opendevreview | Jacob Anders proposed openstack/sushy master: Improve handling of empty-string eTags https://review.opendev.org/c/openstack/sushy/+/956204 | 05:53 |
| rpittau | good morning ironic! o/ | 06:36 |
| queensly[m] | Good morning o/ | 06:56 |
| opendevreview | Merged openstack/ironic master: doc: Use dnsmasq to run tftp server https://review.opendev.org/c/openstack/ironic/+/955579 | 07:49 |
| abongale | good morning ironic o/ | 08:40 |
| opendevreview | Merged openstack/ironic master: Drop redundant geattr https://review.opendev.org/c/openstack/ironic/+/956161 | 09:32 |
| janders | TheJulia thank you for your feedback w/r/t empty string eTags. I would value your feedback on https://review.opendev.org/c/openstack/sushy/+/956204 if you have time | 11:40 |
| frickler | a customer wants/needs to use NFS for redfish virtual-media attachment due to hardware restrictions. does this need further discussion before submitting some patches? https://bugs.launchpad.net/ironic/+bug/2119212 | 12:21 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent bugfix/11.1: Update .gitreview for bugfix/11.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/956235 | 12:59 |
| TheJulia | frickler: I suspect as long as its not insane, we would be okay with it and I think that was the prior discussion so submit a patch? | 12:59 |
| janders | TheJulia further to the empty-string-etag query, I had had the reporter retest with the ^^^ patch and it does seem to resolve his issue. Which is not to say the proposed solution is perfect but it does seem to make a positive difference. | 13:10 |
| TheJulia | janders: looking at it, I might condense it down a little bit more | 13:12 |
| TheJulia | one lining the getting of the value and the comparison | 13:13 |
| janders | no worries, happy to re-spin it tomorrow - thank you! | 13:17 |
| TheJulia | janders: thanks! | 13:18 |
| rpittau | frickler: as far as it's not SMB :) | 13:55 |
| opendevreview | Merged openstack/ironic master: Add manual clean and automated verify steps to set BMC clock via Redfish Manager https://review.opendev.org/c/openstack/ironic/+/953477 | 14:16 |
| TheJulia | heh | 14:32 |
| frickler | yeah, thx, just checking now whether they'll want to submit patches themselves or have me do it | 14:36 |
| TheJulia | ok | 15:05 |
| cardoe | Can we workflow https://review.opendev.org/c/openstack/networking-baremetal/+/945500 ? | 17:55 |
| cardoe | There's actually a bunch of stuff with multiple +2's on it. | 17:55 |
| TheJulia | I workflowed it | 17:57 |
| cardoe | https://review.opendev.org/c/openstack/ironic/+/945515 and https://review.opendev.org/c/openstack/ironic/+/945217 and https://review.opendev.org/c/openstack/ironic-python-agent/+/954245 and https://review.opendev.org/c/openstack/networking-generic-switch/+/955798 | 17:57 |
| cardoe | That's from ironic-week-prio | 17:57 |
| cardoe | I'd love if at least https://review.opendev.org/c/openstack/ironic/+/955537 and it's child landed for the next release. I don't have a release note for it and I guess I could... my release note is ultimately in the commit above it which still fails which enables hooks and rules for redfish (which was one of our release goals) | 17:59 |
| cardoe | But 955537 fixes redfish inspection enough that node.properties isn't out of whack with what regular inspection does today. e.g. we stop setting the cpu_arch to i686 in node.properties | 18:00 |
| cardoe | I'm trying real hard to get the redfish inspect done. but -ETIME | 18:00 |
| TheJulia | I'll try to shift to code review before my next meeting | 18:01 |
| TheJulia | I've got one change I need to make in the mean time | 18:01 |
| cardoe | Happy to review whatever ya need. | 18:02 |
| TheJulia | I'm stuck on getting a futurist feature released at the moment. https://review.opendev.org/c/openstack/futurist/+/955217 blocks our eventlet stuffs at the moment | 18:08 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Replace GreenThreadPoolExecutor in conductor https://review.opendev.org/c/openstack/ironic/+/952939 | 18:17 |
| TheJulia | regarding schema validation stuffs, I think a review jam would be a good idea | 18:54 |
| opendevreview | Merged openstack/networking-generic-switch master: Autogenerate list of NGS compatible devices https://review.opendev.org/c/openstack/networking-generic-switch/+/955798 | 19:49 |
| opendevreview | Merged openstack/ironic-python-agent master: Hard stop on image download duration threshold https://review.opendev.org/c/openstack/ironic-python-agent/+/954245 | 20:03 |
| opendevreview | Queensly Kyerewaa Acheampongmaa proposed openstack/ironic master: [docs] Update manual clean and verify steps https://review.opendev.org/c/openstack/ironic/+/955730 | 20:16 |
| opendevreview | Merged openstack/networking-baremetal master: Clean up baremetal agents on node delete https://review.opendev.org/c/openstack/networking-baremetal/+/945500 | 20:29 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!