| opendevreview | Merged openstack/ironic master: DB & Object layer for node.shard https://review.opendev.org/c/openstack/ironic/+/864236 | 00:26 |
|---|---|---|
| opendevreview | Merged openstack/ironic master: API support for CRUD node.shard https://review.opendev.org/c/openstack/ironic/+/866235 | 00:27 |
| opendevreview | Steve Baker proposed openstack/ironic master: Respond to rpc requests on stop until hash ring reset https://review.opendev.org/c/openstack/ironic/+/872601 | 01:49 |
| opendevreview | Steve Baker proposed openstack/ironic master: On rpc service stop, wait for node reservation release https://review.opendev.org/c/openstack/ironic/+/872834 | 01:49 |
| opendevreview | Merged openstack/ironic master: Allow port queries by shard list https://review.opendev.org/c/openstack/ironic/+/872361 | 02:12 |
| opendevreview | Steve Baker proposed openstack/ironic master: WIP Expose conductor online boolean for accurate alive https://review.opendev.org/c/openstack/ironic/+/873660 | 02:18 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: WIP: cross-conductor rpc https://review.opendev.org/c/openstack/ironic-specs/+/873662 | 02:24 |
| TheJulia | ^^^ idea w/r/t baremetal sig last week | 02:24 |
| opendevreview | Verification of a change to openstack/ironic master failed: Add support for filtering for sharded nodes https://review.opendev.org/c/openstack/ironic/+/872472 | 02:46 |
| opendevreview | Verification of a change to openstack/metalsmith stable/zed failed: Use a network cache in Instance https://review.opendev.org/c/openstack/metalsmith/+/873276 | 02:59 |
| opendevreview | Verification of a change to openstack/ironic master failed: Add support for filtering for sharded nodes https://review.opendev.org/c/openstack/ironic/+/872472 | 03:05 |
| TheJulia | ugh some narly errors in https://zuul.opendev.org/t/openstack/build/4ae6a9b6dbbc415296a50753a3e0d87f | 03:48 |
| * TheJulia guesses new setuptools | 03:48 | |
| arne_wiebalck | Good morning, Ironic! | 07:23 |
| rpittau | good morning ironic! o/ | 07:55 |
| zigo | Hi. | 08:46 |
| zigo | FYI, proliantutils.tests.redfish.resources.system.test_system.HPESystemTestCase.test_update_persistent_boot_uefi_target() is broken under py 3.11: | 08:46 |
| zigo | https://bugs.debian.org/1030469 | 08:46 |
| zigo | Sorry, but I wont have time to file bug or investigate, I'll just blacklist the unit test in my package, so I would appreciate if someone picks the issue... | 08:46 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add mac address to the fakedriver node https://review.opendev.org/c/openstack/sushy-tools/+/873606 | 09:07 |
| rpittau | zigo: that fails in any version of Python, the test looks old, try with a more recent version of proliantutils | 09:11 |
| zigo | rpittau: Thanks. | 09:50 |
| opendevreview | Jakub Jelinek proposed openstack/ironic master: Erase swift inventory entry on node deletion https://review.opendev.org/c/openstack/ironic/+/871394 | 10:58 |
| iurygregory | good morning | 11:42 |
| kubajj | Morning everyone | 11:58 |
| * TheJulia tries to wake up | 14:38 | |
| kubajj | TheJulia: morning, if you would like something easy to do while waking up, may I suggest https://review.opendev.org/c/openstack/ironic/+/871394 ? 😉 | 14:41 |
| TheJulia | heh | 14:42 |
| TheJulia | okay | 14:42 |
| rpittau | lol | 14:49 |
| rpittau | good morning TheJulia :) | 14:49 |
| TheJulia | kubajj: done | 15:05 |
| kubajj | TheJulia: thanks | 15:05 |
| opendevreview | Michal Nasiadka proposed openstack/bifrost stable/yoga: Allow for almalinux and rocky installations https://review.opendev.org/c/openstack/bifrost/+/873581 | 15:43 |
| mnasiadka | Hello | 15:44 |
| mnasiadka | I posted ^^ - because Kolla needs that in order to backport Rocky Linux 9 support to stable/yoga - is that mergable? | 15:44 |
| TheJulia | mnasiadka: ack, that seems like a slightly "gray area" sort of backport, but your logic makes sense so +2 from me | 15:47 |
| mnasiadka | TheJulia: thanks :) | 15:48 |
| opendevreview | Merged openstack/metalsmith stable/zed: Use a network cache in Instance https://review.opendev.org/c/openstack/metalsmith/+/873276 | 15:49 |
| kubajj | dtantsur, or JayF: I don't know if you're around, but could I get a +2 for https://review.opendev.org/c/openstack/ironic/+/871394 ? I slightly modified it from the version dtantsur review in the morning to fix the nits | 16:42 |
| dtantsur | looking | 16:49 |
| dtantsur | kubajj: approved, thanks for your patience! please do check TheJulia's comment (can be fixed in a follow-up). | 16:51 |
| kubajj | dtantsur: will do. Need to create a follow up to fix db as well. | 16:52 |
| rpittau | good night! o/ | 16:59 |
| JayF | I apparently have COVID, I'm going to be out for a while. | 17:33 |
| iurygregory | take care JayF | 17:35 |
| TheJulia | eek, hopefully you feel better soon JayF | 17:56 |
| opendevreview | Merged openstack/ironic master: Erase swift inventory entry on node deletion https://review.opendev.org/c/openstack/ironic/+/871394 | 18:39 |
| kubajj | oops, hope it's going to be alright, JayF | 18:56 |
| kubajj | dtantsur, TheJulia: any suggestions on how to deal with https://opendev.org/openstack/ironic/src/branch/master/ironic/db/sqlalchemy/api.py#L2595 ? The nodeInventoryNotFound has been modified in the last change to take node as a parameter and says "Node inventory record for node %(node)s could not be found." which doesn't really work with Inventory_id | 19:04 |
| TheJulia | kubajj: I'd just modify it I guess | 19:31 |
| opendevreview | Verification of a change to openstack/ironic master failed: Add configurable delays to the fake drivers https://review.opendev.org/c/openstack/ironic/+/861127 | 20:00 |
| opendevreview | Harald Jensås proposed openstack/metalsmith stable/zed: Get ports by 'binding:host_id' query filter https://review.opendev.org/c/openstack/metalsmith/+/873588 | 20:36 |
| opendevreview | Harald Jensås proposed openstack/metalsmith stable/zed: list_instances - cache allocations https://review.opendev.org/c/openstack/metalsmith/+/873589 | 20:37 |
| opendevreview | Harald Jensås proposed openstack/metalsmith stable/yoga: Use a network cache in Instance https://review.opendev.org/c/openstack/metalsmith/+/873770 | 20:37 |
| opendevreview | Harald Jensås proposed openstack/metalsmith stable/xena: Use a network cache in Instance https://review.opendev.org/c/openstack/metalsmith/+/873771 | 20:38 |
| opendevreview | Harald Jensås proposed openstack/metalsmith stable/wallaby: Use a network cache in Instance https://review.opendev.org/c/openstack/metalsmith/+/873772 | 20:38 |
| opendevreview | Merged openstack/ironic master: Add support for filtering for sharded nodes https://review.opendev.org/c/openstack/ironic/+/872472 | 21:28 |
| TheJulia | \o/ | 21:29 |
| opendevreview | Verification of a change to openstack/ironic master failed: Add configurable delays to the fake drivers https://review.opendev.org/c/openstack/ironic/+/861127 | 22:47 |
| opendevreview | Verification of a change to openstack/ironic master failed: Add configurable delays to the fake drivers https://review.opendev.org/c/openstack/ironic/+/861127 | 23:38 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!