| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-06-15 | |||
| 06:51:37 | opendevreview | wanghongtao proposed openstack/nova master: Fix operator precedence in limit_check https://review.opendev.org/c/openstack/nova/+/993297 | |
| 07:26:21 | opendevreview | Shalini Srivastava proposed openstack/nova master: Filter invalid kwargs in volume_api.create to prevent TypeError https://review.opendev.org/c/openstack/nova/+/993306 | |
| 07:39:36 | opendevreview | Kamil Sambor proposed openstack/nova master: Restore 'fork' start method for daemon mode on Python 3.14 https://review.opendev.org/c/openstack/nova/+/987815 | |
| 07:57:59 | opendevreview | Ashish Gupta proposed openstack/placement master: tests: Add connection parameter to Database and PlacementFixture https://review.opendev.org/c/openstack/placement/+/993106 | |
| 09:17:42 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Speed up release notes builds https://review.opendev.org/c/openstack/nova/+/989212 | |
| 09:21:24 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Speed up release notes builds https://review.opendev.org/c/openstack/nova/+/989212 | |
| 09:35:46 | opendevreview | Joan Gilabert proposed openstack/nova master: Add mtty/mdpy support for testing fake mdevs https://review.opendev.org/c/openstack/nova/+/898100 | |
| 09:35:47 | opendevreview | Joan Gilabert proposed openstack/nova master: Rename vtpm job and add mtty support for vpgu test https://review.opendev.org/c/openstack/nova/+/922140 | |
| 10:32:58 | opendevreview | Shalini Srivastava proposed openstack/nova master: Filter invalid kwargs in volume_api.create to prevent TypeError https://review.opendev.org/c/openstack/nova/+/993306 | |
| 11:17:43 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron Ports https://review.opendev.org/c/openstack/nova/+/969298 | |
| 11:19:54 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron security-groups https://review.opendev.org/c/openstack/nova/+/981141 | |
| 11:33:26 | opendevreview | Joan Gilabert proposed openstack/nova master: Rename vtpm job and add mtty support for vpgu test https://review.opendev.org/c/openstack/nova/+/922140 | |
| 12:25:02 | bauzas | sean-k-mooney[m]: gibi; could either of you look at https://review.opendev.org/q/topic:%22bug/2153425%22 ? I'm all good | |
| 12:28:48 | opendevreview | Joan Gilabert proposed openstack/nova master: Rename vtpm job and add mtty support for vpgu test https://review.opendev.org/c/openstack/nova/+/922140 | |
| 12:45:51 | sean-k-mooney | bauzas: sorry was on a call but i can take a look quickly | |
| 12:46:20 | sean-k-mooney | bauzas: so i have been watching the updates on this passivly | |
| 12:46:35 | sean-k-mooney | bauzas: this is not quite enough to fix numa vswitch correct? | |
| 12:47:01 | sean-k-mooney | its related but this is only for the live migration path | |
| 12:47:06 | bauzas | it should be good | |
| 12:47:14 | sean-k-mooney | we still have the issue with the compute not havig it on spwawn | |
| 12:47:21 | bauzas | at least we have a customer asking for it | |
| 12:47:25 | sean-k-mooney | or cold migrate | |
| 12:48:00 | sean-k-mooney | bauzas: what i mean is this will help with live migration but there is more work needed to make sure that all code path include the limits | |
| 12:49:18 | bauzas | we can ask that for a follow-up | |
| 12:49:23 | sean-k-mooney | bugs.launchpad.net/nova/+bug/2145135 we alos have a gap for https://bugs.launchpad.net/nova/+bug/1855332 | |
| 12:49:40 | sean-k-mooney | bauzas: sure im not saying its inscope fo those change | |
| 12:49:55 | sean-k-mooney | but you were previsoly working on bugs.launchpad.net/nova/+bug/2145135 a month or two ago | |
| 12:50:02 | sean-k-mooney | that why im bringing the topic up | |
| 12:50:28 | sean-k-mooney | anyway ill review it now i just want to clear on the scope | |
| 12:52:14 | bauzas | ack thanks | |
| 12:53:16 | sean-k-mooney | just an fyi the expecation in the bug are also incorrect https://bugs.launchpad.net/nova/+bug/2153425 | |
| 12:53:21 | sean-k-mooney | setting only `{‘hw:numa_nodes’: ‘1’}` | |
| 12:53:42 | sean-k-mooney | does not claim any cpu or memroy in the host numa tracker but it does end up with the vm being pinned | |
| 12:53:50 | sean-k-mooney | to a host numa cell | |
| 12:54:22 | sean-k-mooney | we do not expect any numa blancing to happen with that config unless you also request hw:mem_page_size in teh falvor or image | |
| 12:58:12 | sean-k-mooney | bauzas: so im sorry to say but thet bug is technially inviald and its a psudo feature request. | |
| 13:00:00 | bauzas | but the issue still remains | |
| 13:00:10 | sean-k-mooney | the flavor is not valid | |
| 13:00:39 | sean-k-mooney | we can add supprot for numa blancin on shared cpus only but that was never supproted by nova | |
| 13:00:47 | sean-k-mooney | we supprot it for pinned cpus | |
| 13:00:50 | sean-k-mooney | and for guest memory | |
| 13:01:01 | sean-k-mooney | but not for shared cpus | |
| 13:01:08 | sean-k-mooney | so im not agains supproting that | |
| 13:01:33 | sean-k-mooney | but the expecation today is that we wont consider the cpu allcation ration at the numa node level today | |
| 13:09:51 | bauzas | honestly, I don't know what to say :( | |
| 13:10:31 | sean-k-mooney | bauzas: im not agaisnt building thei fucntionlatiy into nova but its really a new feature | |
| 13:11:07 | sean-k-mooney | and if we are to do it we need to do it for spawn and cold migrate and unshleve ectra | |
| 13:12:47 | sean-k-mooney | thisis not the first tirm that i have had this converatoion with operators or custoemrs and pointed out that simply requesting hw:numa_node and nothing else is only vaild if your usign file backed memory as we do not do numa aware shared cpu placement so we will not blance between numa node | |
| 13:13:05 | sean-k-mooney | to be clear even with file backed memory it not a good idea | |
| 13:13:51 | sean-k-mooney | if other really want to tack this as a bug we need to add supprot for this on spwan and other move operaiotn and more fucntional tests to cover that | |
| 13:23:10 | sean-k-mooney | https://bugs.launchpad.net/nova/+bug/1792985 and https://bugs.launchpad.net/nova/+bug/1439247 are relevent context for why the current flavor is invlaid | |
| 13:24:22 | sean-k-mooney | usign file backed memory prevetn the OOM kill but without also susign cpu pinnign it does nto adress the cpu blancing part | |
| 13:24:47 | sean-k-mooney | which is what ye were tryign to address with that new bug | |
| 13:50:12 | opendevreview | Joan Gilabert proposed openstack/nova master: Rename vtpm job and add mtty support for vpgu test https://review.opendev.org/c/openstack/nova/+/922140 | |
| 14:00:02 | opendevreview | Merged openstack/nova stable/2025.1: Add reproducer test for bug 2105896 https://review.opendev.org/c/openstack/nova/+/989535 | |
| 15:26:48 | dansmith | sean-k-mooney: unrelated to the numa aspect, why is it not right to set self.limits before we call that compute-via-RPC check? it seems that the code expects limits to be set,. | |
| 15:26:58 | dansmith | but I'm not sure when else they would be, but I'm sure I'm missing something | |
| 15:27:46 | sean-k-mooney | dansmith: that part isned nessarly wrong but the expecation that nova shoudl blance based on that is. also nova shoudl not be multiplying the core or memory on a given numa node by the ratios | |
| 15:27:55 | Uggla | reminder: upstream meeting in ~30mn | |
| 15:28:45 | sean-k-mooney | well the memofy shoudl not be multipled the cores could be viable | |
| 15:29:36 | dansmith | sean-k-mooney: yeah okay I was going to say.. the assumption in the patch being wrong makes sense, not arguing with that | |
| 15:30:06 | dansmith | it just looks to me like in the force case we probably need to be setting self.limits | |
| 15:30:35 | sean-k-mooney | dansmith: so there is a seprate bug in teh conductor | |
| 15:30:43 | sean-k-mooney | where the limt are nto passed to the compute | |
| 15:30:50 | dansmith | yeah | |
| 15:30:56 | sean-k-mooney | so the cshcdule does nto actully select the placmenet on the host | |
| 15:30:58 | sean-k-mooney | the compute does | |
| 15:31:09 | sean-k-mooney | so untile we etenbr recreate them there ro pass them | |
| 15:31:28 | sean-k-mooney | the two could still disagree | |
| 15:31:30 | dansmith | "select the placement" meaning the numa placement? | |
| 15:31:39 | dansmith | we're still calling scheduler even in the force case right? | |
| 15:32:01 | sean-k-mooney | if you mean force live migration i belive if you use the old api where that exist | |
| 15:32:14 | sean-k-mooney | we only check that the host exist and skipp all the filters | |
| 15:32:27 | sean-k-mooney | but if you use the non force microverion where you can pass a host | |
| 15:32:33 | sean-k-mooney | then we run true the filters | |
| 15:33:03 | dansmith | sean-k-mooney: L100 here https://review.opendev.org/c/openstack/nova/+/990212/5/nova/conductor/tasks/live_migrate.py# | |
| 15:33:07 | sean-k-mooney | so if you use the old foce api then we just trust you that it fits and send it to the compute | |
| 15:33:48 | dansmith | oh I guess in that case we don't even have limits to set | |
| 15:34:16 | sean-k-mooney | right because we are currently relying on a sideffect fo the numa topltoy filter to create them | |
| 15:34:25 | dansmith | yeah, okay | |
| 15:34:28 | sean-k-mooney | which is the bug with numa vswthces | |
| 15:36:27 | sean-k-mooney | without checking i woudl have to assume we are effectivly applying our defaults or no multipler at all when the limits are not pass today? | |
| 15:37:08 | sean-k-mooney | which while not ideal for memory and disk its at least <=1.0 now | |
| 15:37:52 | bauzas | sean-k-mooney: the problem is that we say 'no' to something is a tribal knowledge miss | |
| 15:37:59 | sean-k-mooney | in practice we are relying on placment to enfoce the multipler globally on the compute node | |
| 15:38:18 | bauzas | sean-k-mooney: if the scheduler provides limits and then the conductor doesn't use it, then it's a bug | |
| 15:38:27 | sean-k-mooney | bauzas: this is somethign we have discussed about changign a few time and when i have suggested it we wer econcerd about upgrade impact | |
| 15:38:36 | bauzas | but I understand your point, the scheduler shouldn't accept this | |
| 15:38:43 | sean-k-mooney | bauzas: dansmith just to be clear | |
| 15:38:53 | sean-k-mooney | im not agaisnt fixign the fact we shoudl pass limit when we dont | |
| 15:39:22 | sean-k-mooney | but im not ok with impliy hw:numa_nodes=1 with nothign else is a vlaid numa flavor that shoudl be balanced | |
| 15:39:36 | sean-k-mooney | at elast not until that works for all code paths | |
| 15:39:46 | bauzas | so we should provide a HTTP400 if an operator creates a wrong flavor then | |
| 15:39:56 | bauzas | at least when creating the instance | |
| 15:40:00 | sean-k-mooney | we coudl btu we rejected that in the past | |
| 15:40:15 | sean-k-mooney | we dicuss this when stephen was addign the falvor metadata validation | |
| 15:40:39 | sean-k-mooney | the minitum valid numa flavor to day has hw:mem_page_size set to anything | |
| 15:41:01 | bauzas | but that's a tribal knowledge, right? | |
| 15:41:02 | sean-k-mooney | everythign else builds form that baseline otherwise your vm will get OOM killed eventually | |
| 15:41:32 | sean-k-mooney | bauzas: sort of | |