| *** haleyb is now known as haleyb|out | 00:31 | |
| opendevreview | Merged openstack/nova stable/2024.2: hypervisors: Optimize uptime retrieval for better performance https://review.opendev.org/c/openstack/nova/+/960410 | 00:45 |
|---|---|---|
| opendevreview | melanie witt proposed openstack/nova-specs master: Re-propose vTPM live migration https://review.opendev.org/c/openstack/nova-specs/+/961564 | 01:08 |
| opendevreview | yangjianfeng proposed openstack/nova master: Support reusing existing port binding during instance live-migration https://review.opendev.org/c/openstack/nova/+/961593 | 06:29 |
| 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 |
| 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:42 |
| 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:43 |
| 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 | 07:47 |
| *** EugenMayer4401809 is now known as EugenMayer440180 | 08:23 | |
| opendevreview | Merged openstack/nova stable/2024.1: hypervisors: Optimize uptime retrieval for better performance https://review.opendev.org/c/openstack/nova/+/960411 | 08:49 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Regression test for ImagePropsWeigher due to missing context https://review.opendev.org/c/openstack/nova/+/961615 | 10:12 |
| *** ykarel_ is now known as ykarel | 11:43 | |
| opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Mention socket in pci numa affinity extra spec help https://review.opendev.org/c/openstack/nova/+/961634 | 11:57 |
| opendevreview | Josephine Seifert proposed openstack/nova stable/2024.2: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/961655 | 13:23 |
| bauzas | dansmith: ping, if you are free of meetings :) | 14:03 |
| dansmith | bauzas: just a sec | 14:04 |
| bauzas | cool | 14:04 |
| dansmith | bauzas: so, fill_faults() and fill_metadata() are operations on the list object, just like .save() on an instance or anything else | 14:05 |
| dansmith | they expect to use the context that the object maintains | 14:06 |
| bauzas | I see, those are not getters | 14:06 |
| 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 |
| dansmith | it's not wrong, it's just not the intended workflow | 14:06 |
| dansmith | so just create the list like insts = InstanceList(context, ...) and then it will have it internally | 14:07 |
| bauzas | sure, but then I wonder which context to pass | 14:07 |
| 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 |
| bauzas | context for others, we speak about https://bugs.launchpad.net/nova/+bug/2125052 | 14:07 |
| bauzas | yup, I figured it, so I probably need to instantiate a fully new admin context | 14:08 |
| bauzas | we have the user request context stored in the requestspec, but as you said, that can't work | 14:08 |
| bauzas | okay, I think I have everything I need | 14:09 |
| bauzas | I'll write the bugfix | 14:09 |
| dansmith | yup | 14:09 |
| dansmith | I also definitely think it makes sense to have this enabled in nova-next, if not more of the weighers | 14:10 |
| 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 |
| dansmith | even if they're simple, if we're not running them we're not testing them | 14:10 |
| dansmith | pass it on create, IIRC.. that other usage is likely very old (or misguided) | 14:10 |
| 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:11 |
| 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:12 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Regression test for ImagePropsWeigher due to missing context https://review.opendev.org/c/openstack/nova/+/961615 | 14:31 |
| 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 |
| 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 |
| dansmith | already did? | 14:47 |
| bauzas | well, that's a one-liner addition, right? | 14:53 |
| bauzas | the regression suite was written this morning | 14:54 |
| dansmith | what? | 14:54 |
| bauzas | the functional test I meant | 14:56 |
| dansmith | I'm not sure what you're saying | 14:57 |
| dansmith | I already +2d the regression and left a comment on the other one | 14:57 |
| 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 |
| bauzas | that's why it only took me 10 mins to upload the fix itself | 15:03 |
| bauzas | (I was just replying to to 'already did ?') | 15:04 |
| dansmith | weren't you asking for review? | 15:04 |
| dansmith | that's what I was responding to: "already did...review" | 15:04 |
| bauzas | oh sorry about the misunderstanding then :) | 15:09 |
| gibi | I'm +2 on the first patch and the second looks good too I will +2 once Dan's comment is fixed | 15:18 |
| bauzas | cool, lemme provide a new patchset then | 15:22 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Add admin context when filling metadata on ImagePropsWeigher https://review.opendev.org/c/openstack/nova/+/961673 | 15:25 |
| opendevreview | Merged openstack/nova master: Regression test for ImagePropsWeigher due to missing context https://review.opendev.org/c/openstack/nova/+/961615 | 16:09 |
| opendevreview | melanie witt proposed openstack/nova master: Add functional reproducer for bug 2125030 https://review.opendev.org/c/openstack/nova/+/961734 | 19:03 |
| opendevreview | melanie witt proposed openstack/nova master: Add functional reproducer for bug 2125030 https://review.opendev.org/c/openstack/nova/+/961734 | 19:07 |
| 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 | 20:01 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!