| opendevreview | Steve Baker proposed openstack/bifrost master: ansible-lint enforce fqcn[action] https://review.opendev.org/c/openstack/bifrost/+/881306 | 05:12 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/bifrost master: ansible-lint enforce fqcn[action-core] https://review.opendev.org/c/openstack/bifrost/+/881307 | 05:12 |
| opendevreview | Steve Baker proposed openstack/bifrost master: ansible-lint enforce key-order[task] https://review.opendev.org/c/openstack/bifrost/+/881308 | 05:12 |
| opendevreview | Steve Baker proposed openstack/bifrost master: ansible-lint enforce meta-no-tags https://review.opendev.org/c/openstack/bifrost/+/881309 | 05:12 |
| opendevreview | Steve Baker proposed openstack/bifrost master: ansible-lint suppress jinja[spacing] https://review.opendev.org/c/openstack/bifrost/+/881330 | 05:12 |
| opendevreview | Steve Baker proposed openstack/bifrost master: ansible-lint enforce all name rules https://review.opendev.org/c/openstack/bifrost/+/881331 | 05:12 |
| opendevreview | Steve Baker proposed openstack/bifrost master: ansible-lint enforce no-free-form https://review.opendev.org/c/openstack/bifrost/+/881332 | 05:12 |
| opendevreview | Steve Baker proposed openstack/bifrost master: Remove extra symbols accidentally added https://review.opendev.org/c/openstack/bifrost/+/879547 | 05:31 |
| opendevreview | Steve Baker proposed openstack/bifrost master: ansible-lint enforce fqcn[action] https://review.opendev.org/c/openstack/bifrost/+/881306 | 05:31 |
| opendevreview | Steve Baker proposed openstack/bifrost master: ansible-lint enforce fqcn[action-core] https://review.opendev.org/c/openstack/bifrost/+/881307 | 05:31 |
| opendevreview | Steve Baker proposed openstack/bifrost master: ansible-lint enforce key-order[task] https://review.opendev.org/c/openstack/bifrost/+/881308 | 05:31 |
| opendevreview | Steve Baker proposed openstack/bifrost master: ansible-lint enforce meta-no-tags https://review.opendev.org/c/openstack/bifrost/+/881309 | 05:31 |
| opendevreview | Steve Baker proposed openstack/bifrost master: ansible-lint suppress jinja[spacing] https://review.opendev.org/c/openstack/bifrost/+/881330 | 05:31 |
| opendevreview | Steve Baker proposed openstack/bifrost master: ansible-lint enforce all name rules https://review.opendev.org/c/openstack/bifrost/+/881331 | 05:31 |
| opendevreview | Steve Baker proposed openstack/bifrost master: ansible-lint enforce no-free-form https://review.opendev.org/c/openstack/bifrost/+/881332 | 05:31 |
| frickler | stevebaker[m]: TheJulia: those jobs are failing because tooz 4.0.0 no longer supports py3.8 (focal). time to bump testing to distros matching the current TC plan, i.e. jammy | 06:05 |
| stevebaker[m] | Ah thanks frickler | 06:06 |
| rpittau | good morning ironic! o/ | 07:10 |
| rpittau | JayF: thanks! yeah, let's merge those and do a release first? so we can move ironic to jammy | 07:10 |
| rpittau | TheJulia stevebaker[m] dtantsur iurygregory when you have a moment today please approve the virtualpdu patches so we can create a new python 3.x compatible release | 07:10 |
| stevebaker[m] | looking | 07:12 |
| stevebaker[m] | rpittau: here is one more you could review https://review.opendev.org/c/openstack/virtualpdu/+/881276 | 07:16 |
| rpittau | stevebaker[m]: then I'm going to abandon my upgrade to hacking 5.x https://review.opendev.org/c/openstack/virtualpdu/+/880914 | 07:18 |
| opendevreview | Merged openstack/virtualpdu master: Remove ignore F811 flake8 rule https://review.opendev.org/c/openstack/virtualpdu/+/880913 | 07:19 |
| stevebaker[m] | I'm surprised gerrit didn't flag that as a conflict | 07:19 |
| rpittau | yep, weird | 07:19 |
| rpittau | but I was being conservative, I wanted to go directly to 6.x :) | 07:19 |
| rpittau | I'm going to abandon a bunch of old changes in virtualpdu | 07:20 |
| stevebaker[m] | yeah good plan | 07:25 |
| opendevreview | Merged openstack/virtualpdu master: Remove useless config files https://review.opendev.org/c/openstack/virtualpdu/+/880915 | 07:28 |
| opendevreview | Merged openstack/virtualpdu master: Add doc8 test to pep8 https://review.opendev.org/c/openstack/virtualpdu/+/880917 | 07:28 |
| opendevreview | Verification of a change to openstack/virtualpdu master failed: Update pep8 deps - hacking to 6.0.0 https://review.opendev.org/c/openstack/virtualpdu/+/881276 | 07:28 |
| rpittau | there you go :D | 07:29 |
| rpittau | gerrit realized the conflict | 07:29 |
| frickler | identical patches don't conflict, eventually one of them will just become an empty patch when rebased on top of the other | 07:30 |
| rpittau | yeah, but in this case the conflict is legit | 07:31 |
| opendevreview | Riccardo Pittau proposed openstack/virtualpdu master: Update pep8 deps - hacking to 6.0.0 https://review.opendev.org/c/openstack/virtualpdu/+/881276 | 07:31 |
| rpittau | stevebaker[m]: if you have a moment plesae reapprove https://review.opendev.org/c/openstack/virtualpdu/+/881276, I just simply rebased it so it's ok to +W, thanks | 07:52 |
| stevebaker[m] | done | 07:53 |
| opendevreview | Merged openstack/virtualpdu master: Update pep8 deps - hacking to 6.0.0 https://review.opendev.org/c/openstack/virtualpdu/+/881276 | 08:05 |
| Sandzwerg[m] | Might be useful considering the pypi access discussion from some time ago: https://blog.pypi.org/posts/2023-04-23-introducing-pypi-organizations/ | 08:25 |
| rpittau | Sandzwerg[m]: interesting, I guess would be worth discussing that during the weekly meeting? | 08:40 |
| Sandzwerg[m] | Yeah. Not sure if the whole Openstack community would go a single organization or each project | 08:46 |
| rpittau | this is probably something for the TC, probably JayF can advise here | 08:55 |
| opendevreview | Vanou Ishii proposed openstack/ironic master: [iRMC] Fix parse_driver_info bug enforcing SNMP v3 under FIPS mode https://review.opendev.org/c/openstack/ironic/+/881358 | 10:52 |
| opendevreview | Vanou Ishii proposed openstack/ironic master: [iRMC] Fix typo of Python string format in log message https://review.opendev.org/c/openstack/ironic/+/881362 | 11:04 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/yoga: Align iRMC driver with Ironic's default boot_mode https://review.opendev.org/c/openstack/ironic/+/866626 | 11:45 |
| *** iurygregory_ is now known as iurygregory | 11:55 | |
| iurygregory | good morning ironic | 12:01 |
| TheJulia | good morning | 12:56 |
| rpittau | hey TheJulia :) | 12:58 |
| rpittau | FYI I've requested a release of virtualpdu here https://review.opendev.org/c/openstack/releases/+/881367 | 12:59 |
| rpittau | until we get the new version we can't upgrade the base jobs of ironic to jammy merging https://review.opendev.org/c/openstack/ironic/+/869052, meaning everything will fail because of the tooz incompatibility | 12:59 |
| rpittau | Happy Monday! | 12:59 |
| iurygregory | rpittau, +1 | 13:02 |
| rpittau | thanks iurygregory :) | 13:02 |
| opendevreview | Verification of a change to openstack/bifrost master failed: remove unused variables https://review.opendev.org/c/openstack/bifrost/+/874518 | 14:36 |
| TheJulia | rpittau: fun! | 14:47 |
| dtantsur | rpittau: please make the release team aware of the urgency (if not already) | 14:54 |
| dtantsur | good morning TheJulia, iurygregory | 14:54 |
| rpittau | dtantsur: yeah, if that doesn't get attention before EOD I'm going to poke them | 14:54 |
| JayF | o/ | 15:00 |
| JayF | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Apr 24 15:00:27 2023 UTC and is due to finish in 60 minutes. The chair is JayF. 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 |
| JayF | Happy Monday, folks. Who all is here? | 15:00 |
| dtantsur | o/ | 15:00 |
| rpittau | o/ | 15:00 |
| matfechner | o/ | 15:00 |
| JayF | #note Standing reminder: hashtag patches that are ready for review with ironic-week-prio, and cores, please review things in the dashboard for priority. | 15:01 |
| JayF | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:01 |
| TheJulia | o/ | 15:01 |
| JayF | No action items from last meeting to follow up on; skipping that agenda item. | 15:01 |
| JayF | #topic Review Ironic CI Status | 15:02 |
| JayF | How are things? I saw rpittau commenting we need https://review.opendev.org/c/openstack/ironic/+/869052 to land to fix jobs? | 15:02 |
| rpittau | yeah | 15:02 |
| rpittau | we need jammy | 15:02 |
| rpittau | I asked for a virtualpdu release, that should allow us to move forward | 15:02 |
| JayF | by asked for, I assume you mean PR -> releases? | 15:03 |
| rpittau | yes here's the change https://review.opendev.org/c/openstack/releases/+/881367 | 15:03 |
| JayF | got it, thanks for that | 15:03 |
| JayF | anything else gate-wise for us to be aware of? | 15:03 |
| hjensas | o/ | 15:04 |
| JayF | Guess not. Moving on! | 15:05 |
| JayF | #topic pypi maintainer audit | 15:05 |
| JayF | I got another response, and followed up on folks. I'm going to pull this item from our agenda because the folks left who need to take action haven't been to one of these in years :D | 15:06 |
| rpittau | JayF: btw about that Sandzwerg[m] today pointed out this interesting change in pypi https://blog.pypi.org/posts/2023-04-23-introducing-pypi-organizations/ | 15:06 |
| JayF | I saw that, it's interesting but not really a solution to the base problem we have | 15:07 |
| rpittau | ok | 15:07 |
| JayF | that might allow infra to do more secure things, but really our problem right now is logical: humans have access to publish things, and noly robots should :) | 15:07 |
| JayF | #topic VirtualPDU | 15:08 |
| JayF | I think this is a leftover? | 15:08 |
| rpittau | that doesn't sound eery at all :D | 15:08 |
| JayF | rpittau: it's very eerie, and it's why we've focused on it :| | 15:08 |
| JayF | VirtualPDU is moved, the github redirect works since I pushed a patch | 15:08 |
| JayF | I think there's still some love needed in that repo but we've cleared a 'good enough' bar, I think. | 15:08 |
| JayF | Any comments on vpdu? | 15:09 |
| rpittau | thanks for pushing that forward, I think we're in a good place with vpdu at the moment | 15:09 |
| JayF | yeah there's some things we can do if folks have time | 15:09 |
| JayF | fix the docs links in the readme, make docs publish w/openstack logos, etc | 15:09 |
| JayF | but nothing left that's urgent or functional related aiui | 15:09 |
| rpittau | right, I can have a look at that during the week | 15:10 |
| JayF | thanks o/ | 15:10 |
| JayF | #topic Open Discussion | 15:10 |
| JayF | anything else? | 15:10 |
| JayF | Aight, that concludes the weekly meeting. Have a good one folks o/ | 15:12 |
| JayF | #endmeeting | 15:12 |
| opendevmeet | Meeting ended Mon Apr 24 15:12:12 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:12 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-04-24-15.00.html | 15:12 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-04-24-15.00.txt | 15:12 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-04-24-15.00.log.html | 15:12 |
| TheJulia | Just as a general FYI, we've had some folks chasing down issues at RH with the embedded Matrox/Aspeed graphics controllers on server class machines. Specifically the "nomodeset" kernel command line option is hyper bad for performance upon any graphics update. | 15:26 |
| rpittau | good night folks! o/ | 15:39 |
| opendevreview | Ebbex proposed openstack/bifrost master: remove iniparse system packages requirement https://review.opendev.org/c/openstack/bifrost/+/874520 | 17:01 |
| TheJulia | err, there is an issue with proliantutils, it it pins the upper bound of pysnmp to <5.0.0, but when the pypi package takeover goes through, proliantutils will artificially hold us back unless we can get that fixed, but which will likely have to await that happening | 18:09 |
| opendevreview | Chris Krelle proposed openstack/ironic master: Add ablity to power off nodes in clean failed https://review.opendev.org/c/openstack/ironic/+/880165 | 19:12 |
| opendevreview | Jay Faulkner proposed openstack/ironic-specs master: Add 2023.2 Workitems discussed at Ironic PTG https://review.opendev.org/c/openstack/ironic-specs/+/879381 | 21:14 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-specs master: Firmware Interface https://review.opendev.org/c/openstack/ironic-specs/+/878505 | 21:53 |
| TheJulia | NobodyCam: +2'ed, but two quick nits, happy to fix them | 23:09 |
| NobodyCam | Oh I can fix " -> `` and a o -> O | 23:15 |
| opendevreview | Chris Krelle proposed openstack/ironic master: Add ablity to power off nodes in clean failed https://review.opendev.org/c/openstack/ironic/+/880165 | 23:21 |
| NobodyCam | I think that should cover it | 23:21 |
| NobodyCam | :) | 23:21 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Disable MD5 image checksums https://review.opendev.org/c/openstack/ironic-python-agent/+/865190 | 23:55 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!