| *** matt__ is now known as matfechner | 03:24 | |
| rpittau | good morning ironic! o/ | 07:45 |
|---|---|---|
| janders | hey rpittau o/ | 09:29 |
| rpittau | hey janders :) | 09:29 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Remove CentOS Stream 8 leftovers https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/848826 | 09:30 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: [WIP] Move bifrost integration tests to Ubuntu Jammy https://review.opendev.org/c/openstack/bifrost/+/848828 | 09:50 |
| dtantsur | TheJulia: I think we did nuke it in the end | 10:29 |
| janders | dtantsur iurygregory can I please ask you for a second review on https://review.opendev.org/c/openstack/ironic/+/846859 (verify steps fix on older firmwares/iDRACs)? Thank you! | 10:55 |
| dtantsur | putting on the queue | 11:03 |
| janders | TY! | 11:05 |
| iurygregory | good morning Ironic o/ | 11:40 |
| janders | hey iurygregory o/ | 11:41 |
| janders | thank you for the +2 | 11:41 |
| iurygregory | janders, yw =) | 11:42 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: [WIP] Move bifrost integration tests to Ubuntu Jammy https://review.opendev.org/c/openstack/bifrost/+/848828 | 12:28 |
| kubajj | Hi everyone, I have a question about IPA logs. Where is everything stored? I opened one of the ironic-bm-logs/deploy_logs and I can't figure out if the journal is everything. | 12:36 |
| dtantsur | kubajj: the journal is probably the main thing for DIB-built ramdisks | 12:40 |
| kubajj | dtantsur: thanks | 12:43 |
| TheJulia | good morning | 13:01 |
| TheJulia | dtantsur: hmm, based upon some unit test stuff yesterday I guess it was only for non-netboot scenarios, but I guess that is totally okay since netboot is going to go poof | 13:01 |
| dtantsur | TheJulia: for netboot you need kernel/ramdisk to do a normal reboot, hence I did not touch it | 13:04 |
| TheJulia | makes sense now that I'm beginning to sip coffee | 13:04 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Deprecate os_ironic_node_info https://review.opendev.org/c/openstack/bifrost/+/848765 | 13:04 |
| dtantsur | this work is by request of the collection team ^^^ | 13:05 |
| dtantsur | we need to clean up our usage of ironic_url (hence also the desire to nuke redeploy-dynamic) | 13:05 |
| TheJulia | Ahh, makes sense | 13:05 |
| * TheJulia needs https://i.pinimg.com/originals/e8/2f/be/e82fbe370ddc8bc63820f8abc569419a.jpg | 13:05 | |
| dtantsur | +++ | 13:06 |
| rpittau | lol | 13:10 |
| opendevreview | Merged openstack/ironic master: Prevent clear_job_queue and reset_idrac failures on older iDRACs https://review.opendev.org/c/openstack/ironic/+/846859 | 13:15 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Trivial: log which state the node is in https://review.opendev.org/c/openstack/ironic/+/848874 | 14:52 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Move logging out of skip_automated_cleaning https://review.opendev.org/c/openstack/ironic/+/848875 | 15:02 |
| dtantsur | JayF: ^^^ | 15:02 |
| TheJulia | I <3 that trivial change | 15:03 |
| dtantsur | It has a pretty interesting backstory. A person is hitting this error because inspection takes for them exactly 30 minutes, i.e. the inspect wait timeout. So inspector tries to update ports at the same moment as Ironic moves the node to "inspect failed". | 15:04 |
| TheJulia | ugh, all sorts of fun races around there | 15:05 |
| dtantsur | yup. I'm still puzzled why it takes so long. | 15:05 |
| dtantsur | see you tomorrow folks o/ | 15:12 |
| rpittau | I wonder if it's pbr not ready for Python 3.10 or viceversa | 15:17 |
| arne_wiebalck | janders: I *think* sending multiple etags is fine ... did I add this? If I did, I claim that I checked the standard to make sure this is ok :) | 15:23 |
| TheJulia | rpittau: your the second person I've seen remark as such in the last few weeks | 15:29 |
| rpittau | heh having issues with bifrost on jammy, I guess it's early | 15:30 |
| rpittau | oook my head hurts enough, see you tomorrow o/ | 15:45 |
| arne_wiebalck | bye everyone o/ | 16:15 |
| EmilienM | we have an interesting (and valid) request on Gophercloud, if anyone wants to have a look too: https://github.com/gophercloud/gophercloud/issues/2431 | 16:59 |
| JayF | That's not a library we support, is it? | 17:03 |
| TheJulia | but it is the support to talk to our api | 17:03 |
| JayF | Is the implication it's our API bug? Or that we should be invested just because we should be invested. | 17:04 |
| TheJulia | so it has always been "next". I wonder where "nodes_links" | 17:04 |
| TheJulia | came from | 17:04 |
| JayF | I'm onboard with the latter, just unclear | 17:04 |
| TheJulia | so, Stephen/Joe wrote that section of code | 17:05 |
| TheJulia | i suspect it was just not an inherent need to understand the pagination and most of their references were much smaller lists of nodes | 17:06 |
| JayF | We should, at a minimum, comment on the issue and endorse the alternate code linked if it's accurate | 17:07 |
| JayF | I suspect if we just say "this alternate implementation LGTM signed, Ironic dev" it might be sufficient :) | 17:08 |
| TheJulia | Yeah, I'm looking at it now | 17:08 |
| JayF | I am willing to make that comment, the alternate code looks sane to me but I have very little context in gophercloud | 17:08 |
| JayF | (but do write golang for my day job) | 17:08 |
| TheJulia | commented | 17:18 |
| TheJulia | Yeah, I went back and checked the apiwg stuffs as well | 17:18 |
| TheJulia | I'm just thinking it would have been prevented if there was more transparency from API docs to details like pagination | 17:18 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Do not require stage2 for anaconda with standalone https://review.opendev.org/c/openstack/ironic/+/835769 | 19:54 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Make anaconda non-image deploys sane https://review.opendev.org/c/openstack/ironic/+/834709 | 19:57 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Do not require stage2 for anaconda with standalone https://review.opendev.org/c/openstack/ironic/+/835769 | 19:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
