| *** dtantsur_ is now known as dtantsur | 01:32 | |
| opendevreview | Kamil Gustab proposed openstack/sushy master: Adds attributes for IPs for Ethernet Interfaces. https://review.opendev.org/c/openstack/sushy/+/930814 | 06:15 |
|---|---|---|
| opendevreview | Kamil Gustab proposed openstack/sushy master: Adds attributes for IPs for Ethernet Interfaces. https://review.opendev.org/c/openstack/sushy/+/930814 | 06:23 |
| rpittau | good morning ironic! o/ | 07:24 |
| opendevreview | Kamil Gustab proposed openstack/sushy master: Adds attributes for IPs for Ethernet Interfaces. https://review.opendev.org/c/openstack/sushy/+/930814 | 08:19 |
| grami[m] | Hi All hopfully I got things right this time could I ask again for this to be merged https://review.opendev.org/c/openstack/networking-generic-switch/+/926886 | 08:27 |
| opendevreview | Merged openstack/tenks master: Change default boot mode to UEFI https://review.opendev.org/c/openstack/tenks/+/827553 | 09:36 |
| opendevreview | Graeme Moss proposed openstack/networking-generic-switch master: Add Supermicro switches to allow for supported write config https://review.opendev.org/c/openstack/networking-generic-switch/+/926886 | 10:05 |
| grami[m] | As this has been my second time trying to add something to openstack I will say this the only people that has shown a inch of respect is @julia and @Riccardo. I'm not a coder and never will be I don't follow or understand a bunch of stuff but I can now see why people just don't want to even try post a patch or fix to openstack anymore as the two word demands that get put into new comers is trouble. I now see why some many people just | 10:13 |
| grami[m] | fork and handle the changes them selves which is really not good at all. So to the people that won't take the time to try and teach people how to do things right please just don't type it's really just pushes people away from contributing and that's really not what you should be doing. | 10:13 |
| dtantsur | grami[m]: I understand your frustration, but you'll be the first to complain if we drop the quality and just take everything in without much consideration. | 10:13 |
| dtantsur | Always two sides of coins. | 10:14 |
| grami[m] | it's not about quality I get that it's about how you respond not explaining what needs to be done to get the quality up to standard | 10:14 |
| grami[m] | Riccardo message is a perfect example of how to handle it. | 10:15 |
| dtantsur | grami[m]: you can always ask follow-up questions. But nobody knows your background, and we cannot make assumptions. | 10:15 |
| dtantsur | grami[m]: I will also be happy to point you to hundreds of contributors who take the first critical review much easier than you. | 10:16 |
| iurygregory | good morning ironic | 11:00 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Replace image_format_inspector with its oslo.utils version https://review.opendev.org/c/openstack/ironic/+/929904 | 14:18 |
| dtantsur | JayF: this ^^ is without an allowlist of inspectors (I'd avoid increasing the scope since I'll have a week of PTO from Wed) | 14:25 |
| JayF | I'll have a look | 14:27 |
| JayF | I'm also PTO from like Thursday to Tuesday. My little brother is getting married | 14:27 |
| dtantsur | Nice! | 14:27 |
| * dtantsur will just spend some time in the forest | 14:27 | |
| JayF | Well, assuming the roads to his wedding venue are passable. They picked some nice place up in the North Carolina mountains, luckily not one of the places that has been majorly impacted by the hurricane but the whole west side of the state is screwed up right now | 14:27 |
| dtantsur | NC, ouch | 14:28 |
| dtantsur | good luck! | 14:28 |
| opendevreview | Kamil Gustab proposed openstack/sushy master: Adds attributes for IPs for Ethernet Interfaces. https://review.opendev.org/c/openstack/sushy/+/930814 | 14:43 |
| JayF | Yeah I'm not going to lie if it was for anything less than a wedding I would already have canceled the trip because I'm not going to feel that good traveling in that area next week | 14:52 |
| TheJulia | good morning | 14:52 |
| cardoe | grami[m]: let me know how I can help you get it over the finish line. | 14:54 |
| TheJulia | ++ | 14:54 |
| TheJulia | I haven't been abel to look at the log much today or any patches, I'm dealing with a customer who wants their 4k disks to behave perfecty as if support had been developed years ago | 14:55 |
| iurygregory | good luck TheJulia | 14:55 |
| cardoe | ugh. Love that stuff. Reminds me the docs on that needed a rebase. But Jay is gonna yell at me. I’m on PTO and my wife is driving. | 14:58 |
| rpittau | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Sep 30 15:00:10 2024 UTC and is due to finish in 60 minutes. The chair is rpittau. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:00 |
| cardoe | https://review.opendev.org/c/openstack/ironic/+/927518 this one. | 15:00 |
| rpittau | Hello everyone! | 15:00 |
| rpittau | Welcome to our weekly meeting! | 15:00 |
| rpittau | The meeting agenda can be found here: | 15:00 |
| rpittau | https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_September_30.2C_2024 | 15:00 |
| dtantsur | o/ | 15:00 |
| cardoe | \o | 15:00 |
| TheJulia | o/ | 15:00 |
| JayF | o/ | 15:00 |
| rpittau | let's give a minute or two for other people to join | 15:01 |
| masghar | o/ | 15:01 |
| rpittau | #topic Announcements/Reminders | 15:02 |
| rpittau | #info Standing reminder to review patches tagged ironic-week-prio and to hashtag any patches ready for review with ironic-week-prio: | 15:02 |
| rpittau | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:02 |
| iurygregory | o/ | 15:02 |
| rpittau | we have some failry new patches | 15:02 |
| rpittau | also some months old ones | 15:02 |
| cardoe | Can I get access to toss the weekly prio tag on stuff? | 15:02 |
| rpittau | from June | 15:02 |
| rpittau | cardoe: that's for cores only | 15:03 |
| iurygregory | rpittau, I think everyone can add the tag to patches | 15:03 |
| TheJulia | only if you create the patch | 15:03 |
| TheJulia | we can likely talk with opendev folks about the acl | 15:03 |
| * dtantsur has sent an email with a proposal that might change the tagging issue - cores please read | 15:03 | |
| rpittau | well, interesting :) | 15:03 |
| TheJulia | but we need to remember to do talk to them as well | 15:03 |
| TheJulia | also dmitry++ | 15:03 |
| opendevreview | Merged openstack/ironic-inspector master: Remove workaround for eventlet < 0.27.0 https://review.opendev.org/c/openstack/ironic-inspector/+/930791 | 15:04 |
| cardoe | Yeah I can add it to my own patches but not to others. You guys talked about it last week. | 15:04 |
| iurygregory | I added the ACL, I will double check | 15:04 |
| rpittau | dtantsur: yep, saw that, going to reply after the meeting | 15:04 |
| TheJulia | cardoe: life is a firehose some weeks :) | 15:04 |
| cardoe | I’m willing to help try and tag so | 15:04 |
| rpittau | anyway, we probably need to decide what to do with the monnths old prio patches | 15:04 |
| rpittau | there's at least 3 in the list | 15:05 |
| rpittau | 2 of them are in merge conflict | 15:05 |
| dtantsur | In the past, we would drop patches and let the author or someone else to re-add them once ready | 15:06 |
| rpittau | let's maybe have a look at them this week | 15:06 |
| JayF | rpittau: when I was core, I'd often just drop the HT and comment to re-add it once they have it fixed up | 15:06 |
| JayF | s/core/ptl/ | 15:06 |
| rpittau | yeah, I'm just saying that we need to check them to leave a comment :) | 15:07 |
| rpittau | anyway! | 15:07 |
| rpittau | #info let's look at the epoxy schedule! | 15:07 |
| rpittau | #link https://releases.openstack.org/epoxy/schedule.html | 15:07 |
| rpittau | nothing really new there :) | 15:08 |
| rpittau | dalmatian is officially out this week | 15:08 |
| iurygregory | \o/ | 15:08 |
| rpittau | and we'll have the openinfra live this thursday | 15:08 |
| rpittau | I will present the dalmatian highlights it for the ironic project | 15:09 |
| TheJulia | cool, thanks rpittau! | 15:09 |
| cardoe | I’m willing to help with dropping and adding the hashtag on patches if they aren’t ready. | 15:09 |
| rpittau | my pleasure :) | 15:09 |
| rpittau | #info the next OpenInfra PTG which will take place October 21-25, 2024 virtually! | 15:10 |
| rpittau | Ironic PTG has been scheduled in mitaka virtual room, more details in the PTG etherpad | 15:10 |
| rpittau | #link https://etherpad.opendev.org/p/ironic-ptg-october-2024 | 15:10 |
| rpittau | I still have to send the mail for the APAC-friendly slot, but we have some time | 15:11 |
| rpittau | same for adjusting the topics based on the time | 15:11 |
| TheJulia | cardoe: your awesome btw | 15:11 |
| rpittau | I'll ask again if anyone is willing to talk to neutron people for the joint session, we have at least one topic that requires that | 15:12 |
| cardoe | Right back at ya TheJulia | 15:13 |
| rpittau | alright I'll see if I have time this week to reach out to them | 15:13 |
| rpittau | anything else to announce/remind us of ? | 15:14 |
| cardoe | rpittau: happy to help with neutron | 15:14 |
| TheJulia | I'll be out next week and likely moderately part of the time the week following | 15:14 |
| TheJulia | well "out" as involving metal tubes and trains | 15:15 |
| dtantsur | I'll be out Wed-Mon | 15:15 |
| rpittau | cardoe: sure thing, thanks | 15:16 |
| rpittau | ok moving on | 15:16 |
| rpittau | #topic Review Ironic CI status | 15:16 |
| rpittau | didn't see anbything particularly broken last week, except some time outs | 15:17 |
| rpittau | anything to report? | 15:17 |
| cardoe | It seemed pretty solid this past week. | 15:17 |
| rpittau | great! | 15:17 |
| JayF | I am out some of next week, but will be available for lamost all PTG times | 15:18 |
| rpittau | perfect | 15:18 |
| rpittau | #topic Discussion topics | 15:18 |
| rpittau | #topic PTG, other joint sessions besides neutron? | 15:18 |
| JayF | cardoe: when you schedule neutron session ^ don't worry too much about me except closer to PDT-normal-hours the more awake I'll be :D | 15:19 |
| TheJulia | I'm curious if we'll have quorum next week | 15:19 |
| rpittau | TheJulia: probably not, I may also be out | 15:19 |
| JayF | let me look I think there was one more | 15:19 |
| TheJulia | Neutron definitely makes sense to try and chat with them. | 15:19 |
| JayF | oh, images | 15:19 |
| TheJulia | howso? | 15:19 |
| JayF | we might want to see if there's a nova/glance images session and have someone in there | 15:19 |
| rpittau | cardoe: btw the topic for the joint session is https://etherpad.opendev.org/p/ironic-ptg-october-2024#L86 | 15:19 |
| JayF | because there are specs in glance right now to define new image types and language around iamges | 15:20 |
| JayF | and frankly I think Ironic may have special needs there | 15:20 |
| TheJulia | I think participation makes a lot of sense there | 15:20 |
| JayF | if they are having a PTG session we need a person in that session, whether it's "joint" or not | 15:20 |
| rpittau | so both nova and glance ? | 15:20 |
| JayF | rpittau: like I said, dunno if it needs to be joint so much as we need a core to commit to rep us if there's a PTG session | 15:20 |
| JayF | and probably someone other than me should review that spec | 15:21 |
| rpittau | I see | 15:21 |
| rpittau | I haven't see glance/nova schedule yet | 15:21 |
| TheJulia | Well, if they are having a joint session, it would be best if we were also "in the room" or had "someone in the room to represent the nova.virt.ironic driver and ironic | 15:21 |
| JayF | I begrudgingly put a +1 on it -- the only reason it's begrudging is because reality is sad, and the content reflects that :) | 15:21 |
| rpittau | glance has no schedule yet | 15:21 |
| rpittau | nova has similar schedule to us | 15:21 |
| rpittau | except they have one more hour each day | 15:22 |
| rpittau | JayF: probably need to see what exactly the topics we have interest in are and when they're schduled for | 15:23 |
| JayF | yeah, just as long as it doesn't overlap one of our hot topics someone should be able to slip away | 15:23 |
| rpittau | maybe try to steer them in the hour that does not overlap | 15:23 |
| JayF | I am not the person to do that coordination though as I'm gonna be fully offline more or less late this week/early next | 15:23 |
| rpittau | JayF: no worries for that | 15:24 |
| rpittau | I will talk to nova at least | 15:24 |
| rpittau | I think we're good for the joint sessions | 15:25 |
| rpittau | anything else to discuss ? | 15:26 |
| TheJulia | Plan 04951 to take over the world? | 15:27 |
| rpittau | heh :) | 15:27 |
| TheJulia | </silly> | 15:27 |
| * dtantsur is scared to ask what happened with the previous 4950 plans | 15:27 | |
| rpittau | ok, moving on! :D | 15:27 |
| masghar | Go for it :D | 15:27 |
| rpittau | #topic Bug Deputy Updates | 15:27 |
| rpittau | #info rpittau was bug deputy | 15:27 |
| TheJulia | dtantsur: I didn't cover what base unit that was in ;) | 15:27 |
| dtantsur | TheJulia: it's not going to make the number smaller though since 9 is present | 15:28 |
| cardoe | I just read Andy Weir’s latest book. So I’m gonna guess base 6. | 15:28 |
| TheJulia | dtantsur: base-16 includes 9 ;) | 15:28 |
| rpittau | oh gosh | 15:28 |
| TheJulia | We're geeks, moving on | 15:29 |
| dtantsur | TheJulia: right, but it makes the number even huger! | 15:29 |
| dtantsur | yes | 15:29 |
| rpittau | anyhow :D | 15:29 |
| dtantsur | Let's pretend we're serious adults for some more minutes | 15:29 |
| JayF | 09 means the same in base 10 and base 256 | 15:29 |
| rpittau | I don't ahve anything to report on bugs or RFEs | 15:29 |
| rpittau | anyone else wants to add anything ? | 15:29 |
| JayF | I will note that if you wanna make progress on anything | 15:29 |
| JayF | I learned recently, maybe others know this, about the "heat" sort for launchpad | 15:30 |
| JayF | sorting by hottest actually gives you an interesting list of features | 15:30 |
| JayF | so maybe a thing to look at if someone has time | 15:30 |
| rpittau | any volunteer for the bug deputy this week ? | 15:30 |
| JayF | plus we still have untriaged stuff in ironic-adjacent repos we probably shuld focus on -- I'll look next time I can take bug deputy otherwise (post ptg) | 15:30 |
| masghar | I can take it | 15:30 |
| rpittau | thanks masghar :) | 15:30 |
| dtantsur | masghar++ | 15:30 |
| masghar | no problem! | 15:31 |
| rpittau | and I believe that is all folks! | 15:31 |
| rpittau | #endmeeting | 15:32 |
| opendevmeet | Meeting ended Mon Sep 30 15:32:05 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:32 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-09-30-15.00.html | 15:32 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-09-30-15.00.txt | 15:32 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-09-30-15.00.log.html | 15:32 |
| rpittau | thanks everyone :) | 15:32 |
| JayF | So a thing I was saving for post meeting | 15:32 |
| JayF | https://wiki.sipeed.com/hardware/en/kvm/NanoKVM/introduction.html :-O | 15:32 |
| rpittau | cardoe: Hail Mary is really a masterpiece btw :) | 15:32 |
| JayF | IPMI supported out of box; redfish is in the todo | 15:32 |
| dtantsur | WOW | 15:32 |
| JayF | they did a video on this for LTT ShortCircuit | 15:33 |
| cardoe | rpittau: so the hashtag ask was more to help ya out with nudging some patches out if they were back in the contributor court or needed a rebase. Also bring in stuff that’s got a +2 and just waiting on workflow. | 15:33 |
| rpittau | really nice | 15:33 |
| JayF | and I'm about to spend my whole day's pay on outfitting my server room lol | 15:33 |
| JayF | https://pikvm.org/ is apparently an altnerative, but is a lot more pricey | 15:33 |
| JayF | pikvm already does redfish too, apparently | 15:34 |
| TheJulia | JayF: cool cool, I saw that a few weeks ago and thought it was neat | 15:34 |
| dtantsur | JayF: definitely write a blog post | 15:34 |
| TheJulia | ++ | 15:34 |
| rpittau | cardoe: yeah, I got it, but at the moment I don't think we have a way to allow non-cores to assign hashtags to other's patches | 15:34 |
| TheJulia | for ironicbaremetal.org too, please! | 15:34 |
| TheJulia | rpittau: maybe worth asking #opendev if we create another group? | 15:34 |
| JayF | we'll see, my content creation juice is quite accounted for these days :D | 15:34 |
| JayF | rpittau: I think we should de-acl hashtags in ironic overall | 15:35 |
| JayF | rpittau: just let people set 'em on whatever patches, we can lock the door back later in case of abuse | 15:35 |
| iurygregory | we can update the acl for the hashtag | 15:35 |
| rpittau | TheJulia, JayF, I think we have something like that under discussion at the moment? :) | 15:35 |
| JayF | (and learn that someone has bad judgement in the least harmful way possible lol) | 15:35 |
| JayF | rpittau: yeah, but I'm saying, we shouldn't ACL hashtags at all | 15:35 |
| * JayF put a comment on that thread during meeting btw | 15:36 | |
| * dtantsur takes a break | 15:36 | |
| rpittau | JayF: you know I'm a bit conservative on that, but I would not say no if there's a majority agreement | 15:36 |
| iurygregory | https://github.com/openstack/project-config/blob/8377a1d94a7d26975956dfd89bf989f0b0f126b8/gerrit/acls/openstack/ironic.config#L6 | 15:37 |
| iurygregory | we just need to update this basically =) | 15:38 |
| rpittau | iurygregory: cool, you want to take that on? :) | 15:39 |
| iurygregory | rpittau, sure | 15:39 |
| rpittau | thanks! | 15:39 |
| cid | ATM, are new changesets that affect API version still under Dalmatian? | 15:42 |
| JayF | cid: since we have a stable/2024.2 now (dalmation), that means master is 2025.1 (epoxy) | 15:42 |
| cid | Okay, just double checking. | 15:43 |
| JayF | okay $250 for 5 of those nanokvm pros shipped to the US, as long as they actually arrive that's crazy good pricing | 15:43 |
| JayF | and I am over the moon about this existing | 15:43 |
| JayF | I might finally run an Ironic in a VM or something for my small handful of local machines | 15:43 |
| JayF | now to nerd-snipe someone into turning the redfish on their OSS firmware from a todo to a todone :D | 15:44 |
| iurygregory | weird, I had the feeling we should have editHashtags in each [access "refs"] ... but we only have in [access "refs/heads/*"] | 15:46 |
| iurygregory | rpittau https://review.opendev.org/c/openstack/project-config/+/930930 | 15:49 |
| rpittau | thanks iurygregory, +1 | 15:51 |
| cardoe | Ooo. Core can abandon patches too. | 15:54 |
| clarkb | iurygregory: you only need it under the subpaths if you set them to be exclusive permissions | 15:54 |
| clarkb | since refs/heads/* applies to everything and edithashtags isn't exclusive in your config ti should work | 15:55 |
| JayF | cardoe: yeah, we probably *won't* give that to non-cores though :) | 15:55 |
| JayF | cardoe: although you can abandon your own | 15:55 |
| iurygregory | clarkb, oh good to know! | 15:55 |
| iurygregory | tks =) | 15:55 |
| cardoe | JayF: yeah definitely don’t want that opened. But that means we can clean up some stuff. | 15:56 |
| JayF | abandon is a very big hammer, even when I was PTL I didn't use it unless something was very very old and I 100% knew the person was gone or disinterested | 15:57 |
| JayF | let me put it this way, I don't think we're going to run out of room in the "status:open" part of the gerrit DB :D | 15:57 |
| TheJulia | An exceptionally big hammer | 15:58 |
| rpittau | good night! o/ | 16:09 |
| Pcmalih_ | Hi | 16:10 |
| opendevreview | Graeme Moss proposed openstack/networking-generic-switch master: Add Supermicro switches to allow for supported write config https://review.opendev.org/c/openstack/networking-generic-switch/+/926886 | 16:20 |
| opendevreview | Graeme Moss proposed openstack/networking-generic-switch master: Add Supermicro switches to allow for supported write config https://review.opendev.org/c/openstack/networking-generic-switch/+/926886 | 16:38 |
| grami[m] | JayF: thanks for the +2 but I thought I would fix the pep8 E405 errors if you don't mind checking and giving your thumbs up. | 16:40 |
| JayF | I saw the post here and followed up | 16:41 |
| JayF | also put a link to some help for doing that locally (the ironic unit test docs) | 16:41 |
| JayF | honestly we should probably do a better job of linking to ironic contributor docs from the adjacent projects | 16:42 |
| JayF | that actually is achievable in short order, I might JFDI | 16:42 |
| grami[m] | Thanks :) | 16:42 |
| opendevreview | Jay Faulkner proposed openstack/networking-generic-switch master: Link to Ironic dev-quickstart in contributing doc https://review.opendev.org/c/openstack/networking-generic-switch/+/930943 | 18:32 |
| JayF | ^ if folks like the shape of that I'll spread it around | 18:32 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: [WIP] devstack: split into VENVs https://review.opendev.org/c/openstack/ironic/+/930776 | 18:54 |
| opendevreview | cid proposed openstack/ironic master: [WIP] Add support for node name in port creation https://review.opendev.org/c/openstack/ironic/+/930944 | 19:34 |
| JayF | oooh so close on the venv change! | 19:39 |
| JayF | virtualbmc worked in the ipmi job from inside the venv | 19:39 |
| TheJulia | awesome | 20:21 |
| opendevreview | Jay Faulkner proposed openstack/sushy master: contributor: links to ironic dev docs, launchpad https://review.opendev.org/c/openstack/sushy/+/930946 | 20:24 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: [WIP] devstack: split into VENVs https://review.opendev.org/c/openstack/ironic/+/930776 | 20:40 |
| JayF | so fun story, I bet our CI does not work if you're set to use virtualbmc/virtualpdu/sushytools but are *not* configured to pull them from git | 20:55 |
| JayF | in that case, there's nothing installing the bindep.txt contents from those repos (as I discovered troubleshooting ^^^^) | 20:55 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: [WIP] devstack: split into VENVs https://review.opendev.org/c/openstack/ironic/+/930776 | 20:56 |
| opendevreview | Jay Faulkner proposed openstack/sushy master: contributor: links to ironic dev docs, launchpad https://review.opendev.org/c/openstack/sushy/+/930946 | 21:15 |
| opendevreview | Jay Faulkner proposed openstack/virtualpdu master: Vendor pysnmp v5.0.33 into virtualpdu https://review.opendev.org/c/openstack/virtualpdu/+/928823 | 22:43 |
| JayF | sushy still failing, I'm a little lost on why it's not seeing libvirt-python | 22:43 |
| JayF | but I'm seeing if the virtualpdu patch is enough to make this happy | 22:43 |
| JayF | (for snmp) | 22:43 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!