| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-04-02 | |||
| 08:23:25 | sean-k-mooney | the only neutron driver that might work woudl be the macvtap driver | |
| 08:23:43 | sean-k-mooney | and that effectivly unmaintaiend | |
| 08:24:30 | sean-k-mooney | so unless you go linux on m2+ i dont see a path to kvm/hardware virt supprot | |
| 08:24:36 | tafkamax | ok thx for info | |
| 10:03:02 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet with threading in novncproxy https://review.opendev.org/c/openstack/nova/+/976089 | |
| 10:03:02 | opendevreview | Kamil Sambor proposed openstack/nova master: [vncproxy]Handle ssl.wrap_socket removal in py312 https://review.opendev.org/c/openstack/nova/+/955915 | |
| 10:03:03 | opendevreview | Kamil Sambor proposed openstack/nova master: Change nova-alt-configurations job https://review.opendev.org/c/openstack/nova/+/983179 | |
| 10:05:39 | opendevreview | Kamil Sambor proposed openstack/nova master: [vncproxy]Handle ssl.wrap_socket removal in py312 https://review.opendev.org/c/openstack/nova/+/955915 | |
| 10:05:40 | opendevreview | Kamil Sambor proposed openstack/nova master: Change nova-alt-configurations job https://review.opendev.org/c/openstack/nova/+/983179 | |
| 10:05:40 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet with threading in novncproxy https://review.opendev.org/c/openstack/nova/+/976089 | |
| 14:28:54 | gibi | JayF: o/ would NoVNCConsoleTestJSON tempest tests work in an job with ironic virt driver, like in ironic-tempest-bios-ipmi-autodetect ? https://github.com/openstack/tempest/blob/9307558eb0a4248fbbc02b9489132b6156ffef61/tempest/api/compute/servers/test_novnc.py#L26 | |
| 14:29:36 | gibi | I'm asking this as we are touching stuff in that codepath here https://review.opendev.org/c/openstack/nova/+/955915 and it would be nice to see if this does not break ironic novnc integration | |
| 14:59:58 | JayF | gibi: stevebaker or TheJulia might know a thing about that | |
| 15:00:52 | JayF | gibi: I ping'd steve in the Ironic channel | |
| 15:01:08 | TheJulia | I’m basically out today, but I can look tomorrow. I think Steve is out due to the holiday as well. | |
| 15:02:36 | gibi | thanks. it is not urgent. | |
| 15:02:41 | gibi | enjoy your time off | |
| 15:02:41 | TheJulia | That being said, I think we exercise it in the gate job now so changes shouldn’t break it, but I can look deeply when not sitting in a doctors waiting room | |
| 15:02:53 | TheJulia | Eh, not a fun day off :( | |
| 15:03:07 | gibi | ohh, sorry :( | |
| 15:03:08 | TheJulia | But thanks! | |
| 15:03:58 | gibi | so far I haven't find an ironic job where the result contained the vnc test cases but mabye I'm looking at the wrong way | |
| 15:07:59 | JayF | gibi: I'd say this, especially this early in the cycle: put up a depend-on change in Ironic, if our CI passes and you broke us, that's an us problem and we have months to unearth it | |
| 15:11:15 | gibi | JayF: OK, I can do that easily. Thanks for the idea | |
| 15:44:26 | sean-k-mooney | gibi: do we have new resulte with teh autotest job | |
| 15:44:41 | sean-k-mooney | *autodetect job | |
| 15:45:07 | sean-k-mooney | when i was review ign the nova patch i had to manually trace it as it was not easy to prove that it worked otherwise | |
| 15:45:27 | gibi | sean-k-mooney: yes we have | |
| 15:45:45 | gibi | but that job does not include the tempest vnc test | |
| 15:45:51 | sean-k-mooney | https://zuul.opendev.org/t/openstack/build/0a8f54843e134d21afe261089421c5ad | |
| 15:45:55 | gibi | hence my above disucssion with th eironic folks | |
| 15:47:02 | sean-k-mooney | that sound like a regression becuase it used too | |
| 15:48:20 | sean-k-mooney | maybe that was before we dorped the depns on ectra | |
| 15:49:02 | sean-k-mooney | i did review at least one tempet run which i tough was on th enova patch wit it workign end ot end before | |
| 15:56:14 | sean-k-mooney | gibi: https://review.opendev.org/c/openstack/ironic/+/983219/1 did you push an entirly empty commit? | |
| 15:56:32 | sean-k-mooney | in gerrit that litcally only has a commit message for me | |
| 15:57:26 | sean-k-mooney | it did tirgger jobs in anycase im jsut wonderign if that is a gerrit rendering bug | |
| 15:58:35 | gibi | sean-k-mooney: it is an empty commit yes | |
| 15:58:49 | gibi | I can update that to have some change in it | |
| 15:58:57 | sean-k-mooney | no need | |
| 15:59:09 | sean-k-mooney | i just didnt know you coudl have entirly empty commits | |
| 15:59:28 | gibi | git by default rejectes it but you can use --allow-empty :) | |
| 15:59:50 | sean-k-mooney | oh ok | |
| 16:00:24 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/942528/comments/42b2fbf7_78b47e56 was where i taced this before by th eway | |
| 16:01:05 | sean-k-mooney | so in the past tempest was callign the console | |
| 16:02:00 | sean-k-mooney | no matches in the autodetect version | |
| 16:07:06 | sean-k-mooney | gibi: ok https://github.com/openstack/ironic-tempest-plugin/commit/29cd41eb3659138baa8d3eab06797fe2189ac7dc | |
| 16:07:24 | sean-k-mooney | the test was reverted so we need ot unrevert it | |
| 16:07:37 | sean-k-mooney | or rever tthe nova feature for partity :P | |
| 16:08:00 | gibi | :P | |
| 16:10:34 | sean-k-mooney | so you shoudl be able to test it with a ~DNM to nova with a depens-on to https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/982077 | |
| 16:11:25 | sean-k-mooney | or a dnm on that repo with a depens on agains the nova change either way works | |
| 16:13:37 | sean-k-mooney | actully no | |
| 16:13:56 | sean-k-mooney | well. that does not appre to currenlty be runign the test as a sperate test | |
| 16:14:08 | sean-k-mooney | it might be part of ironic_tempest_plugin.tests.scenario.test_baremetal_basic_ops.BaremetalBasicOpsAndRescue | |
| 16:14:13 | sean-k-mooney | as a sideffect | |
| 16:14:32 | sean-k-mooney | ill check but this shoudl be a sperate test so its clearly visabel in the tempest report | |
| 16:15:22 | sean-k-mooney | ok its embded in the largere test not its own thing... | |
| 16:15:36 | sean-k-mooney | which was partly why it was so hard to debug before | |
| 16:22:25 | sean-k-mooney | gibi: on second tought this sound like a next week us problem. i left a -1 on the ironic-tempst-patch since it currenly not actully validating the console works | |
| 16:24:23 | gibi | Thanks. | |
| 16:24:45 | gibi | It is definitely not earlier than next week us problem for me ow | |
| 16:24:46 | gibi | now | |
| 16:25:38 | sean-k-mooney | im checkign ont thing while im at it | |
| 16:25:53 | sean-k-mooney | we had 2 jobs in diffent pipelien one was testign with ironic sharding | |
| 16:26:01 | sean-k-mooney | and the other was testing without it to make sure both work | |
| 16:26:56 | sean-k-mooney | ok this one does not have the shared key defiend so hopefully our other ironic job still does | |
| 16:27:19 | opendevreview | Clif Houck proposed openstack/nova master: [WIP] perf(ironic): eliminate O(N²) ProviderTree deepcopy at startup (B6) https://review.opendev.org/c/openstack/nova/+/980676 | |
| 16:27:28 | sean-k-mooney | ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa | |
| 16:28:35 | opendevreview | Clif Houck proposed openstack/nova master: perf(ironic): eliminate O(N²) ProviderTree deepcopy at startup (B6) https://review.opendev.org/c/openstack/nova/+/980676 | |
| 16:29:49 | sean-k-mooney | ah its ironic-tempest-ipa-wholedisk-direct-tinyipa-multinode-shard | |
| 16:29:53 | clif | JayF: sean-k-mooney ^ I cleaned up that commit a bit and now tests pass for me locally. The change looks reasonable to me and I'm somewhat convinced that it provides a speedup | |
| 16:29:55 | sean-k-mooney | and that in the weekly perodic | |
| 16:31:01 | sean-k-mooney | ack so you pulled that out onto master directly cool | |
| 16:31:09 | clif | yep | |
| 16:31:38 | clif | unit tests needed a small change to pass due to new arguments to expected calls | |
| 16:32:04 | clif | my caveat is I'm still very new to the nova codebase in general | |
| 16:32:07 | sean-k-mooney | ah makes sense | |
| 16:32:58 | sean-k-mooney | this likely need more test coverage but did you have a chace to do any manual testing of this with a real or simulated ironic? | |
| 16:33:24 | sean-k-mooney | jay mention there is a db simulator thing in ironic but i didnt have a change to dig into it | |
| 16:33:40 | clif | I haven't done manual testing with it no, I can take a closer look at test coverage | |
| 16:35:39 | sean-k-mooney | we liekly dont need/want the simulator in this change but we might want to add some simpel unit test that create a repesteive provide tyree cache with 1, 10, 100 nodes and do some very simple scale tests | |
| 16:36:26 | JayF | going from "working ironic devstack" -> "working ironic devstack with lots of fake driver nodes" is trivial by hand | |
| 16:36:33 | JayF | I am actually working today on hooking it up via devstack config | |
| 16:36:48 | sean-k-mooney | oh cool | |
| 16:37:35 | sean-k-mooney | so that should be simpler to tst we litclly jsut need to stop and start nova and see how long it take to compelte if so | |
| 16:37:49 | sean-k-mooney | nova -> the nova-compute | |
| 16:37:55 | JayF | yeah | |
| 16:38:06 | JayF | honestly the culture here has always seemed to only count proof if it runs in CI | |
| 16:38:12 | JayF | so I wans't even going to attempt a manual test | |
| 16:38:26 | sean-k-mooney | well yes if its not in ci we assume its broken | |
| 16:38:45 | sean-k-mooney | but i jsut wanted to be sure we saw a speedup before we spent too much time on this | |
| 16:38:54 | sean-k-mooney | i.e. to make sure it really not a test artifact | |
| 16:39:20 | sean-k-mooney | so a one of manual test to confirm build confindence in the aproch | |
| 16:42:01 | clif | I don't mind attempting manual testing, I will say the change logically makes sense to me. Doing less work should *almost* always result in a speed increase | |
| 16:42:48 | sean-k-mooney | clif: yep what im hoping the manaul test will show is how big or small this is to the overall tiem | |
| 16:42:52 | sean-k-mooney | in a real system | |
| 16:43:10 | sean-k-mooney | the next speed up to try woudl be hte mutli threading patch | |
| 16:43:17 | sean-k-mooney | those are where i think most of it will come form | |
| 16:43:26 | sean-k-mooney | out of the inifally few | |