| opendevreview | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic-inspector/+/906935 | 03:43 |
|---|---|---|
| rpittau | good morning ironic! o/ | 08:06 |
| iurygregory | good morning ironic | 11:00 |
| adam-metal3 | good morning ironic | 11:27 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Add inspection PXE filter https://review.opendev.org/c/openstack/ironic/+/907991 | 12:04 |
| dtantsur | hjensas: hi! If you have a minute for a quick sanity-check on ^^, I'd really appreciate it. I haven't dealt with PXE filters for years. | 12:04 |
| hjensas | dtantsur: I'll take a look. | 12:05 |
| hjensas | dtantsur: looks good, I left a comment on purging the hostdir. | 14:04 |
| dtantsur | hjensas: it's a good question, I'm essentially fine to go any way there. | 14:05 |
| hjensas | dtantsur: I think the risk is that we purge the directory and dnsmasq is left with a host record that is "allowed" - if that host is no longer managed by ironic the host will boot to inspection. | 14:14 |
| TheJulia | derekh_: https://bugs.launchpad.net/ironic/+bug/2044013 | 14:23 |
| TheJulia | derekh_: https://review.opendev.org/c/openstack/ironic/+/900964 | 14:27 |
| TheJulia | derekh_: https://bugs.launchpad.net/ironic/+bug/2032380 | 14:27 |
| TheJulia | derekh_: in-review: https://review.opendev.org/c/openstack/ironic/+/900965 | 14:28 |
| TheJulia | derekh_: https://github.com/openstack/ironic-specs/blob/master/specs/approved/httpboot.rst | 14:28 |
| dtantsur | Do we assign everything to Derek? :) | 14:53 |
| TheJulia | sharing context :) | 15:09 |
| TheJulia | dtantsur: replied to your comment on the mutlipath change | 15:27 |
| TheJulia | Some interesting reading out there on the subject :) | 15:27 |
| dtantsur | ack, will check once I free my head of terraform, libvirt and such | 15:28 |
| TheJulia | yeouch | 15:28 |
| rpittau | good night! o/ | 17:13 |
| TheJulia | goodnight | 17:17 |
| JayF | welcome cid! | 19:02 |
| JayF | dtantsur: cid is the MLH fellow who will be working with us for the next 12 weeks to improve ironic-bug-dashboard | 19:02 |
| TheJulia | Greetings! | 19:03 |
| cid | Thanks a lot. Very glad to be here | 19:09 |
| iurygregory | welcome cid =) | 19:38 |
| cid | Thanks! | 19:38 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: troubleshooting adiction for sync period https://review.opendev.org/c/openstack/ironic/+/908342 | 20:09 |
| JayF | TheJulia: quick review, I have a couple comments if you wanna revise real quick | 20:12 |
| TheJulia | on ^, I pinged hjensas directly since he was more aware of that than I am | 20:12 |
| JayF | my comments are style moreso than substance fwiw | 20:13 |
| JayF | but do as you desire with them :D | 20:13 |
| TheJulia | ack, I was more trying to convey I'm going to hold off until I hear from him since I think I understand the issue, but his brain immediately jumped to extend the sync interval in a case and didn't convey the extra context how his brain got there in notes :) | 20:16 |
| TheJulia | I just self -1'ed that one :) | 20:18 |
| TheJulia | and hopefully https://review.opendev.org/c/openstack/ironic/+/908203 merges first :) | 20:19 |
| opendevreview | Verification of a change to openstack/bifrost master failed: Deprecate ironic-inspector support https://review.opendev.org/c/openstack/bifrost/+/905192 | 20:47 |
| JayF | doh, that's my fault, it'll land on a recheck | 20:48 |
| JayF | didn't see it had a depends-on in the commit and tried to land it lol | 20:48 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/896570 has been waiting a while for additioanl core reviews | 20:50 |
| JayF | (child node power off) | 20:51 |
| opendevreview | Merged openstack/ironic master: trivial: fix api-ref generation script https://review.opendev.org/c/openstack/ironic/+/907638 | 20:58 |
| opendevreview | Verification of a change to openstack/ironic stable/wallaby failed: [stable-only] Fix tempest checkout for older versions https://review.opendev.org/c/openstack/ironic/+/907610 | 22:18 |
| opendevreview | Merged openstack/ironic-lib master: Split common qemu-img functions from disk_utils https://review.opendev.org/c/openstack/ironic-lib/+/906371 | 22:18 |
| opendevreview | Merged openstack/ironic-python-agent stable/2023.2: Fix referencing to the raid_device var which is not set https://review.opendev.org/c/openstack/ironic-python-agent/+/903547 | 22:28 |
| opendevreview | Jay Faulkner proposed openstack/ironic-tempest-plugin master: WIP: Basic API tests for sharding https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/906749 | 22:35 |
| opendevreview | Merged openstack/ironic-python-agent master: Make inspection URL optional if the collectors are provided https://review.opendev.org/c/openstack/ironic-python-agent/+/904026 | 23:06 |
| opendevreview | Jay Faulkner proposed openstack/ironic-tempest-plugin master: WIP: Basic API tests for sharding https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/906749 | 23:52 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!