| JayF | ironic-tempest-ipa-wholedisk-direct-tinyipa-multinode https://zuul.opendev.org/t/openstack/build/56bb08753ff64c68b63d0f19d450974e : SUCCESS in 1h 44m 01s | 00:15 |
|---|---|---|
| JayF | I'm going to be honest, i'm going to not even look at the log, recheck the ironic side of this, and take the wins when they come lol | 00:16 |
| JayF | I'll obviously check tomorrow :D | 00:16 |
| iurygregory | WIN \o/ | 00:59 |
| opendevreview | Merged openstack/sushy master: Add firmware information on NetworkAdapter resource https://review.opendev.org/c/openstack/sushy/+/950923 | 01:14 |
| opendevreview | Takashi Kajinami proposed openstack/ironic-ui master: Drop environments for nose https://review.opendev.org/c/openstack/ironic-ui/+/950940 | 05:58 |
| adam-metal3 | hello ironic o/ | 06:14 |
| rpittau | good morning ironic! o/ | 06:50 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [CI] Fix libvirt network names in metal3 job logs collection https://review.opendev.org/c/openstack/ironic/+/950950 | 07:23 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] Run metal3 integration job using UEFI boot (default) https://review.opendev.org/c/openstack/ironic/+/939694 | 07:24 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [CI] metal3 job back to voting https://review.opendev.org/c/openstack/ironic/+/950333 | 07:25 |
| opendevreview | Matt Crees proposed openstack/bifrost master: Support IPA download with authenticated requests https://review.opendev.org/c/openstack/bifrost/+/945324 | 07:57 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Drop explicit executor argument https://review.opendev.org/c/openstack/ironic/+/950957 | 08:01 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Build ironic-python-agent DIB image with CentOS Stream 10 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/950965 | 09:10 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Build ironic-python-agent DIB image with CentOS Stream 10 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/950965 | 09:11 |
| opendevreview | Queensly Kyerewaa Acheampongmaa proposed openstack/sushy-tools master: [WIP] Test updateability of Redfish DateTime fields using PATCH https://review.opendev.org/c/openstack/sushy-tools/+/950925 | 09:35 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Drop duplicate logic https://review.opendev.org/c/openstack/ironic/+/950983 | 12:34 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] Run metal3 integration job using UEFI boot (default) https://review.opendev.org/c/openstack/ironic/+/939694 | 12:49 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Remove warning filter for old oslo.db https://review.opendev.org/c/openstack/ironic/+/950987 | 12:54 |
| TheJulia | good morning folks! Going back to JayF's comment yesterday, https://review.opendev.org/c/openstack/networking-generic-switch/+/950559 could use some reviews | 13:52 |
| TheJulia | JayF: clarity needed on https://review.opendev.org/c/openstack/networking-generic-switch/+/950559 | 13:54 |
| JayF | TheJulia: looking at the ordering of things, we 100% can't do a restart in extra | 14:06 |
| JayF | TheJulia: literally ironic was creating nodes before we got around to our neutron restart | 14:06 |
| JayF | TheJulia: which made me realize: I think we get a restart for free after we put configs in place | 14:06 |
| TheJulia | so the same very race condition I sort of realize already exists | 14:06 |
| TheJulia | just in a different shape at present | 14:06 |
| TheJulia | wheeeeeeeee | 14:06 |
| TheJulia | In that case, cool I guess | 14:07 |
| JayF | TheJulia: the job results seem to back that; I was going to update my TODO to say something like "Ideally we'd validate a restart has happened if it hasn't already" | 14:07 |
| opendevreview | Merged openstack/ironic master: Drop duplicate logic https://review.opendev.org/c/openstack/ironic/+/950983 | 14:07 |
| JayF | as long as we reliably win the race, I suggest we land it once I make the comment clearer | 14:07 |
| JayF | then bug against devstack/neutron to provide guidance on how out-of-tree plugins are to configure | 14:07 |
| JayF | the comment update will also serve as a second run for more data | 14:07 |
| TheJulia | fwiw, I rechecked it like 15 minutes ago for that data point collection | 14:09 |
| JayF | I'll let the underlying job return results, even if not reported to zuul, before I kick it with a patchset then | 14:09 |
| TheJulia | k | 14:09 |
| JayF | cardoe: once CI is fixed; ngs/nbm need the pre-commit update as well. If you won't have time for it lmk and I can try to make time. | 14:15 |
| JayF | ngs-tempest-dlm already passed, waiting for the longer job | 14:23 |
| opendevreview | Jay Faulkner proposed openstack/networking-baremetal master: Remove explicitly use of eventlet https://review.opendev.org/c/openstack/networking-baremetal/+/947985 | 14:24 |
| opendevreview | Jay Faulkner proposed openstack/networking-baremetal master: Remove explicit use of eventlet https://review.opendev.org/c/openstack/networking-baremetal/+/947985 | 14:27 |
| cardoe | JayF: I think I already made some patches to add it to nbm. | 14:31 |
| cardoe | They are in the queue. | 14:31 |
| JayF | oh? | 14:31 |
| JayF | oh awesome, I'll look once CI is sane | 14:31 |
| JayF | I saw there was a tox-codepsell job failing there | 14:31 |
| cardoe | CI was broken. | 14:32 |
| JayF | s/was/is/ | 14:35 |
| TheJulia | CI is very much like a Schrödinger's Cat. | 14:48 |
| TheJulia | iurygregory: do you have a downstream bug w/r/t idrac 10 gear? I want to put a thing on my team's backlog so we're at least semi-aware of it and can follow-up | 15:39 |
| opendevreview | Jay Faulkner proposed openstack/networking-generic-switch master: ci: workaround neutron's move to uwsgi only https://review.opendev.org/c/openstack/networking-generic-switch/+/950559 | 15:46 |
| JayF | TheJulia + other cores: ^ is approvable | 15:47 |
| TheJulia | Anyone have our ironic removal etherpad link handy? | 16:31 |
| dtantsur | I hope you meant https://etherpad.opendev.org/p/ironic-eventlet-removal | 16:31 |
| dtantsur | TheJulia: ^^ :) | 16:31 |
| TheJulia | are we officially eventlet free in IPA now?! | 16:33 |
| JayF | have been for weeks | 16:33 |
| JayF | cid finished that up with the knowledge drop about cheroot from dtantsur | 16:34 |
| TheJulia | cool cool | 16:35 |
| JayF | jfyi I'm publishing https://www.youtube.com/watch?v=eSivOQth1ms today about OSSA-2025-001 | 16:35 |
| TheJulia | Regarding Ironic, I guess I'm wondering what items I might be able to add to the downstream backlog to support the upstream work. I guess having a clearer picture might help. Like I know 10/11 on the etherpad are links under review, we likely need to be a little more verbose on high level status so it is clear what could be jumped on to help or is not actiely being looked at/considered right now | 16:38 |
| JayF | I think that's the struggle overall, isn't it? That many of those items are just opening the door to finding out if there are real nasty issues lurking within | 16:38 |
| TheJulia | Somewhat yeah | 16:39 |
| JayF | so it's hard to break down in a reasonable way due to not having enough info | 16:39 |
| * JayF is really bad at ^^^ that stuff even in the simpler cases where we're not ripping out a decade of tech debt in a month or six | 16:39 | |
| TheJulia | like item 12, I suspect needs a quick revisit, but we might be super close to having eventlet mostly purged from unit testing aside from the stuff which will obviously need to change as code is updated/refactored as part of the earlier items | 16:40 |
| TheJulia | super hard to really convey that in any short way other than "maybe 85-90% done, needs a revisit once 10/11 are done" | 16:40 |
| TheJulia | I guess, what is the biggest hurdle and how can we decompose that | 16:41 |
| TheJulia | Like #1, I feel like there was a change to start that?! | 16:41 |
| JayF | #1 is literally just importing a conf module and mapping it into ironic-specific configs, right? | 16:42 |
| TheJulia | https://review.opendev.org/c/openstack/ironic/+/943678 is likely the patch I was thinking of | 16:43 |
| dtantsur | TheJulia: double-checking my findings by someone else deeply familiar with Ironic (aka you) would help a ton | 16:43 |
| TheJulia | so... I guess as a starting point, I'll make a spike to chew on and try and generate more items | 16:44 |
| TheJulia | That will also help raise awareness on folks in my team who don't live on IRC | 16:44 |
| dtantsur | yep | 16:44 |
| TheJulia | Hi, I'm Julia, and I'm an IrcCloud customer. | 16:44 |
| JayF | I'm also working to finalize a GR-OSS commitment of contribution for these items this week | 16:44 |
| TheJulia | (easier than saying: "Hi, I'm Julia and I'm an IRC addict") | 16:45 |
| JayF | I kinda determined for my sabbatical I might just have to drop OFTC entirely | 16:45 |
| JayF | because I don't have the discipline to detach otherwise | 16:45 |
| TheJulia | JayF: knowing that about yourself speaks volumes and does indeed say, you need to drop off IRC for your sabbatical | 16:48 |
| TheJulia | I... mostly have the disipline to detach, but yeah | 16:49 |
| JayF | like, sometimes we have interesting problems, and part of taking time away is to let the desire to solve those kinda interesting problems build back up again lol | 16:54 |
| TheJulia | ++ | 16:55 |
| JayF | although like I told dtantsur in DM, I might have a small sushy-tools adjacent project I try (a nanokvm backend) | 16:55 |
| JayF | but that's only happening if sushy-tools as-is runs on risc-v, because that's a whole rabbithole I want no part of if not lol | 16:55 |
| dtantsur | if they can run Fedora on risc-v, there is some Python at least | 16:56 |
| JayF | doing power control on them is GPIO-trivial https://github.com/sipeed/NanoKVM/blob/main/server/config/hardware.go#L20 | 16:56 |
| JayF | oh, the existing stuff has python on it | 16:56 |
| JayF | I'm mainly thinking about libraries | 16:56 |
| dtantsur | bcrypt needs rust (which, again, is commonly used on risc-v but worth checking your stuff) | 16:59 |
| dtantsur | everything else is very straightforward | 16:59 |
| JayF | nice | 17:02 |
| rm_work[m] | I asked before about debug logging, about to enable the overall loglevel=debug ... but was there any real HARM in setting the ipmi.debug flag? I saw some comments about it possibly printing misleading stuff? | 17:10 |
| JayF | it shows ipmitool output | 17:11 |
| JayF | ipmi as a protocol is dumb enough that a plaintext reading of almost anything is misleading | 17:11 |
| JayF | I think that's 99% of what that's warning about. | 17:11 |
| rm_work[m] | lol ok | 17:12 |
| rm_work[m] | is there any reason I wouldn't want to enable that though just in case it shows something useful? like might it print passwords or something dumb that would end up in our log storage? | 17:13 |
| JayF | Yes. | 17:14 |
| JayF | You only want to enable it in specific cases when you are troubleshooting IPMI. | 17:14 |
| JayF | Ironic debug logging is noisy; I would never suggest enabling it unconditionally in prod | 17:14 |
| rm_work[m] | k, it's just that I literally have to wait a month almost between changes here | 17:15 |
| rm_work[m] | so if I miss this window tonight and turns out I want it on... | 17:15 |
| rm_work[m] | I get to wait like three weeks | 17:15 |
| JayF | then it sounds like you need better hardware validation labs :| | 17:15 |
| JayF | there is no real reason to think you'd need ipmi debugging info if ipmitool works | 17:16 |
| rm_work[m] | kk | 17:17 |
| TheJulia | Yeah, I'd only enable ipmi debugging, which *also* if memory serves asks ipmitool to also do debug logging | 17:24 |
| TheJulia | And honestly... ipmi use should really be avoided | 17:24 |
| JayF | lol | 17:41 |
| JayF | whoops wrong window | 17:41 |
| JayF | I just published a video summary of OSSA-2025-001 to the GR-OSS youtube channel if anyone wants to have a look: https://www.youtube.com/watch?v=eSivOQth1ms | 18:09 |
| JayF | ah, I said that in here earlier, it's actually public now :) | 18:09 |
| TheJulia | A firebirds jersey | 18:15 |
| JayF | with "Wright" on the back :) | 18:21 |
| TheJulia | heh | 18:36 |
| opendevreview | Merged openstack/networking-generic-switch master: ci: workaround neutron's move to uwsgi only https://review.opendev.org/c/openstack/networking-generic-switch/+/950559 | 18:54 |
| JayF | \o/ | 18:58 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: [ci] Restore ovn-ipv6 job to voting https://review.opendev.org/c/openstack/ironic/+/951021 | 18:59 |
| JayF | ^^ | 19:00 |
| TheJulia | woot | 19:05 |
| opendevreview | Mithun Krishnan Umesan proposed openstack/networking-generic-switch master: Improve NGS documentation <Jira: OSPRH-15369> https://review.opendev.org/c/openstack/networking-generic-switch/+/951022 | 19:22 |
| opendevreview | Mithun Krishnan Umesan proposed openstack/networking-generic-switch master: Improve NGS documentation. Some minor nitpicks and changes https://review.opendev.org/c/openstack/networking-generic-switch/+/951022 | 19:29 |
| opendevreview | Jay Faulkner proposed openstack/networking-generic-switch master: Remove explicit use of eventlet https://review.opendev.org/c/openstack/networking-generic-switch/+/951026 | 19:46 |
| opendevreview | Jay Faulkner proposed openstack/networking-baremetal master: Remove explicit use of eventlet https://review.opendev.org/c/openstack/networking-baremetal/+/947985 | 19:52 |
| JayF | https://github.com/openstack/devstack/blob/master/lib/apache#L242 devstack developers get a gold star for using all iniset in the create_uwsgi_config \o/ | 19:54 |
| * JayF goes to do the NGS followup | 19:54 | |
| opendevreview | Jay Faulkner proposed openstack/networking-generic-switch master: Make devstack runs slightly more deterministic https://review.opendev.org/c/openstack/networking-generic-switch/+/951027 | 19:56 |
| iurygregory | TheJulia, hey o/ for some reason my notifications for irc was disable, so I missed your ping earlier I will send you the bug on slack o/ | 20:23 |
| JayF | cardoe: https://review.opendev.org/q/project:openstack/networking-generic-switch+status:open what am I missing? I don't see patches here? | 22:10 |
| iurygregory | Just to check, anyone planning on submitting a Getting Started with Ironic to OIS? I don't want to jump on anyone foots | 22:39 |
| JayF | I have one submission in for OIS; based on my "Invisible Work of OpenStack" blogpost series | 22:40 |
| JayF | networking-baremetal gate is broken on what appears to be devstack plugin changes, too | 22:41 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!