| arne_wiebalck | Good morning, Ironic! And all the best for 2024! | 07:33 |
|---|---|---|
| TheJulia | good morning! | 13:54 |
| opendevreview | Takashi Kajinami proposed openstack/ironic-lib master: Drop lower-constraints.txt (again) https://review.opendev.org/c/openstack/ironic-lib/+/904993 | 14:43 |
| JayF | Getting my computer connected to a power source this morning, I forgot to charge it overnight. The meeting will be starting shortly. | 15:00 |
| TheJulia | ack | 15:00 |
| JayF | #startmeeting ironic | 15:02 |
| opendevmeet | Meeting started Mon Jan 8 15:02:01 2024 UTC and is due to finish in 60 minutes. The chair is JayF. 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 |
| JayF | Welcome to the Ironic weekly meeting! | 15:02 |
| JayF | #topic Announcements/Reminder | 15:02 |
| TheJulia | o/ | 15:02 |
| JayF | As always, make sure to add hashtag: ironic-week-prio to your patches if they are ready for review priority, and when reviewing check that dashboard first https://tinyurl.com/ironic-weekly-prio-dash | 15:02 |
| JayF | Only action items from last meeting was informing community of cancelled meetings, which I did, skipping that agenda item. | 15:03 |
| JayF | #topic Caracal Release Schedule | 15:03 |
| JayF | Caracal-2 is this week. We need to ensure we've got releases out of libraries we want released. | 15:03 |
| JayF | Please someone on release liason team make some time to look at this, I will take time if required but am still catching up myself. from holiday | 15:05 |
| JayF | #topic OpenInfra Meetup June 6 2024 | 15:05 |
| JayF | It looks like this may be developing into a colocated Ironic meetup alongside the OpenInfra meetup. | 15:06 |
| JayF | It's probably too early to set anything in stone, but I know I'm working on getting travel approved to this event and I suggest others in the community do so as well. | 15:06 |
| TheJulia | Which event is that, for those which might not be fully aware of events schedule? | 15:06 |
| JayF | The meetup at CERN listed here | 15:07 |
| JayF | #link https://openinfra.dev/blog/openinfra-events-2024 | 15:07 |
| JayF | I'll note it's not showing up as a separate event on the events page (yet) | 15:07 |
| dtantsur | This is very unfortunate, it's hard to get an approval for something that is barely planned\ | 15:08 |
| JayF | Consider this an early warning system. | 15:08 |
| TheJulia | Near impossible for me, until there are public announcements | 15:08 |
| JayF | So basically you need it to be an official event with announcements if anyone RH is going to go, is what I'm hearing? | 15:09 |
| JayF | An unofficial/more under the radar meetup among cores/cores emeritus and maybe some users is off the table then? | 15:09 |
| dtantsur | Only as a private vacation in my case | 15:10 |
| JayF | ack | 15:10 |
| JayF | I'll email arne_wiebalck and see if we can make something more official out of it, I'm nervous to get too involved in planning until *my* travel is set in stone, too. | 15:10 |
| TheJulia | I can have the initial discussion, but without things to back it up, it becomes a little difficult to justify | 15:11 |
| arne_wiebalck | hi o/ | 15:11 |
| JayF | arne_wiebalck: tl;dr if we want people to come to a CERN meetup around Ironic, we'll probably have to make it something more formal. I think my travel approval is nearly complete; when that breaks free I'll email you if that's OK? | 15:12 |
| arne_wiebalck | JayF: sure ... you mean as part of the OpenStack meetup we plan here? | 15:13 |
| JayF | yeah | 15:13 |
| JayF | we talked about it in here a couple weeks ago IIRC | 15:13 |
| arne_wiebalck | with me? :-D | 15:14 |
| JayF | Yes? | 15:14 |
| arne_wiebalck | first day back at work, still staging memory back in :) | 15:14 |
| JayF | Well, my memory is bad to start with so now you have me wondering if I dreamed all this up :D | 15:14 |
| JayF | but I wouldn't have put it on the agenda based on a whim lol | 15:15 |
| arne_wiebalck | heh | 15:15 |
| arne_wiebalck | ok | 15:15 |
| JayF | Either way, we should figure out something or figure out NOT something :D | 15:15 |
| JayF | moving on for meeting purposes though | 15:15 |
| JayF | #topic Ironic CI Status | 15:15 |
| arne_wiebalck | ok, let;s take this offline | 15:15 |
| JayF | Any comments on Ironic CI Status? | 15:16 |
| JayF | I've not seen much about the current state, but not much is moving | 15:17 |
| JayF | I know TheJulia has a lot of interesting PRs up that I'm itching to review but haven't gotten to yet :) | 15:17 |
| TheJulia | CI seems to be okay for the most part, I have concerns about the grenade job, but largely related to the policy default change as we're getting an error we *shouldn't* be hitting | 15:18 |
| TheJulia | in other words, unrelated to default running state | 15:18 |
| JayF | If you want a second pair of eyes on that, the rest of my day is meeting-clear, just reach out. | 15:19 |
| JayF | I suspect we'll have more on CI as things pick back up; going to move on for now. | 15:19 |
| JayF | #topic Bug Deputy | 15:19 |
| JayF | Would anyone like to volunteer to be Bug Deputy for 1/8-1/15? | 15:19 |
| TheJulia | I am tempted to just hold a node because it suggests that the configuration is wrong or there is a bug in python-ironicclient. Which is unknown at the moment | 15:19 |
| JayF | ++ hold the node, lets figure it out especially if it could be a client bug | 15:20 |
| JayF | We can do another week of bug deputy by collective like we did over holiday; hopefully we'll get someone who can volunteer next week. | 15:22 |
| JayF | #topic RFE Review | 15:22 |
| JayF | > [Dmitry] https://bugs.launchpad.net/ironic/+bug/2046428 Move configdrive to an auxiliary table | 15:22 |
| JayF | #link https://bugs.launchpad.net/ironic/+bug/2046428 | 15:22 |
| JayF | This was already approved. I must have missed removing it from the Agenda. | 15:23 |
| JayF | #undo | 15:23 |
| opendevmeet | Removing item from minutes: #link https://bugs.launchpad.net/ironic/+bug/2046428 | 15:23 |
| TheJulia | request to hold the node made | 15:23 |
| JayF | #topic Open Discussion | 15:23 |
| JayF | this could go under RFE or Open Discussion, but I filed https://bugs.launchpad.net/ironic-python-agent/+bug/2047654 and https://review.opendev.org/q/topic:%22codespell%22+repo:openstack/ironic-python-agent over the holiday | 15:24 |
| TheJulia | I have one item, specifically as a result of the http boot interface work, I started looking at a more opportunistic test behavior | 15:24 |
| JayF | curious how folks would feel about integrating this across the repo | 15:24 |
| TheJulia | with regards to tempest | 15:24 |
| TheJulia | no objection to codespell | 15:25 |
| JayF | TheJulia: what's the test behavior | 15:26 |
| TheJulia | with regards to the tempest change I was talking about, I've posted https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/902171 which generally is not tied to a specific driver, so the same test can any interface in the valid hardware type list | 15:26 |
| JayF | I think it's just me and you so unlikely to get real progress on codespell consensus, but riccardo has it +2'd so maybe it's generally OK | 15:26 |
| JayF | TheJulia: yes, that's EXACTLY the thing I had in mind when we were talking during the PTG | 15:27 |
| TheJulia | Allowing the boot_interface to be changed as a result and triggering the test. The alternative is a bunch of static configs. The plus side is doing this should allow us to rip out some of the existing jobs | 15:27 |
| JayF | TheJulia: about how we don't need to dictate what interface runs for X when testing the interface for Y | 15:27 |
| TheJulia | it would move the interface's testing mainly to the standalone jobs too which are meant to be a bit more light weight | 15:28 |
| JayF | Is there some downside to this I'm not seeing? | 15:29 |
| TheJulia | Maybe naming :) | 15:30 |
| TheJulia | We will end up removing some of the scenario jobs where we have a tendancy to do more environmental type testing, but the reality is that we can have competing configurations work just fine under the same job at this point | 15:33 |
| TheJulia | To say "have a redfish and have an ipmi machine" is sort of out of scope of just the tests, but changing just the tests allows us to have better coverage and reduce the overall job count | 15:33 |
| JayF | that sounds pretty much excellent | 15:33 |
| JayF | I think we're pretty much to the end of the meeting, anything else for the record before I close it up? | 15:35 |
| JayF | #endmeeting | 15:37 |
| opendevmeet | Meeting ended Mon Jan 8 15:37:09 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:37 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-01-08-15.02.html | 15:37 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-01-08-15.02.txt | 15:37 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-01-08-15.02.log.html | 15:37 |
| JayF | whoa stealth dtantsur lol | 15:38 |
| JayF | landing the codespell in IPA | 15:38 |
| dtantsur | :) | 15:38 |
| JayF | you all want me to add that to the CI gate too? | 15:38 |
| dtantsur | I'm a bit on the fence about it | 15:38 |
| dtantsur | It can frustrate contributors.. | 15:38 |
| JayF | Yeah, I think it's useful tool to have at hand, and good to apply and have available to ask people to run against extra bad PRs | 15:39 |
| JayF | but I'm very OK with leaving it outta the gate for starters | 15:39 |
| dtantsur | Can be non-voting so that we see what is happening to it | 15:39 |
| JayF | only downside to that is it'd require a new job, versus just adding to the existing lint job | 15:39 |
| JayF | uses more resources, but that may not be a big deal | 15:39 |
| dtantsur | I suspect another tox job won't be a big problem | 15:40 |
| JayF | ack; I'll take that approach | 15:40 |
| JayF | will put the polish on the IPA change today, but will do the other repos over time | 15:40 |
| JayF | use it as an onboarding tool for adamcarthur5 + when I need a win after a hard day lol | 15:40 |
| dtantsur | :D | 15:41 |
| JayF | dtantsur: just noting, there's no guarantee we have an MLH fellow this cycle, but if we do, I'll likely set them loose on "Ironic bug dashboard improvements"; if you have any specific things on a wishlist please let me know | 15:45 |
| JayF | It's sorta buggy around some of the filtering now, so ensuring all that works as expected (maybe add unit tests? idk) would at least be a start | 15:46 |
| dtantsur | Nothing specific.. I'd love it transformed into something more immediately actionable, if you know what I mean. So that "what should I do with it" is not a question. | 15:46 |
| dtantsur | And yeah, polish filtering | 15:46 |
| JayF | and if we had someone with any UI chops at all "make it pretty" might not be a bad ask, either lol | 15:46 |
| dtantsur | yeah, absolutely! | 15:46 |
| * dtantsur is allergic to frontend work | 15:46 | |
| TheJulia | dtantsur: question for you, I've got an odd failure on https://5c6df17b3ed6fc20e575-67452faa46fddf50e0cf1e389c2c8597.ssl.cf2.rackcdn.com/902009/3/check/metalsmith-integration-glance-centos9-legacy/3e64d69/job-output.txt when metalsmith attempts to make an allocation. What is your first thought of what the issue could be (given it is on the second deployment of the job...) | 15:57 |
| TheJulia | err, no, first. It went through cleaning | 16:00 |
| dtantsur | yeah, figured that much | 16:00 |
| TheJulia | so in that case, it is an issue of access rights, the job is running without sufficient access to actually see the node, which makes sense depending on what OS_CLOUD env var is present | 16:02 |
| dtantsur | This is the query https://opendev.org/openstack/ironic/src/branch/master/ironic/conductor/allocations.py#L108-L118 | 16:02 |
| TheJulia | can't schedule what your user cannot see based upon policy defaults | 16:02 |
| dtantsur | so yeah, can be the 'project' filter | 16:02 |
| TheJulia | yup, it is not set on the node, so yeah, that makes sense | 16:04 |
| TheJulia | dtantsur: do you remember if the sdk hunts for parameters in order when looking at defaults? | 16:08 |
| dtantsur | which defaults? sorry, a bit unclear | 16:08 |
| TheJulia | credentials to talk to the "cloud" | 16:08 |
| TheJulia | what we would normally hint to which to use through using OS_CLOUD | 16:09 |
| dtantsur | I believe parameters in clouds.yaml are in a mapping => not ordered? | 16:09 |
| TheJulia | oh, I see it | 16:10 |
| TheJulia | playbooks/integration/centos9-integration.yaml | 16:10 |
| TheJulia | it is explicitly set to devstack-admin | 16:10 |
| TheJulia | So, I guess the path forward is *either* to make that var able to be set, *or* just set an owner upfront | 16:11 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Drop rootwrap support https://review.opendev.org/c/openstack/ironic/+/904836 | 17:02 |
| opendevreview | Julia Kreger proposed openstack/metalsmith master: CI: Ask ironic devstack to set node owner https://review.opendev.org/c/openstack/metalsmith/+/905012 | 17:06 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM: Change to enforced policy by default https://review.opendev.org/c/openstack/ironic/+/902009 | 17:10 |
| TheJulia | I *think* that will just do the trick | 17:12 |
| TheJulia | at least, for everything but grenade | 17:13 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Support several API and Inspector URLs https://review.opendev.org/c/openstack/ironic-python-agent/+/903999 | 17:18 |
| dtantsur | JayF: done ^^ | 17:18 |
| dtantsur | meanwhile, seeking opinions on the direction in https://review.opendev.org/c/openstack/ironic-python-agent/+/904026 vs a new boolean option (ipa-inspect=1 or something) | 17:20 |
| opendevreview | Merged openstack/ironic-python-agent master: [codespell] Fix spelling issues in IPA https://review.opendev.org/c/openstack/ironic-python-agent/+/904404 | 17:22 |
| opendevreview | Merged openstack/ironic-python-agent master: Add tox target and configuration for codespell https://review.opendev.org/c/openstack/ironic-python-agent/+/904405 | 17:22 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] Make inspection URL optional if the collectors are provided https://review.opendev.org/c/openstack/ironic-python-agent/+/904026 | 17:32 |
| JayF | dtantsur: might be worth hitting the list, for visibility to operators on that change ... at least from my perspective it seems like the best route is not adding ipa-inspect=1 and just using your patch mostly as suggested, but that has behavior change impacts that might be undesired for operators | 18:15 |
| JayF | dtantsur: and how things like K-A configure by default might be a good data point | 18:15 |
| opendevreview | Verification of a change to openstack/ironic-lib master failed: Drop lower-constraints.txt (again) https://review.opendev.org/c/openstack/ironic-lib/+/904993 | 18:33 |
| * TheJulia glares at CI | 18:33 | |
| opendevreview | Merged openstack/virtualbmc master: Handle any error from libvirt operations https://review.opendev.org/c/openstack/virtualbmc/+/835104 | 18:49 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM: Change to enforced policy by default https://review.opendev.org/c/openstack/ironic/+/902009 | 19:19 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Fix system scoped manageable node network failure https://review.opendev.org/c/openstack/ironic/+/905022 | 19:19 |
| TheJulia | well! looks like it was grenade being grenade setting a few unexpected environment variables | 19:21 |
| opendevreview | Winicius Allan Bezerra da Silva proposed openstack/sushy master: Allows System to access VirtualMedia in Sushy https://review.opendev.org/c/openstack/sushy/+/904463 | 21:32 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM: Change to enforced policy by default https://review.opendev.org/c/openstack/ironic/+/902009 | 22:05 |
| opendevreview | Julia Kreger proposed openstack/ironic master: RBAC: Fix allocation check https://review.opendev.org/c/openstack/ironic/+/905038 | 22:05 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!