Wednesday, 2025-07-02

opendevreviewJulia Kreger proposed openstack/ironic master: WIP: Set the backend to threading.  https://review.opendev.org/c/openstack/ironic/+/95368300:20
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: Set the backend to threading.  https://review.opendev.org/c/openstack/ironic/+/95368301:30
TheJuliaso, looks like some of the tests which invoke the service are actually failing/going sideways test wise because aspects like startup checks blow up.01:31
rpittaugood morning ironic! o/05:41
rpittauuhmmmm why is https://review.opendev.org/c/openstack/bifrost/+/948245 not merged? and what "ready to submit" means?!? :/07:02
jandersgood morning Ironic o/08:13
jandersquick Q for my testing worflow: is bifrost generally reboot-proof? (relevant to some of the testing I am about to attempt)08:14
dtantsurTheJulia: I got the impression that we get a session per thread because oslo.db has some sort of a thread-local cache for session. I might be horribly wrong.08:30
dtantsurMmm, does anyone know how to increase backlog in irccloud? It's tiny, I miss some responses from the day before apparently08:55
opendevreviewAbhishek Bongale proposed openstack/ironic-tempest-plugin master: Add Tempest tests for inspection rules in Ironic  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/95176108:57
fricklerrpittau: it needs a rebase, the gerrit status is somewhere between misleading and wrong. priteau asked about that the other day already. I just guess that in order to submit the rebase you'll also need to add a DCO footer now09:13
rpittaufrickler: ok, thanks09:18
opendevreviewRiccardo Pittau proposed openstack/bifrost master: Default ansible to version 10.x  https://review.opendev.org/c/openstack/bifrost/+/94824509:19
iurygregorygood morning ironic12:22
TheJuliadtantsur: some of the tests... I was able to identify it is because of self._start_service running and fixing issues due to conductor code firing up12:48
dtantsurTheJulia: are you talking about the database still or about the later issue with tests?12:49
TheJuliastill seems to present as db issues, ultimately I suspect its going to require some evaluation of failing tests to determine if the issue is actually that the test has some unexpected behavior occuring which is presenting as database failures12:51
* TheJulia proceeds to try and wake up for meeting in a half hour12:52
dtantsurjanders: missed your question: we expect bifrost to be reboot-proof, but its testenv setup is not.12:52
dtantsurspecifically, we kill libvirt's dnsmasq during installation, but it will start back on reboot12:53
dtantsurTheJulia: I wonder if we're doing something unsafe in tests themselves (dunno, passing sessions around)12:54
TheJuliadtantsur: that is what I'm thinking, but I want to get a long lived conductor up to see how it behaves12:56
TheJuliahmm, no errors12:56
TheJuliastill seems to be sitting there happily power syncing12:57
dtantsurhappy power syncing \o/12:58
TheJuliaWe've got a weird issue with the "none" transport option, if you could take a look at it, that should get us a little close. I need to take a closer look at the single process stuff today12:59
TheJuliaI... at least now that I've kind of fixed some tests which I could get reliably erroring with just a single test run which now seem rock solid, it feels like we're on an improving track13:00
opendevreviewSyed Haseeb Ahmed proposed openstack/ironic master: Add interface name to inventory for inspection hooks  https://review.opendev.org/c/openstack/ironic/+/95341413:32
opendevreviewMahnoor Asghar proposed openstack/ironic master: Fix names of steps in comments as per function  https://review.opendev.org/c/openstack/ironic/+/95395914:14
opendevreviewMahnoor Asghar proposed openstack/ironic master: Fix names of steps in comments as per function  https://review.opendev.org/c/openstack/ironic/+/95396014:16
TheJuliamasghar: did you mean to do two separate changes?14:18
TheJuliaat a glance, it looks like the same change14:18
iurygregoryTheJulia, tks for the review yesterday o/14:29
masgharTheJulia, no, the first one had two change ids so I abandoned it14:52
masghar(Its the same change yes)14:53
TheJuliaack14:59
TheJuliawhen that happens, generally it is tracking on the last change ID, you yeah, sort of makes sense to just abandon then15:00
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: Set the backend to threading.  https://review.opendev.org/c/openstack/ironic/+/95368315:50
opendevreviewIury Gregory Melo Ferreira proposed openstack/sushy master: Don't fail to eject vmedia in case there is no vmedia  https://review.opendev.org/c/openstack/sushy/+/95273017:26
iurygregorydtantsur, should be fixed now ^ 17:26
opendevreviewVerification of a change to openstack/ironic master failed: Fix Sensor Data values  https://review.opendev.org/c/openstack/ironic/+/95339518:12
cardoeSo I think… https://review.opendev.org/c/openstack/ironic/+/952168 is more correct. But still speaking with the neutron folks about the fact that in the L2 case the port should have some kind of identification to the segment it’s bound on.18:35
cardoeI’ll be honest I haven’t run that patch set on actual metal yet. I plan to.18:36
TheJuliaI'll try to take a look soon, but might not be until next week18:55
TheJuliaI'm very heads down on eventlet stuffs18:55
TheJuliaOn a plus side, real threads with ironic :)18:55
TheJuliadtantsur: an update, a ton of our issues are largely we're just trying to do threads on some actions because of test style and invoking executors, and just a lot of substrate complexity tests don't always need, peeling some of it off or limiting it to a syncrhonous executor seems to be the way to go. Just working through each item one by one as I find them18:57
TheJuliaalso, stuff like undo a thing which is supposed to also be undone previously but .... yeah. I think eventlet was hiding a lot of stuff which is now surfacing18:58
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: Set the backend to threading.  https://review.opendev.org/c/openstack/ironic/+/95368320:24
TheJuliadtantsur: tomorrow lests chat regarding threads/databases and our tests20:41
TheJuliacid: you'll likely want to join the discussion as well20:47
TheJuliaI'll just try and get us on a call tomorrow and we'll go from there20:48

Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!