| opendevreview | Takashi Kajinami proposed openstack/ironic-python-agent master: Remove packages for TinyIPA https://review.opendev.org/c/openstack/ironic-python-agent/+/1000665 | 00:30 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/ironic-python-agent master: Apply formatter for existing release note https://review.opendev.org/c/openstack/ironic-python-agent/+/1000666 | 00:39 |
| opendevreview | Julia Kreger proposed openstack/ironic master: trivial: Remove x11vnc -ncache flag inflating framebuffer height https://review.opendev.org/c/openstack/ironic/+/1000667 | 00:43 |
| opendevreview | Takashi Kajinami proposed openstack/ironic-python-agent master: Maintain plugin requirements as optional dependencies https://review.opendev.org/c/openstack/ironic-python-agent/+/1000668 | 00:46 |
| TheJulia | stevebaker[m]: I noticed some funky numbers w/r/t vnc while playing with using it, https://review.opendev.org/c/openstack/ironic/+/1000667 seems to be the fix. | 00:58 |
| TheJulia | also, confusing when I turend off scaling for novnc's webui, it suddenly became clear that it was a super tall screen which made no sense | 01:01 |
| opendevreview | Merged openstack/python-ironicclient master: Add openstack-python3-next-jobs job https://review.opendev.org/c/openstack/python-ironicclient/+/997754 | 01:05 |
| opendevreview | Merged openstack/python-ironicclient master: Remove Python 3.10 support https://review.opendev.org/c/openstack/python-ironicclient/+/997755 | 01:08 |
| opendevreview | Takashi Kajinami proposed openstack/ironic-python-agent master: Remove burnin-network-kazoo extra https://review.opendev.org/c/openstack/ironic-python-agent/+/998721 | 03:16 |
| opendevreview | Takashi Kajinami proposed openstack/ironic-python-agent master: Maintain plugin requirements as optional dependencies https://review.opendev.org/c/openstack/ironic-python-agent/+/1000668 | 04:16 |
| rm_work | cardoe: aha yeah I saw the github notification. I'll take a loot at doing that soon, I wanted to get a little more real testing in first... I DM'd you | 04:18 |
| rm_work | *take a look | 04:18 |
| opendevreview | Takashi Kajinami proposed openstack/ironic-python-agent-builder master: Remove unused variable https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/1000674 | 04:20 |
| opendevreview | Takashi Kajinami proposed openstack/ironic-python-agent-builder master: Remove note about old pip's behavior https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/1000675 | 04:23 |
| opendevreview | Merged openstack/ironic master: trivial: Remove x11vnc -ncache flag inflating framebuffer height https://review.opendev.org/c/openstack/ironic/+/1000667 | 05:21 |
| dtantsur | Looking for a 2nd +2 on https://review.opendev.org/c/openstack/ironic/+/997998 please | 08:43 |
| opendevreview | Merged openstack/ironic master: Add state, target_provision_state, and duration_seconds to node history https://review.opendev.org/c/openstack/ironic/+/989994 | 09:00 |
| opendevreview | Merged openstack/ironic master: Populate node_history state fields https://review.opendev.org/c/openstack/ironic/+/993208 | 09:00 |
| opendevreview | Merged openstack/ironic master: Refresh the scaling guide in the reference architecture https://review.opendev.org/c/openstack/ironic/+/997998 | 09:22 |
| opendevreview | Takashi Kajinami proposed openstack/ironic-python-agent master: Remove burnin-network-kazoo extra https://review.opendev.org/c/openstack/ironic-python-agent/+/998721 | 09:23 |
| opendevreview | Merged openstack/ironic-python-agent master: Remove packages for TinyIPA https://review.opendev.org/c/openstack/ironic-python-agent/+/1000665 | 09:46 |
| opendevreview | Merged openstack/ironic-python-agent master: Apply formatter for existing release note https://review.opendev.org/c/openstack/ironic-python-agent/+/1000666 | 10:07 |
| opendevreview | Esther Domfeh proposed openstack/python-ironicclient master: Add new fields to node history detailed view https://review.opendev.org/c/openstack/python-ironicclient/+/1000705 | 10:25 |
| opendevreview | Esther Domfeh proposed openstack/python-ironicclient master: Add new fields to node history detailed view https://review.opendev.org/c/openstack/python-ironicclient/+/1000705 | 10:30 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix BIOS periodic task race that skips subsequent steps https://review.opendev.org/c/openstack/ironic/+/1000706 | 10:57 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix BIOS periodic task race that skips subsequent steps https://review.opendev.org/c/openstack/ironic/+/1000406 | 11:01 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix BIOS periodic task race that skips subsequent steps https://review.opendev.org/c/openstack/ironic/+/1000406 | 11:32 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix BIOS periodic task race that skips subsequent steps https://review.opendev.org/c/openstack/ironic/+/1000406 | 11:43 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix BIOS periodic task race that skips subsequent steps https://review.opendev.org/c/openstack/ironic/+/1000406 | 11:50 |
| janders | cardoe long time no chat o/ I wanted to ask you for help reviewing a follow up to https://review.opendev.org/c/openstack/ironic/+/977569 we worked on together a while back. From my downstream reports, it seems this change may have broken a day0 scenario where we do BIOS settings AND firmware updates. I proposed | 12:09 |
| janders | https://review.opendev.org/c/openstack/ironic/+/1000406 which should fix it (first iteration was successfully lab tested, there's been few more since but idea remains). I would be curious to hear your thoughts on this follow up, thanks in advance | 12:09 |
| cardoe | We’re running the version of the change where I had posted some tests. I know it morphed quite a bit after that and there have been a number of other changes to it in follow up patches as well. I think rpittau has an open one as well. | 12:29 |
| cardoe | I’ll gladly review what you’ve got though. | 12:31 |
| TheJulia | good morning | 13:46 |
| cardoe | janders: I agree with dtantsur's comments | 13:48 |
| TheJulia | Jay's portgroup networking support for our latest stuff has recently been updated and can use more eyes: https://review.opendev.org/c/openstack/ironic-specs/+/999448 | 13:53 |
| cardoe | Not sure if https://review.opendev.org/c/openstack/ironic/+/1000059 is something I should backport. We're gonna backport it to our install. | 14:28 |
| TheJulia | I think it is fine as long as policy and security effect still holds, but I've not had time to wrap my head around/verify | 14:33 |
| dtantsur | Well, strictly speaking we were supposed to microversion that change and not backport it :) Since we already YOLO a bit, I'll pretend I haven't seen this discussion :) | 14:43 |
| TheJulia | I guess I kind of read it as "this was not intended", at least at a cursory glance | 14:53 |
| cardoe | It was supposed to be available with the original introduction of the API from my reading of the code. | 14:54 |
| cardoe | It appears that when we added the schema validation bits we made a mistake. | 14:54 |
| JayF | yeah, I wouldn't have thought to microversion it since it was unintended behavior | 14:54 |
| cardoe | So it was there for 1 release and then it disappeared. | 14:55 |
| JayF | I would be +1 to fix + backport | 14:55 |
| dtantsur | Okay, then I agree for sure. I assumed it was an addition | 14:56 |
| dtantsur | Well, the PR does not make it clear that what it's fixing is a regression, not a missing feature | 14:57 |
| dtantsur | If the commit message and the release note mentioned something along the lines of "it was there for 1 release and then it disappeared", it would be an easy one | 14:57 |
| JayF | you are correct :) +1 sounds like cardoe has concrete feedback | 14:58 |
| cardoe | I can rewrite the note. | 14:58 |
| dtantsur | thx! | 14:59 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: Add spec for sonic and nvue driver replacements. https://review.opendev.org/c/openstack/ironic-specs/+/998363 | 15:50 |
| opendevreview | cid proposed openstack/ironic master: Maintenance: add the node_maintenance table https://review.opendev.org/c/openstack/ironic/+/1000751 | 16:59 |
| opendevreview | cid proposed openstack/ironic master: Maintenance: NodeMaintenance object and DB-API https://review.opendev.org/c/openstack/ironic/+/1000752 | 16:59 |
| opendevreview | cid proposed openstack/ironic master: Maintenance: mirror the boolean into node_maintenance https://review.opendev.org/c/openstack/ironic/+/1000753 | 16:59 |
| opendevreview | cid proposed openstack/ironic master: Maintenance: document maintenance in node history https://review.opendev.org/c/openstack/ironic/+/1000754 | 16:59 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Do not require a container steps file to build https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/998390 | 19:02 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Verify registry TLS by default in the podman element https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/999178 | 19:06 |
| opendevreview | Jay Jahns proposed openstack/ironic stable/2026.1: Redfish: retry transient 409 conflict on power-on https://review.opendev.org/c/openstack/ironic/+/1000767 | 19:13 |
| JayF | Does anyone have any OOB contact for shermanm? Trying to get https://review.opendev.org/c/openstack/ironic/+/999701 tested for cid so we can errata the OSSA | 20:24 |
| * JayF found him on Scientific SIG slack | 20:27 | |
| cardoe | TheJulia: no no specs for you docs | 20:32 |
| TheJulia | cardoe: can I get a translation for that? | 20:32 |
| JayF | I think he wants more prose reviewed rather than uploaded into gerrit ;) | 20:33 |
| JayF | lol | 20:33 |
| cardoe | random brain synapse to English? | 20:33 |
| cardoe | JayF: yes. | 20:33 |
| cardoe | TheJulia: for some reason I just got the notification for the ironic-spec push and was just trying to say less specs and more docs review/writing | 20:34 |
| JayF | yeah but that requires toddlers to not exist, in the colloquial | 20:34 |
| JayF | (that's a mildly inside joke) | 20:34 |
| TheJulia | cardoe: oh, I... umm... rebased? | 20:35 |
| TheJulia | I'll blame my corgi | 20:35 |
| TheJulia | and the RV next to us getting new tires today which is making him quite upset | 20:35 |
| JayF | https://review.opendev.org/c/openstack/security-doc/+/1000784 RFR -- this is the OSSN around Container Hardware Manager | 22:02 |
| JayF | stevebaker[m]: https://review.opendev.org/c/openstack/security-doc/+/1000784 LN 33. This is a nontrivial portion of the reason I was -1 to adding podman/docker by default. With these fixes merged, as long as we only make the change for 2026.2+, I can be onboard | 22:44 |
| stevebaker[m] | JayF: OK thanks, much appreciated | 22:51 |
| JayF | IPA-B is branched, right? | 22:53 |
| JayF | That is possible, to add something to ramdisk N but not <N? | 22:54 |
| JayF | I think so... | 22:54 |
| stevebaker[m] | Yes I see branches | 23:09 |
| janders | cardoe noted w/r/t https://review.opendev.org/c/openstack/ironic/+/1000406, will re-spin it | 23:15 |
| janders | thank you | 23:15 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!