rpittau | good morning ironic! o/ | 06:58 |
---|---|---|
kubajj | Good morning Ironic | 08:26 |
opendevreview | Adam Rozman proposed openstack/ironic master: implement basic-auth support for user image download process https://review.opendev.org/c/openstack/ironic/+/890411 | 10:15 |
iurygregory | good morning Ironic | 11:17 |
adam-metal3 | Hi Ironic, just a kind advertisement I have pushed basic auth support related changes, if you are interested please take a look: https://review.opendev.org/c/openstack/ironic/+/890411 , https://review.opendev.org/c/openstack/ironic-python-agent/+/890272 and thanks for the comments JayF | 12:13 |
* dtantsur puts on his queue | 12:22 | |
* iurygregory tags both patches with the hashtag | 12:22 | |
rpittau | thanks adam-metal3 have them in my queue :) | 12:27 |
kubajj | dtantsur: hi, do I understand this https://specs.openstack.org/openstack/ironic-specs/priorities/2023-2-workitems.html#improving-deploy-kernel-ramdisk-config correctly, that we would like to have something like a flag in the config - let's say arch_separate - that would signal ironic that instead of ironic/ironic/conf/agent.py the ironic/ironic/conf/agent_arm.py parameters would be used for ARM? | 12:36 |
dtantsur | kubajj: I personally didn't envision it to go so far as to duplicate all options | 12:41 |
dtantsur | kubajj: we have prior art that looks like this https://opendev.org/openstack/ironic/src/branch/master/ironic/conf/pxe.py#L74-L80 | 12:42 |
dtantsur | I thought we just introduce deploy_kernel_by_arch and deploy_ramdisk_by_arch (and bootloader_by_arch?) | 12:42 |
dtantsur | actually, having overrides in something like [pxe:aarch64] would be cool, but I'm not sure it can be expressed in oslo.config | 12:42 |
kubajj | dtantsur: oh, so we want just those config parameters that would say: we are using our own custom ramdisk for arm | 12:45 |
dtantsur | kubajj: yeah. Currently you can set deploy ramdisks globally or per node. The former does not really work for the multi-arch case, the latter is annoying. | 12:48 |
kubajj | dtantsur: ok, thanks | 12:48 |
TheJulia | good morning | 13:51 |
iurygregory | good morning TheJulia | 14:58 |
JayF | Good morning, meeting time :) | 14:59 |
iurygregory | o/ | 15:00 |
masghar | o/ | 15:00 |
kubajj | o/ | 15:00 |
* iurygregory was too fast | 15:00 | |
rpittau | o/ | 15:01 |
* TheJulia wonders if the meeting needs coffee | 15:01 | |
iurygregory | JayF, are you going to start? :D | 15:01 |
rpittau | \o/ | 15:01 |
JayF | #startmeeting ironic | 15:01 |
opendevmeet | Meeting started Mon Aug 7 15:01:24 2023 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 | o/ | 15:01 |
TheJulia | o/ | 15:01 |
JayF | TheJulia: I was actually starting a pot :) | 15:01 |
iurygregory | o/ | 15:01 |
kubajj | o/ | 15:01 |
masghar | o/ | 15:01 |
rpittau | that was fun :) | 15:01 |
TheJulia | JayF: hopefully enough to share with the channel! :) | 15:01 |
JayF | I'm more thinking lets all be on top of it | 15:01 |
JayF | so when it's done I can have a cup without rushing away ;) | 15:02 |
JayF | #topic Announcements/Reminder | 15:02 |
JayF | Standing reminder: tag your patches #ironic-week-prio, review things using the priority dashboard | 15:02 |
JayF | bonus add for today: the reverse is true; if your patch is no longer ready for review, just drop the tag until it is again | 15:02 |
JayF | so that our dashboard is a clean way to know what things need attn | 15:02 |
JayF | Next event on our schedule in the agenda is the Bobcat Non-client library freeze coming up 8/24 | 15:03 |
JayF | please ensure your sushy changes and other library code is landed well before then for Bobcat release | 15:03 |
JayF | Moving on | 15:03 |
JayF | There were no action items last meeting; skipping agenda item. | 15:03 |
JayF | #topic Review Ironic CI Status | 15:03 |
JayF | So I know rpittau had an item here; bifrost is broken by centos stream, it seems? | 15:04 |
rpittau | yep | 15:04 |
rpittau | the BZ to fix that is ON_QA hence the question | 15:04 |
JayF | I don't know what that means | 15:04 |
rpittau | the package is ready, just need to be approved | 15:04 |
rpittau | and it will be published | 15:04 |
rpittau | once it's published there's no need for the workaround | 15:05 |
JayF | This only impacts bifrost CI, yeah? | 15:05 |
rpittau | correct | 15:05 |
JayF | My $.02 is to keep the workaround in your pocket, but AFAICT we don't have much pressing bifrost stuff | 15:05 |
JayF | so I'd lean slightly towards waiting for the fix | 15:05 |
JayF | but that's a mild preference | 15:05 |
TheJulia | Is it blocking the ironic gate at all, or has this apparent workaround kept us moving forward? | 15:06 |
rpittau | the workaround is on hold | 15:06 |
JayF | TheJulia: I think we only test CentOS Stream in Bifrost gate; that's what I was trying to ask earlier | 15:06 |
rpittau | TheJulia: bifrost ci is broken up to zed at the moment | 15:06 |
rpittau | the other projects look good so far | 15:06 |
rpittau | it's because we only have cs9 nodes in bifrost | 15:07 |
JayF | I' | 15:07 |
JayF | **I'll note that I think the virt side of the stack had to land a similar workaround | 15:07 |
JayF | at a devstack level | 15:07 |
TheJulia | looks like all the bifrost jobs on the ironic gate are non-voting, 2 of 3 work | 15:07 |
rpittau | well yeah, technically the issue is on devstack | 15:07 |
TheJulia | regardless, seems like we're good to move on then? | 15:08 |
rpittau | I haven't seen any impact in other projects besides bifrost | 15:08 |
JayF | yeah I'd say fix it if we get something pressing on bifrost | 15:08 |
JayF | otherwise let it sit | 15:08 |
rpittau | yep, agred | 15:09 |
JayF | Is there any other notable things going on with CI? Aside from bifrost; I think we're as happy as we've been in a while | 15:09 |
rpittau | nothing else AFAICS | 15:09 |
JayF | Moving on | 15:10 |
JayF | #topic Review Ongoing 2023.2 Workstreams | 15:10 |
JayF | I'll note, generally, we're reaching the homestretch of the release; things we want to land need to get up for review quickly so we can make sure they hit | 15:10 |
JayF | we've already had a super productive cycle though | 15:10 |
JayF | #link https://etherpad.opendev.org/p/IronicWorkstreams2023.2 | 15:11 |
JayF | Giving another minute for comments on in-progress items | 15:12 |
kubajj | should I include the improving of config if I don't know if I'll complete it before the freeze? | 15:12 |
JayF | kubajj: first of all, lets try to get it in this cycle :) the freeze you have to worry about for that is 8/31-ish | 15:13 |
JayF | so you have weeks to complete that | 15:13 |
kubajj | oh, okay | 15:13 |
JayF | somewhere between 8/31 and 10/4 we have to have our final release cut | 15:13 |
JayF | 8/31 is when the other projects will stop taking major changes | 15:13 |
JayF | #topic OpenStack User Survey Updates | 15:14 |
JayF | I added this item to the agenda because here | 15:14 |
JayF | #link https://lists.openstack.org/pipermail/openstack-discuss/2023-July/034589.html | 15:14 |
JayF | Projects were asked to come up with any project-specific questions for the survey | 15:15 |
JayF | Ironic has none, currentyl | 15:15 |
JayF | We probably should? But I'm not sure what exactly, so I made this item to 1) increase awareness and see if anyone was interested in drafting questions and 2) give folks a chance to have input before I just go add something if nobody else has interest | 15:15 |
TheJulia | does anyone remember our ideas from last year | 15:16 |
iurygregory | I thought we had questions in the survey lol | 15:16 |
TheJulia | the ones that came up after we would have had to send them in? | 15:16 |
JayF | I don't even recall this at all last cycle tbh | 15:16 |
JayF | and if we have questions in the survey, they are missing from the google doc linked there | 15:17 |
JayF | or, always possible; I just missed it :D | 15:17 |
TheJulia | well, it would have been last june/july | 15:17 |
JayF | I created | 15:19 |
JayF | #link https://etherpad.opendev.org/p/ironic-user-survey-questions-2023 | 15:19 |
JayF | We can use that to coordinate; I'll make sure to get something to them by EOW either way | 15:19 |
iurygregory | ++ | 15:19 |
JayF | #topic Open Discussion | 15:19 |
JayF | floor is open for any other items | 15:19 |
opendevreview | Julia Kreger proposed openstack/ironic master: Support sha256/sha512 with the ilo firmware upgrade logic https://review.opendev.org/c/openstack/ironic/+/882164 | 15:20 |
JayF | Last call? | 15:21 |
TheJulia | regarding service steps | 15:22 |
TheJulia | I've not had time recently to work on it too much, but I'm wondering if it would be one of those "lets just merge it with imperfect testing" and sort it out next cycle sort of things | 15:23 |
JayF | I think we should make sure it lands this cycle; but can we be explicit about what further testing we want to make on it? | 15:23 |
TheJulia | some unit testing + tempest, but the through unit tests work just fine | 15:24 |
JayF | I'm not terribly worried about it not getting a temptest test, frankly | 15:24 |
TheJulia | so decent confidence, just there may be a obvious missing piece somewhere hwich we could just treat as a bug later | 15:24 |
JayF | with that context, I wouldn't hold it up | 15:24 |
TheJulia | okay, cool | 15:24 |
JayF | as long as it works when manually tested | 15:24 |
TheJulia | I'll try to polish it and get a client patch up this week | 15:25 |
JayF | awesome | 15:26 |
JayF | That's exciting, if you need specific help please ask | 15:26 |
JayF | I'm invested in being sure that lands in bobcat | 15:26 |
JayF | Anything else for open discussion? | 15:27 |
TheJulia | JayF: a client patch would always be appreciated :) | 15:27 |
JayF | lets land the API side then :D | 15:27 |
TheJulia | okay then! | 15:28 |
TheJulia | :) | 15:28 |
TheJulia | I have two things to follow-up on and I can shift gears to taht | 15:28 |
JayF | the other item that one or both of us owe | 15:29 |
JayF | is I think johnthetubaguy wanted us to test sharding | 15:29 |
JayF | and we know Nova will be strict about that B-3 deadline | 15:29 |
JayF | and if client for service steps is a fast follow that's not that bad | 15:29 |
JayF | so I think I might focus on getting my nova-set-lessee patch tested, and then sharding tested, so that can get in | 15:29 |
* JayF finally wrangled a devstack late last week | 15:30 | |
TheJulia | I'm not sure we have anything there that would be needed/blocking for that on our side | 15:30 |
TheJulia | at least, that I'm conciously aware of, if we do, we should put it on the etherpad | 15:31 |
TheJulia | otherwise it will just get lost in my short term recall | 15:31 |
JayF | I'm saying I think John-the-person wanted us to sanity check the changes | 15:31 |
JayF | and test in devstack | 15:31 |
TheJulia | ahh, ok | 15:31 |
TheJulia | reasonable | 15:32 |
JayF | not that there's some pending technical action so much as one or both of us expressing confidence in the changes | 15:32 |
JayF | I have a devstack up and plans this week to use it to test that and my nova-set-lessee change | 15:32 |
JayF | Either way, lots of stuff to do :) hopefully can keep pointed in a direction and doing good stuff | 15:32 |
JayF | for now I think the meeting is mostly petered out though, I'm going to close it? | 15:33 |
TheJulia | sounds good to me | 15:33 |
JayF | #endmeeting | 15:33 |
opendevmeet | Meeting ended Mon Aug 7 15:33:55 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:33 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-08-07-15.01.html | 15:33 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-08-07-15.01.txt | 15:33 |
opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-08-07-15.01.log.html | 15:33 |
rpittau | good night! o/ | 15:35 |
dtantsur | TheJulia: we have implemented the automatic checksum algo detection, haven't we? | 16:20 |
TheJulia | I believe so | 16:20 |
TheJulia | https://docs.openstack.org/releasenotes/ironic-python-agent/unreleased.html#relnotes-9-5-0 | 16:21 |
dtantsur | \o/ | 16:21 |
dtantsur | thx | 16:21 |
TheJulia | np | 16:34 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: minor: fix release notes file path https://review.opendev.org/c/openstack/ironic-python-agent/+/890694 | 19:57 |
*** dking is now known as Guest8209 | 20:13 | |
opendevreview | Merged openstack/ironic-python-agent master: minor: fix release notes file path https://review.opendev.org/c/openstack/ironic-python-agent/+/890694 | 20:25 |
*** Guest8209 is now known as dking | 20:25 | |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/2023.1: Log the number of bytes downloaded https://review.opendev.org/c/openstack/ironic-python-agent/+/890696 | 20:43 |
opendevreview | Jay Faulkner proposed openstack/ironic master: DNM: Testing sharding against Ironic https://review.opendev.org/c/openstack/ironic/+/888277 | 21:31 |
* TheJulia closes the text editor for service steps tests and takes migraine meds because a front has moved in and brought a migraine | 23:05 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!