opendevreview | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic-inspector/+/906935 | 04:15 |
---|---|---|
rpittau | good morning ironic! o/ | 08:22 |
opendevreview | Riccardo Pittau proposed openstack/ironic master: Bump proliantutils for pyasn1 compatibility https://review.opendev.org/c/openstack/ironic/+/913522 | 09:15 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Drop fallback to unmanaged inspection for virtual media and UEFI boot https://review.opendev.org/c/openstack/ironic/+/913523 | 09:27 |
opendevreview | OpenStack Release Bot proposed openstack/bifrost stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/bifrost/+/913530 | 10:24 |
opendevreview | OpenStack Release Bot proposed openstack/bifrost stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/bifrost/+/913531 | 10:24 |
opendevreview | OpenStack Release Bot proposed openstack/bifrost master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/bifrost/+/913532 | 10:24 |
opendevreview | OpenStack Release Bot proposed openstack/networking-baremetal stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/networking-baremetal/+/913535 | 10:25 |
opendevreview | OpenStack Release Bot proposed openstack/networking-baremetal stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/networking-baremetal/+/913536 | 10:25 |
opendevreview | OpenStack Release Bot proposed openstack/networking-baremetal master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/networking-baremetal/+/913537 | 10:25 |
opendevreview | Merged openstack/ironic-inspector master: Update bug tracker url https://review.opendev.org/c/openstack/ironic-inspector/+/912733 | 10:41 |
opendevreview | Merged openstack/ironic-python-agent stable/2023.2: Use assert_not_called https://review.opendev.org/c/openstack/ironic-python-agent/+/912760 | 11:07 |
opendevreview | Merged openstack/ironic-python-agent master: Update regex to detect closed branch https://review.opendev.org/c/openstack/ironic-python-agent/+/912736 | 11:07 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Do not rely on predefined UUIDs in tests https://review.opendev.org/c/openstack/ironic/+/881485 | 11:35 |
opendevreview | Merged openstack/bifrost master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/bifrost/+/913532 | 12:01 |
opendevreview | Merged openstack/networking-baremetal stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/networking-baremetal/+/913535 | 12:02 |
opendevreview | Merged openstack/networking-baremetal stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/networking-baremetal/+/913536 | 12:09 |
opendevreview | Merged openstack/networking-baremetal master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/networking-baremetal/+/913537 | 12:09 |
opendevreview | Kamil Gustab proposed openstack/sushy master: Adds Ethernet Interfaces for manager. https://review.opendev.org/c/openstack/sushy/+/909451 | 12:22 |
iurygregory | FYI The CFP for the #OpenInfraSummit Asia is now open! Submissions for the CFP can be submitted in both English and Korean. The CFP closes May 29, 2024 at 11:59 PM KST! | 12:25 |
TheJulia | good morning folks | 13:23 |
TheJulia | I'm going to be away this week attending to a family matter. If folks need me, well... ping me, but no guarantees I'll be available | 13:27 |
dtantsur | good luck! | 13:30 |
rpittau | TheJulia: good luck | 13:41 |
opendevreview | Merged openstack/bifrost stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/bifrost/+/913530 | 13:41 |
opendevreview | Merged openstack/bifrost stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/bifrost/+/913531 | 13:42 |
cid | TheJulia: good luck | 14:02 |
rpittau | iurygregory: new version of proliantutils is out, we can move on with https://review.opendev.org/c/openstack/ironic/+/913522 | 14:05 |
iurygregory | rpittau, awesome! | 14:35 |
iurygregory | good luck TheJulia! | 14:37 |
opendevreview | Merged openstack/ironic master: Fix data length exceeding limit error https://review.opendev.org/c/openstack/ironic/+/910973 | 14:40 |
iurygregory | rpittau, do we have a patch for the other requirement? the fujitsu one I think | 14:44 |
rpittau | iurygregory: https://review.opendev.org/c/x/python-scciclient/+/913303 | 14:47 |
rpittau | or you mean on the ironic side ? | 14:47 |
iurygregory | rpittau, was wondering about both tbh =) | 14:49 |
rpittau | ok, on the ironic side not yet | 14:49 |
iurygregory | ack | 14:50 |
JayF | well that's problematic, right? | 14:54 |
JayF | we can't land scciclient patch | 14:54 |
JayF | and we need to release today | 14:54 |
dtantsur | Reminder: we don't HAVE to bump requirements for bug fixes | 14:59 |
JayF | yeah | 14:59 |
dtantsur | It can be nice towards an operator to designate a very critical bug fix this way | 14:59 |
dtantsur | but it's by no means a required or even standard practice | 14:59 |
drannou | Hello ironic! | 15:00 |
JayF | hello, meeting time :D | 15:01 |
rpittau | o/ | 15:01 |
JayF | #startmeeting irronic | 15:01 |
opendevmeet | Meeting started Mon Mar 18 15:01:06 2024 UTC and is due to finish in 60 minutes. The chair is JayF. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
opendevmeet | The meeting name has been set to 'irronic' | 15:01 |
iurygregory | o/ | 15:01 |
dtantsur | o/ | 15:01 |
rpittau | o/ | 15:01 |
JayF | #endmeeting | 15:01 |
opendevmeet | Meeting ended Mon Mar 18 15:01:18 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:01 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/irronic/2024/irronic.2024-03-18-15.01.html | 15:01 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/irronic/2024/irronic.2024-03-18-15.01.txt | 15:01 |
opendevmeet | Log: https://meetings.opendev.org/meetings/irronic/2024/irronic.2024-03-18-15.01.log.html | 15:01 |
JayF | #startmeeting ironic | 15:01 |
opendevmeet | Meeting started Mon Mar 18 15:01:21 2024 UTC and is due to finish in 60 minutes. The chair is JayF. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
opendevmeet | The meeting name has been set to 'ironic' | 15:01 |
rpittau | that was quick :D | 15:01 |
JayF | Lets try this again, this time with the proper number of R's | 15:01 |
iurygregory | o/ | 15:01 |
JayF | o/\ | 15:01 |
rpittau | o/ | 15:01 |
JayF | #topic Announcements/Reminder | 15:02 |
dtantsur | o/ | 15:02 |
JayF | #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:02 |
JayF | #info Project Teams Gathering (PTG) will be held from Monday, April 8 to Friday, April 12 2024 | 15:02 |
JayF | #link https://etherpad.opendev.org/p/ironic-ptg-april-2024 | 15:02 |
JayF | #link https://ptg.opendev.org/ptg.html | 15:02 |
JayF | #info Ironic Meetup/BareMetal SIG June 5, OpenInfra Days June 6 @ CERN. Signup at https://indico.cern.ch/event/1378171/ and https://indico.cern.ch/event/1376907/ | 15:02 |
JayF | #topic Caracal Release schedule | 15:02 |
JayF | it's R-2 week | 15:02 |
JayF | Final RCs are due 3/28 (that's our final-final-final-final Ironic deadline, but we shouldn't even get close to that imo) | 15:03 |
iurygregory | we won't =) | 15:03 |
JayF | and the Caracal release is April 3rfd | 15:03 |
JayF | I think rpittau will officially be PTL by then, but JFYI: I will be gone release week on vacation, and will be doing my best to be completely detached | 15:03 |
JayF | That week of April 1 | 15:04 |
rpittau | JayF: I'll try not to burn eveyrtinhg down my first week :P | 15:04 |
JayF | lol I have full confidence in your ability to handle anything :D | 15:04 |
JayF | #topic Review Ironic CI Status | 15:04 |
iurygregory | ++ | 15:04 |
JayF | How is CI? | 15:05 |
iurygregory | we merged some workarounds last week | 15:05 |
iurygregory | I think it's working "ok" | 15:05 |
JayF | I have a note here "metalsmith legacy job is foobar" FWIW, it's "FUBAR" (it's an acronym :D) | 15:05 |
JayF | I forget what the F means, but the last part is "up beyond all recognition" | 15:05 |
iurygregory | I was about to ask | 15:06 |
TheJulia | Well, we recognize it, we don't recognize value of excessive partition testing :) | 15:06 |
TheJulia | and it is rooted in tinycore | 15:06 |
TheJulia | so...... | 15:06 |
JayF | TheJulia: I'm pondering a ... small add-on to our IPA on Gentoo for CI project lol | 15:06 |
JayF | Really, we should ensure the partiion image issues get on PTG etherpad | 15:07 |
JayF | would be a good choice for a group pairing session if we have time | 15:07 |
rpittau | JayF: I use the military acronym | 15:07 |
JayF | rpittau: aiui "FUBAR" is from the military but I wouldn't know firsthand | 15:07 |
JayF | I'm going to move on so we can finish the meeting :D | 15:07 |
JayF | #topic Virtual PTG Planning | 15:07 |
TheJulia | JayF: I'm packing up to drive to new mexico for a family emergency, so very likely won't be this week unless you want to pair up at an odd hour | 15:07 |
JayF | We have times planned, it's at ptg.opendev.org/ptg | 15:08 |
JayF | TheJulia: I mean during PTG week, not now | 15:08 |
TheJulia | oh, cool, yeah | 15:08 |
JayF | Please take some time over the next three weeks to look at PTG topics on the etherpad | 15:08 |
JayF | Anything else about etherpad? | 15:08 |
JayF | er, ptg | 15:08 |
JayF | #topic Bug Deputy | 15:09 |
rpittau | I'll leave the topic list open for at least another week | 15:09 |
JayF | Cool, ty | 15:09 |
JayF | I was bug deputy. My time was basically stolen (CPU-steal-style) by other issues around the community and personally | 15:09 |
JayF | I am going to keep bug deputy if nobody else wants it, and focus on ironic bugs this week | 15:10 |
JayF | and try to get at least a 50% record of promised deputying vs actual deputying :| | 15:10 |
JayF | No RFEs to review, skipping topic. | 15:10 |
JayF | #topic Open Discussion | 15:11 |
JayF | Going to loose the meeting and let us get back to work. Thanks all o/ | 15:12 |
JayF | #endmeeting | 15:12 |
opendevmeet | Meeting ended Mon Mar 18 15:12:32 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:12 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-03-18-15.01.html | 15:12 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-03-18-15.01.txt | 15:12 |
opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-03-18-15.01.log.html | 15:12 |
JayF | fungi: I accidentally made an "irronic" meeting with a typo, is there any cleanup that's needed? | 15:12 |
TheJulia | irronic is the next project ;) | 15:13 |
iurygregory | :D | 15:13 |
TheJulia | We'll take a diesel engine and add a mechanical state machine | 15:13 |
drannou | Quick question, I'm playing a lot with ironic and nova, and I have a strange behavior: when I'm spawning, ironic is checking the glance properties with an empty context, so at the end it's causing an ERROR because project_id could not be verified (common/glance_service/image_service.py). Did I miss a configuration somewhere ? Cannot found it in the documentation | 15:14 |
drannou | of course, no problem with "public" images | 15:15 |
fungi | JayF: there are so many typo meetings at https://meetings.opendev.org/meetings/ that i'm not sure there's much point ;) | 15:15 |
JayF | Well, I didn't wanna leave it laying on the floor without saying something. You own the broom and the floor so I'll leave it up to you all if you wanna clean up or not :) | 15:15 |
fungi | we'll just let it become part of the floor, i think | 15:16 |
TheJulia | drannou: you *likely* have image server config misconfigured in ironic.conf, maybe trying system scope when a base project scope is needed. Seems like a bug but unsure of where glance is at in the grand scheme | 15:16 |
fungi | maybe someday someone will want to go through and sweep up all of the empty-ish ones at once, but unlikely | 15:16 |
TheJulia | "in accordance with the prophecy" | 15:19 |
drannou | Well I was not very clear, may be it's a little bit different : ironic can take glance informations, but does NOT have the customer context, so it cannot compare to nova customer request | 15:20 |
JayF | drannou: can you lay out your use case + issue in a bug ticket? I'm not sure if it's expected behavior or not, but if it's a missing feature, it's something that could potentially be scoped into some related work I'm planning for next cycle | 15:21 |
drannou | JayF: I can, but it's so basic (simply an 'openstack server create' with a shared image) that I would be surpised to be the only one having that | 15:22 |
JayF | Ack; I would be surprised too, but I don't have any direct knowledge or way to test right now. I know there are some pain points where right now, Ironic has no information about who is making the call in Nova to provision an instance | 15:23 |
JayF | so it's hard for me to see /how/ this would work in the current world, even thoguh I have no knowledge that it does/does not work | 15:23 |
drannou | https://opendev.org/openstack/ironic/src/branch/master/ironic/common/glance_service/service_utils.py#L120 I clearly see it here. When I debug the code, I'm arriving there with No auth_token and no context.project_id | 15:24 |
JayF | yeah, when called from nova we don't have project in the context, I'm fairly certain | 15:24 |
TheJulia | hmmmm it should just work with a public image | 15:24 |
JayF | the work I'm doing next cycle is to help resolve that for another use case, but this seems like it'd be an additional win | 15:25 |
JayF | TheJulia: it's a private/shared image aiui | 15:25 |
TheJulia | oh | 15:25 |
TheJulia | that would do it | 15:25 |
drannou | TheJulia: it's the case, with public image it's perfectly working | 15:25 |
TheJulia | ahh, okay, yeah | 15:25 |
JayF | drannou: file a bug, I'll mark it down on the feature discussion we're having for PTG | 15:25 |
TheJulia | that has long been a thorn in our side because we don't get the actual request context | 15:25 |
JayF | drannou: I'm not promising we'll get it next cycle, I never do, but it should be intensely easier after the work I've planned is done | 15:25 |
drannou | JayF: ok let's do it | 15:25 |
TheJulia | we need some sort of "is this thing valid and can we, as a service, please has it like the lolcat we need to be. | 15:26 |
JayF | TheJulia: we're going to have metadata from nova as part of ironic-guest-metadata to enable automatic_lessee | 15:26 |
JayF | TheJulia: I'm not wrong to think we could plumb that in other places, am I? | 15:26 |
TheJulia | yeah, but that won't include the original request token | 15:26 |
TheJulia | slightly complicated, but there might be a path to leverage it actually | 15:26 |
JayF | yeah, I don't think it'll be free | 15:27 |
JayF | but I should get more than half of the way there with my existing planned work | 15:27 |
JayF | and it'll be the harder half (interface w/nova) | 15:27 |
drannou | So, as a work arround, I will locally return True if the image is shared or private | 15:27 |
drannou | nova has already made all the test before | 15:28 |
drannou | It means that the code in service_utils only work with "ironic only" deployment ? | 15:28 |
JayF | Nova doesn't pass on any user context when it calls into Ironic to deploy today | 15:30 |
JayF | so it's not possible to make it work with integrated openstack until we implement https://blueprints.launchpad.net/nova/+spec/ironic-guest-metadata which is loosely scheduled for early next cycle | 15:31 |
TheJulia | Packing up, I'll be in the car all day | 15:31 |
JayF | o/ | 15:31 |
drannou | JayF: ok ok. So I have to open a bug ? | 15:31 |
JayF | If you would, it'd be very helpful. bugs.launchpad.net/ironic | 15:32 |
drannou | TheJulia: Big thks, o/ | 15:32 |
JayF | link it here when you're done and I'll triage it and add it to the PTG discussion around guest metadata | 15:32 |
TheJulia | +++++ | 15:36 |
drannou | JayF: https://bugs.launchpad.net/ironic/+bug/2058259 | 16:00 |
Sandzwerg[m] | drannou which rights does the ironic user have? We gave ours image_admin and we can use private images, pretty sure shared images as well. | 16:01 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Remove legacy Query support from _paginate_query https://review.opendev.org/c/openstack/ironic/+/891505 | 16:04 |
dtantsur | TheJulia: ^^^ finally got to finishing it | 16:04 |
drannou | Sandzwerg[m]: mmm when I re read the code, I don't see how you would pass the "getattr(image, 'owner', None) == context.project_id" | 16:06 |
drannou | but I will check exactly the ironic rights | 16:07 |
JayF | drannou: https://etherpad.opendev.org/p/ironic-ptg-april-2024 starting at line 132, ty for documenting it | 16:08 |
JayF | Sandzwerg[m]: drannou: you should talk about versions you're running for this conversation, I suspect :D | 16:08 |
drannou | JayF: yes may be :) 2023.1 for me | 16:10 |
Sandzwerg[m] | We're still on xena, but added that some versions prior. We had issues with non-public images before as well | 16:10 |
Sandzwerg[m] | I don't think we added any code changes to make it work, but need to check | 16:10 |
Nisha_Agarwal | rpittau, hi. proliantutils 2.16.2 is released with the changes | 16:18 |
rpittau | Nisha_Agarwal: thank you! | 16:24 |
Nisha_Agarwal | rpittau, you are welcome :) and thanks for taking care pointing the incompatibility mismatch between proliantutils and ironic environment | 16:27 |
rpittau | no problem! hopefully we won't have other surprises with pysnmp in the next cycle :) | 16:28 |
JayF | ration your hope, Riccardo ;) | 16:28 |
JayF | lol | 16:28 |
rpittau | TheJulia, JayF, can you please look at https://review.opendev.org/c/openstack/ironic/+/913522 when you get a chance? | 16:29 |
rpittau | lol | 16:29 |
JayF | patch release version of proliantutils changing the requirement is yikes | 16:29 |
JayF | I wonder if that'll cause some blowback in earlier Ironic's since we don't top the version | 16:29 |
JayF | I suspect pip will figure it out | 16:29 |
JayF | rpittau: landed it ^ but we should be aware of this generally, in case we need to backport a requirements block for the new version | 16:30 |
rpittau | JayF: I believe we've removed the top cap recently | 16:31 |
rpittau | it was capped in zed at least | 16:31 |
JayF | ack | 16:31 |
JayF | I'm just saying we should have ear to the ground for related bugs :D | 16:32 |
rpittau | oh yeah | 16:32 |
opendevreview | Merged openstack/ironic master: Bump proliantutils for pyasn1 compatibility https://review.opendev.org/c/openstack/ironic/+/913522 | 16:46 |
iurygregory | \o/ | 16:47 |
rpittau | one's done, hopefully scciclient patch will merge soon | 16:50 |
iurygregory | I was generating the release for ironic, for some reason it add changes to the cycle highlithgs lol | 16:58 |
iurygregory | https://paste.opendev.org/show/bKcv8ytxjvoKVaLIXeFY/ O.o | 16:59 |
rpittau | iurygregory: seems like formatting issue | 17:02 |
rpittau | anyway | 17:02 |
rpittau | time to split | 17:02 |
rpittau | good night! o/ | 17:02 |
iurygregory | good night! | 17:03 |
iurygregory | funny that we will release 2024.1 and our version is 24.1 :D | 17:03 |
dtantsur | heh, funny indeed | 17:04 |
dtantsur | I hope people don't start expecting such correlations | 17:04 |
iurygregory | yep! | 17:05 |
dtantsur | Does anyone have any security-based objections to passing node UUID to IPA via kernel params? (cc JayF) I want to finally get rid of MAC requirements for virtual media. | 17:05 |
iurygregory | we won't have to tell the MAC? =O | 17:23 |
dtantsur | yeah, no MAC for virtual media at all | 17:25 |
iurygregory | I like the idea | 17:26 |
NobodyCam | I have a crazy question, Has anyone ever tested the upper limit of concurrent ironic provisions thru nova, I expect the limiting factor is dnsMasq | 17:37 |
JayF | dtantsur: no objections ever for virtual media | 17:56 |
JayF | dtantsur: and probably not for pxe but I need to think on it more | 17:56 |
JayF | dtantsur: would we still validate macs are same as node, even if it has node id? | 18:07 |
dtantsur | JayF: we may or may not. Depending on your considerations. But I want to make the MAC address completely optional for IPA to operate. | 18:25 |
opendevreview | cid proposed openstack/ironic master: Special treatment of .json is now disabled for nodes with .json extension. https://review.opendev.org/c/openstack/ironic/+/913467 | 18:51 |
JayF | dtantsur: basically I struggle to see how it's an issue if we continue to ensure Mac addresses match. | 18:54 |
JayF | dtantsur: if we don't, I think there might be some gremlins to think about | 18:54 |
dtantsur | At least inspection does check that everything provided matches | 18:55 |
JayF | E.g. port swap causing a machine to misprovision in a pxe case | 18:55 |
JayF | Most of these concerns are moot with a bmc involved like in the virtual media case | 18:55 |
iurygregory | JayF, this is the only new change we have in NGS https://github.com/openstack/networking-generic-switch/commit/73f50d98eb0a1df843272dc71dfda877d2ff0c9b do you think we need a release? (seems like a bugfix for tempest testing...) | 22:31 |
JayF | It's *probably* fine, and if it's not, we can trivially backport it | 22:32 |
JayF | no reason to preemptively kick it | 22:32 |
iurygregory | ack | 22:34 |
iurygregory | JayF, can you try running "tox -e venv -- new-release caracal ironic feature --stable-branch" to see if it also changes the content on cycle-highlights? | 22:52 |
* iurygregory brb dinner | 22:53 | |
JayF | I saw this after I'm EOD, I'll happily try tomorrow or if I have spare time this evening | 23:15 |
iurygregory | ack | 23:16 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!