| kubajj | Good morning Ironic o/ | 06:58 |
|---|---|---|
| rpittau | good morning ironic! o/ | 07:10 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Use sparkingly new metalsmith cs9 job https://review.opendev.org/c/openstack/ironic/+/892680 | 10:22 |
| iurygregory | good morning Ironic | 11:19 |
| opendevreview | Maksim Malchuk proposed openstack/ironic-python-agent-builder stable/2023.1: Extend the DIB_CHECKSUM variable usage https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/892913 | 12:25 |
| mmalchuk | morning Ironic o/ | 12:30 |
| rpittau | if anyone has a moment please review https://review.opendev.org/c/openstack/ironic/+/892680 it's CI change that is long due | 12:40 |
| iurygregory | rpittau, I'm ok with +W since is a CI only change (if you are in a hurry) | 12:46 |
| rpittau | thanks iurygregory :) | 12:46 |
| opendevreview | Maksim Malchuk proposed openstack/ironic-python-agent-builder stable/zed: Add checksum generation support https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/892925 | 13:03 |
| opendevreview | Maksim Malchuk proposed openstack/ironic-python-agent-builder stable/zed: Extend the DIB_CHECKSUM variable usage https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/892926 | 13:03 |
| kubajj | Does anybody have any comments or opinions about https://review.opendev.org/c/openstack/ironic/+/890819 ? | 13:09 |
| TheJulia | kubajj: haven't had a chacne to look yet | 13:23 |
| TheJulia | btw folks, I'm out sick today | 13:23 |
| * TheJulia stumbles away to make coffee so she can return to low power mode | 13:23 | |
| kubajj | TheJulia: no worries, hope you get better soon | 13:25 |
| iurygregory | take care TheJulia | 13:26 |
| opendevreview | Riccardo Pittau proposed openstack/metalsmith master: Remove CentOS Stream 8 jobs https://review.opendev.org/c/openstack/metalsmith/+/892965 | 13:39 |
| opendevreview | Maksim Malchuk proposed openstack/ironic-python-agent-builder stable/yoga: Add checksum generation support https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/892966 | 13:42 |
| opendevreview | Maksim Malchuk proposed openstack/ironic-python-agent-builder stable/yoga: Extend the DIB_CHECKSUM variable usage https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/892967 | 13:42 |
| opendevreview | Maksim Malchuk proposed openstack/ironic-python-agent-builder stable/xena: Add checksum generation support https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/892968 | 13:42 |
| opendevreview | Maksim Malchuk proposed openstack/ironic-python-agent-builder stable/xena: Extend the DIB_CHECKSUM variable usage https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/892969 | 13:42 |
| mmalchuk | take care TheJulia | 13:47 |
| rpittau | seems like metal 3 job is green again https://review.opendev.org/c/openstack/ironic/+/892797 looking for +W there | 14:23 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Exclude .pyc encoding files. https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/892706 | 14:24 |
| iurygregory | rpittau, done | 14:28 |
| rpittau | tnx iurygregory :) | 14:37 |
| *** JasonF is now known as JaqyF | 14:54 | |
| *** JaqyF is now known as JayF | 14:54 | |
| JayF | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Aug 28 15:00:18 2023 UTC and is due to finish in 60 minutes. The chair is JayF. 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 |
| rpittau | o/ | 15:00 |
| JayF | Good morning Ironic'ers: A reminder we operate under the OpenInfra Foundation CoC https://openinfra.dev/legal/code-of-conduct | 15:00 |
| iurygregory | o/ | 15:00 |
| JayF | Agenda is available at https://wiki.openstack.org/wiki/Meetings/Ironic | 15:00 |
| kubajj | o/ | 15:00 |
| JayF | I'll note my internet is not behaving well this morning; if I disappear in the meeting someone please take over | 15:01 |
| JayF | #chair rpittau | 15:01 |
| opendevmeet | Current chairs: JayF rpittau | 15:01 |
| JayF | #chair iurygregory | 15:01 |
| opendevmeet | Current chairs: JayF iurygregory rpittau | 15:01 |
| JayF | #note 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 |
| JayF | We are reaching the end of the release cycle. Please prioritize reviewing things that are close so we can get as much stuff into the release as possible :) | 15:01 |
| JayF | #note Bobcat client library freeze is in three days; please take notice | 15:02 |
| JayF | Also please update the PTG etherpad with anything you have to discuss in October during our chat: https://etherpad.opendev.org/p/ironic-ptg-october-2023 | 15:02 |
| JayF | I think that's all the announcements :) | 15:03 |
| JayF | #topic Previous action items | 15:03 |
| JayF | I had an action item to review and take action on Ironic-related zuul config errors; I have a change up here: https://review.opendev.org/c/openstack/python-ironicclient/+/892622 to fix stable/yoga ironicclient; and an email out to the mailing list announcing all other impacted branches will be retired on Sept 1 or later. Please comment there if you have an objection. | 15:04 |
| JayF | #link https://review.opendev.org/c/openstack/python-ironicclient/+/892622 | 15:04 |
| JayF | That was the only action item. Giving a minute for comment before moving on. | 15:05 |
| rpittau | I jsut approved it | 15:05 |
| JayF | Awesome | 15:06 |
| JayF | moving on | 15:06 |
| JayF | #topic Review CI Status | 15:06 |
| JayF | How is CI? | 15:06 |
| JayF | Other than that metal3 CI blip on Friday was there anything else? | 15:07 |
| iurygregory | there is the issue with python-ironicinspectorclient | 15:08 |
| iurygregory | the functional job is broken, still trying to figure out what is happening there | 15:08 |
| rpittau | we should merge https://review.opendev.org/c/openstack/ironic/+/892680 ASAP so we can merge https://review.opendev.org/c/openstack/metalsmith/+/892965 and finally release metalsmith :) | 15:09 |
| JayF | there have been approximately no changes in that repo, yeah? | 15:09 |
| iurygregory | JayF, correct | 15:09 |
| iurygregory | https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/891823 this was the patch that mentioned our gate was broken | 15:09 |
| JayF | yeah we have to make that pass then :( | 15:10 |
| iurygregory | and seems like it's broken till zed =X | 15:10 |
| iurygregory | https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/856419 | 15:10 |
| iurygregory | since we couldn't merge this one | 15:10 |
| JayF | I'll note that's a repo on the way out; I'm not that opposed to us removing the func job if we're sure the client itself still works | 15:10 |
| iurygregory | the weird things is the test that is failing | 15:11 |
| iurygregory | according to our test we should give an exception in case we try to abort inspection if is not started | 15:11 |
| iurygregory | but we return 202 :D | 15:12 |
| iurygregory | testtools.matchers._impl.MismatchError: <bound method ClientV1.abort of <ironic_inspector_client.v1.ClientV1 object at 0x7f2472a1b010>> returned <Response [202]> | 15:12 |
| JayF | iurygregory: I also saw complaints about function-py310 not existing, which is true: https://github.com/openstack/python-ironic-inspector-client/blob/master/tox.ini#L45 | 15:12 |
| iurygregory | just a heads-up, we can move forward with the meeting | 15:12 |
| JayF | yeah, we should | 15:12 |
| iurygregory | https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/891976 | 15:12 |
| JayF | good stuff | 15:13 |
| JayF | ty | 15:13 |
| iurygregory | yeah, but in the patch I pushed it was given the actual error =) | 15:13 |
| JayF | #topic Review ongoing 2023.2 Workstreams | 15:13 |
| JayF | I'm going to note I think it's unlikely sharding will land in Nova this cycle. I will do what I can but it hasn't had enough review focus since the patches were put up. I will try to get eyes on it early next cycle when there is more developer availability :( | 15:14 |
| JayF | https://etherpad.opendev.org/p/IronicWorkstreams2023.2 | 15:15 |
| JayF | although at this point; if you have changes outstanding for this release | 15:15 |
| JayF | they should be close if you want them in :( | 15:15 |
| JayF | Looks like no comments on workstreams in progress. Moving on. | 15:17 |
| JayF | #topic Branch retirement for Zuul config errors | 15:17 |
| opendevreview | Merged openstack/python-ironicclient stable/yoga: [stable-only] Fix zuul config; comment failing CI https://review.opendev.org/c/openstack/python-ironicclient/+/892622 | 15:17 |
| JayF | As mentioned above, in response to action item to fix zuul config errors in Ironic; I'm proposing mass retirement of some branches: https://lists.openstack.org/pipermail/openstack-discuss/2023-August/034854.html | 15:18 |
| JayF | please comment there if you have any concerns | 15:18 |
| JayF | #topic RFE Review | 15:18 |
| JayF | rpittau: you have the floor | 15:18 |
| rpittau | JayF: no concerns and I think I left a comment last week about the bugfix branches | 15:18 |
| JayF | ty :) | 15:19 |
| JayF | you wanna talk about your RFE? | 15:19 |
| rpittau | JayF: yes, thanks! | 15:19 |
| rpittau | I'd like to propose a change to allow to expose ironic functionalities to be able to attach/detach non-bootable isos to provisioned nodes | 15:20 |
| rpittau | the RFE should have all the info https://bugs.launchpad.net/ironic/+bug/2033288 but please let me know if there are questions/doubts/concerns | 15:20 |
| * iurygregory reads | 15:20 | |
| JayF | I'm mainly curious about the use case, it'd be nice to have one in the bug. | 15:20 |
| rpittau | the changes should be minimal as the features are there already, jsut need to be exposed | 15:21 |
| TheJulia | Interesting! | 15:21 |
| rpittau | JayF: right! I think I have that, just forgot to add it to the RFE! :D | 15:21 |
| TheJulia | It does sort of make sense to expose, sort of like setting boot mode manually | 15:21 |
| TheJulia | Err, boot device | 15:22 |
| JayF | Yeah, I can think of a case or two but it's nice to know the actual real world reason we're doing a thing :) | 15:22 |
| rpittau | yeah | 15:22 |
| * TheJulia returns to the Covid infected world of trying to rest | 15:22 | |
| JayF | if anything my only question is if there are other low-hanging items like this we should implement | 15:22 |
| iurygregory | agree, I think the missing bit is just the use case (but I think it makes sense ++) | 15:23 |
| rpittau | probably :D | 15:23 |
| JayF | going to mark it approved based on lazy consensus here | 15:23 |
| rpittau | thanks JayF , I'll add the use case soon(TM) | 15:24 |
| JayF | #topic Open Discussion | 15:24 |
| JayF | Anything else for the meeting? | 15:24 |
| JayF | Last call? | 15:26 |
| JayF | #endmeeting | 15:26 |
| opendevmeet | Meeting ended Mon Aug 28 15:26:48 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:26 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-08-28-15.00.html | 15:26 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-08-28-15.00.txt | 15:26 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-08-28-15.00.log.html | 15:26 |
| rpittau | bye everyone, see you tomorrow! o/ | 15:27 |
| kubajj | bye rpittau o/ | 15:30 |
| JayF | rpittau: iurygregory: Do one of you want to hold the meeting next week? Or we can cancel for US labor day | 15:55 |
| iurygregory | JayF, I can do it | 16:18 |
| opendevreview | Jakub Jelinek proposed openstack/ironic master: Introduce default kernel/ramdisks by arch https://review.opendev.org/c/openstack/ironic/+/890819 | 16:24 |
| opendevreview | Jakub Jelinek proposed openstack/ironic master: Introduce default kernel/ramdisks by arch https://review.opendev.org/c/openstack/ironic/+/890819 | 16:25 |
| opendevreview | Verification of a change to openstack/ironic master failed: Revert "Move metal3 integration job to non-voting temporarily" https://review.opendev.org/c/openstack/ironic/+/892797 | 17:37 |
| *** awb_ is now known as awb` | 17:43 | |
| opendevreview | Merged openstack/ironic master: Use sparkingly new metalsmith cs9 job https://review.opendev.org/c/openstack/ironic/+/892680 | 19:08 |
| espenfl | Hi there. When reinstalling Bifrost to test a new branch or feature, what do you typically do? Removing the venv (e.g. /opt/stack/bifrost) is okey. Then you kill the dnsmasqn and the ironic and ironic-inspector process? Is there more to this? | 19:42 |
| JayF | I suspect there's a non-destructive way to get from point a to point b there | 19:43 |
| JayF | probably just pip installing the newer bifrost into the venv then restarting services? | 19:44 |
| JayF | depends really on how you use it | 19:44 |
| JayF | some folks use bifrost to provision a persistent Ironic; some folks use it as a "oneshot" to deploy a bunch of nodes | 19:44 |
| espenfl | Right, okey, thanks. We use it mainly to ease into Kayobe. So want to get the basics in order in Bifrost first. We are at that point, but there are some issues in the most recent code base we want to get to the bottom of and hence need to play a bit with versions. | 20:01 |
| JayF | This is the place to expand on such issues if there's anything speciifc you can share :) | 20:03 |
| JayF | I am not the biggest expert on bifrost, but I know a lot about Ironic-proper and running it in production, so maybe we can skip the middle question and solve the original problem? | 20:03 |
| clarkb | pip should cache all the wheels it fetches/builds for dependencies. This means there isn't much reason to not start fresh | 20:08 |
| clarkb | the difference between starting fresh and working with a delta becomes very minimal from a time perspective | 20:08 |
| JayF | I guess that's a reasonable point; I hadn't thought about it from that perspective. If you keep the DB, keep the config, keep the data dirs, you can swap venvs with versions | 20:09 |
| JayF | although the "keep the DB" may not work as well if you need DB migration :) | 20:09 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/891912 can we please land this? Someone? It's a massive improvement, I want to do follow ups but first I'd rather us have at least one working documented devstack ironic+nova config in our docs :) | 20:52 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/884783 is Julia's service steps stuff, has my +2 needs further review and we need to land this in bobcat | 20:55 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add service steps call to agent logic https://review.opendev.org/c/openstack/ironic/+/891229 | 20:57 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/891229 now has my +2 too (I just pushed a one-character whitespace fix for flake8) | 20:58 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/890819 another one that needs more love | 21:14 |
| JayF | we gotta start getting this stuff in the gate or else we're going to be a part of the problem when the gate crush happens | 21:14 |
| opendevreview | Jay Faulkner proposed openstack/networking-generic-switch master: Fix delete_network_postcommit KeyError https://review.opendev.org/c/openstack/networking-generic-switch/+/888051 | 21:43 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/892790 trivial needs +2A to bump requirement for python-scciclient | 21:44 |
| opendevreview | Merged openstack/ironic master: Fail on node lookup if it is locked https://review.opendev.org/c/openstack/ironic/+/892414 | 23:04 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!