| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: tox: Enable codespell by default and limit it to relevant files https://review.opendev.org/c/openstack/networking-generic-switch/+/958154 | 01:41 |
|---|---|---|
| rpittau | good morning ironic! o/ | 07:47 |
| rpittau | interesting, there are some specific performance issues in noble jobs compared to cs10 jobs in bifrost, especially for vmedia | 08:53 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Remove tinyipa support and switch to debian IPA https://review.opendev.org/c/openstack/bifrost/+/964404 | 08:56 |
| iurygregory | good morning ironic | 11:24 |
| himani__ | Happy Monday ironic ^^ | 11:35 |
| opendevreview | Verification of a change to openstack/ironic master failed: ci: remove snmp ci job https://review.opendev.org/c/openstack/ironic/+/963215 | 12:46 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Remove tinyipa support and switch to debian IPA https://review.opendev.org/c/openstack/bifrost/+/964404 | 13:43 |
| TheJulia | good morning everyone | 13:57 |
| opendevreview | Riccardo Pittau proposed openstack/tenks master: [WIP] Modernize and fix CI https://review.opendev.org/c/openstack/tenks/+/968200 | 14:13 |
| cardoe | good morning everyone | 14:13 |
| opendevreview | Riccardo Pittau proposed openstack/tenks master: [WIP] Modernize and fix CI https://review.opendev.org/c/openstack/tenks/+/968200 | 14:18 |
| opendevreview | Riccardo Pittau proposed openstack/tenks master: [WIP] Modernize and fix CI https://review.opendev.org/c/openstack/tenks/+/968200 | 14:19 |
| opendevreview | Riccardo Pittau proposed openstack/tenks master: [WIP] Modernize and fix CI https://review.opendev.org/c/openstack/tenks/+/968200 | 14:21 |
| opendevreview | Riccardo Pittau proposed openstack/tenks master: [WIP] Modernize and fix CI https://review.opendev.org/c/openstack/tenks/+/968200 | 14:23 |
| opendevreview | Riccardo Pittau proposed openstack/tenks master: [WIP] Modernize and fix CI https://review.opendev.org/c/openstack/tenks/+/968200 | 14:47 |
| TheJulia | good morning Ironic! | 15:00 |
| TheJulia | Who is driving our Ironic meeting this week? | 15:00 |
| TheJulia | I guess I'm going to be running it | 15:01 |
| TheJulia | Does everyone have a large cup of coffee? | 15:02 |
| womax | good morning/afternoon | 15:02 |
| TheJulia | #startmeeting ironic | 15:02 |
| opendevmeet | Meeting started Mon Nov 24 15:02:33 2025 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:02 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:02 |
| alegacy | o/ | 15:02 |
| TheJulia | Welcome to the Ironic meeting for November 24th, 2025. Our agenda can be found on the wiki. | 15:02 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_November_24.2C_2025 | 15:03 |
| TheJulia | Big question for the moment: do we have a quorum ? | 15:03 |
| kubajj | o/ | 15:03 |
| TheJulia | Or is everyone going and getting coffee like the agenda says? :) | 15:03 |
| cid | I think so :D | 15:04 |
| cid | o/ | 15:04 |
| TheJulia | I know Jay said he was semi-taking today off, although I entirely forgot something I discussed with him last week. | 15:05 |
| TheJulia | I believe Dmitry is out as well this week. | 15:05 |
| TheJulia | Since it looks like we're basically right below where I'd feel comfortable going through the full agenda with what feels like quorum, do we have anything which requires specific inspection or discussion by attendees at this time? | 15:06 |
| cid | True. Jay proposed a review jam session after the weekly meeting. | 15:07 |
| cid | None from my end. | 15:07 |
| alegacy | I've got some new review comments to address on my end so i'll be busy with that for the next day or so. | 15:08 |
| TheJulia | Ahh, yes. In part I know alegacy pushed a bunch of revisions as well on ?Friday? | 15:08 |
| TheJulia | alegacy: okay, do you need review focus today? | 15:08 |
| opendevreview | Riccardo Pittau proposed openstack/tenks master: [WIP] Modernize and fix CI https://review.opendev.org/c/openstack/tenks/+/968200 | 15:08 |
| dtantsur | I'm not out, only slow :) | 15:08 |
| alegacy | if there are more review comments i'll take 'em. :) | 15:08 |
| rpittau | o/ | 15:09 |
| TheJulia | Huh, I thought you were out... | 15:09 |
| TheJulia | alegacy: fair enough | 15:09 |
| dtantsur | I'm here until the very holidays | 15:09 |
| TheJulia | do we want to roll through the agenda, or just continue on our laid back path we're on today? | 15:09 |
| dtantsur | I'm fine being laid back | 15:09 |
| TheJulia | Speaking of holidays, I raised a question, End of year meeting skips | 15:10 |
| TheJulia | Any thoughts on skipping 12/22 and 12/29 ? | 15:10 |
| kubajj | I think it is reasonable | 15:11 |
| cid | ++ | 15:11 |
| rpittau | sounds good | 15:11 |
| dtantsur | yep | 15:12 |
| TheJulia | Sounds like we have consensus on that then | 15:12 |
| TheJulia | I'll take the action item to send an email out | 15:12 |
| TheJulia | Do we have any bugs to discuss or other items to discuss? | 15:12 |
| * TheJulia takes silence as no | 15:14 | |
| cid | Mostly no! I triaged this bug with a high priority. | 15:15 |
| cid | https://bugs.launchpad.net/ironic/+bug/2132246 | 15:15 |
| * TheJulia looks | 15:15 | |
| cid | I bet it's valid, but not confirmed : https://bugs.launchpad.net/ironic/+bug/2132246 | 15:15 |
| TheJulia | If what someone else mentioned to me last week is true, it is totally valid | 15:15 |
| TheJulia | we swapped the clients out and didn't check shared image behavior explicitly when we did | 15:16 |
| cid | Makes sense. | 15:16 |
| TheJulia | Anything else? | 15:17 |
| cid | Not really | 15:17 |
| TheJulia | Okay, thanks! | 15:17 |
| TheJulia | Anything else for us to [plot|discuss|design|confer] ? | 15:18 |
| womax | Hi totally new here, dunno if it is the good times to talk about it, but i was recently working on adding support to generate temp url for image stored on s3 backend and wanted to try to contribute it upstream. And i was asking myself a few questions | 15:19 |
| womax | Mainly if temp url generation should stay only in ironic or should I suggest change to move this part to glance maybe, to be able to use it in nova for example | 15:20 |
| TheJulia | Welcome womax! I think I'll end the meeting if your alright with it, and your free to ask questions anytime | 15:20 |
| womax | ok I'm good with that | 15:20 |
| TheJulia | Thanks everyone! Have a wondeful week! | 15:20 |
| TheJulia | #endmeeting | 15:20 |
| opendevmeet | Meeting ended Mon Nov 24 15:20:49 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:20 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-11-24-15.02.html | 15:20 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-11-24-15.02.txt | 15:20 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-11-24-15.02.log.html | 15:20 |
| cardoe | blah I missed the meeting sorry | 15:21 |
| TheJulia | womax: So S3 backend with Glance? Or directly use S3 ? | 15:21 |
| cardoe | cid, TheJulia: https://bugs.launchpad.net/ironic/+bug/2132246 is fixed by https://review.opendev.org/c/openstack/ironic/+/968087 | 15:21 |
| TheJulia | cardoe: moar coffee | 15:21 |
| cardoe | If someone +2/+W that, I'll backport it. | 15:21 |
| womax | so currently glance already support s3 but ironic is unable to boot image that are stored in a s3 store because temp url generation in swift specific | 15:22 |
| womax | I made a few patches on my end to change that but doing that I took quite a bit of code from glance_store mainly to handle config file | 15:22 |
| TheJulia | cardoe: just to confirm, member.id value is the project UUID? | 15:23 |
| TheJulia | womax: so, what is actually required to generate the S3 tempurl? | 15:23 |
| TheJulia | With swift, its a shared secret in advnace | 15:23 |
| womax | some credentials, endpoints and bucket name | 15:23 |
| womax | it close to what swift required but the main problem from my understanding of the code is that ironic doesn't really care about which store the image is in in case glance is confiured with multistore | 15:24 |
| TheJulia | Following the swift pattern, it sounds like you'd need to add image service client code to sort of directly understand and handle it to generate the tempurl if it is S3. Maybe that is the right path? | 15:25 |
| TheJulia | Is a glance client able to determine if it is stored in S3? | 15:25 |
| cardoe | TheJulia: per the docs yes | 15:25 |
| opendevreview | Merged openstack/ironic master: fix: sphinx-lint unbalanced backticks https://review.opendev.org/c/openstack/ironic/+/968148 | 15:25 |
| womax | from what i found it depends what glance exposing in image info | 15:25 |
| womax | in my case I only found glance only sending me the store name but maybe i did not found the right way to do it | 15:26 |
| TheJulia | Hmm, that does make it a bit difficult | 15:27 |
| womax | and yeah i did make my change in image service | 15:27 |
| cardoe | TheJulia: "the appropriate identifier to use for the member_id is the consumer’s project ID" | 15:28 |
| womax | basically what i did is add configuration options like glance to define store and update image service to try to load the store glance give based on ironic config and then generate it | 15:28 |
| womax | that why i was wondering wether such change should stay in ironic or maybe i should also go see the glance team to move the generation in glance (where configuration is already present) and just retrieve in ironic | 15:29 |
| TheJulia | that doesn't seem like an awful approach, really. | 15:29 |
| TheJulia | One service can't retrieve configuration details of another service | 15:29 |
| TheJulia | so configuration duplication is really the only way | 15:30 |
| womax | I understand that what I meant is maybe just generate a temp url in glance and then send it to ironic as glance has already everything it need to do the generation | 15:30 |
| womax | But I agree a simpler approach would be configuration duplication | 15:31 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: ci: disable vnc on IPA jobs https://review.opendev.org/c/openstack/ironic-python-agent/+/968078 | 15:31 |
| TheJulia | That could be viable, I mean, it would be nice to be able to ask glance to "give us tempurls" in general | 15:32 |
| womax | Ok so I guess I'll go discuss with the glance to get their opinions on this, and see where it goes | 15:34 |
| JayF | TheJulia: cid: I'm here if we wanted to go through reviews. Maybe +5M to give me time to make coffee :) | 15:34 |
| womax | thank you for your time | 15:35 |
| TheJulia | womax: thank you for your efforts! Keep us apprised! Thanks! | 15:35 |
| opendevreview | Verification of a change to openstack/ironic master failed: ci: remove snmp ci job https://review.opendev.org/c/openstack/ironic/+/963215 | 15:35 |
| TheJulia | JayF: Sure, I will warn you, my coffee is being retireved and has yet to be sipped | 15:35 |
| cardoe | TheJulia: even though https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/951681 failed it seemed like a good thing we wanted? So I dragged it out of the wood work. | 15:36 |
| TheJulia | cardoe: hmm.. I got soemthing wrong there | 15:36 |
| cardoe | sure but the concept seemed good | 15:37 |
| JayF | https://us06web.zoom.us/j/89014244066?pwd=5PiI85z5DpGSEC0L1CqkqeFicw8fao.1 no rush just hanging out until we get folks around | 15:37 |
| TheJulia | looking for headphones | 15:37 |
| opendevreview | Verification of a change to openstack/ironic master failed: Remove inspector inspect interface https://review.opendev.org/c/openstack/ironic/+/967476 | 15:40 |
| cid | omw | 15:41 |
| JayF | cardoe: cid: review jam? We need 3 for edit-and-land-quorum | 15:41 |
| dtantsur | oh, review jam? I can join if you give me a few minutes | 15:42 |
| cardoe | I'd be down. Trying to troubleshoot this 2025.2 env | 15:42 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Switch graphical console image to ubuntu by default https://review.opendev.org/c/openstack/ironic/+/964201 | 15:45 |
| opendevreview | Marcus Furlong proposed openstack/sushy master: Don't require Boot and Actions for Systems https://review.opendev.org/c/openstack/sushy/+/967715 | 15:45 |
| opendevreview | Riccardo Pittau proposed openstack/tenks master: [WIP] Modernize and fix CI https://review.opendev.org/c/openstack/tenks/+/968200 | 15:46 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Remove tinyipa support and switch to debian IPA https://review.opendev.org/c/openstack/bifrost/+/964404 | 15:51 |
| JayF | iurygregory: https://review.opendev.org/c/openstack/ironic/+/955484/12/ironic/drivers/modules/redfish/management.py please make sure to review this | 15:52 |
| cardoe | JayF: so that image issue we're seeing... it appears that our instance_master_path and our tftp_master_path are different. It's downloading the image to the tftp_master_path and then throwing the exception because it expects the file to be in the instance_master_path | 15:57 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Minimize json-rpc client logging https://review.opendev.org/c/openstack/ironic/+/958119 | 15:59 |
| JayF | cardoe: is https://review.opendev.org/c/openstack/ironic/+/967965/2#message-185c8672f7b643acca5e874865e598e1faf339c3 a -1, essentially? | 16:04 |
| cardoe | yeah sorry. I should have done that. | 16:05 |
| JayF | clif: please be sure to apply this followup: https://review.opendev.org/c/openstack/ironic/+/962598/11#message-dcb4932cf93b6b6f045596e6a7054598b7fdef5d | 16:07 |
| opendevreview | cid proposed openstack/python-ironicclient master: A new `instance_name` field to the node object https://review.opendev.org/c/openstack/python-ironicclient/+/955102 | 16:10 |
| opendevreview | Verification of a change to openstack/ironic master failed: Remove inspector inspect interface https://review.opendev.org/c/openstack/ironic/+/967476 | 16:17 |
| opendevreview | Merged openstack/ironic master: fix: grammar Ideall -> Ideally https://review.opendev.org/c/openstack/ironic/+/968149 | 16:23 |
| opendevreview | Verification of a change to openstack/ironic master failed: fix: remove unnecessary trailing slash https://review.opendev.org/c/openstack/ironic/+/968150 | 16:23 |
| opendevreview | Verification of a change to openstack/ironic master failed: chore: disable pycodestyle and pyflakes double runs https://review.opendev.org/c/openstack/ironic/+/968151 | 16:23 |
| opendevreview | Verification of a change to openstack/ironic master failed: chore: fix spelling mistakes https://review.opendev.org/c/openstack/ironic/+/968152 | 16:23 |
| opendevreview | Verification of a change to openstack/ironic master failed: chore: bump to latest pre-commit tools https://review.opendev.org/c/openstack/ironic/+/968153 | 16:23 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: ci: remove metalsmith ci job invocation https://review.opendev.org/c/openstack/ironic/+/963216 | 16:26 |
| JayF | stevebaker[m]: https://review.opendev.org/c/openstack/ironic/+/962242/11 needs a rebase then can land | 16:29 |
| dtantsur | TheJulia: https://review.opendev.org/c/openstack/bifrost/+/961388 (the last failure is because of the oslo.service issue, rechecking now) | 16:36 |
| TheJulia | yeouch, github error took out that chain of patches | 16:36 |
| opendevreview | cid proposed openstack/ironic master: Move configdrive code to configdrive_utils https://review.opendev.org/c/openstack/ironic/+/965880 | 16:36 |
| * dtantsur shakes fist at github | 16:36 | |
| dtantsur | yeah, we had a row of failures in the metal3 CI too | 16:37 |
| TheJulia | yeah | 16:40 |
| opendevreview | Merged openstack/ironic master: fix: glance image member lookup resulted in an empty list always https://review.opendev.org/c/openstack/ironic/+/968087 | 16:40 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Revert "ci: stabilize ironic-standalone-redfish" https://review.opendev.org/c/openstack/ironic/+/956515 | 16:50 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2025.2: fix: glance image member lookup resulted in an empty list always https://review.opendev.org/c/openstack/ironic/+/968226 | 16:50 |
| opendevreview | Merged openstack/ironic master: ci: disable multinode subnode running n-cpu https://review.opendev.org/c/openstack/ironic/+/956117 | 17:03 |
| opendevreview | Riccardo Pittau proposed openstack/tenks master: [WIP] Modernize and fix CI https://review.opendev.org/c/openstack/tenks/+/968200 | 17:19 |
| opendevreview | Verification of a change to openstack/ironic master failed: Trait Based Networking Filter Expression Parsing and Base Models https://review.opendev.org/c/openstack/ironic/+/961498 | 17:21 |
| opendevreview | Verification of a change to openstack/ironic master failed: Configuration file for Trait Based Networking https://review.opendev.org/c/openstack/ironic/+/962598 | 17:21 |
| opendevreview | Riccardo Pittau proposed openstack/tenks master: [WIP] Modernize and fix CI https://review.opendev.org/c/openstack/tenks/+/968200 | 17:31 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Add simple-init by default https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/966192 | 17:33 |
| clif | JayF: aren't we also changing the format of the config file to be JSON instead of YAML? | 17:43 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: Fix incorrect command when unplugging bond subports https://review.opendev.org/c/openstack/networking-generic-switch/+/959423 | 17:45 |
| cardoe | My eyes are floating looking at this image cache / download code. | 17:57 |
| TheJulia | cardoe: why are you causing yourself pain doing that? | 17:57 |
| cardoe | We cannot get a box to kick on 2025.2 cause of this error | 17:58 |
| cardoe | skrobul and I are trying to figure out why | 17:58 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Downgrade the lock on vif detach https://review.opendev.org/c/openstack/ironic/+/968061 | 18:01 |
| TheJulia | cardoe: what error? and btw ^^^ is for y'all | 18:02 |
| opendevreview | Merged openstack/ironic-python-agent master: ci: disable vnc on IPA jobs https://review.opendev.org/c/openstack/ironic-python-agent/+/968078 | 18:02 |
| cardoe | https://gist.github.com/skrobul/64798d4ff8987ee3fcce93f8f1b991e5 | 18:03 |
| cardoe | Line 54 of the conductor logs | 18:03 |
| cardoe | https://github.com/rackerlabs/understack/releases/download/understack-images-20251016184704/ubuntu-noble.qcow2 is image 5048de36-4c01-4047-b638-195e639cb1e3 | 18:05 |
| cardoe | Which is just disk image builder of the Ubuntu Noble with a broadcom NIC driver being added by a DIB | 18:05 |
| cardoe | I'm wondering if John Garbutt didn't see the periodic timer coming along and cleaning up the image cache directory cause it doesn't take into account files that are currently locked and will just delete anything. | 18:07 |
| TheJulia | all sorts of fun errors, looks like you ahve a machine which doesn't have version attributes in the bios settings | 18:08 |
| TheJulia | so, hmmm | 18:08 |
| TheJulia | converted should only be a thing when files are converted | 18:08 |
| cardoe | yeah cid made a patch and I +W'd it last week | 18:08 |
| TheJulia | but, I don't remember exactly | 18:09 |
| cardoe | So the Ubuntu image is qcow2... I should probably just upload it raw | 18:09 |
| TheJulia | nothing there, at a glance makes me think image_to_raw is getting invoked | 18:10 |
| TheJulia | but that is expected | 18:10 |
| TheJulia | oh, more files | 18:11 |
| cardoe | https://www.irccloud.com/pastebin/LCNKchR2/ | 18:11 |
| cardoe | That's what the image checker thing gives from oslo.utils | 18:11 |
| TheJulia | taht sure does sort of look like the cache got waxed mid-stream | 18:12 |
| cardoe | We slapped a root logger on all of this and qemu-img doesn't get called. | 18:14 |
| cardoe | or at least utils.execute() didn't throw its logging messages | 18:14 |
| TheJulia | so, it gets checked | 18:15 |
| TheJulia | ... weird | 18:16 |
| TheJulia | That explains the error, but... how | 18:16 |
| TheJulia | because that would explain the unlink/removal error as well | 18:16 |
| TheJulia | well, sort of if it was going down that path | 18:16 |
| TheJulia | weird, I'd add more logging just to be on the safe side that we're actually getting to qemu_img.convert_image | 18:17 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: list detailed ports by node https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/712831 | 18:31 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: split error messages in image cache for more clarity https://review.opendev.org/c/openstack/ironic/+/968246 | 18:40 |
| opendevreview | Clif Houck proposed openstack/ironic master: Generate network plan based on trait based networking config https://review.opendev.org/c/openstack/ironic/+/964895 | 18:41 |
| opendevreview | Clif Houck proposed openstack/ironic master: Trait Based Networking Simulator https://review.opendev.org/c/openstack/ironic/+/966202 | 18:41 |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Add configuration options for trait based networking https://review.opendev.org/c/openstack/ironic/+/968054 | 18:41 |
| cardoe | TheJulia: so getting the exception from the _fetch() and not the os.link() | 19:00 |
| TheJulia | so the actual write operation? | 19:01 |
| JayF | clif: that wasn't the original intent, but if you switch to Jason, it completely removes the concern from the standalone networking patches about it introducing a json-based config file. Just please make sure that the spec gets updated as well. As long as TheJulia has no objection | 19:05 |
| clif | I thought we had a conversation at some point last week about changing the format to JSON which I'm fine with | 19:06 |
| TheJulia | I don't remember any discussion about using json, but we've tried to weed json out in favor of yaml over thea ges | 19:07 |
| TheJulia | the ages | 19:07 |
| clif | aha | 19:07 |
| JayF | I want it to be consistent for networking features across the project. I don't care what direction it goes. | 19:07 |
| clif | erg | 19:07 |
| clif | Then I'm going to keep YAML for now and will change the format if there's consensus it should change | 19:08 |
| clif | Since I've got +2s on the config file change | 19:08 |
| clif | also since the spec calls for YAML | 19:08 |
| opendevreview | Julia Kreger proposed openstack/bifrost master: DNM: Test chagne to evaluate CI's state https://review.opendev.org/c/openstack/bifrost/+/968248 | 19:09 |
| JayF | Yeah. We'll use code review on the other standalone changes to make sure that they are revised to use yaml or oslo.config. | 19:11 |
| cardoe | Are you guys still reviewing? | 19:19 |
| cardoe | clif: so jsonschema maybe added to the confusion but the point was jsonschema can validate yaml as well | 19:29 |
| opendevreview | Merged openstack/ironic master: ci: remove metalsmith ci job invocation https://review.opendev.org/c/openstack/ironic/+/963216 | 19:33 |
| clif | cardoe: you also metioned pydantic, do you have a preference? | 19:45 |
| cardoe | I'm just suggesting we don't open code parsing of data. Whatever we use is fine | 19:46 |
| opendevreview | Verification of a change to openstack/ironic master failed: Trait Based Networking Filter Expression Parsing and Base Models https://review.opendev.org/c/openstack/ironic/+/961498 | 19:50 |
| cardoe | YAY | 20:28 |
| cardoe | Found it | 20:28 |
| cardoe | Holy exception swallowing batmans | 20:28 |
| TheJulia | Which batman?! | 20:34 |
| TheJulia | These are important questions for an important moment | 20:34 |
| stevebaker[m] | TheJulia: boop | 20:34 |
| TheJulia | stevebaker[m]: https://review.opendev.org/c/openstack/bifrost/+/961388 | 20:34 |
| opendevreview | Merged openstack/ironic master: Minimize json-rpc client logging https://review.opendev.org/c/openstack/ironic/+/958119 | 20:42 |
| TheJulia | dtantsur: oh, I get it, your using podman in rootless mode but I bet its not uniformly available. :\ | 20:43 |
| TheJulia | Could we instead make it just opt-in, or maybe start out for just centos10? | 20:45 |
| cardoe | or not... | 20:46 |
| TheJulia | cardoe: not... batman? | 20:48 |
| cardoe | well it switched to wait call-back but then never had iPXE configs setup for the box to boot from | 20:49 |
| cardoe | But got past the image issue | 20:49 |
| cardoe | its related to turning off deep image inspection which was problematic recently cause it was identifying an image differently than ironic did previously | 20:50 |
| TheJulia | did it recognize the qcow2 as raw or something? | 20:53 |
| cardoe | I believe it's the VMware images that foul it up. | 20:53 |
| TheJulia | single file vmdks or other? | 20:54 |
| cardoe | No it's the ESXi installer raw image | 20:55 |
| TheJulia | oh, hmm | 20:55 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix: use the correct path to the image when deep image inspection is off https://review.opendev.org/c/openstack/ironic/+/968258 | 21:06 |
| cardoe | That's at least the image issue | 21:09 |
| cardoe | yay successful kick with my pile o' backports. | 21:56 |
| opendevreview | Verification of a change to openstack/ironic master failed: fix: remove unnecessary trailing slash https://review.opendev.org/c/openstack/ironic/+/968150 | 21:56 |
| cardoe | So the image issue is actually when using macOS tools to convert the image to raw | 21:59 |
| TheJulia | Interesting | 22:00 |
| TheJulia | what is getting returned on the conversion? | 22:00 |
| cardoe | I'm gonna have to run the oslo.utils helper myself on an image. I'm just relaying what I'm getting told. | 22:01 |
| TheJulia | ahh, okay | 22:01 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: WIP VXLAN networking https://review.opendev.org/c/openstack/ironic-specs/+/959401 | 22:02 |
| TheJulia | hmm... still doesnt' do the linked file, wut | 22:10 |
| opendevreview | Merged openstack/ironic master: Trait Based Networking Filter Expression Parsing and Base Models https://review.opendev.org/c/openstack/ironic/+/961498 | 22:15 |
| opendevreview | Verification of a change to openstack/ironic master failed: Configuration file for Trait Based Networking https://review.opendev.org/c/openstack/ironic/+/962598 | 23:10 |
| opendevreview | Steve Baker proposed openstack/ironic master: Add a kubernetes provider for console container https://review.opendev.org/c/openstack/ironic/+/962242 | 23:10 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!