| iurygregory | TheJulia, reading your messages now.. | 01:14 |
|---|---|---|
| iurygregory | my focus was mostly on the vmedia path (the issues I've found were there, I wasn't able to test the non-vmedia patch because the machines are in other labs at redhat so we would need to deal with some changes ... | 01:18 |
| iurygregory | I like the idea of https://review.opendev.org/c/openstack/sushy/+/958578, but maybe we still need some changes in sushy/oem/dell to handle the Target and ImportBuffer? | 01:21 |
| TheJulia | I'm unaware of what you mean by input buffer. Furthermore, are you thinking the target change doesn't work? | 03:49 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: [IPE] Support iDRAC driver metrics https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/954870 | 03:50 |
| opendevreview | Merged openstack/ironic master: Fix insufficient mocking in rpc_service tests https://review.opendev.org/c/openstack/ironic/+/958517 | 11:39 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: [IPE] Support iDRAC driver metrics https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/954870 | 12:52 |
| TheJulia | a quick review to clean up n-g-s CI: https://review.opendev.org/c/openstack/ironic/+/958583 | 12:56 |
| opendevreview | Merged openstack/ironic master: Launch API in the same process as conductor for singleprocess Ironic https://review.opendev.org/c/openstack/ironic/+/958462 | 12:59 |
| dtantsur | done | 13:15 |
| TheJulia | thanks! | 13:20 |
| opendevreview | John Garbutt proposed openstack/ironic master: WIP: Choose prefered vmedia boot device https://review.opendev.org/c/openstack/ironic/+/958082 | 13:31 |
| opendevreview | John Garbutt proposed openstack/ironic master: WIP: Request boot from USB for virtual media when required https://review.opendev.org/c/openstack/ironic/+/958655 | 13:31 |
| opendevreview | Merged openstack/ironic master: Reduce the number of RPC calls to traits API https://review.opendev.org/c/openstack/ironic/+/958226 | 13:45 |
| opendevreview | Merged openstack/ironic master: ci: disable the novnc service on multinode https://review.opendev.org/c/openstack/ironic/+/958583 | 14:12 |
| dtantsur | TheJulia: quick q: the oci:// support: does it work for kernel/ramdisk too? | 14:21 |
| JayF | yes | 14:32 |
| JayF | it's supposed to | 14:32 |
| dtantsur | Nice! We've just had another round of "where do we put IPA images" in the Metal3 meeting :D | 14:33 |
| JayF | if my recollection is somehow wrong, we should hook it up anyway | 14:37 |
| opendevreview | cid proposed openstack/ironic master: Follow-up: Fix improper HTTP status code usage https://review.opendev.org/c/openstack/ironic/+/958663 | 14:39 |
| TheJulia | I'm 98% sure I put IPA in OCI as well | 16:32 |
| dtantsur | nice! | 16:45 |
| opendevreview | John Garbutt proposed openstack/ironic master: WIP: Request boot from USB for virtual media when required https://review.opendev.org/c/openstack/ironic/+/958655 | 17:45 |
| JayF | https://bugs.launchpad.net/futurist/+bug/2121545 this may impact us | 18:18 |
| TheJulia | eh, its an artifact of queued or not | 18:25 |
| TheJulia | FWIW, https://github.com/openstack/ironic/blob/master/ironic/conductor/base_manager.py#L382-L409C13 we flag the shutdown well before, eventually we'll have new work cease coming in. I'd *anticipate* the queue to empty to running tasks in that time... | 19:09 |
| JayF | I mentioned it here mainly because I knew queuing behavior was a hot topic | 19:13 |
| cardoe | dtantsur: where are you gonna put the images? | 19:15 |
| TheJulia | JayF: hey, I got dmtiry to put in writing he things our way is way better overall ;) | 19:33 |
| TheJulia | I'll file that under success | 19:33 |
| JayF | which our way :) | 19:33 |
| JayF | there are lots of our ways | 19:33 |
| TheJulia | our custom rejector | 19:33 |
| JayF | ah, yeah | 19:33 |
| cardoe | I would love to reliably get rid of the local http server for conductor. | 19:39 |
| JayF | Huh? | 19:43 |
| JayF | Conductor only runs http server if json-rpc is enabled. | 19:43 |
| TheJulia | cardoe: do you mean for files? | 19:44 |
| TheJulia | for vmedia/booting/etc | 19:44 |
| cardoe | Yeah. There’s too much weird with swift. | 19:44 |
| TheJulia | I'd honestly like to remove swift support | 19:45 |
| TheJulia | so... *shrug* | 19:45 |
| cid | JayF, TheJulia, a possible fix to the above bug... https://review.opendev.org/c/openstack/futurist/+/958689 | 20:22 |
| cardoe | That's why I was asking a few weeks back if we had anything capturing what all could use the local http server. | 20:34 |
| cardoe | Was gonna try to propose some future state. | 20:34 |
| TheJulia | cid: I *feel* like the queue needs to be ensured flushed and/or reject new work being added. Anyhow, just a thought. | 20:35 |
| TheJulia | cardoe: ipxe/httpboot is all I can really think of. | 20:36 |
| TheJulia | cardoe: finally sat down and read through your neutron-spec change. I guess it makes sense if ranges can be allocated for reserved vlans which would facilitate it, and require an ml2 plugin to do the $rest. | 20:41 |
| cardoe | I need to probably expand on the wording and make it clearer | 20:43 |
| TheJulia | Upfront, I thoguht you were just trying to do spine/leaf mapping for single vlans | 20:43 |
| TheJulia | then when I got futher on, it sort of all clicked together for me | 20:43 |
| TheJulia | But, the base problem is also going to be to get others on to that same page | 20:44 |
| cardoe | Yep. That's where I'm stuck. | 20:46 |
| cid | TheJulia, right. | 20:49 |
| cid | hmmm, sake of a race condition(?) | 20:49 |
| * cid seems the shutdown method already takes care of new submissions after shutdown begins. | 20:49 | |
| * cid Likely a possibility | 20:50 | |
| TheJulia | Yeah, I'm not super sure it really is based upon my reading, but I'm going across 4 topics at the moment | 20:51 |
| cid | ++ | 20:51 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: Minor doc generation fixes https://review.opendev.org/c/openstack/networking-generic-switch/+/958498 | 21:18 |
| JayF | Monday is Labor Day in the US. Should we cancel the ironic meeting? | 21:48 |
| TheJulia | ... we're sort of on release autopilot at this point unless there is something really pressing I suspect it might be okay to do so?! | 21:49 |
| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Minor doc generation fixes https://review.opendev.org/c/openstack/networking-generic-switch/+/958498 | 23:44 |
| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Ignore newly enforced pep8 rules https://review.opendev.org/c/openstack/networking-generic-switch/+/958708 | 23:44 |
| stevebaker[m] | hey it looks like there are some newly enforced flake8 rules running in ngs. I have proposed ignoring them for now | 23:45 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!