rpittau | good morning ironic! o/ | 08:19 |
---|---|---|
janders | good morning rpittau and Ironic o/ | 08:53 |
rpittau | hey janders :) | 08:54 |
dtantsur | morning folks | 11:13 |
iurygregory | good morning rpittau janders dtantsur and Ironic o/ | 11:19 |
janders | hey dtantsur iurygregory o/ | 11:24 |
*** Guest5324 is now known as rcastillo | 13:07 | |
TheJulia | good morning | 13:21 |
rpittau | good morning TheJulia :) | 13:31 |
TheJulia | it is too quiet today | 13:41 |
TheJulia | I guess everyone is taking time off? | 13:41 |
rpittau | seems like zuul is on vacation | 13:43 |
rpittau | opendevreview is very quiet here | 13:43 |
TheJulia | I'd appreciate reviews on https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/849752/ | 13:47 |
TheJulia | Turns out our tempest plugin has a bug | 13:48 |
TheJulia | rpittau: thanks | 14:05 |
dtantsur | I'll have to take tomorrow off: it's going to be +38 | 14:14 |
TheJulia | yeouch | 14:14 |
iurygregory | OMG | 14:15 |
dtantsur | ya.. combined with the virus, it will not going to provide an acceptable working environment | 14:17 |
* dtantsur is just out of a cold shower | 14:17 | |
TheJulia | cold water is a myth here... at least during the summer time | 14:18 |
TheJulia | we have warm and water heater supported hot. | 14:18 |
dtantsur | heh, I imagine | 14:19 |
dtantsur | TheJulia: I wonder if https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/849752/1/ironic_tempest_plugin/tests/api/admin/test_nodes.py should be using self.addCleanup to ensure it gets executed even on failures | 14:21 |
TheJulia | so looks like the ipa bugfix 8.4 has a depedency library issue. It is expecting a version of neutron_lib and neutron is failing to start. | 14:22 |
TheJulia | because present neutron lib doesn't have the necessary bits anymore | 14:22 |
TheJulia | I guess | 14:22 |
dtantsur | .... | 14:23 |
iurygregory | TheJulia, you can ignore 8.4 | 14:23 |
TheJulia | dtantsur: likely now that I think about it, I can change it | 14:23 |
iurygregory | I'm requesting to delete some bugfix branches this week | 14:23 |
iurygregory | IPA 8.4 is one | 14:23 |
TheJulia | iurygregory: so I can abandon the proposed ipa patch which is workflowed? | 14:23 |
TheJulia | https://review.opendev.org/c/openstack/ironic-python-agent/+/842402 | 14:23 |
iurygregory | well, up to you =) I don't think CI will pass based on what you said... | 14:23 |
dtantsur | maybe we should actually physically delete branches we no longer maintain | 14:24 |
TheJulia | I thought we were... | 14:24 |
TheJulia | going to that is | 14:24 |
iurygregory | dtantsur, yeah - was going to request infra to do this (I think this is the way) | 14:24 |
dtantsur | yeah, the admins can do it | 14:24 |
TheJulia | iurygregory: afaik yes, but we need to ensure we hae any outstanding patches abandoned | 14:24 |
dtantsur | they'll probably ask us if we want the tip of the branch tagged somehow | 14:25 |
iurygregory | TheJulia, yeah | 14:25 |
TheJulia | dtantsur: ++ | 14:25 |
dtantsur | if we don't tag it, all untagged commits are lost | 14:25 |
TheJulia | yup | 14:25 |
dtantsur | okay, time to kill some netboot | 14:26 |
iurygregory | something like 8.4-eol tag? | 14:26 |
TheJulia | yeah | 14:26 |
dtantsur | up to us. we may even decide that losing commits is okay. but 8.4-eol sounds good to me. | 14:26 |
iurygregory | ack | 14:26 |
opendevreview | Merged openstack/ironic master: Fix compatibility with jsonschema>=4.0.0 https://review.opendev.org/c/openstack/ironic/+/849882 | 14:37 |
opendevreview | Merged openstack/ironic bugfix/20.2: Fix markup typo in Redfish driver docs https://review.opendev.org/c/openstack/ironic/+/849811 | 14:37 |
opendevreview | Merged openstack/ironic stable/yoga: Fix markup typo in Redfish driver docs https://review.opendev.org/c/openstack/ironic/+/849810 | 14:37 |
opendevreview | Merged openstack/ironic master: Remove workaround for Python < 3.7.4 https://review.opendev.org/c/openstack/ironic/+/850095 | 14:37 |
dtantsur | JFYI iurygregory, TheJulia: removing netboot means removing support for trusted boot | 14:41 |
TheJulia | ++ | 14:41 |
TheJulia | I've wanted to rip out trusted boot support for a while | 14:41 |
TheJulia | it is ia32 only with tboot and stuffs | 14:41 |
iurygregory | ack =) | 14:42 |
dtantsur | TheJulia: does kickstart deploy use instance kernel/ramdisk? | 14:46 |
dtantsur | I'm trying to figure out if it's safe to remove various boot_option!='local' | 14:46 |
TheJulia | yes, it does | 14:46 |
dtantsur | ack | 14:46 |
TheJulia | because deploy_kernel|ramdisk does cleaning between instances | 14:47 |
dtantsur | TheJulia: no, wait, I mean the *instance* image kernel/ramdisk | 14:47 |
dtantsur | like for netboot/ramdisk interface | 14:47 |
TheJulia | yes | 14:47 |
TheJulia | the plus side, the use becomes restricted to ramdisk and anaconda | 14:48 |
dtantsur | ture, that's a big plus, especially as far as the testing surface is concerned | 14:48 |
TheJulia | and delineation | 14:48 |
TheJulia | granted, it might be a while before any thoughts of netboot are completely delineated, but it is a start to rip it out for direct | 14:49 |
iurygregory | #startmeeting ironic | 15:00 |
opendevmeet | Meeting started Mon Jul 18 15:00:00 2022 UTC and is due to finish in 60 minutes. The chair is iurygregory. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
opendevmeet | The meeting name has been set to 'ironic' | 15:00 |
iurygregory | Hello ironicers o/ | 15:00 |
rpittau | o/ | 15:00 |
matfechner | o/ | 15:00 |
TheJulia | o/ | 15:00 |
ajya | o/ | 15:00 |
erbarr | o/ | 15:00 |
iurygregory | welcome to our weekly meeting =) | 15:00 |
dtantsur | o/ | 15:01 |
rloo | o/ | 15:01 |
iurygregory | #topic Announcements / Reminder | 15:01 |
iurygregory | forgot to mention, you can find the agenda in the wiki https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting =) | 15:01 |
iurygregory | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
erbarr | Reminder, this past weekend Dell CI was offline, it is now back online. So if any reviews need a Dell CI run, add a "dell-recheck" comment | 15:02 |
iurygregory | #info Revise Ironic Vision meeting - Tomorrow 14:00 UTC based on the doodle results | 15:02 |
iurygregory | this is the result from the the doodle, thanks to everyone who provided input on it | 15:02 |
iurygregory | I will add a link for the meeting in the etherpad we will use to discuss | 15:03 |
* iurygregory will create the etherpad in a few | 15:03 | |
TheJulia | ugh, I think I have a conflict | 15:03 |
iurygregory | TheJulia, ouch .-. | 15:03 |
TheJulia | yup | 15:03 |
iurygregory | hummm | 15:04 |
TheJulia | :( | 15:05 |
TheJulia | sorry | 15:05 |
iurygregory | maybe we can do two meetings ? | 15:05 |
TheJulia | I don't see why not | 15:05 |
iurygregory | just wondering the best way to address the issue... | 15:05 |
iurygregory | having one a bit later (so APAC folks can join etc) | 15:06 |
rloo | i think it is important for TheJulia to be there :) | 15:06 |
TheJulia | I've done visioning exercises a few times which makes it kind of easy for me to help facilitate and orient the flow of discussion | 15:08 |
rpittau | we hsould probably postpone | 15:08 |
iurygregory | the other options based on the doodle would be | 15:08 |
iurygregory | tomorrow starting at 15:00 UTC (but I think TheJulia can't join - based on the doodle) | 15:09 |
TheJulia | hold on | 15:09 |
iurygregory | or Friday 15:00 UTC | 15:09 |
iurygregory | ok | 15:09 |
TheJulia | I can do 1500 tomorrow | 15:10 |
iurygregory | ok o/ | 15:11 |
* TheJulia suspects she set the wrong time window in off by 1 hour | 15:11 | |
iurygregory | it's "#undo" to remove the info item right? | 15:11 |
TheJulia | yes | 15:11 |
iurygregory | #undo | 15:11 |
opendevmeet | Removing item from minutes: #info Revise Ironic Vision meeting - Tomorrow 14:00 UTC based on the doodle results | 15:11 |
iurygregory | #info Revise Ironic Vision meeting - Tomorrow 15:00 UTC based on the doodle results | 15:12 |
iurygregory | I will also send an email to openstack-discuss with the details + etherpad | 15:12 |
iurygregory | #info Dell CI is now back online. So if any reviews need a Dell CI run, add a "dell-recheck" comment | 15:13 |
iurygregory | #info Feel free to add topics for the PTG | 15:13 |
iurygregory | #link https://etherpad.opendev.org/p/ironic-ptg-planing-Columbus-OH | 15:14 |
iurygregory | Does anyone have something to add ? | 15:14 |
TheJulia | I'll be at a conference wednesday->sunday next week | 15:15 |
TheJulia | so I will likely be slightly difficult to get ahold of | 15:16 |
iurygregory | TheJulia, ack, enjoy the conference o/ | 15:16 |
iurygregory | ok, moving on | 15:17 |
iurygregory | #topic Review subteam status reports | 15:18 |
iurygregory | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:18 |
iurygregory | starting around L75 | 15:18 |
TheJulia | I guess project scoped manager is something that should be relatively easy for us to move forward on, but it is not getting review traction. :\ | 15:22 |
TheJulia | line 95 | 15:22 |
iurygregory | will make it a priority for me to review it | 15:23 |
TheJulia | we should try and review the openconfig patches harald has up this week | 15:26 |
iurygregory | I don't see any of them in the ironic-week-prio (we should add them) | 15:27 |
TheJulia | ++ | 15:27 |
iurygregory | ok, let's move to the next topic | 15:27 |
iurygregory | #topic Deciding on priorities for the coming week | 15:28 |
iurygregory | #link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio | 15:28 |
iurygregory | Any patches we should add to our list? | 15:28 |
iurygregory | \o/ | 15:28 |
TheJulia | aside from harald's openconfig stuffs, I don't think so | 15:29 |
iurygregory | I will try to look at open patches without the hashtag and add some (if there is any ofc) | 15:30 |
TheJulia | ack | 15:30 |
iurygregory | TheJulia, if you have harald's patches handy can you add the hashtag to it? | 15:31 |
TheJulia | I suspect someone might need to reach out to ilo's maintainers w/r/t https://review.opendev.org/c/openstack/ironic/+/804486 | 15:31 |
TheJulia | outstanding since March | 15:31 |
TheJulia | iurygregory: sure | 15:31 |
dtantsur | I think I may have bifrost stuff to review | 15:33 |
iurygregory | TheJulia, done (added a comment and attention in the patch) | 15:33 |
iurygregory | dtantsur, sure =) | 15:33 |
dtantsur | https://review.opendev.org/c/openstack/bifrost/+/849652 https://review.opendev.org/c/openstack/bifrost/+/849074 https://review.opendev.org/c/openstack/bifrost/+/849563 https://review.opendev.org/c/openstack/bifrost/+/841876 | 15:33 |
dtantsur | also https://review.opendev.org/c/openstack/ironic-lib/+/849661 is minor but nice | 15:34 |
dtantsur | also https://review.opendev.org/c/openstack/python-ironicclient/+/842986 has been around for a while | 15:34 |
TheJulia | harald's patches are done | 15:34 |
dtantsur | also docs https://review.opendev.org/c/openstack/ironic/+/847136 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/846609 :D | 15:34 |
dtantsur | heh, I have quite a few patches open :D | 15:34 |
TheJulia | looks like we have a few patches on networking-generic-switch too | 15:36 |
dtantsur | REVIEW ALL THE THINGS | 15:36 |
TheJulia | ++ | 15:36 |
TheJulia | we really need to, we've been so heads down with CI breaks and interrupts that we've not had time to really just go through the outstanding items | 15:37 |
iurygregory | ++ | 15:37 |
iurygregory | I've added the patches Dmitry mentioned, since I will check our projects to see open patches I will add them to the list | 15:39 |
TheJulia | I've tagged a few of the networking-generic-switch items, a couple others seem like there may be other things going on or are pending response/revision | 15:39 |
iurygregory | acj | 15:40 |
iurygregory | ack* | 15:40 |
iurygregory | ok, moving on o/ | 15:40 |
iurygregory | #topic Baremetal SIG | 15:40 |
iurygregory | #link https://etherpad.opendev.org/p/bare-metal-sig | 15:41 |
iurygregory | we probably don't have a lot to add to it, but the presentation from last week "Scott Solkhon & Jamie Poole: "Bare metal Kubernetes at G-Research" | 15:41 |
iurygregory | was amazing =) | 15:41 |
iurygregory | soon the recording will be available =) | 15:42 |
TheJulia | Awesome. I need to reach out to a potential for august | 15:42 |
TheJulia | or two potentials, It is a calendar invite for me for later today | 15:42 |
iurygregory | nice \o/ | 15:43 |
iurygregory | we don't have any RFE to review (skipping) | 15:43 |
iurygregory | #topic Open discussion | 15:43 |
iurygregory | Do we have any topics we would like to discuss? | 15:43 |
rpittau | just wondering if we should add the list of the "supported" bugfix branches to the etherpad | 15:44 |
iurygregory | wondering if the etherpad would be the right place | 15:44 |
rpittau | well it's fast enough to change | 15:45 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Remove support for trusted boot https://review.opendev.org/c/openstack/ironic/+/850239 | 15:45 |
iurygregory | but we should add =) | 15:45 |
dtantsur | JFYI folks ^^^ | 15:45 |
iurygregory | rpittau, right! | 15:45 |
rpittau | considering that they change often | 15:45 |
iurygregory | we don't need to wait for CI =D | 15:45 |
iurygregory | humm that makes sense | 15:45 |
rpittau | I'll add a small section to the etherpad if there are no objections | 15:46 |
iurygregory | none from me =) | 15:47 |
TheJulia | dtantsur: thanks! | 15:47 |
iurygregory | rpittau, just do it :D | 15:48 |
rpittau | ok | 15:48 |
TheJulia | needful for sanity over asking for forgivness/permission any day of the week | 15:48 |
rpittau | :) | 15:48 |
iurygregory | so I think that's all for today's meeting | 15:49 |
iurygregory | I will run the meeting next week | 15:49 |
iurygregory | thanks everyone \o/ | 15:49 |
TheJulia | thanks iurygregory | 15:49 |
iurygregory | #endmeeting | 15:50 |
opendevmeet | Meeting ended Mon Jul 18 15:50:54 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:50 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-07-18-15.00.html | 15:50 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-07-18-15.00.txt | 15:50 |
opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-07-18-15.00.log.html | 15:50 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: WIP finally remove netboot https://review.opendev.org/c/openstack/ironic/+/850240 | 15:55 |
rpittau | bye everyone o/ | 15:58 |
TheJulia | goodnight | 15:58 |
dtantsur | o/ | 16:12 |
opendevreview | Merged openstack/bifrost master: CI: use openstacksdk from source if it's cloned https://review.opendev.org/c/openstack/bifrost/+/849563 | 16:36 |
opendevreview | Merged openstack/bifrost stable/yoga: Prevent the enroll/deploy commands from running without venv https://review.opendev.org/c/openstack/bifrost/+/841876 | 17:00 |
opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add Nokia SRL release note https://review.opendev.org/c/openstack/networking-generic-switch/+/850245 | 17:15 |
opendevreview | Merged openstack/ironic-python-agent-builder master: Document building an image with patches https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/846609 | 17:26 |
opendevreview | Merged openstack/ironic stable/wallaby: Improve dependency resolution in driver-requirements.txt https://review.opendev.org/c/openstack/ironic/+/849491 | 17:27 |
opendevreview | Merged openstack/ironic master: Docs: specify what to do with the created images https://review.opendev.org/c/openstack/ironic/+/847136 | 17:42 |
opendevreview | Merged openstack/bifrost master: Wait for cleaning to be over when enrolling https://review.opendev.org/c/openstack/bifrost/+/849652 | 17:52 |
opendevreview | Merged openstack/bifrost master: Fix upgrade on CentOS and with Keystone enabled https://review.opendev.org/c/openstack/bifrost/+/849074 | 18:40 |
opendevreview | Merged openstack/networking-generic-switch master: Support Nokia SR Linux https://review.opendev.org/c/openstack/networking-generic-switch/+/848661 | 19:05 |
*** tosky_ is now known as tosky | 19:45 | |
opendevreview | Merged openstack/python-ironicclient master: Fix logging in the baremetal CLI https://review.opendev.org/c/openstack/python-ironicclient/+/842986 | 20:10 |
cbouchar | TheJulia: Could you or someone else forward the meeting invite details for tomorrow's meeting on ironic plans? I registered to get openstack-discuss today but not received anything yet. | 20:22 |
opendevreview | Merged openstack/ironic stable/xena: Fix Redfish RAID for non-immediate controllers https://review.opendev.org/c/openstack/ironic/+/849999 | 21:07 |
TheJulia | cbouchar: hmm | 21:07 |
TheJulia | cbouchar: details are at https://etherpad.opendev.org/p/revise-ironic-vision | 21:08 |
TheJulia | https://docs.openstack.org/ironic/latest/contributor/vision.html <-- what we came up with last time we visioned | 21:10 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP: ident and handle redfish pw change case https://review.opendev.org/c/openstack/ironic/+/850259 | 23:06 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP: ident and handle redfish pw change case https://review.opendev.org/c/openstack/ironic/+/850259 | 23:15 |
janders | good morning Ironic o/ | 23:30 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!