| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-09-17 | |||
| 11:10:58 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove unnecessary action method prefix https://review.opendev.org/c/openstack/nova/+/958863 | |
| 12:23:48 | gibi | can I get some stable reviews on https://review.opendev.org/q/topic:%22bug/2115905%22 ? | |
| 12:24:52 | opendevreview | Takashi Kajinami proposed openstack/nova stable/2024.2: libvirt: Disable VMCoreInfo device for SEV-encrypted instances https://review.opendev.org/c/openstack/nova/+/959913 | |
| 12:25:44 | opendevreview | Takashi Kajinami proposed openstack/nova stable/2024.1: libvirt: Disable VMCoreInfo device for SEV-encrypted instances https://review.opendev.org/c/openstack/nova/+/959914 | |
| 15:00:26 | MichielPiscaer[m] | Hi, I'm searching for some documentation on about live-migration? I would like to know what decision tree it uses to determine the destination host when I let "Openstack" choose the host. | |
| 15:05:06 | sean-k-mooney | MichielPiscaer[m]: so there isnt a descion tree as such | |
| 15:05:42 | sean-k-mooney | the saem proceess that is used to select a hsot when intially creat the vm is performed by the schdluer with the only excption being the current hsot is excluded | |
| 15:06:20 | sean-k-mooney | MichielPiscaer[m]: so https://docs.openstack.org/nova/latest/admin/scheduling.html is the correct entry point in our docs to review | |
| 15:07:01 | sean-k-mooney | effectivly when you ask nova to migrate a vm, the api calls the conductor which then calls the schdluer to select a destination | |
| 15:07:55 | sean-k-mooney | the schduler use the info in teh reqeust spec form the rpc call to first call the placement service for a set of allcaiton candate based on reqcoue and trait request computeed form the flavor and image | |
| 15:08:24 | sean-k-mooney | the the filters run over the candiate host aplying difent policies liek numa affinity | |
| 15:08:46 | sean-k-mooney | finally teh weigher sort the restulant list of hosts | |
| 15:09:05 | sean-k-mooney | with default config we will then take the first host form the list | |
| 15:09:23 | sean-k-mooney | but you can apply a ranadomn suffel over the top n candiates via config | |
| 15:09:45 | sean-k-mooney | when you pass a host its mostly the same wrokflow the only real change is we only consier that host as a candiate | |
| 15:10:39 | sean-k-mooney | MichielPiscaer[m]: so there is no finite descison tree baked in in code. it depend on the filter, weghers adn cofnig option you define | |
| 15:29:14 | opendevreview | Takashi Kajinami proposed openstack/nova stable/2024.1: libvirt: Disable VMCoreInfo device for SEV-encrypted instances https://review.opendev.org/c/openstack/nova/+/959914 | |
| 15:31:38 | MichielPiscaer[m] | Thanks for the answer | |
| 15:52:31 | opendevreview | Merged openstack/nova stable/2025.1: hypervisors: Optimize uptime retrieval for better performance https://review.opendev.org/c/openstack/nova/+/960409 | |
| 21:10:03 | opendevreview | Merged openstack/nova stable/2024.2: libvirt: Disable VMCoreInfo device for SEV-encrypted instances https://review.opendev.org/c/openstack/nova/+/959913 | |
| 22:28:18 | opendevreview | Merged openstack/nova master: nova-conductor puts instance in error state https://review.opendev.org/c/openstack/nova/+/901655 | |
| #openstack-nova - 2025-09-18 | |||
| 00:45:50 | opendevreview | Merged openstack/nova stable/2024.2: hypervisors: Optimize uptime retrieval for better performance https://review.opendev.org/c/openstack/nova/+/960410 | |
| 01:08:47 | opendevreview | melanie witt proposed openstack/nova-specs master: Re-propose vTPM live migration https://review.opendev.org/c/openstack/nova-specs/+/961564 | |
| 06:29:45 | opendevreview | yangjianfeng proposed openstack/nova master: Support reusing existing port binding during instance live-migration https://review.opendev.org/c/openstack/nova/+/961593 | |
| 06:50:52 | opendevreview | yangjianfeng proposed openstack/nova master: Support reusing existing port binding during instance live-migration https://review.opendev.org/c/openstack/nova/+/961593 | |
| 07:42:10 | opendevreview | Julien LE JEUNE proposed openstack/nova stable/2025.1: Adds regression test for bug LP#2044235 https://review.opendev.org/c/openstack/nova/+/961601 | |
| 07:43:48 | opendevreview | Julien LE JEUNE proposed openstack/nova stable/2024.2: Adds regression test for bug LP#2044235 https://review.opendev.org/c/openstack/nova/+/961602 | |
| 07:47:10 | opendevreview | Julien LE JEUNE proposed openstack/nova stable/2024.1: Adds regression test for bug LP#2044235 https://review.opendev.org/c/openstack/nova/+/961603 | |
| 08:49:39 | opendevreview | Merged openstack/nova stable/2024.1: hypervisors: Optimize uptime retrieval for better performance https://review.opendev.org/c/openstack/nova/+/960411 | |
| 10:12:52 | opendevreview | Sylvain Bauza proposed openstack/nova master: Regression test for ImagePropsWeigher due to missing context https://review.opendev.org/c/openstack/nova/+/961615 | |
| 11:57:34 | opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Mention socket in pci numa affinity extra spec help https://review.opendev.org/c/openstack/nova/+/961634 | |
| 13:23:21 | opendevreview | Josephine Seifert proposed openstack/nova stable/2024.2: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/961655 | |
| 14:03:55 | bauzas | dansmith: ping, if you are free of meetings :) | |
| 14:04:01 | dansmith | bauzas: just a sec | |
| 14:04:04 | bauzas | cool | |
| 14:05:58 | dansmith | bauzas: so, fill_faults() and fill_metadata() are operations on the list object, just like .save() on an instance or anything else | |
| 14:06:09 | dansmith | they expect to use the context that the object maintains | |
| 14:06:31 | bauzas | I see, those are not getters | |
| 14:06:33 | dansmith | since you're sort of yanking InstanceList into existence (and not getting one as the result of a DB query), you don't have a context set on the object itself | |
| 14:06:49 | dansmith | it's not wrong, it's just not the intended workflow | |
| 14:07:07 | dansmith | so just create the list like insts = InstanceList(context, ...) and then it will have it internally | |
| 14:07:27 | bauzas | sure, but then I wonder which context to pass | |
| 14:07:34 | dansmith | but you need to use an admin context, not the one for the current boot request, since that one won't be able to read all the instances you're going to survey | |
| 14:07:44 | bauzas | context for others, we speak about https://bugs.launchpad.net/nova/+bug/2125052 | |
| 14:08:12 | bauzas | yup, I figured it, so I probably need to instantiate a fully new admin context | |
| 14:08:38 | bauzas | we have the user request context stored in the requestspec, but as you said, that can't work | |
| 14:09:11 | bauzas | okay, I think I have everything I need | |
| 14:09:18 | bauzas | I'll write the bugfix | |
| 14:09:45 | dansmith | yup | |
| 14:10:07 | dansmith | I also definitely think it makes sense to have this enabled in nova-next, if not more of the weighers | |
| 14:10:17 | bauzas | dansmith: fancy question, is it better to hydrate a list of instances by passing the context in InstanceList creator or do exactly liek https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L339 ? | |
| 14:10:21 | dansmith | even if they're simple, if we're not running them we're not testing them | |
| 14:10:51 | dansmith | pass it on create, IIRC.. that other usage is likely very old (or misguided) | |
| 14:11:47 | bauzas | dansmith: I don't disagree but if so, we need to find the right weigher configs in order to have predictable and easy-to-understand scheduling | |
| 14:12:48 | dansmith | I surely hope that the weighers won't change the behavior in a way that makes the tests not work, especially on single-node jobs :) | |
| 14:31:55 | opendevreview | Sylvain Bauza proposed openstack/nova master: Add admin context when filling metadata on ImagePropsWeigher https://review.opendev.org/c/openstack/nova/+/961673 | |
| 14:31:55 | opendevreview | Sylvain Bauza proposed openstack/nova master: Regression test for ImagePropsWeigher due to missing context https://review.opendev.org/c/openstack/nova/+/961615 | |
| 14:47:11 | bauzas | gibi: dansmith: given you still have context about the bug (oh oh oh, pun intended), reviews would be appreciated before we all forget https://review.opendev.org/c/openstack/nova/+/961673 | |
| 14:47:36 | dansmith | already did? | |
| 14:53:51 | bauzas | well, that's a one-liner addition, right? | |
| 14:54:14 | bauzas | the regression suite was written this morning | |
| 14:54:32 | dansmith | what? | |
| 14:56:37 | bauzas | the functional test I meant | |
| 14:57:04 | dansmith | I'm not sure what you're saying | |
| 14:57:24 | dansmith | I already +2d the regression and left a comment on the other one | |
| 15:03:27 | bauzas | let me rephrase: my morning, I investigated the cause of the bug and I tried to reproduce it using a functional test, which I uploaded | |
| 15:03:58 | bauzas | that's why it only took me 10 mins to upload the fix itself | |
| 15:04:19 | bauzas | (I was just replying to to 'already did ?') | |
| 15:04:27 | dansmith | weren't you asking for review? | |
| 15:04:43 | dansmith | that's what I was responding to: "already did...review" | |
| 15:09:06 | bauzas | oh sorry about the misunderstanding then :) | |
| 15:18:30 | gibi | I'm +2 on the first patch and the second looks good too I will +2 once Dan's comment is fixed | |
| 15:22:27 | bauzas | cool, lemme provide a new patchset then | |
| 15:25:22 | opendevreview | Sylvain Bauza proposed openstack/nova master: Add admin context when filling metadata on ImagePropsWeigher https://review.opendev.org/c/openstack/nova/+/961673 | |
| 16:09:26 | opendevreview | Merged openstack/nova master: Regression test for ImagePropsWeigher due to missing context https://review.opendev.org/c/openstack/nova/+/961615 | |
| 19:03:43 | opendevreview | melanie witt proposed openstack/nova master: Add functional reproducer for bug 2125030 https://review.opendev.org/c/openstack/nova/+/961734 | |
| 19:07:07 | opendevreview | melanie witt proposed openstack/nova master: Add functional reproducer for bug 2125030 https://review.opendev.org/c/openstack/nova/+/961734 | |
| 20:01:34 | opendevreview | Merged openstack/nova stable/2024.1: hardware: Fix image_meta.id within get_mem_encryption_constraint https://review.opendev.org/c/openstack/nova/+/960050 | |
| #openstack-nova - 2025-09-19 | |||
| 06:57:58 | gibi | bauzas: Hi! if you have a bit of time this backport will look familiar to you https://review.opendev.org/q/topic:%22bug/2115905%22 :)h | |
| 07:00:25 | bauzas | lolilol | |
| 07:01:33 | jlejeune | gibi: hello, same request for mine https://review.opendev.org/c/openstack/nova/+/961601 :) | |
| 07:07:25 | gibi | jlejeune: on it | |
| 07:08:07 | jlejeune | thanks :) | |
| 07:11:26 | gibi | jlejeune: maybe its me but I only see the regression test being backported there. do you have the actual fix backported on top of that as well? | |
| 07:12:55 | jlejeune | yes but I haven't yet create the changes, doing | |
| 07:14:44 | gibi | OK. I'm +2 on the regression test backports. Feel free to ping me when the fix backports are up. | |
| 07:14:57 | jlejeune | perfect | |
| 07:36:11 | opendevreview | Julien LE JEUNE proposed openstack/nova stable/2025.1: Adds regression test for bug LP#2044235 https://review.opendev.org/c/openstack/nova/+/961601 | |
| 07:36:12 | opendevreview | Julien LE JEUNE proposed openstack/nova stable/2025.1: nova-conductor puts instance in error state https://review.opendev.org/c/openstack/nova/+/961774 | |
| 07:40:47 | opendevreview | Julien LE JEUNE proposed openstack/nova stable/2024.2: nova-conductor puts instance in error state https://review.opendev.org/c/openstack/nova/+/961775 | |
| 07:41:55 | opendevreview | Julien LE JEUNE proposed openstack/nova stable/2024.1: nova-conductor puts instance in error state https://review.opendev.org/c/openstack/nova/+/961778 | |
| 07:42:57 | jlejeune | gibi: here there are ^ | |
| 07:47:33 | opendevreview | Merged openstack/nova stable/2025.1: Reproduce bug/2115905 https://review.opendev.org/c/openstack/nova/+/960241 | |
| 08:02:55 | gibi | jlejeune: thanks | |
| 08:06:16 | gibi | jlejeune: one small suggestion, you can put all these to the same gerrit topic that was used for the original bugfix so it is easy to see the list of backports in a single query | |
| 08:30:53 | jlejeune | gibi: indeed, can I add a topic after having created the change ? | |
| 08:33:30 | gibi | jlejeune: yes on the web UI you can add it by hand | |
| 08:40:04 | jlejeune | nice, done :) | |
| 08:46:44 | gibi | jlejeune: thanks | |
| 08:47:11 | gibi | elodilles: there is a simple clean backport series from jlejeune in https://review.opendev.org/q/topic:%22bug-2044235%22 if you have time :) | |