| arne_wiebalck | Good morning, Ironic! | 06:22 |
|---|---|---|
| MahnoorAsghar | Good morning Arne! | 06:26 |
| arne_wiebalck | Hey MahnoorAsghar o/ | 06:27 |
| MahnoorAsghar | o/ | 06:31 |
| dtantsur | good morning folks | 07:08 |
| dtantsur | bifrost yoga still broken, blocking kolla folks. please review: https://review.opendev.org/c/openstack/bifrost/+/842302 | 07:10 |
| rpittau | good morning ironic! o/ | 07:53 |
| *** matt__ is now known as matfechner | 09:58 | |
| opendevreview | Verification of a change to openstack/bifrost stable/yoga failed: Use libvirt-python from the distribution on Red Hat systems https://review.opendev.org/c/openstack/bifrost/+/842302 | 10:35 |
| dtantsur | I wonder if older branches are affected.. we may need to start backports from the older version that supported CS8 | 10:37 |
| iurygregory | good morning Ironic | 10:54 |
| dtantsur | hey iurygregory | 11:04 |
| iurygregory | o/ | 11:05 |
| opendevreview | Merged openstack/tenks master: CI: Removing tenks job queue https://review.opendev.org/c/openstack/tenks/+/842607 | 11:05 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Add a metalsmith job with legacy boot https://review.opendev.org/c/openstack/ironic-python-agent/+/842085 | 11:19 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: CI: Removing ironic job queue https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/842606 | 12:33 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Add yoga tests, stop testing victoria https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/840508 | 12:33 |
| TheJulia | good morning | 12:37 |
| TheJulia | o/ MahnoorAsghar | 12:38 |
| dtantsur | TheJulia: good morning! will you be available to check https://review.opendev.org/c/openstack/ironic-python-agent/+/837784 (backport of the multipath issue) today? | 12:57 |
| dtantsur | we're a bit under pressure with it, I'll take a look as well | 12:57 |
| opendevreview | Verification of a change to openstack/bifrost stable/yoga failed: Use libvirt-python from the distribution on Red Hat systems https://review.opendev.org/c/openstack/bifrost/+/842302 | 13:02 |
| dtantsur | so, metalsmith is broken. because JSON parsing of the output no longer works. because openstacksdk started issuing a warning. and it's virtually impossible to make Python to log into stderr instead of stdout. | 13:15 |
| dtantsur | it's Monday, I want to drink AND scream | 13:15 |
| rpittau | I can share my alcoholic cough syrup | 13:16 |
| rpittau | dtantsur: I had a look at the wallaby backport for the multipath patch, if we agree on waiting for the backports to the other stable branches, I can approve before crawling to bed | 13:20 |
| iurygregory | I will be working on the other backports as soon as I have finished my meetings today | 13:23 |
| dtantsur | rpittau: I think we can approve it as an exception and try to get xena/yoga/etc in asap | 13:26 |
| rpittau | alright, let's do it | 13:26 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent stable/yoga: Multipath Hardware path handling https://review.opendev.org/c/openstack/ironic-python-agent/+/842942 | 13:55 |
| opendevreview | Aija Jauntēva proposed openstack/ironic master: Fix Redfish RAID to update raid_config https://review.opendev.org/c/openstack/ironic/+/842979 | 14:12 |
| iurygregory | ofc zuul will fail https://zuul.opendev.org/t/openstack/status#837784 | 14:30 |
| iurygregory | =( | 14:30 |
| iurygregory | why would wallaby try to install oslo.log 5.0.0 O.o | 14:32 |
| iurygregory | 2022-05-23 14:08:07.907730 | controller | 2022-05-23 14:08:07.907 | The conflict is caused by: | 14:32 |
| iurygregory | 2022-05-23 14:08:07.907910 | controller | 2022-05-23 14:08:07.907 | ironic-python-agent 8.6.1.dev10 depends on oslo.log>=3.36.0 | 14:32 |
| iurygregory | 2022-05-23 14:08:07.907929 | controller | 2022-05-23 14:08:07.907 | The user requested (constraint) oslo-log===5.0.0 | 14:32 |
| opendevreview | Verification of a change to openstack/ironic-python-agent stable/wallaby failed: Multipath Hardware path handling https://review.opendev.org/c/openstack/ironic-python-agent/+/837784 | 14:39 |
| iurygregory | (╯°□°)╯︵ ┻━┻ | 14:41 |
| dtantsur | weeeeird | 14:52 |
| dtantsur | 5.0.0>=3.36.0 for sure | 14:53 |
| dtantsur | a mirror glitch? | 14:53 |
| opendevreview | Dmitry Tantsur proposed openstack/python-ironicclient master: Fix logging in the baremetal CLI https://review.opendev.org/c/openstack/python-ironicclient/+/842986 | 14:53 |
| dtantsur | folks, this ^^^ (when released) will hopefully fix the metalsmith job | 14:53 |
| dtantsur | until then, not sure what to do. probably hacks something together.. | 14:53 |
| TheJulia | blarg | 14:56 |
| rpittau | iurygregory: that's the requirement for tempest, which is wrong as it points to master instead of wallaby | 14:57 |
| rpittau | s/requirement/upper-constraint | 14:58 |
| iurygregory | WTF | 14:58 |
| iurygregory | O.o | 14:58 |
| iurygregory | so we need to update the jobs to point to the right version? | 14:58 |
| dtantsur | tempest is branchless | 14:58 |
| rpittau | just had a quick look, not sure what's going on | 14:58 |
| iurygregory | I put recheck because it was weird... | 14:59 |
| rpittau | indeeed tempest is branchless but shouldn't it use u-c from stable branches based on hte job ? | 14:59 |
| iurygregory | let's see, but I will try to take a look | 14:59 |
| iurygregory | it should... | 14:59 |
| iurygregory | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon May 23 15:00:11 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 | o/ | 15:00 |
| dtantsur | o/ | 15:00 |
| rpittau | o/ | 15:00 |
| rpioso | o/ | 15:00 |
| matfechner | o/ | 15:00 |
| ajya | o/ | 15:00 |
| iurygregory | Hello everyone, welcome to our weekly meeting! | 15:00 |
| kamlesh6808c | o/ | 15:00 |
| iurygregory | you can find the agenda for the meeting in the wiki | 15:01 |
| iurygregory | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
| iurygregory | #topic Announcements / Reminder | 15:01 |
| TheJulia | o/ | 15:01 |
| iurygregory | #info Please provide feedback to the Zed Themes patch | 15:02 |
| iurygregory | #link https://review.opendev.org/c/openstack/ironic-specs/+/841061 | 15:02 |
| JayF | o/ | 15:02 |
| stendulker_ | o/ | 15:02 |
| * iurygregory has no idea why is on merge-conflict will check later... | 15:02 | |
| iurygregory | #info Dinner in Berlin Etherpad - Please vote in the restaurant/pubs till EOW =) | 15:03 |
| opendevreview | Verification of a change to openstack/bifrost stable/yoga failed: Use libvirt-python from the distribution on Red Hat systems https://review.opendev.org/c/openstack/bifrost/+/842302 | 15:03 |
| iurygregory | #link https://etherpad.opendev.org/p/ironic-dinner-ois2022Berlin | 15:03 |
| iurygregory | For those who will be in the OIS Berlin, we are planning to have a dinner, if you are interested please add you name in the etherpad and vote for the day you are available and the restaurants/pubs | 15:04 |
| iurygregory | does anyone have anything else to add for announcements/reminder? | 15:04 |
| iurygregory | ok, moving on | 15:06 |
| iurygregory | we don't have any action item so skipping | 15:07 |
| iurygregory | #topic Review subteam status reports | 15:07 |
| iurygregory | #link https://etherpad.opendev.org/p/IronicWhiteBoard | 15:07 |
| iurygregory | starting L65 | 15:07 |
| iurygregory | I think we can move on | 15:13 |
| dtantsur | yeah, seems so | 15:14 |
| iurygregory | #topic Deciding on priorities for the coming week | 15:14 |
| iurygregory | #link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio | 15:14 |
| iurygregory | I will be adding some backports I'm working on later today | 15:15 |
| TheJulia | Seems reasonable | 15:15 |
| kamlesh6808c | Can we put priority tag for this patch https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/838462 | 15:16 |
| dtantsur | please add https://review.opendev.org/c/openstack/ironic/+/842574 as well | 15:16 |
| iurygregory | kamlesh6808c, by any chance do you have some log execution that shows this problem happening? it would be good to have a story with it and add in the commit message =) | 15:17 |
| iurygregory | added both patches to the list | 15:18 |
| kamlesh6808c | iurygregory, Yes.Would do that.Thanks! | 15:18 |
| iurygregory | Tks! | 15:19 |
| iurygregory | we have some bifrost patches to fix CI I think | 15:19 |
| TheJulia | Has the tag been added to them? | 15:21 |
| iurygregory | no, I'm trying to find the patches to see if they got reviews / or add the tag | 15:22 |
| dtantsur | https://review.opendev.org/c/openstack/bifrost/+/842302 keeps failing in the gate | 15:22 |
| iurygregory | yeah I just saw this patch, I added the hashtag to it, just in case we need to fix something in case of a config problem etc | 15:24 |
| iurygregory | I think we can move on, if anyone has patches that will need review let us know or add the hashtag =) | 15:25 |
| iurygregory | #topic Baremetal SIG | 15:25 |
| iurygregory | #link https://etherpad.opendev.org/p/bare-metal-sig | 15:25 |
| iurygregory | arne_wiebalck, do you have any updates you would like to share? | 15:26 |
| iurygregory | I think Arne is busy today =) | 15:28 |
| TheJulia | I suspect many are busy today | 15:28 |
| iurygregory | yeah | 15:28 |
| TheJulia | Also, presentation review deadlines and such are likely coming up for internal approvals in some orgs | 15:28 |
| iurygregory | skipping rfe topic since we don't have any | 15:29 |
| iurygregory | #topic Open discussion | 15:29 |
| iurygregory | Does anyone have something that would like to discuss? | 15:30 |
| TheJulia | So, I've been asked to turn the "lets fix this nasty nova bug patch" in a spec for discussion | 15:31 |
| ebbex | a review on https://review.opendev.org/c/openstack/bifrost/+/836172 ? | 15:31 |
| TheJulia | because there are conflicting views on it as well as major differences of opinion on how it should be handled | 15:31 |
| TheJulia | #link https://review.opendev.org/c/openstack/nova-specs/+/842015 | 15:31 |
| TheJulia | If anyone wants to take a look | 15:31 |
| iurygregory | ebbex, for you case we can add the patch in the list of priority to review I would say | 15:32 |
| * iurygregory looks at the spec link | 15:33 | |
| ebbex | iurygregory: thanks :) | 15:34 |
| arne_wiebalck | sorry, was in another meeting: nothing for the SIG really :) | 15:36 |
| iurygregory | TheJulia, the bug is https://bugzilla.redhat.com/show_bug.cgi?id=2017980 ? | 15:38 |
| TheJulia | iurygregory: amongst them, yes | 15:39 |
| TheJulia | there are multiple bugs wrapped into one issue | 15:39 |
| TheJulia | Each with implications, and not fixing them together just kind of pushes the problem around the plate as it were | 15:39 |
| iurygregory | oh the other bugs in launchpad right? | 15:39 |
| TheJulia | yeah | 15:39 |
| TheJulia | There may be more bugs in RHBZ as well, I didn't go dig through the depths of it when I was writing the document up | 15:40 |
| TheJulia | I only used what was at my finger tips | 15:40 |
| iurygregory | gotcha, no worries | 15:40 |
| * iurygregory will need more coffee in a few minutes... | 15:41 | |
| TheJulia | coffee++ | 15:41 |
| iurygregory | the idea is to try to get some ironic folks to look at it and provide feedback to see if we can get a consensus on the path nova/ironic should take in the multiple issues we have documented in the spec? | 15:42 |
| TheJulia | That would be ideal yes | 15:43 |
| TheJulia | Nova has some... differing perceptions on acceptability, such as a knob to enable the bugfix is generally viewed as unacceptable | 15:44 |
| TheJulia | They also want to lean on only fixing the issues/state when manual action has occured | 15:44 |
| iurygregory | yeah =( | 15:44 |
| iurygregory | woot? O.o | 15:45 |
| TheJulia | Or automatic from explicit state changes. There are a number of worries there, so it is important to try and navigate them | 15:45 |
| TheJulia | And maybe the usage is a little different | 15:45 |
| iurygregory | do they have a way to identify it was a manual action? | 15:45 |
| TheJulia | arne_wiebalck: it would be useful if somebody from CERN could chime in | 15:45 |
| TheJulia | iurygregory: what I mean is "nova-manage ironic fix-cluster-host-fields" | 15:46 |
| TheJulia | or something along those lines | 15:46 |
| TheJulia | or explicitly set the compute services to disabled and then and only then would rebalance occur... or soemthing along those lines | 15:46 |
| TheJulia | All rather, abstract at the moment | 15:47 |
| iurygregory | I think I understood | 15:47 |
| iurygregory | I've added myself in the spec and I will try to look after finishing all backports I need to push | 15:48 |
| TheJulia | Thanks | 15:48 |
| iurygregory | np =) | 15:51 |
| iurygregory | I will run the meeting next week o/ | 15:51 |
| iurygregory | Thanks everyone! | 15:51 |
| iurygregory | #endmeeting | 15:52 |
| opendevmeet | Meeting ended Mon May 23 15:52:50 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:52 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-05-23-15.00.html | 15:52 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-05-23-15.00.txt | 15:52 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-05-23-15.00.log.html | 15:52 |
| arne_wiebalck | TheJulia: sorry ... chime in for ? | 16:04 |
| * arne_wiebalck is reading the backlog | 16:04 | |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent master: Use json for lsblk output https://review.opendev.org/c/openstack/ironic-python-agent/+/775391 | 16:05 |
| rpittau | alright, not sure I'll be active a lot online the rest of the week, we'll keep an eye on things anyway | 16:07 |
| rpittau | by everyone! o/ | 16:07 |
| arne_wiebalck | TheJulia: https://review.opendev.org/c/openstack/nova-specs/+/842015 ... I will try to have a look! | 16:12 |
| iurygregory | bye rpittau | 16:19 |
| arne_wiebalck | bye everyone o/ | 16:22 |
| dtantsur | bloody hell, ironicclient is still testing py3.6? | 16:24 |
| iurygregory | why wallaby is doing this OGW | 16:35 |
| iurygregory | =( | 16:35 |
| iurygregory | yoga seems to be fine... WHY WALLABY?! | 16:40 |
| iurygregory | (╯°□°)╯︵ ┻━┻ ┻━┻ ┻━┻ | 16:41 |
| * iurygregory - more coffee is required... | 16:41 | |
| dtantsur | what a lovely Monday | 16:41 |
| iurygregory | do we need diskimage in the src jobs for ipa? I've checked a diff from installed things when it was green/red) | 16:44 |
| dtantsur | depends on the image type, I assume | 16:48 |
| iurygregory | pip3-freeze diff (green|red)https://www.diffchecker.com/LVeqFbqN | 16:48 |
| iurygregory | ipa-tempest-uefi-redfish-vmedia-src | 16:48 |
| iurygregory | it's a bit weird that wasn't using before, but now it's | 16:49 |
| iurygregory | O.o | 16:49 |
| iurygregory | or maybe it's because of the cloud provider! | 16:49 |
| * iurygregory checks other failures | 16:49 | |
| opendevreview | Verification of a change to openstack/bifrost stable/yoga failed: Use libvirt-python from the distribution on Red Hat systems https://review.opendev.org/c/openstack/bifrost/+/842302 | 17:32 |
| iurygregory | ERROR: Package 'ironic-python-agent' requires a different Python: 3.6.8 not in '>=3.8' | 17:34 |
| iurygregory | O.o | 17:34 |
| iurygregory | why this is in wallaby wtf O.o | 17:34 |
| iurygregory | https://zuul.opendev.org/t/openstack/build/e84ce495c61f4251bbd1a7586555be64/log/job-output.txt#5398 | 17:41 |
| iurygregory | i like this! | 17:41 |
| opendevreview | Verification of a change to openstack/python-ironicclient master failed: CI: Removing ironic job queue https://review.opendev.org/c/openstack/python-ironicclient/+/842600 | 17:42 |
| opendevreview | Merged openstack/python-ironicclient master: Add Python3 zed unit tests https://review.opendev.org/c/openstack/python-ironicclient/+/832245 | 19:27 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Auto-populate lessee for deployments https://review.opendev.org/c/openstack/ironic/+/818641 | 23:21 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!