| opendevreview | OpenStack Release Bot proposed openstack/bifrost master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/bifrost/+/918254 | 06:21 |
|---|---|---|
| opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/ironic-inspector/+/918256 | 06:21 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/918259 | 06:22 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/918261 | 06:22 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/ironic-python-agent/+/918263 | 06:23 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/ironic-ui/+/918265 | 06:23 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/ironic/+/918267 | 06:23 |
| opendevreview | OpenStack Release Bot proposed openstack/metalsmith master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/metalsmith/+/918269 | 06:24 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-baremetal master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/networking-baremetal/+/918271 | 06:24 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-generic-switch master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/networking-generic-switch/+/918273 | 06:24 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironic-inspector-client master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/918276 | 06:24 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironicclient master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/python-ironicclient/+/918278 | 06:25 |
| opendevreview | OpenStack Release Bot proposed openstack/sushy master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/sushy/+/918280 | 06:25 |
| rpittau | good morning ironic! o/ | 07:01 |
| opendevreview | Merged openstack/ironic-specs master: Add work items for 2024.2 Dalmatian development cycle https://review.opendev.org/c/openstack/ironic-specs/+/916295 | 08:00 |
| opendevreview | Merged openstack/ironic master: doc: Fix wrong example of [ipmi] cipher_suite_versions https://review.opendev.org/c/openstack/ironic/+/918158 | 08:34 |
| opendevreview | Verification of a change to openstack/ironic-python-agent stable/2023.1 failed: Unmount config drives https://review.opendev.org/c/openstack/ironic-python-agent/+/917852 | 09:19 |
| opendevreview | Merged openstack/python-ironicclient master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/python-ironicclient/+/918278 | 09:23 |
| opendevreview | Merged openstack/sushy master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/sushy/+/918280 | 09:23 |
| opendevreview | Merged openstack/networking-generic-switch master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/networking-generic-switch/+/918273 | 09:25 |
| opendevreview | Merged openstack/ironic-python-agent master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/ironic-python-agent/+/918263 | 09:26 |
| opendevreview | Merged openstack/python-ironic-inspector-client master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/918276 | 09:28 |
| opendevreview | Merged openstack/ironic-prometheus-exporter master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/918259 | 09:28 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/918261 | 09:28 |
| opendevreview | Merged openstack/ironic-ui master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/ironic-ui/+/918265 | 09:33 |
| opendevreview | Merged openstack/bifrost master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/bifrost/+/918254 | 09:35 |
| dtantsur | rpittau, TheJulia, JayF, wdyt about instantly EOL'ing some of our deliverables instead of this ^^^? | 09:47 |
| dtantsur | I think bifrost is the obvious candidate, but also ironic-ui, and clients are also not super useful | 09:48 |
| dtantsur | IPA-builder can technically work across branches | 09:48 |
| dtantsur | metalsmith | 09:48 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Remove agent_ipmitool/pxe_ipmitool tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/917902 | 09:53 |
| opendevreview | Merged openstack/ironic-inspector master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/ironic-inspector/+/918256 | 09:53 |
| opendevreview | Merged openstack/ironic master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/ironic/+/918267 | 09:53 |
| opendevreview | Verification of a change to openstack/ironic stable/2023.1 failed: ci: stable-only: explicitly pin centos build https://review.opendev.org/c/openstack/ironic/+/918118 | 10:29 |
| opendevreview | Merged openstack/metalsmith master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/metalsmith/+/918269 | 10:44 |
| opendevreview | Merged openstack/ironic master: docs: document stance on partition image use https://review.opendev.org/c/openstack/ironic/+/915388 | 10:49 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Add inspection rules https://review.opendev.org/c/openstack/ironic/+/918303 | 10:52 |
| opendevreview | Merged openstack/networking-baremetal master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/networking-baremetal/+/918271 | 11:06 |
| rpittau | dtantsur: I guess that would work too, but I'm not sure we can avoid the unmaintained status anyway as it's Openstack-wide | 11:29 |
| dtantsur | rpittau: there have been a discussion about skipping unmaintained status for some projects like openstacksdk | 11:31 |
| rpittau | dtantsur: having a precedent helps, I see if I can start a discussion in that direction for some of our projects | 12:19 |
| JayF | Let me check with my downstream too | 12:42 |
| JayF | But we don't use bifrost or metalsmith at least for sure | 12:42 |
| dtantsur | Bifrost won't even work correctly without some modifications | 12:49 |
| iurygregory | good morning Ironic | 12:50 |
| TheJulia | good morning | 13:05 |
| TheJulia | Regarding instantly EOL-ing some of our deliverables, specifically the non-service/component ones, I'm good with doing so | 13:07 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: Intermediate networking service https://review.opendev.org/c/openstack/ironic-specs/+/916126 | 14:48 |
| TheJulia | Updates from discovery and increased understanding over the weekend | 14:48 |
| rpittau | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon May 6 15:00:31 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 |
| iurygregory | o/ | 15:00 |
| masghar | o/ | 15:00 |
| TheJulia | good morning | 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 | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_May_6.2C_2024 | 15:00 |
| dtantsur | o/ | 15:00 |
| rpittau | #topic Announcements/Reminders | 15:01 |
| rpittau | #info Standing reminder to review patches tagged ironic-week-prio and to hashtag any patches ready for review with ironic-week-prio: https://tinyurl.com/ironic-weekly-prio-dash | 15:01 |
| rpittau | #info Work items for 2024.2 have been merged https://review.opendev.org/c/openstack/ironic-specs/+/916295 | 15:01 |
| rpittau | #info Ironic Meetup/BareMetal SIG June 5, OpenInfra Days June 6 @ CERN. | 15:02 |
| rpittau | Signup is now closed | 15:02 |
| rpittau | #info 2024.2 Dalmatian Release Schedule https://releases.openstack.org/dalmatian/schedule.html | 15:02 |
| rpittau | anything else to announce/remind ? | 15:02 |
| iurygregory | #info CFP for OIS Asia'24 closes on May 29 | 15:02 |
| rpittau | thanks iurygregory | 15:02 |
| iurygregory | yw | 15:03 |
| rpittau | I do have an announcement, I will be out for a couple of weeks, I'll be back on Monday May 26 | 15:04 |
| rpittau | I will skip two meetings, so need one/two volunteer(s) to run the next 2 meetings :) | 15:04 |
| TheJulia | I've got nothing planned so I guess I can | 15:04 |
| dtantsur | I'll be out for 2 Mondays as well | 15:04 |
| rpittau | thanks TheJulia :) | 15:04 |
| rpittau | #info TheJulia will run the meetings on 13 and 20 of May | 15:05 |
| iurygregory | I can run | 15:05 |
| iurygregory | oh nvm =D | 15:05 |
| rpittau | thanks iurygregory, I guess you can alternate with TheJulia :) | 15:05 |
| iurygregory | yeah =) for sure | 15:06 |
| rpittau | ok, moving on! | 15:06 |
| rpittau | #topic Review Ironic CI status | 15:06 |
| rpittau | haven't seen anything particular in CI last week | 15:07 |
| JayF | I believe Julia landed some fixes for stable branches in IPA | 15:07 |
| JayF | Other than that I've not seen anything new | 15:07 |
| TheJulia | Yeah, CI seemed mostly okay last week with the exception being some stable branches being a little cranky | 15:08 |
| rpittau | true, thanks TheJulia! | 15:08 |
| rpittau | now that I think about it, I've seen some failures on grenade jobs in stable releases, maybe due to zed going into unmaintained mode | 15:08 |
| rpittau | we may need to remove the job in zed to be able to land the latest changes in the branch, just simple branch updates | 15:09 |
| rpittau | I don't see any discussion topics for this week, do we want to jump directly to Bug Deputy updates ? | 15:10 |
| TheJulia | Yeah, oldest grenade always needs to be disabled | 15:10 |
| rpittau | #action disable grenade in zed | 15:11 |
| TheJulia | Bug Deputy update sounds like it is next | 15:11 |
| rpittau | #topic Bug Deputy Updates | 15:12 |
| rpittau | TheJulia: anything to reprt? :) | 15:12 |
| TheJulia | Two items to note | 15:12 |
| TheJulia | First, I cleaned up ironic-inspector bugs, mostly marked stuff that was super old to indicate we were not planning on fixing the items | 15:12 |
| TheJulia | I also was able to close out a few bugs in ironic as already fixed or no longer applicable | 15:13 |
| rpittau | great, thanks! | 15:13 |
| JayF | I'll take it this week | 15:14 |
| iurygregory | I can be the bug dep after next week (ie, the week from May 20 - 24) | 15:14 |
| rpittau | thanks JayF I was going to ask :D | 15:14 |
| rpittau | and thanks iurygregory :) | 15:14 |
| rpittau | I guess we can move to | 15:15 |
| rpittau | #topic RFE Review | 15:15 |
| rpittau | we have 2 RFEs proposed by rozzi but I don't see him here today | 15:15 |
| rpittau | they look both reasonable to me | 15:16 |
| rpittau | and I saw the development already started | 15:16 |
| TheJulia | I concur, I think the block device list combo one makes sense | 15:16 |
| rpittau | yeah, absolutely | 15:16 |
| TheJulia | that seems more like a bug to me, tbh | 15:16 |
| TheJulia | the other, I have some mixed feelings on but I can see the reasoning | 15:17 |
| rpittau | sure on the edge | 15:17 |
| rpittau | I guess the scenario proposed makes sense | 15:17 |
| * iurygregory looks at the RFEs | 15:17 | |
| JayF | Can we link the rfe's in here so that they are in the minutes? I also don't have the agenda up lol | 15:17 |
| rpittau | #link https://bugs.launchpad.net/ironic-python-agent/+bug/2061437 | 15:17 |
| rpittau | #https://bugs.launchpad.net/ironic-python-agent/+bug/2061362 | 15:17 |
| rpittau | #link https://bugs.launchpad.net/ironic-python-agent/+bug/2061362 | 15:18 |
| opendevreview | Verification of a change to openstack/ironic stable/2023.1 failed: ci: stable-only: explicitly pin centos build https://review.opendev.org/c/openstack/ironic/+/918118 | 15:18 |
| JayF | 2061362, I'm not sure I'm on board with approving that | 15:18 |
| JayF | We have methods to disable the step entirely which seems to be the correct move if you have a disc that you do not want cleaned | 15:19 |
| dtantsur | Mixed feelings here too. We can disable the step or we can skip a disk by using hints. | 15:19 |
| iurygregory | we can also skip cleaning a given disk | 15:19 |
| iurygregory | yeah | 15:19 |
| TheJulia | Based upon the described case, it sounds like they have shared luns across hosts | 15:20 |
| rpittau | mmm I can see your point too, I'm more on the verge of accepting honestly | 15:20 |
| JayF | Is the actual feature a way to skip shared devices? Like a device hint that points to any shared devices that they can skip cleaning | 15:20 |
| TheJulia | so, if we can better understand the exact case in which the device is locked, that might help us understand how this feature could be scoped and accepted as an "optional" thing | 15:20 |
| JayF | I'm okay with a declarative way of saying don't clean this. I'm less okay with saying just ignore certain types of errors during cleaning | 15:20 |
| TheJulia | can't necessarilly know what the device is or if it will be in use in a shared lab | 15:20 |
| dtantsur | I'd even be okay with "skip any remote device" as an option | 15:21 |
| JayF | I do think it's not unreasonable to ask someone to use a custom hardware manager for a lab case if it is a unique use case | 15:21 |
| JayF | Same dtantsur | 15:21 |
| masghar | I think their trouble is having many disks and a random one randomly failing | 15:22 |
| TheJulia | skipping "remote" might make sense, not sure I'd ever let a customer use it though :) | 15:22 |
| TheJulia | but that is always a documentation thing | 15:22 |
| rpittau | JayF, dtantsur, when you have a minute please add a comment in the RFE, I guess we need more details on that? | 15:22 |
| dtantsur | masghar: it's not a normal situation there? we're curious why they fail. | 15:22 |
| JayF | masghar: another way of looking at that is should ironic say a machine is available if it has a random disk failure | 15:22 |
| masghar | dtantsur, JayF, I see...but then their whole fleet can't be cleaned? | 15:23 |
| rpittau | I will approve the first one | 15:23 |
| dtantsur | masghar: I'd be really curious what is going on in this case | 15:24 |
| dtantsur | Like, in my whole career in OpenShift, I don't remember a single such case | 15:24 |
| masghar | dtantsur: true, I guess disk failures are too serious to ignore | 15:24 |
| dtantsur | A harsher way to put that would be "if you don't care if some of your disks don't get cleaned, why enable cleaning at all?" | 15:25 |
| TheJulia | to me, it reads as if there are quorum disks in the environment | 15:25 |
| TheJulia | and we hit one while another node is up which shares the quorum disk, and we fail at that point | 15:26 |
| masghar | oh I see! | 15:26 |
| dtantsur | Sure but... why have cleaning enabled? | 15:26 |
| dtantsur | Metadata cleaning (the only type supported in metal3) is designed to prevent software conflicts, such as Ceph not liking old Ceph partitions | 15:27 |
| JayF | Or why not as part of your process, skip cleaning on shared/remote discs. Even if the feature was oriented that way, we'd likely approve it as skipping cleaning on remote devices optionally | 15:27 |
| TheJulia | dtantsur: I think because they are re-cycling the environment | 15:27 |
| rpittau | the error is not going to be ignored though, just not fail the entire cleanup | 15:27 |
| rpittau | but I guess at that point excluding the disk (removing it physically or logically) would be better than leaving it there in any case, and then redoing the cleaning | 15:27 |
| dtantsur | Yeah, but if you leave random data around on a shared disk, you still hit software problems | 15:27 |
| TheJulia | dtantsur: so if you have a host already cleaning, and it grabs the quorum disk first, it has the lock | 15:27 |
| TheJulia | another one comes along and will fail | 15:27 |
| TheJulia | dtantsur: if the other node is being cleaned, the data gets wiped out, but another node doesn't know about it | 15:28 |
| dtantsur | Or maybe it uses the disk, and you try to clean it :) I think this environment should have no cleaning or some very custom cleaning | 15:28 |
| dtantsur | exactly | 15:28 |
| dtantsur | how is cleaning the way we do it safe there? | 15:28 |
| TheJulia | we kind of need to take a step back from the problem | 15:28 |
| TheJulia | and provide greater clarity | 15:28 |
| TheJulia | is there a way to identify this as an "acceptable" failure | 15:29 |
| TheJulia | for that device, or not | 15:29 |
| TheJulia | for example, I ran 8GB quorum disks in clusters years ago shared across multiple nodes, the lock holder was always the leader | 15:29 |
| dtantsur | Right, yeah. We're guessing already, we should get back to Adam | 15:29 |
| TheJulia | ++ | 15:29 |
| rpittau | yep, I agree we need more clarification at this point | 15:29 |
| dtantsur | I think the option to ignore errors should be our last resort | 15:31 |
| JayF | Yeah basically we open ourselves up to a security vulnerability if someone can figure out how to break our agent using something crazy on disk | 15:31 |
| dtantsur | Well, this is metadata cleaning | 15:31 |
| dtantsur | For full cleaning, I'd be hard -2 on even considering that | 15:32 |
| TheJulia | JayF: if a disk is locked, you might not be able to read it depending on the lock | 15:32 |
| JayF | Yeah, I guess for metadata cleaning the worst cases that will get very low quality bugs around people disabling error checking and then being shocked that their unclean disc can't be deployed to | 15:32 |
| TheJulia | yeah | 15:32 |
| JayF | I still think we should round trip some questions about details. I'm not a hard and fast opposed no matter what with that nuance in mind though | 15:33 |
| TheJulia | ++ | 15:33 |
| dtantsur | agreed | 15:33 |
| rpittau | ++ | 15:34 |
| rpittau | please just add a comment in the RFE to get more info from Adam | 15:35 |
| TheJulia | += | 15:35 |
| TheJulia | ++ | 15:35 |
| JayF | I'll toss that comment in here in the next 15 minutes | 15:36 |
| rpittau | thanks JayF :) | 15:36 |
| JayF | If y'all want to just let me do it so that we don't crush Adam with five things that say the same thing LOL | 15:36 |
| rpittau | sounds good :D | 15:36 |
| * dtantsur nods | 15:37 | |
| rpittau | we can probably move on | 15:37 |
| rpittau | is there anything for open discussion ? | 15:37 |
| JayF | Ah that Nova bug I linked earlier in the ironic driver, it's worth a look if anyone has some time | 15:38 |
| JayF | I'm mainly curious if we need to add tempest coverage on the nova driver side | 15:39 |
| JayF | for things like reboot | 15:39 |
| rpittau | I think I lost it | 15:39 |
| rpittau | can you link it here? | 15:39 |
| JayF | https://review.opendev.org/c/openstack/nova/+/918195 | 15:39 |
| rpittau | thanks | 15:39 |
| JayF | #link https://launchpad.net/bugs/2064826 | 15:39 |
| JayF | tl;dr we broke nova reboot last cycle, CI didn't catch it | 15:39 |
| rpittau | whoops | 15:39 |
| JayF | it's being fixed but I wanted to see if it is an oversight or an explicit decision to not test this | 15:39 |
| JayF | if an oversight, I'll see if I can swing a test for it | 15:40 |
| JayF | but always like asking since I know we are shaving our test surfaces, don't wanna readd something that was intentionally removed | 15:40 |
| TheJulia | afaik, our basic ops plugin doesn't exercise reoot | 15:40 |
| TheJulia | reboot | 15:40 |
| TheJulia | doing reboot testing as part of it would also add some time to plugin execution | 15:41 |
| TheJulia | ... Mileage may vary, of course as with all things tempest plugin wise | 15:41 |
| JayF | It sorta goes back to my statement a cycle ago ... I wish we could just have a fake ironic node for some of these interactions/tests | 15:41 |
| JayF | but I think that's more than I'm willing to take on right now, so I might just swing at a reboot test and see what the actual impact is in minutes | 15:41 |
| TheJulia | well, we sort of can but the test can't ping something entirely fake | 15:42 |
| TheJulia | your ether doing something entirely fake, or your doing stuff which is a full scenario | 15:42 |
| TheJulia | integrative is scenario inherently | 15:42 |
| JayF | well, Nova<>Ironic driver, some of those things are just "did we tell Ironic to do the thing" | 15:43 |
| TheJulia | doing stuff in the middle leads fixes like https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/918001 having to come along | 15:43 |
| JayF | heh, fair | 15:43 |
| JayF | I'll keep wishing for utopia and living in the real world instead then :P | 15:43 |
| TheJulia | heh | 15:43 |
| TheJulia | adding to our basic ops likely isn't a huge issue, just add a knob for the time | 15:44 |
| TheJulia | if a reasonable one doesn't already exist | 15:44 |
| JayF | ack | 15:44 |
| rpittau | thanks for reporting that JayF | 15:44 |
| rpittau | anything else we want to discuss today? | 15:45 |
| dtantsur | At some point, we should probably delineate stuff we test in Nova and stuff we test in Ironic | 15:45 |
| dtantsur | otherwise, our testing matrix will keep being chaotic. | 15:45 |
| JayF | dtantsur: that's a line of chat Sean Mooney and I had last cycle; a tempest scenario set specifically designed for nova driver, and to run it more frequently | 15:46 |
| TheJulia | Yeah, I'm not a fan of doing an extra reboot when exercising rescue | 15:46 |
| TheJulia | and then doing that across however many tests that ends up triggering on | 15:46 |
| JayF | but realistically I'm already majorly overcommitted this cycle | 15:46 |
| rpittau | I guess this needs a longer discussion and involving the nova team | 15:47 |
| rpittau | we need to see if we can find the resources during this cycle | 15:47 |
| TheJulia | well | 15:47 |
| TheJulia | They are not going to really grasp the current scale | 15:47 |
| JayF | Sean is sorta our ambassador to the nova team in some ways these days | 15:48 |
| JayF | but I don't wanna engage a chat at that level if I don't have time to take action items outta it | 15:48 |
| rpittau | yep, exactly | 15:48 |
| TheJulia | but we do need to likely take a look and sort out if we've got appropriate coverage with the plugin | 15:48 |
| * JayF has had a problem with saying yes more times than he delivered a "yes" result | 15:48 | |
| dtantsur | we know this feeling | 15:50 |
| rpittau | we can probably reevaluate the situation during the cycle | 15:50 |
| rpittau | looks like we're closing | 15:51 |
| rpittau | thanks everyone | 15:51 |
| rpittau | #endmeeting | 15:51 |
| opendevmeet | Meeting ended Mon May 6 15:51:55 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:51 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-05-06-15.00.html | 15:51 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-05-06-15.00.txt | 15:51 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-05-06-15.00.log.html | 15:51 |
| rpittau | bye everyone, see you in 3 weeks! o/ | 16:07 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add fake ironic python agent (fake_ipa) https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 16:23 |
| ikoliveira | TheJulia and iurygregory thanks for supporting me, your recommendations solved my issues with deploying baremetal nodes and now we are able to use it. Thank you so much! | 18:14 |
| TheJulia | \o/ | 18:15 |
| TheJulia | Congrats! | 18:15 |
| ikoliveira | \o/ | 18:15 |
| JayF | woohoo | 18:19 |
| *** dking is now known as Guest4281 | 19:06 | |
| *** Guest4281 is now known as dking | 19:06 | |
| opendevreview | Merged openstack/ironic stable/2023.1: ci: stable-only: explicitly pin centos build https://review.opendev.org/c/openstack/ironic/+/918118 | 19:07 |
| opendevreview | Maximilian Brandt proposed openstack/ironic-python-agent master: Add bmc_subnet information to introspection and Consolidated BMC information retrieval logic into a single function _get_bmc_info(). https://review.opendev.org/c/openstack/ironic-python-agent/+/918340 | 19:13 |
| opendevreview | Maximilian Brandt proposed openstack/ironic-python-agent master: Add bmc_subnet information to introspection. https://review.opendev.org/c/openstack/ironic-python-agent/+/918340 | 19:16 |
| *** Guest2968 is now known as diablo_rojo | 20:38 | |
| iurygregory | ikoliveira, glad it worked \o/ now you can add ironic to the production cloud =D | 20:39 |
| NobodyCam | Good evening Ironic folks, | 23:18 |
| NobodyCam | any updated doc on getting unit test going? | 23:19 |
| TheJulia | NobodyCam: does jut running "tox" not sort things out for unit tests? | 23:26 |
| TheJulia | s/jut/just/ | 23:27 |
| NobodyCam | `failed with pass_env values cannot contain whitespace, use comma to have multiple values in a single line, invalid values found 'http_proxy HTTP_PROXY https_proxy HTTPS_PROXY no_proxy NO_PROXY'` | 23:27 |
| NobodyCam | tox.ini looks okay | 23:27 |
| NobodyCam | I was looking at: https://docs.openstack.org/ironic/latest/contributor/dev-quickstart.html#unit | 23:28 |
| TheJulia | so you've got an older version with a newer tox which is incompatible | 23:29 |
| TheJulia | https://www.irccloud.com/pastebin/PneTjf2s/ | 23:30 |
| opendevreview | cid proposed openstack/ironic master: Flexible IPMI credential persistence method configuration https://review.opendev.org/c/openstack/ironic/+/917229 | 23:30 |
| NobodyCam | oh | 23:31 |
| NobodyCam | Thank you! | 23:33 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!