| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: NetworkAdapter resource ID changes unpredictably across reboots https://review.opendev.org/c/openstack/ironic/+/972421 | 01:54 |
|---|---|---|
| rpittau | good morning ironic! happy friday! o/ | 08:18 |
| abongale | good morning o/ | 09:09 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-prometheus-exporter master: Refactor redfish parser to eliminate code duplication https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/975891 | 11:02 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-prometheus-exporter master: Fix resource management in ipmi.py parser https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/975892 | 11:12 |
| opendevreview | Jakub Jelinek proposed openstack/ironic master: Fix loading inventory data from Swift https://review.opendev.org/c/openstack/ironic/+/975898 | 11:42 |
| kubajj | dtantsur: we figured it out https://bugs.launchpad.net/ironic/+bug/2140909 it was my fault 🥲 | 11:43 |
| opendevreview | Merged openstack/ironic master: Fix loading inventory data from Swift https://review.opendev.org/c/openstack/ironic/+/975898 | 15:18 |
| dtantsur | JFYI folks, I'll be out for 2 weeks | 17:17 |
| -opendevstatus- NOTICE: Gerrit on review.opendev.org will experience a short outage while we upgrade it to 3.11.8 | 17:50 | |
| JayF | dtantsur: have a good break | 18:07 |
| * JayF will be out a week from today thru up to 2 weeks | 18:08 | |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Relax assertion in test_get_all_shards https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/973647 | 18:38 |
| cardoe | So looks like I’m out (peeking in occasionally), TheJulia is out, dtantsur is out and JayF is out. | 19:13 |
| JayF | I'm out a week from tomorrow, starting :) | 19:15 |
| JayF | having minor surgery to correct RSI on my right hand | 19:15 |
| JayF | a week from today | 19:15 |
| JayF | lol | 19:15 |
| JayF | then I'm off until I can type for 8 hours again, more or less | 19:15 |
| opendevreview | Clif Houck proposed openstack/ironic master: Remove shellinabox https://review.opendev.org/c/openstack/ironic/+/975864 | 19:58 |
| TheJulia | https://review.opendev.org/q/topic:ironic-vxlan reviews would always be appreciated in the mean time :) | 21:15 |
| JayF | I have at least a couple hours set aside this weekend to go through some reviews. Depending on how behind I am on general reviews, I might try to context up on your VXlan chain | 21:24 |
| opendevreview | Verification of a change to openstack/ironic master failed: Address remaining review comments for rpc methods https://review.opendev.org/c/openstack/ironic/+/971184 | 22:04 |
| TheJulia | If there is anything which is *wow*, or which causes your brain to click, let me know. I still need to draft up something for the ironic repo. | 22:34 |
| *** gmaan is now known as gmaan_afk | 22:46 | |
| jrosser | TheJulia: for the vxlan stuff, how tightly knitted in is the choice for ingress replication, i.e if at some future point adding a dial to allow PIM in the underlay for BUM traffic was needed | 23:21 |
| jrosser | my specific context being that a bunch of my use cases for ironic need bare metal nodes to support workloads with extremely large amounts of multicast traffic | 23:22 |
| TheJulia | jrosser: not terribly, really. Like Claude originally suggested multicast grouping for Cisco devices, but we limited to ingress only on ciscos for simplicity. The other switch driver, and largely all in the ml2 plugin doing switch config. | 23:38 |
| TheJulia | The ingress replication is largely for spreading Mac/l2vni info around though, I’m under how fitting that into a different config model would really pan out | 23:41 |
| opendevreview | Merged openstack/ironic master: Prevent multiple attach actions being generated for the same port https://review.opendev.org/c/openstack/ironic/+/974520 | 23:58 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!