| *** ianw has joined #openstack-ironic | 00:11 | |
| *** tosky has quit IRC | 01:02 | |
| *** ociuhandu has joined #openstack-ironic | 01:12 | |
| *** ociuhandu has quit IRC | 01:26 | |
| openstackgerrit | Rotan proposed openstack/ironic-inspector master: Fix the MismatchError https://review.opendev.org/c/openstack/ironic-inspector/+/780219 | 01:34 |
|---|---|---|
| *** zzzeek has quit IRC | 02:09 | |
| *** zzzeek has joined #openstack-ironic | 02:11 | |
| *** zzzeek has quit IRC | 02:36 | |
| *** zzzeek has joined #openstack-ironic | 02:39 | |
| *** rcernin has quit IRC | 02:50 | |
| *** rcernin has joined #openstack-ironic | 02:55 | |
| *** rcernin has quit IRC | 03:12 | |
| openstackgerrit | Rotan proposed openstack/ironic-inspector master: Fix the MismatchError https://review.opendev.org/c/openstack/ironic-inspector/+/780219 | 03:17 |
| *** ricolin has quit IRC | 03:26 | |
| *** rcernin has joined #openstack-ironic | 03:31 | |
| *** rcernin has quit IRC | 03:34 | |
| *** rcernin has joined #openstack-ironic | 03:34 | |
| *** ricolin has joined #openstack-ironic | 04:12 | |
| *** baha has joined #openstack-ironic | 04:16 | |
| *** ricolin has quit IRC | 04:33 | |
| *** mkrai has joined #openstack-ironic | 04:59 | |
| *** k_mouza has joined #openstack-ironic | 05:16 | |
| *** k_mouza has quit IRC | 05:21 | |
| *** jamesdenton has quit IRC | 05:38 | |
| *** jamesdenton has joined #openstack-ironic | 05:40 | |
| *** anuradha1904 has joined #openstack-ironic | 06:10 | |
| openstackgerrit | Rotan proposed openstack/ironic-inspector master: [WIP]Support AEP, sst at introspection data https://review.opendev.org/c/openstack/ironic-inspector/+/768138 | 06:38 |
| arne_wiebalck | Good morning, ironic! | 06:41 |
| stevebaker | good evening | 06:43 |
| *** ociuhandu has joined #openstack-ironic | 06:56 | |
| *** viks____ has joined #openstack-ironic | 07:06 | |
| *** k_mouza has joined #openstack-ironic | 07:12 | |
| *** k_mouza has quit IRC | 07:16 | |
| *** rcernin has quit IRC | 07:17 | |
| *** ociuhandu has quit IRC | 07:44 | |
| *** ociuhandu has joined #openstack-ironic | 07:46 | |
| *** ociuhandu has quit IRC | 07:51 | |
| *** ociuhandu has joined #openstack-ironic | 08:04 | |
| openstackgerrit | liuzhuangzhuang proposed openstack/ironic master: update grub file name for uefi boot mode https://review.opendev.org/c/openstack/ironic/+/780481 | 08:05 |
| *** jamesdenton has quit IRC | 08:13 | |
| *** jamesdenton has joined #openstack-ironic | 08:14 | |
| *** ociuhandu has quit IRC | 08:17 | |
| *** priteau has joined #openstack-ironic | 08:17 | |
| *** ociuhandu has joined #openstack-ironic | 08:18 | |
| *** rpittau|afk is now known as rpittau | 08:22 | |
| rpittau | good morning ironic! o/ | 08:22 |
| *** ociuhandu has quit IRC | 08:23 | |
| *** ociuhandu has joined #openstack-ironic | 08:48 | |
| *** tosky has joined #openstack-ironic | 08:53 | |
| *** lucasagomes has joined #openstack-ironic | 08:57 | |
| *** ociuhandu has quit IRC | 08:59 | |
| *** ociuhandu has joined #openstack-ironic | 08:59 | |
| *** derekh has joined #openstack-ironic | 09:06 | |
| *** k_mouza has joined #openstack-ironic | 09:12 | |
| *** k_mouza has quit IRC | 09:16 | |
| *** rcernin has joined #openstack-ironic | 09:36 | |
| *** mkrai has quit IRC | 09:39 | |
| *** mkrai has joined #openstack-ironic | 09:39 | |
| *** rcernin has quit IRC | 09:40 | |
| *** k_mouza has joined #openstack-ironic | 09:56 | |
| *** derekh has quit IRC | 09:57 | |
| *** derekh has joined #openstack-ironic | 09:57 | |
| *** derekh has quit IRC | 10:29 | |
| *** derekh has joined #openstack-ironic | 10:29 | |
| *** dtantsur|afk is now known as dtantsur | 10:31 | |
| dtantsur | morning ironic | 10:31 |
| rpittau | good morning dtantsur :) | 10:35 |
| arne_wiebalck | Preparing the upgrade to Ussuri, I ran into the fact that localhost is not allowed as a callback URL for the inspector in ironic.conf anymore. We have this set like this since ever (so that I do not remember why :-), but I was wondering why this is not an allowed config any longer? | 10:41 |
| openstackgerrit | Moshiur Rahman proposed openstack/ironic-python-agent-builder master: Fix: IPA image building with OpenSuse. https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/778726 | 10:41 |
| *** openstackgerrit has quit IRC | 10:43 | |
| *** k_mouza has quit IRC | 11:02 | |
| *** k_mouza_ has joined #openstack-ironic | 11:02 | |
| *** bfournie has quit IRC | 11:07 | |
| janders | good morning / afternoon Ironic o/ | 11:08 |
| janders | janders:bees 1:0 so far | 11:08 |
| dtantsur | arne_wiebalck: it's used in PXE configuration | 11:08 |
| dtantsur | Ussuri allowed ironic to drive PXE configuration for inspector | 11:09 |
| *** bfournie has joined #openstack-ironic | 11:21 | |
| rpittau | hey janders :) | 11:21 |
| rpittau | dtantsur: when you have time can you please check my reply to your comment in https://review.opendev.org/c/openstack/ironic-python-agent/+/779812 ? | 11:32 |
| dtantsur | rpittau: just read. I cannot agree with blocking stable/ussuri fixes on a publishing job. I agree that we need visibility, but we can achieve it via a stable job on IPA-builder. Adding another source of breakages to stable/ussuri doesn't sound good to me. | 11:34 |
| rpittau | dtantsur: ok, just to clarify, do you mean adding an "ussuri" job to ipa-builder ? | 11:35 |
| dtantsur | yep | 11:35 |
| dtantsur | which is the expected source of breakage, right? | 11:35 |
| rpittau | dtantsur: yes, alright, makes sense. We still need to publish a new tinyipa ramdisk for ussuri as the current one is broken, would a dummy patch be ok? Otherwise we need something to backport :) | 11:37 |
| rpittau | or mmmmm I could move the job from ipa to ipa-builder and we do both at the same time, kind of | 11:38 |
| iurygregory | good morning Ironic o/ | 11:40 |
| janders | hey iurygregory o/ | 11:41 |
| rpittau | hey iurygregory :) | 11:41 |
| *** jtomasek has joined #openstack-ironic | 12:03 | |
| dtantsur | rpittau: it's handy to keep the job non-voting for tracking purposes. otherwise a dummy change or a smart backport - both are fine :) | 12:15 |
| arne_wiebalck | dtantsur: Thanks! As you may remember we do not rely on Ironic for PXE, so excuse my ignorance, but why is a localhost callback URL an issue for this functionality? | 12:19 |
| * arne_wiebalck should probably also look into *why* we are doing this in our deployment | 12:20 | |
| dtantsur | well, ironic doesn't know that you don't use its PXE :) | 12:20 |
| arne_wiebalck | dtantsur: right, I am not arguing what we are doing is a good idea :) | 12:20 |
| dtantsur | you can set https://docs.openstack.org/ironic/latest/configuration/config.html#inspector.callback_endpoint_override to something non-localhost | 12:21 |
| arne_wiebalck | using the hostname works as well | 12:21 |
| dtantsur | then you can keep the inspector endpoint as localhost | 12:21 |
| dtantsur | yeah | 12:21 |
| arne_wiebalck | using 'localhost' with your config option does not work | 12:21 |
| arne_wiebalck | (if that is what you meant) | 12:22 |
| dtantsur | the new option must be NOT localhost | 12:22 |
| arne_wiebalck | right | 12:22 |
| dtantsur | it defaults to endpoint_override/endpoint from keystone | 12:22 |
| dtantsur | and thus causes ^^^ to be non-localhost too | 12:22 |
| dtantsur | but if you set it to something meaningful, you can keep using localhost for endpoint_override/in keystone | 12:22 |
| dtantsur | (this is IIRC, ussuri was a while ago) | 12:22 |
| arne_wiebalck | ok | 12:23 |
| arne_wiebalck | but what breaks in Ironic/PXE when you use localhost for this override? | 12:23 |
| arne_wiebalck | (just for interest) | 12:23 |
| arne_wiebalck | out of interest | 12:23 |
| arne_wiebalck | I mean 'localhost', not the localhost :) | 12:24 |
| arne_wiebalck | I *think* we do this since in our PXE we pass the host with the conductor/inspector to the IPA, and when the conductor then talks to some other inspector, things break ... but this config is pretty old, so I am not sure | 12:26 |
| arne_wiebalck | ooh, this is the URL used by the node to post things back to the inspector? | 12:28 |
| arne_wiebalck | which is used in the PXE config to be passed as the kernel param? | 12:29 |
| * arne_wiebalck begins to see why localhost makes no sense :) | 12:29 | |
| *** jtomasek has quit IRC | 12:38 | |
| *** jtomasek has joined #openstack-ironic | 12:40 | |
| TheJulia | good morning | 12:45 |
| janders | good morning TheJulia | 12:46 |
| *** derekh has quit IRC | 12:50 | |
| *** k_mouza_ has quit IRC | 12:50 | |
| *** ricolin_ has joined #openstack-ironic | 12:50 | |
| *** ricolin_ has quit IRC | 12:50 | |
| *** k_mouza has joined #openstack-ironic | 12:50 | |
| *** ricolin has joined #openstack-ironic | 12:51 | |
| *** derekh has joined #openstack-ironic | 12:51 | |
| *** mkrai has quit IRC | 12:54 | |
| *** zaneb has joined #openstack-ironic | 12:58 | |
| *** zaneb has quit IRC | 13:03 | |
| *** zaneb has joined #openstack-ironic | 13:03 | |
| dtantsur | morning TheJulia | 13:04 |
| dtantsur | arne_wiebalck: exactly :) | 13:04 |
| arne_wiebalck | dtantsur: and this param is/was also used for something else and then split into two config options? | 13:06 |
| arne_wiebalck | 13:06 | |
| dtantsur | arne_wiebalck: endpoint_override itself is also used for ironic to talk to inspector. localhost is fine for this use case. | 13:06 |
| arne_wiebalck | dtantsur: exactly, this is what we use it for | 13:06 |
| arne_wiebalck | dtantsur: and the new callback_endpoint_override param can be used to tweak the callback URL passed to the IPA and needs to be set to allow 'localhost' in endpoint_override? | 13:09 |
| dtantsur | arne_wiebalck: correct | 13:12 |
| *** rloo has joined #openstack-ironic | 13:12 | |
| arne_wiebalck | dtantsur: ok, thanks, all clear now! | 13:12 |
| *** jamesdenton has quit IRC | 13:14 | |
| *** jamesdenton has joined #openstack-ironic | 13:15 | |
| *** ricolin has quit IRC | 13:15 | |
| dtantsur | TheJulia: FYI "baremetal:deploy_template:update deprecated without deprecated_reason or deprecated_since. This will be an error in a future release" | 13:20 |
| dtantsur | (and many more) | 13:20 |
| dtantsur | related, "baremetal:events:post should not configure deprecated_reason or deprecated_since as these should be configured on the DeprecatedRule indicated by deprecated_rule. This will be an error in a future release" | 13:20 |
| dtantsur | dunno what it all means and where exactly deprecated_reason/since have to go | 13:21 |
| *** ociuhandu has quit IRC | 13:28 | |
| *** ociuhandu has joined #openstack-ironic | 13:29 | |
| janders | see you tomorrow Ironic o/ | 13:30 |
| dtantsur | o/ | 13:31 |
| *** tosin has joined #openstack-ironic | 13:31 | |
| rpittau | dtantsur: the unusual chain of changes is 780557 -> 780556 -> 780558 | 13:38 |
| *** ociuhandu has quit IRC | 13:39 | |
| *** ociuhandu has joined #openstack-ironic | 13:42 | |
| *** ricolin has joined #openstack-ironic | 13:45 | |
| TheJulia | hmm, there should be a reason for most already set | 13:46 |
| TheJulia | interesting | 13:46 |
| TheJulia | dtantsur: btw, I updated https://review.opendev.org/c/openstack/ironic/+/778340 as we discussed since anyone can technically be an owner of an allocation even those without baremetal | 13:49 |
| dtantsur | k will try to get to it | 13:49 |
| *** ociuhandu has quit IRC | 13:50 | |
| *** ociuhandu has joined #openstack-ironic | 13:57 | |
| rpittau | mmm gerritbot is on PTO ? | 13:57 |
| dtantsur | why not? it's early spring, the nature is waking up | 13:58 |
| TheJulia | heh | 13:59 |
| arne_wiebalck | more like hibernation: it is snowing over here :) | 14:01 |
| rpittau | :) | 14:01 |
| rpittau | do we have review jam today ? | 14:01 |
| *** ociuhandu has quit IRC | 14:02 | |
| TheJulia | yes! Anyone else want to join? | 14:02 |
| TheJulia | https://meetpad.opendev.org/ironic | 14:02 |
| *** ociuhandu has joined #openstack-ironic | 14:03 | |
| *** ociuhandu has quit IRC | 14:08 | |
| *** ociuhandu has joined #openstack-ironic | 14:08 | |
| *** mkrai has joined #openstack-ironic | 14:18 | |
| *** kaifeng has joined #openstack-ironic | 14:56 | |
| *** openstackgerrit has joined #openstack-ironic | 14:58 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: API to force manual cleaning without booting IPA https://review.opendev.org/c/openstack/ironic/+/769793 | 14:58 |
| openstackgerrit | Merged openstack/ironic-specs master: Pre-defined system hw configuration in single step https://review.opendev.org/c/openstack/ironic-specs/+/740721 | 14:58 |
| dtantsur | ajya: FYI ^^^ | 14:58 |
| TheJulia | I mentioned it on the review jam | 15:00 |
| TheJulia | #startmeeting ironic | 15:00 |
| TheJulia | o/ | 15:00 |
| openstack | Meeting started Mon Mar 15 15:00:19 2021 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| *** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
| openstack | The meeting name has been set to 'ironic' | 15:00 |
| rpittau | o/ | 15:00 |
| rpioso | \o | 15:00 |
| kaifeng | o/ | 15:00 |
| ajya | o/ | 15:00 |
| ajya | dtantsur: ack | 15:00 |
| mraineri | o/ | 15:00 |
| TheJulia | Good morning everyone! And happy beginnings to Daylight Savings Time season! | 15:00 |
| bdodd | o/ | 15:00 |
| rloo | o/ | 15:00 |
| erbarr | o/ | 15:00 |
| TheJulia | Our agenda is on the wiki | 15:01 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
| arne_wiebalck | o/ | 15:01 |
| dtantsur | o/ | 15:01 |
| TheJulia | #topic Announcements / Reminders | 15:01 |
| *** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)" | 15:01 | |
| TheJulia | #info This week is R-4 in the Wallaby release schedule. | 15:01 |
| TheJulia | #info Projects which branch for RC1 will branch next week during R-3. | 15:02 |
| TheJulia | #info Ironic will likely be able to hold off on releasing until R-2, but devstack being changed may force us to release early that week. | 15:02 |
| TheJulia | #info *final* Wallaby releases will be the week of R-1 for all projects. | 15:03 |
| TheJulia | #info PTG is April 19th to 23rd. | 15:03 |
| TheJulia | tl;dr - Please register and add thoughts/ideas to the etherpad. | 15:03 |
| TheJulia | #link https://april2021-ptg.eventbrite.com/ | 15:03 |
| TheJulia | #link https://etherpad.opendev.org/p/ironic-xena-ptg | 15:04 |
| openstackgerrit | Moshiur Rahman proposed openstack/ironic-python-agent-builder master: Fix: IPA image building with OpenSuse. https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/778726 | 15:04 |
| TheJulia | I've signed us up for 4 days with a 1 hour session on friday to be informal social discussion. Also, for the first few days of the PTG, I've also selected an APAC friendlier time window for cross-over of discussions. | 15:05 |
| * dtantsur likes the "bring your cats" part | 15:05 | |
| ajya | TheJulia: can you tell more about changes to devstack, what's the impact? | 15:05 |
| TheJulia | ajya: Devstack has branch setting changes that will go into effect or possibly breaking changes as soon as traditional projects are branched. We've seen it take upwards of a few weeks, and we've also seen them break us pretty much immediately. | 15:05 |
| dtantsur | ajya: it's more about two things happening in R-2: requirements branching (and switching master to xena) and grenade testing Wallaby->master. | 15:05 |
| TheJulia | We just need to be cognizant of it because often it forces us to release as-is. | 15:06 |
| TheJulia | dtantsur++ | 15:06 |
| ajya | thanks | 15:06 |
| TheJulia | Does anyone have anything else to announce this week? Hopefully everyone will be taking a little time off soon? | 15:06 |
| *** jtomasek has quit IRC | 15:07 | |
| dtantsur | August :) (but I've just had a bit) | 15:07 |
| TheJulia | dtantsur: I mean, one can bring dogs. Or iguanas. Or stuffed animals. It is all good :) | 15:08 |
| TheJulia | A month off sounds glorious | 15:08 |
| TheJulia | We had no action items last week, so I think we can proceed to reviewing subteam status reports | 15:08 |
| dtantsur | I have a Pusheen! | 15:09 |
| TheJulia | \o/ | 15:09 |
| rpittau | I'll bring my Cthulhu plush then! | 15:09 |
| TheJulia | Last call for any other announcements or reminders? | 15:09 |
| TheJulia | Cthulhu, always a classic! | 15:10 |
| TheJulia | #topic Review subteam status reports | 15:10 |
| *** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)" | 15:10 | |
| TheJulia | #link https://etherpad.opendev.org/p/IronicWhiteBoard | 15:10 |
| TheJulia | Starting around line 293 | 15:10 |
| TheJulia | bdodd: thank you for following up on that | 15:11 |
| TheJulia | well, my question w/r/t redfish raid | 15:11 |
| bdodd | TheJulia: Aija is the one that investigated that | 15:13 |
| * iurygregory is late o/ | 15:13 | |
| TheJulia | bdodd: well, thank you ajya! | 15:13 |
| TheJulia | iurygregory: :) | 15:13 |
| TheJulia | ajya: rpioso: Looks like config molds requires an update, dated 3/8 | 15:13 |
| TheJulia | rpittau: it *does* look like the anaaconda interface is going to slip | 15:14 |
| rpittau | yep | 15:14 |
| ajya | TheJulia: added | 15:15 |
| TheJulia | rpittau: I mean, we could request ffu, but I talked to JayF and zer0c00l on ?friday? and it seemed like they wanted to get everything up with CI so that means next cycle, at least to me | 15:15 |
| rpittau | I don't think we should rush it to be honest | 15:16 |
| rloo | i agree ++ | 15:16 |
| TheJulia | Yeah, I'd still like to see it but understand | 15:16 |
| TheJulia | I think we're good to proceed? | 15:17 |
| dtantsur | ++ | 15:17 |
| rpittau | yeah | 15:17 |
| zer0c00l | TheJulia: yes please | 15:18 |
| zer0c00l | https://review.opendev.org/c/openstack/requirements/+/780397 | 15:18 |
| TheJulia | #topic Deciding on priorities for the coming week | 15:18 |
| *** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:18 | |
| openstackgerrit | Moshiur Rahman proposed openstack/ironic-python-agent-builder master: Fix: IPA image building with OpenSuse. https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/778726 | 15:18 |
| zer0c00l | Btw i have to get a FFE https://review.opendev.org/c/openstack/requirements/+/780397 | 15:18 |
| TheJulia | zer0c00l: I saw | 15:18 |
| zer0c00l | No idea how to do that, any pointers /links would be helpful! | 15:19 |
| zer0c00l | thanks. | 15:19 |
| TheJulia | zer0c00l: if you want to get it into this release, then I have to send the email and justify it | 15:19 |
| dtantsur | either wait for the branches to happen or ^^^ | 15:19 |
| TheJulia | or that | 15:19 |
| dtantsur | * an email to openstack-discuss with [requirements] in it | 15:19 |
| dtantsur | but given the state of the ironic patch, chances are high you'll hear "no" | 15:20 |
| zer0c00l | okay. We definitely want to get it done with in this cycle into wallaby | 15:20 |
| zer0c00l | i see. | 15:20 |
| zer0c00l | i mean if we can in wallaby, if not next cycle. | 15:21 |
| TheJulia | If xena is okay, then I think that is fine to do at this point | 15:21 |
| zer0c00l | TheJulia: ack. When is the wallaby code freeze? | 15:22 |
| zer0c00l | The config drive feature can be shipped in X release | 15:22 |
| TheJulia | I see it as an early Xena item since the patches are close, but yeah, the FFE + CI need means Wallaby is very unlikely | 15:22 |
| zer0c00l | ok. | 15:22 |
| TheJulia | zer0c00l: Basically 2 weeks until we *have* to release | 15:22 |
| zer0c00l | oh wow | 15:22 |
| TheJulia | yeah | 15:22 |
| zer0c00l | yeah no! | 15:22 |
| TheJulia | heh, okay | 15:22 |
| dtantsur | we're actually past feature frreeze | 15:22 |
| dtantsur | we just don't obey it in ironic | 15:22 |
| TheJulia | ++ | 15:22 |
| zer0c00l | :) | 15:23 |
| TheJulia | So, the deciding on priorites for the week is awkward now, since we have the etherpad and the tagged items in the list | 15:23 |
| dtantsur | I think we only need to use the etherpad for incoming items | 15:24 |
| TheJulia | ++ | 15:24 |
| TheJulia | I have no objetion so I'm adding tags | 15:24 |
| TheJulia | Does anyone have anything else they would like to add to the list for this week? | 15:25 |
| TheJulia | #link https://review.opendev.org/q/hashtag:%2522ironic-week-prio%2522+status:open | 15:25 |
| dtantsur | we may remove the indicators until we have a good discussion (PTG?) | 15:26 |
| TheJulia | sounds good to me | 15:26 |
| TheJulia | removing | 15:26 |
| TheJulia | dtantsur: based upon your last comment, I think you may want to remove your -2 from https://review.opendev.org/c/openstack/ironic/+/779418 | 15:28 |
| rpittau | can we please add the tinycore 12 patches plus the final fixes for ussuri tinyipa ? | 15:28 |
| dtantsur | TheJulia: I would like it to move away from the dangerous approach first | 15:28 |
| TheJulia | rpittau: will this actually fix ussuri gates? | 15:29 |
| rpittau | TheJulia: yes | 15:29 |
| TheJulia | add the tag then | 15:29 |
| rpittau | ok | 15:29 |
| TheJulia | dtantsur: ack | 15:29 |
| TheJulia | Looks like we're good to proceed? | 15:29 |
| rpittau | let's | 15:31 |
| * TheJulia hears crickets | 15:31 | |
| * dtantsur has added indicators on the PTG etherpad | 15:31 | |
| TheJulia | dtantsur: thanks! | 15:31 |
| TheJulia | #topic Discussion | 15:31 |
| *** openstack changes topic to "Discussion (Meeting topic: ironic)" | 15:31 | |
| TheJulia | This week, mraineri brings us a topic regarding the Redfish Forum. | 15:31 |
| TheJulia | #link https://redfishforum.com/ | 15:32 |
| TheJulia | mraineri: The floor is yours | 15:32 |
| mraineri | I'd like to social the Redfish public forum found at https://redfishforum.com/ | 15:32 |
| mraineri | The forum allows anyone to post questions, comments, etc and get direct feedback from the authors of the standard | 15:32 |
| dtantsur | awesome! thanks mraineri | 15:33 |
| mraineri | So, while we do encourage folks to join the DMTF and participate directly, we understand this isn't always possible | 15:33 |
| mraineri | Many of the topics involve things like feature requests, and we do take this sorts of requests and drive them back into Redfish to be standardized | 15:33 |
| TheJulia | It this someone related to the whole thing between OpenBMC developers and the DMTF on twitter? | 15:33 |
| TheJulia | s/someone/something/ | 15:33 |
| mraineri | But we also have topics to help folks write client software as well | 15:33 |
| TheJulia | or /somewhat | 15:33 |
| mraineri | Somewhat; I understand there's a lot of hesitancy around standards in general, but from a Redfish perspective, we're trying to be more open with what we do and how we interact with the rest of the world | 15:34 |
| TheJulia | Sounds good! | 15:34 |
| mraineri | And at the same time, one of the gaps we have within the DMTF is the major participants are on the service side of things; we'd really like more client involvement | 15:35 |
| dtantsur | This has, indeed, been a problem for a long time | 15:35 |
| * dtantsur has questions to ask | 15:35 | |
| mraineri | Yes, please open the flood gates on your question :) | 15:36 |
| TheJulia | dtantsur: in meeting questions, or outside meeting questions :) | 15:36 |
| dtantsur | like why on earth is virtual media and console bound to Managers? | 15:36 |
| TheJulia | or Redfish Forum questions? | 15:36 |
| TheJulia | oh noes | 15:36 |
| mraineri | Myself and a few others review the threads daily usually | 15:36 |
| dtantsur | to redfish people | 15:36 |
| mraineri | dtantsur: we actually moved it recently! | 15:36 |
| dtantsur | \o/ | 15:36 |
| * dtantsur is afraid to ask where though :D | 15:36 | |
| mraineri | Yeah, it took a while, but no one really gave us direct feedback on that for a long time | 15:36 |
| TheJulia | oh no, compatability changes inbound at some point | 15:36 |
| TheJulia | Are we good to proceed? | 15:37 |
| mraineri | Absolutely, which is why we leave breadcrumbs with deprecation text | 15:37 |
| mraineri | And we can help you with how to walk through those types of scenarios to dynamically handle older and newer methods | 15:37 |
| TheJulia | mraineri: well, even then, we've seen how some implement their servers. A schism of a hard break is kind of common in my experience. | 15:37 |
| mraineri | True | 15:38 |
| TheJulia | Anyway, seems like we can move on to the Baremetal SIG | 15:38 |
| dtantsur | is there a thread on this topic on redfishforum? | 15:38 |
| mraineri | But that's also part of what we're there for on the forum; okay, there's a better way of doing something now, but how do I handle existing stuff? | 15:38 |
| mraineri | Specifically for virtual media, no | 15:38 |
| mraineri | But if you ask, I can certainly respond there and make the guidance available to everyone | 15:38 |
| TheJulia | Awesome | 15:39 |
| mraineri | That's all there is to it; if you're trying to do something and you're stuck, feel free to make a post and ask for help | 15:39 |
| rpioso | mraineri: Is it indexed by search engines? | 15:40 |
| mraineri | We'll either guide you down the path we've already outlined in the schema/spec, or we might also scratch our heads and make changes to the standard | 15:40 |
| mraineri | rpioso: I think so | 15:40 |
| TheJulia | As long as it doesn't have a evil robots.txt it should get indexed | 15:40 |
| TheJulia | s/a/an/ | 15:40 |
| *** tosin has quit IRC | 15:40 | |
| openstackgerrit | Merged openstack/ironic master: Update Redfish RAID release note https://review.opendev.org/c/openstack/ironic/+/779552 | 15:41 |
| TheJulia | Now, most relevent things getting linked to, that is a whole different headache with search engines. :) | 15:41 |
| TheJulia | #topic Baremetal SIG | 15:41 |
| *** openstack changes topic to "Baremetal SIG (Meeting topic: ironic)" | 15:41 | |
| TheJulia | I believe the next SIG meeting will be April 12th @ 2PM UTC | 15:41 |
| TheJulia | The topic will be Secure RBAC in Ironic | 15:41 |
| TheJulia | arne_wiebalck: anything else to add? | 15:41 |
| arne_wiebalck | We had iurygregory on the Ironic Prometheus Exporter last week (thanks again!), the video is ready and about to be published. | 15:41 |
| iurygregory | \o/ | 15:42 |
| arne_wiebalck | And we have added some topics to the PTG agenda. | 15:42 |
| TheJulia | \o/ | 15:42 |
| arne_wiebalck | Very few operators around, though :) | 15:42 |
| TheJulia | Hmm... we need to work on that | 15:43 |
| arne_wiebalck | Anyway, that's it I think. | 15:43 |
| TheJulia | Okay then. Well, we have no RFE's listed, so I think that leads us to Open Discussion | 15:43 |
| TheJulia | #topic Open Discussion | 15:43 |
| *** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:43 | |
| TheJulia | Anyone have anything to discuss this week? | 15:44 |
| TheJulia | The list of topics is getting longer for the PTG | 15:45 |
| dtantsur | yep | 15:45 |
| TheJulia | Some really good topics as well | 15:46 |
| TheJulia | Well, if nobody has anything for Open Discussion, I guess it is time to call this meeting a wrap | 15:49 |
| TheJulia | Thanks everyone! | 15:49 |
| TheJulia | Have a wonderful week! | 15:50 |
| TheJulia | #endmeeting | 15:50 |
| *** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-discuss mailing list." | 15:50 | |
| openstack | Meeting ended Mon Mar 15 15:50:08 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:50 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-03-15-15.00.html | 15:50 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-03-15-15.00.txt | 15:50 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-03-15-15.00.log.html | 15:50 |
| rpittau | thanks! | 15:50 |
| *** mkrai has quit IRC | 16:04 | |
| TheJulia | Hi, Wind storm. Please go away. I don't want a migraine today | 16:07 |
| *** another_larsks is now known as larsks | 16:07 | |
| * dtantsur sees https://review.opendev.org/c/openstack/ironic/+/778947 and cries | 16:09 | |
| TheJulia | dtantsur: it is explicitly what was agreed upon | 16:10 |
| TheJulia | an informational only signaling of the capability | 16:10 |
| dtantsur | TheJulia: it's an absolutely correct change | 16:10 |
| TheJulia | otherwise we'll also won't be able to make decisions in tempest :( | 16:10 |
| dtantsur | I'm crying because I've just finished the disable_ramdisk patch, and now a merge conflict is looming :) | 16:10 |
| TheJulia | oh noes | 16:10 |
| TheJulia | :( | 16:10 |
| TheJulia | well, whatever merges first, it is not a big deal | 16:11 |
| TheJulia | 778340 has to merge first so if it becomes 1.71, that is not a big deal | 16:11 |
| dtantsur | TheJulia: looking at https://review.opendev.org/c/openstack/ironic/+/778340/15/ironic/api/controllers/v1/allocation.py#229 | 16:15 |
| dtantsur | it has node_ident, node_uuid and parent_node | 16:15 |
| dtantsur | at least the first two have to be redundant.. | 16:15 |
| dtantsur | I suspect it's node_uuid. I also suspect it's not a blocking issue. | 16:16 |
| TheJulia | one is a user supplied search | 16:16 |
| TheJulia | the other is an override of the subresource being accessed | 16:17 |
| TheJulia | The third is the union of which should be used :\ | 16:17 |
| TheJulia | its because the same method is used by both controllers :( | 16:17 |
| dtantsur | TheJulia: parent_node_ident can be a name, right? | 16:17 |
| dtantsur | I don't think you account for that | 16:18 |
| TheJulia | I think it gets looked up in advance | 16:18 |
| TheJulia | by the parent controller | 16:18 |
| dtantsur | doesn't seem so | 16:19 |
| TheJulia | ugh | 16:19 |
| TheJulia | UGH | 16:20 |
| *** hoonetorg has quit IRC | 16:20 | |
| dtantsur | https://opendev.org/openstack/ironic/src/branch/master/ironic/api/controllers/v1/node.py#L1653 | 16:20 |
| TheJulia | the example is with uuid, but i get your point | 16:20 |
| dtantsur | ah, this is actually okay, but the code is confusing | 16:22 |
| TheJulia | if it was separate helpers doing similar things but distinctly different, it would be easier | 16:23 |
| dtantsur | you overwrite node_uuid on line 198, so there is no need to set it earlier | 16:24 |
| dtantsur | so I think the patch is fine, just confusing | 16:24 |
| TheJulia | yeah | 16:24 |
| TheJulia | It is also a parameter that can be passed in on another controller entry path | 16:25 |
| TheJulia | in that case, it is just overriden for the filter side of it later | 16:25 |
| * TheJulia did not want a migraine today | 16:30 | |
| dtantsur | speaking of migraines, I think we've regressed and started double-JSON-encoding error messages again after the WSME transition | 16:31 |
| openstackgerrit | Richard G. Pioso proposed x/sushy-oem-idrac master: Add resources to put iDRAC in known good state https://review.opendev.org/c/x/sushy-oem-idrac/+/721596 | 16:31 |
| TheJulia | That is a hangover with Whiskey and a migrane all at once | 16:32 |
| dtantsur | whiskey++ migraine-- | 16:32 |
| dtantsur | TheJulia: could you remind me, what are the rights of a member with regards to deployment? | 16:33 |
| TheJulia | They can change provision states and move a machine through the workflow | 16:34 |
| dtantsur | but not unprovision? I'm trying to understand why allocation deletion is forbidden for them. | 16:34 |
| TheJulia | unprovision as well... I think | 16:34 |
| TheJulia | well, different classes of it | 16:34 |
| * TheJulia looks | 16:34 | |
| dtantsur | same question re lessee admins | 16:35 |
| TheJulia | lesee admins only | 16:35 |
| dtantsur | allocation deletion logically belongs in the same basket as unprovisioning | 16:35 |
| dtantsur | does it mean that lessee members cannot unprovision too? | 16:35 |
| TheJulia | but only the creator of the allocaiton can see the allocation | 16:35 |
| TheJulia | lessee members cannot | 16:35 |
| TheJulia | lessee, if htey didn't create the allocation, won't even be able to see the allocation | 16:35 |
| dtantsur | right | 16:36 |
| TheJulia | think of it purely as only whoever creates it is the owner | 16:36 |
| dtantsur | I see a test called lessee_member_cannot_delete_their_allocation: | 16:36 |
| dtantsur | this strikes me as incorrect | 16:36 |
| dtantsur | especially since lessee_member_can_patch_allocation | 16:36 |
| TheJulia | give me a moment | 16:36 |
| TheJulia | so | 16:36 |
| TheJulia | members are explicitly expected to be able to read/update things, but not create or delete | 16:37 |
| dtantsur | allocation deletion ~= unprovisioning | 16:37 |
| dtantsur | allocation is not a physical concept, this rule does not apply to it | 16:37 |
| TheJulia | so for a node itself it is SYSTEM_OR_OWNER_MEMBER_AND_LESSEE_ADMIN | 16:37 |
| TheJulia | owner member can move it around, only lessee admin | 16:37 |
| dtantsur | imagine we finish deployment API, I hope you will agree that deployment deletion == unprovisioning? | 16:37 |
| TheJulia | so if a owner also creates an allocation and they are a member, they are stuck | 16:38 |
| TheJulia | but there is no way to delineate the two on the allocation side | 16:38 |
| TheJulia | short of going back into the node and checking access based upon that if allocated | 16:38 |
| dtantsur | lessee_member_cannot_change_provision_state okay, so members are fine | 16:39 |
| TheJulia | lessees specifically because they don't have any elevated node rights | 16:39 |
| TheJulia | only admins do really | 16:40 |
| TheJulia | and that is tenant admins | 16:40 |
| dtantsur | I don't see a test for it so: if I'm a lessee member, can I create an allocation? | 16:41 |
| TheJulia | hmmmm | 16:41 |
| TheJulia | so | 16:41 |
| TheJulia | so, decouple the lessee concept | 16:42 |
| TheJulia | There is no allocation lessee, nor consultation of lesses in nodes | 16:42 |
| TheJulia | except on visibility as a subnode resource | 16:42 |
| TheJulia | but | 16:42 |
| TheJulia | this says all system members, regardless can create an allocation | 16:42 |
| TheJulia | err | 16:43 |
| TheJulia | wait | 16:43 |
| TheJulia | that is system | 16:43 |
| dtantsur | lessee_admin_can_change_provision_state | 16:43 |
| * TheJulia pull sup the patch | 16:43 | |
| dtantsur | which means they should be able to create allocations.. | 16:43 |
| TheJulia | wait | 16:43 |
| TheJulia | That is system scoped | 16:43 |
| TheJulia | not project scoped | 16:43 |
| TheJulia | so allocation creator, which is system members, or any role scoped admin | 16:44 |
| TheJulia | Since we don't have a node, it is open to all project and their admins only | 16:44 |
| TheJulia | again, no lessee concept for any of this to leverage for creation/deletion | 16:45 |
| dtantsur | What I'm trying to say is: | 16:45 |
| dtantsur | mm, scratch it | 16:46 |
| TheJulia | heh | 16:46 |
| TheJulia | it is not simple since the use model is different | 16:46 |
| dtantsur | I guess the problem is with the confusing name lessee_member_cannot_delete_their_allocation | 16:46 |
| dtantsur | they should not have "their allocations" | 16:46 |
| *** jamesdenton has quit IRC | 16:46 | |
| TheJulia | bingo | 16:46 |
| dtantsur | although... they may be assigned an allocation by an admin | 16:46 |
| TheJulia | yes | 16:46 |
| dtantsur | but they cannot unprovision, so it's fine | 16:47 |
| *** jamesdenton has joined #openstack-ironic | 16:47 | |
| TheJulia | correct | 16:47 |
| openstackgerrit | Merged x/sushy-oem-idrac master: Add resources to put iDRAC in known good state https://review.opendev.org/c/x/sushy-oem-idrac/+/721596 | 16:47 |
| dtantsur | owner_member_cannot_delete_their_allocation | 16:48 |
| dtantsur | okay, this is probably wrong | 16:48 |
| dtantsur | TheJulia: ^^^ owner members can provision/unprovision, right? | 16:48 |
| TheJulia | in test_allocation.py? | 16:48 |
| TheJulia | or the yaml? | 16:49 |
| dtantsur | I'm looking at test_rbac_project_scoped.yaml | 16:49 |
| dtantsur | line 2319 | 16:49 |
| TheJulia | looking | 16:49 |
| dtantsur | (much lines wow wow) | 16:49 |
| TheJulia | hmm | 16:49 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Trivial: add a missing argument to an exception https://review.opendev.org/c/openstack/ironic/+/780270 | 16:49 |
| * TheJulia keeps looking | 16:49 | |
| TheJulia | so the issue here is likely I can't tell a node's owner apart from an allocation owner | 16:50 |
| TheJulia | short of doing extra db queries | 16:50 |
| TheJulia | so the allocation *is* in the member's project | 16:51 |
| TheJulia | ironic/common/policy.py restricts deleting to allocation_admin which points up to line 127 | 16:52 |
| TheJulia | which is defined as system scoped members, or an admin in the project which created the allocation. | 16:52 |
| dtantsur | just change to ALLOCATION_MEMBER? | 16:52 |
| TheJulia | I guess we could, but that also allows any member of the project to nuke the allocation from high orbit | 16:53 |
| dtantsur | TheJulia: they can already do that by unprovisioning, no? | 16:53 |
| TheJulia | only if they are indepedendently defined as an owner of the node | 16:54 |
| TheJulia | so the gray area is an owner of a node, who also uses an allocation | 16:54 |
| TheJulia | or we could just allow allocation members to delete | 16:54 |
| dtantsur | so, the conundrum is that allocation.owner may be node.lessee, not node.owner? | 16:54 |
| TheJulia | may be easier in the grand scheme of things | 16:54 |
| TheJulia | correct | 16:54 |
| TheJulia | or not at all | 16:55 |
| TheJulia | which means they will have an emtpy result set potentially and the allocation failed for very good reasons | 16:55 |
| TheJulia | can't allocate nodes they don't have access to | 16:55 |
| dtantsur | yeah, right | 16:55 |
| dtantsur | unless some admin creates a very interesting allocation | 16:55 |
| TheJulia | yup | 16:56 |
| TheJulia | and in such a case, I think we should label them as evil admins | 16:56 |
| dtantsur | ++ | 16:56 |
| TheJulia | Surely they were playing Voltare "When your evil" | 16:56 |
| dtantsur | heh | 16:57 |
| TheJulia | your likely coming to understand why I've asked for people to carefully review this one :) | 16:57 |
| arne_wiebalck | Hmm ... an ussuri conductor should be ok with a train IPA, no ? | 16:57 |
| dtantsur | TheJulia: yeah.. next question: why is ALLOCATION_CREATOR restricted to project owners? is it because we're handling everything else in Python? | 16:59 |
| dtantsur | arne_wiebalck: give or take tokens.. actually, I think yes | 16:59 |
| TheJulia | dtantsur: you mean line 139 in policy.py? | 16:59 |
| arne_wiebalck | dtantsur: does not seem to be related to tokens, more to the partition UUID and bootloader installation | 17:00 |
| dtantsur | TheJulia: yep | 17:00 |
| TheJulia | yeah, any project member with an admin role | 17:00 |
| dtantsur | arne_wiebalck: there was something like that, I thought I fixed it.. | 17:00 |
| arne_wiebalck | dtantsur: it works with s/w RAID, but not without s/w RAID | 17:00 |
| dtantsur | TheJulia: but set_provision_state is allowed for project members? | 17:01 |
| TheJulia | member not as in the role, but they are a user in the project with admin rights granted, which also gives them the member and reader roles | 17:01 |
| arne_wiebalck | dtantsur: I will have a look ... | 17:01 |
| TheJulia | System members, Owner members, not lessee members | 17:01 |
| dtantsur | TheJulia: right, so owner members must be able to create allocations too? | 17:01 |
| *** lucasagomes has quit IRC | 17:02 | |
| TheJulia | dtantsur: so any user that is not an explicit reader only, can create an allocation then? | 17:02 |
| TheJulia | is that what your seeking? | 17:02 |
| dtantsur | TheJulia: allocation creation accompanies deployment, deletion - undeployment | 17:03 |
| dtantsur | these actions must not have different ACLs | 17:03 |
| TheJulia | they have to by the data model design | 17:03 |
| TheJulia | I mean, the result is basically, anyone can create an allocation, which I guess I can be okay with since it will fail if they don't own baremetal or their project doesn't | 17:03 |
| dtantsur | if I'm a member of a project that owns some nodes, I need to be able to provision, which means I need to be able to allocate | 17:03 |
| TheJulia | only if I use the allocation api to begin with | 17:04 |
| dtantsur | (at some point I want to stop explicit patching of nodes for deployment) | 17:04 |
| dtantsur | right, this ^^ | 17:04 |
| dtantsur | in a concurrent environment you need a sort of locking | 17:04 |
| TheJulia | that is going to have to be a ptg topic | 17:04 |
| rpittau | good night! o/ | 17:04 |
| *** rpittau is now known as rpittau|afk | 17:04 | |
| dtantsur | allocation API was supposed to be a step away from patching nodes for deployment | 17:04 |
| TheJulia | yes, the instance_uuid is not enough | 17:04 |
| TheJulia | but that is a whole different conundrum | 17:05 |
| TheJulia | I can make it members can create, but ultimately then the argument will be why can't lessee members unprovision machines | 17:05 |
| dtantsur | so, my project owns some nodes (node.owner == my_user.project_id). I have a member role. I can provision, right? | 17:06 |
| TheJulia | Owner access is intentionally elevated because they are the "managers of the baremetal" | 17:06 |
| TheJulia | yes, that would be possible | 17:06 |
| dtantsur | okay, then I need to be able to do allocations too | 17:06 |
| dtantsur | (and delete them) | 17:06 |
| dtantsur | lessees.... my head is going to explode | 17:07 |
| TheJulia | Welcome to the club! Population 2 | 17:07 |
| dtantsur | I assumed the goal of the lessee field was to allow someone to deploy/undeploy, but nothing else | 17:07 |
| dtantsur | as we have now, being a member in a project that leases nodes gives you not much | 17:08 |
| TheJulia | basic power options, see it, admins there deploy a thing | 17:08 |
| dtantsur | (which is fine in my book, but dunno) | 17:08 |
| dtantsur | okay, right | 17:08 |
| TheJulia | I think in the end, we may want nova to grant lessee rights to users, but that will be a big giant knob on the virt driver | 17:08 |
| TheJulia | #notatodaything | 17:09 |
| *** anuradha1904 has quit IRC | 17:09 | |
| dtantsur | then they'll need to undeploy, no? | 17:09 |
| dtantsur | okay, let's leave it there for now | 17:09 |
| TheJulia | yeah, they should be able to if they use nova to peel the node back | 17:09 |
| dtantsur | ah, right. right. okay, I'm fine with lessees | 17:09 |
| TheJulia | I think that is why it will need a knob since some (?most?) deployments may not want it | 17:10 |
| dtantsur | it seems like we need to pass our status all the way down to the allocation handling :( | 17:11 |
| dtantsur | which is.. gross | 17:11 |
| TheJulia | I only used the project id so stuff got filtered appropriately down to the existing relationships | 17:12 |
| dtantsur | what's the downside of allowing lessee members to {de,}allocate as a side effect? | 17:12 |
| TheJulia | and things like nova will have elevated rights anyway | 17:12 |
| dtantsur | a (very sad) alternative is to add a new field to allocations whether to permit lessees | 17:13 |
| TheJulia | I don't think there is much of a side effect, although common members won't be able to deploy ndoes... if they use the wrong project I could see this being a headache, but they used the wrong project then | 17:15 |
| dtantsur | TheJulia: okay, I'm about to post all the comments. Would you prefer to update the patch or create a follow-up? | 17:17 |
| TheJulia | I can do a follow-up | 17:17 |
| dtantsur | the most important comments are what we just discussed, I think everything else are nits | 17:17 |
| TheJulia | most of these I've been doing follow-up since there are dependent patches | 17:17 |
| TheJulia | okay cool | 17:18 |
| TheJulia | I may not work on it with the migraine | 17:18 |
| dtantsur | okay, I've approved that | 17:18 |
| TheJulia | at least this afternoon | 17:18 |
| dtantsur | yeah, no worries, let's just fit it into Wallaby | 17:18 |
| openstackgerrit | Merged openstack/ironic-python-agent stable/victoria: Prepare to use tinycore 12 https://review.opendev.org/c/openstack/ironic-python-agent/+/780460 | 17:18 |
| TheJulia | ++ | 17:18 |
| dtantsur | I did not comment, but the docs should likely be updated too | 17:19 |
| TheJulia | to reflect what we just discussed? | 17:20 |
| dtantsur | yep | 17:20 |
| dtantsur | I didn't check, but there was a doc addition on allocations | 17:21 |
| *** k_mouza has quit IRC | 17:21 | |
| TheJulia | yeah, it was highlighting the different behavior | 17:21 |
| TheJulia | so I think it does need a minor change then | 17:21 |
| *** k_mouza has joined #openstack-ironic | 17:21 | |
| TheJulia | and likely notation of a caveat | 17:21 |
| dtantsur | https://review.opendev.org/c/openstack/ironic/+/778340/15/doc/source/admin/secure-rbac.rst#158 | 17:21 |
| TheJulia | "operators may wish to make default policy changes based upon their deployment's security requirements" | 17:22 |
| openstackgerrit | Aija Jauntēva proposed x/sushy-oem-idrac master: Add real-time status check to Lifecycle service https://review.opendev.org/c/x/sushy-oem-idrac/+/776224 | 17:24 |
| *** dtantsur is now known as dtantsur|afk | 17:37 | |
| dtantsur|afk | o/ | 17:37 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Trivial: add a missing argument to an exception https://review.opendev.org/c/openstack/ironic/+/780270 | 17:45 |
| *** jtomasek has joined #openstack-ironic | 17:47 | |
| *** jtomasek has quit IRC | 17:50 | |
| openstackgerrit | Merged openstack/ironic stable/victoria: Prepare to use tinycore 12 for tinyipa https://review.opendev.org/c/openstack/ironic/+/780459 | 17:58 |
| *** derekh has quit IRC | 18:03 | |
| *** k_mouza has quit IRC | 18:04 | |
| *** kaifeng has quit IRC | 18:05 | |
| openstackgerrit | Aija Jauntēva proposed openstack/ironic-specs master: Followup pre-defined system hw conf in single step https://review.opendev.org/c/openstack/ironic-specs/+/780645 | 18:19 |
| openstackgerrit | Aija Jauntēva proposed openstack/ironic-specs master: Followup pre-defined system hw conf in single step https://review.opendev.org/c/openstack/ironic-specs/+/780645 | 18:23 |
| *** jamesdenton has quit IRC | 18:31 | |
| *** jamesdenton has joined #openstack-ironic | 18:32 | |
| arne_wiebalck | it seems my ussuri conductor / train IPA combination tries to use grub2 to install the bootloader (since it fails to see there is an EFI partition and should only call efibootmgr) ... there is a patch in ussuri to be more generous when to avoid grub2, but I don't understand why this worked before on train/train (since the detection is the same) ... tomorrow :) | 18:45 |
| arne_wiebalck | bye everyone o/ | 18:46 |
| *** ociuhandu has quit IRC | 18:49 | |
| *** ociuhandu has joined #openstack-ironic | 18:50 | |
| *** ociuhandu has quit IRC | 18:55 | |
| TheJulia | uhh we did nothing for a long time on whole disk iamges | 19:08 |
| TheJulia | but then some hardware *really* needed EFI partitions | 19:08 |
| TheJulia | err | 19:08 |
| TheJulia | EFI references | 19:08 |
| TheJulia | Also, newer grub2 versions are hard breaking | 19:09 |
| TheJulia | we don't know how far that has gotten since it is in centos-stream | 19:09 |
| *** janders2 has joined #openstack-ironic | 19:09 | |
| *** janders has quit IRC | 19:11 | |
| *** janders2 is now known as janders | 19:11 | |
| *** bburns_ has quit IRC | 19:13 | |
| *** bburns has joined #openstack-ironic | 19:15 | |
| openstackgerrit | Merged openstack/ironic master: Allocation support for project scoped RBAC https://review.opendev.org/c/openstack/ironic/+/778340 | 19:15 |
| TheJulia | \o/ | 19:16 |
| *** ociuhandu has joined #openstack-ironic | 19:22 | |
| *** maelk40 has joined #openstack-ironic | 19:30 | |
| *** mushrushu has quit IRC | 19:31 | |
| *** mushrushu_ has joined #openstack-ironic | 19:31 | |
| *** mushrushu_ is now known as mushrushu | 19:32 | |
| *** maelk4 has quit IRC | 19:32 | |
| *** viks____ has quit IRC | 19:33 | |
| *** ociuhandu has quit IRC | 19:40 | |
| *** ociuhandu has joined #openstack-ironic | 20:07 | |
| arne_wiebalck | TheJulia: Yeah ... I was thinking that newly created partitions are now as tagged as 'boot' (rather than 'efi', which then triggers the grub2 branch) and that this is why it worked before ... does not seem to be the case, though. I will confirm tmrw it works with train/train and take it from there. I'd like to avoid having to update the conductor and the IPA at the very same moment ... | 20:17 |
| openstackgerrit | Merged openstack/ironic-python-agent stable/ussuri: Prepare to use ussuri jobs from ipa-builder https://review.opendev.org/c/openstack/ironic-python-agent/+/780557 | 20:18 |
| openstackgerrit | Merged openstack/ironic-python-agent-builder master: Use tinycore 12 to build tinyipa https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/776587 | 20:25 |
| openstackgerrit | Merged openstack/ironic-python-agent-builder master: Avoid empty dependencies in tox.ini https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/779450 | 20:25 |
| *** ociuhandu has quit IRC | 20:27 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Fix port id vs uuid in ValidateInterfacesHook https://review.opendev.org/c/openstack/ironic-inspector/+/779077 | 20:31 |
| *** ociuhandu has joined #openstack-ironic | 20:34 | |
| *** ociuhandu has quit IRC | 20:47 | |
| openstackgerrit | Merged openstack/ironic master: Trivial: add a missing argument to an exception https://review.opendev.org/c/openstack/ironic/+/780270 | 20:50 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Allow unsupported redfish set_boot_mode https://review.opendev.org/c/openstack/ironic/+/779418 | 20:54 |
| *** bfournie has quit IRC | 20:56 | |
| *** lbragstad has quit IRC | 20:56 | |
| *** zer0c00l has quit IRC | 20:58 | |
| *** shadower has quit IRC | 20:58 | |
| *** frigo has joined #openstack-ironic | 20:58 | |
| *** bfournie has joined #openstack-ironic | 20:58 | |
| *** lbragstad has joined #openstack-ironic | 20:58 | |
| *** zer0c00l has joined #openstack-ironic | 20:58 | |
| *** shadower has joined #openstack-ironic | 20:58 | |
| frigo | \o/ | 20:58 |
| *** frigo has quit IRC | 21:02 | |
| *** ociuhandu has joined #openstack-ironic | 21:19 | |
| *** bfournie has quit IRC | 21:22 | |
| *** bfournie has joined #openstack-ironic | 21:22 | |
| TheJulia | dtantsur|afk: w/r/t https://review.opendev.org/c/openstack/ironic/+/778340/15/ironic/api/controllers/v1/allocation.py#376 can you calrify your ocmment | 21:22 |
| TheJulia | comment | 21:22 |
| *** lbragstad has quit IRC | 21:24 | |
| *** lbragstad has joined #openstack-ironic | 21:28 | |
| *** ociuhandu has quit IRC | 21:39 | |
| *** rcernin has joined #openstack-ironic | 21:57 | |
| *** ociuhandu has joined #openstack-ironic | 22:06 | |
| *** ociuhandu has quit IRC | 22:14 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP: Follow-up to RBAC allocation changes https://review.opendev.org/c/openstack/ironic/+/780677 | 22:32 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Deprecate legacy policies, update project scoped docs https://review.opendev.org/c/openstack/ironic/+/778767 | 22:53 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Increment API version for Secure RBAC https://review.opendev.org/c/openstack/ironic/+/778947 | 22:53 |
| *** ociuhandu has joined #openstack-ironic | 23:00 | |
| *** ociuhandu has quit IRC | 23:05 | |
| *** rloo has quit IRC | 23:49 | |
| *** rloo has joined #openstack-ironic | 23:50 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!