| vanou | good morning ironic | 00:19 |
|---|---|---|
| opendevreview | Merged openstack/ironic-python-agent master: Fix UTF-16 result handling for efibootmgr https://review.opendev.org/c/openstack/ironic-python-agent/+/879897 | 00:49 |
| rpittau | good morning ironic! o/ | 08:18 |
| rpittau | happy to see CI in an acceptable state again :) | 08:18 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Add network interface speed to the inventory https://review.opendev.org/c/openstack/ironic-python-agent/+/879156 | 08:39 |
| dtantsur | rpittau: do we need https://review.opendev.org/c/openstack/ironic/+/869052 still? | 08:40 |
| rpittau | we do, but I will reenable the snmp job | 08:59 |
| rpittau | I put a -W and added a comment | 09:00 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Use jammy for base jobs https://review.opendev.org/c/openstack/ironic/+/869052 | 10:29 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Use jammy for base jobs https://review.opendev.org/c/openstack/ironic/+/869052 | 10:30 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Use jammy for base jobs https://review.opendev.org/c/openstack/ironic/+/869052 | 10:31 |
| iurygregory | good morning Ironic | 11:40 |
| opendevreview | Merged openstack/ironic-python-agent master: Deprecate LLDP in inventory in favour of a new collector https://review.opendev.org/c/openstack/ironic-python-agent/+/881462 | 12:05 |
| dtantsur | TheJulia: hi, could you take a brief look at https://review.opendev.org/c/openstack/ironic-specs/+/878230 and https://review.opendev.org/c/openstack/ironic-specs/+/880941 in case you notice anything outrageous? The former has 2x +2 already, so very close. | 12:37 |
| opendevreview | Riccardo Pittau proposed openstack/virtualpdu master: Use openstackdocs theme for docs https://review.opendev.org/c/openstack/virtualpdu/+/881483 | 12:44 |
| TheJulia | dtantsur: ack, writing up an email at the moment, will take a look shortly | 13:51 |
| TheJulia | dtantsur: w/r/t https://review.opendev.org/c/openstack/ironic-specs/+/878230 I guess the only thing missing is "how an operator would", and I guess the undertone is "they would need to recreate, but my quick skimming doesn't seem to explicitly tell me that | 14:18 |
| TheJulia | dtantsur: some sort of idea on what operators should expect process wise, would be appreciated and be useful, but it can also be up in the air I guess at this point | 14:21 |
| rpittau | TheJulia: I went ahead and asked to add lextudio pysnmp and pyasn1 to openstack requirements, I think it's the right path forward considering that they're all part of the same ecosystem https://review.opendev.org/c/openstack/requirements/+/881539 | 14:21 |
| rpittau | for the packages, the changes from the current RPMs to the new ones are minimal since the code is essentially the same | 14:21 |
| TheJulia | dtantsur: so the plan for https://review.opendev.org/c/openstack/ironic-specs/+/880941/1/specs/approved/merge-inspector.rst is a specific queue that all copies of the new service would listen to instead of the host specific queue style we use today with conductors? | 14:25 |
| TheJulia | rpittau: pysnmpcrypto would also likely be good from a standpoint of vanou's needs | 14:25 |
| TheJulia | jfyi | 14:25 |
| rpittau | ah right, I need to check other packages as well | 14:25 |
| TheJulia | we could let the ecosystem sort out pysnmpcrypto packaging | 14:26 |
| TheJulia | its not like we can go back and retroactively add that | 14:26 |
| rpittau | true | 14:26 |
| opendevreview | Merged openstack/ironic-specs master: Migrate inspection rules from Inspector https://review.opendev.org/c/openstack/ironic-specs/+/878230 | 14:31 |
| dtantsur | TheJulia: re queue: yes | 14:51 |
| dtantsur | I need to think if we can do automated or semi-automated rules migration | 14:52 |
| opendevreview | Merged openstack/virtualbmc master: Use generic openstack python3 tests https://review.opendev.org/c/openstack/virtualbmc/+/880916 | 15:02 |
| opendevreview | Jay Faulkner proposed openstack/ironic-prometheus-exporter stable/wallaby: CI: Remove bad job, remove queue key https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/860181 | 15:20 |
| zigo | I have ironic-inspector-conductor failing with this in the logs: | 15:38 |
| zigo | openstack.exceptions.HttpException: HttpException: 503: Server Error for url: https://<hostname>/v1/nodes?fields=uuid, 503 Service Unavailable: No server is available to handle this request. | 15:38 |
| zigo | FYI, my endpoint is: https://<hostname>/baremetal/v1/nodes?fields=uuid | 15:38 |
| zigo | Looks like the inspector failed to keep the /baremetal path in the URL. How can I convince him my catalogue is correct? :) | 15:38 |
| zigo | Appart from that, my setup with separated inspector-api and conductor looks like working pretty well! :) | 15:40 |
| * zigo goes back home now | 15:40 | |
| zigo | Feel free to answer when I'm left, I always read the backlog of my bouncer. | 15:40 |
| dtantsur | zigo: The URL must be coming from keystoneauth/openstacksdk, unless you specified something explicitly in [ironic] | 15:50 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/2023.1: Fix UTF-16 result handling for efibootmgr https://review.opendev.org/c/openstack/ironic-python-agent/+/881657 | 15:57 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/zed: Fix UTF-16 result handling for efibootmgr https://review.opendev.org/c/openstack/ironic-python-agent/+/881658 | 15:57 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/yoga: Fix UTF-16 result handling for efibootmgr https://review.opendev.org/c/openstack/ironic-python-agent/+/881659 | 15:57 |
| opendevreview | Merged openstack/ironic master: Add ablity to power off nodes in clean failed https://review.opendev.org/c/openstack/ironic/+/880165 | 16:07 |
| rpittau | good night! o/ | 16:17 |
| TheJulia | dtantsur: as long as your thinking of it at this point, I'm good, I know it is not a clean cut thing | 16:26 |
| dtantsur | yep | 16:26 |
| opendevreview | Merged openstack/sushy master: Update pep8 dep hacking to latest: v6 https://review.opendev.org/c/openstack/sushy/+/881273 | 16:53 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Add a non-voting ubuntu arm64 build check job https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/880854 | 17:11 |
| opendevreview | Merged openstack/sushy-tools master: Update pep8 deps - now on latest hacking v6 https://review.opendev.org/c/openstack/sushy-tools/+/881274 | 17:13 |
| JayF | https://review.opendev.org/c/openstack/networking-generic-switch/+/867874 someone please land this before CI breaks again lol | 18:05 |
| JayF | stable + CI fix should only need 1 +2A to land it | 18:05 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DPU modeling - parent_node DB/Model/API https://review.opendev.org/c/openstack/ironic/+/880114 | 18:15 |
| opendevreview | Merged openstack/ironic-lib master: Upgrade to latest hacking - v6 https://review.opendev.org/c/openstack/ironic-lib/+/881260 | 18:15 |
| TheJulia | done | 18:16 |
| JayF | thank you :) | 18:16 |
| TheJulia | JayF: how goes shard support for openstacksdk? | 18:16 |
| JayF | landed | 18:16 |
| TheJulia | sweet | 18:16 |
| JayF | literally zero negative feedback | 18:17 |
| TheJulia | has it been released yet? | 18:17 |
| JayF | one patchset | 18:17 |
| JayF | maybe the first time ever it's happen | 18:17 |
| TheJulia | nice | 18:17 |
| JayF | I've not tried to release it; but they are holding releases right now anyway | 18:17 |
| JayF | was going to check up on that next week | 18:17 |
| TheJulia | ok, so earliest nova shard stuff can start back up is once that is released | 18:17 |
| JayF | (there's some migration of tarballs (?)) | 18:17 |
| TheJulia | okay | 18:17 |
| JayF | I mean, it's the earliest any of it can /land/ | 18:17 |
| JayF | right? | 18:17 |
| TheJulia | well... it may never land if we don't start :) | 18:18 |
| JayF | I also have to test the support again. It occurred to me I never tested filtering, only that the shard attr is on the object | 18:18 |
| JayF | I'm saying, we can use master openstacksdk to develop against, yeah? | 18:18 |
| TheJulia | kind of, but it won't work in CI | 18:18 |
| TheJulia | and even locally we would need to hand-install | 18:19 |
| TheJulia | anyway! | 18:19 |
| JayF | progress being made is the point :) | 18:19 |
| JayF | you could also +1 the spec if you want to :D | 18:19 |
| TheJulia | don't feel bad about filtering, doing the parent node stuff I posted a little while ago, I kept thinking "oh, I need to do this" and dound I did realier on | 18:19 |
| TheJulia | earlier on | 18:19 |
| TheJulia | ... aslo needed to add rbac checks, that was *fun* | 18:20 |
| JayF | honestly my take on projects I'm less familiar with is "I'll do a best effort, and trust the experts to review out if anything is weird" | 18:20 |
| JayF | and so it landing w/o any negative feedback is just candy for anxiety that I missed something LOL | 18:20 |
| opendevreview | Merged openstack/ironic master: Set ironic-grenade to wait 120 seconds https://review.opendev.org/c/openstack/ironic/+/879675 | 18:52 |
| opendevreview | Merged openstack/networking-generic-switch stable/yoga: Fix CI https://review.opendev.org/c/openstack/networking-generic-switch/+/867874 | 20:01 |
| TheJulia | NobodyCam: https://review.opendev.org/c/openstack/ironic/+/880114 from lunch discussion | 20:48 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/xena: Fix UTF-16 result handling for efibootmgr https://review.opendev.org/c/openstack/ironic-python-agent/+/881762 | 21:28 |
| TheJulia | if I can get 2 core reviewers to take a look at ^, it would be appreciated, the efi_utils stuff was created after xena during yoga development | 21:29 |
| TheJulia | after that I'll continue cherry-picking | 21:29 |
| JayF | looking | 21:30 |
| JayF | lgtm | 21:31 |
| JayF | not a lot of things there that could go wrong | 21:31 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/wallaby: Fix UTF-16 result handling for efibootmgr https://review.opendev.org/c/openstack/ironic-python-agent/+/881763 | 21:48 |
| TheJulia | yeah | 21:48 |
| TheJulia | since that, ^ | 21:49 |
| TheJulia | :) | 21:49 |
| TheJulia | ugh, looks like we've still got some lingering sqlalchemy lock issue | 22:09 |
| TheJulia | *looks* like it might be near the hash ring code too :\ | 22:09 |
| opendevreview | Merged openstack/ironic-prometheus-exporter stable/wallaby: CI: Remove bad job, remove queue key https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/860181 | 22:35 |
| opendevreview | Merged openstack/ironic master: Remove all references to the "cpus" property https://review.opendev.org/c/openstack/ironic/+/878421 | 23:10 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: port creation lock https://review.opendev.org/c/openstack/ironic/+/881765 | 23:18 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Remove autocommit, again. https://review.opendev.org/c/openstack/ironic/+/862832 | 23:19 |
| TheJulia | so, my theory is our frequent exclusive lock pattern shoots ourselves in the foot | 23:33 |
| TheJulia | ... *and* we do it in times when we *really* don't need to | 23:33 |
| TheJulia | so if we can dial those back, hopefully happiness | 23:33 |
| TheJulia | anyway, those results will give me a half decent idea if we can reproduce in the same pass/pattern | 23:35 |
| opendevreview | Merged openstack/ironic master: [iRMC] Fix typo of Python string format in log message https://review.opendev.org/c/openstack/ironic/+/881362 | 23:39 |
| vanou | Thanks TheJulia and rpittau for SNMP things. I check candidates you mentioned before. | 23:42 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!