| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-06-16 | |||
| 17:20:41 | sean-k-mooney | thats interesting | |
| 17:20:51 | sean-k-mooney | sory forgot the ... | |
| 17:56:26 | opendevreview | Artom Lifshitz proposed openstack/nova stable/ussuri: functional: Use tempdir for CONF.instances_path https://review.opendev.org/c/openstack/nova/+/846204 | |
| 17:56:27 | opendevreview | Artom Lifshitz proposed openstack/nova stable/ussuri: Add a regression test for bug 1939545 https://review.opendev.org/c/openstack/nova/+/843951 | |
| 17:56:28 | opendevreview | Artom Lifshitz proposed openstack/nova stable/ussuri: compute: Ensure updates to bdms during pre_live_migration are saved https://review.opendev.org/c/openstack/nova/+/843952 | |
| 17:56:29 | opendevreview | Artom Lifshitz proposed openstack/nova stable/ussuri: fup: Make connection_info returned by CinderFixture unique per attachment https://review.opendev.org/c/openstack/nova/+/844606 | |
| 17:56:31 | opendevreview | Artom Lifshitz proposed openstack/nova stable/ussuri: fup: Assert state of connection_info during LM rollback in func tests https://review.opendev.org/c/openstack/nova/+/844607 | |
| 18:29:23 | opendevreview | Balazs Gibizer proposed openstack/nova master: Report PCI inventory to placement basics https://review.opendev.org/c/openstack/nova/+/846187 | |
| 18:29:24 | opendevreview | Balazs Gibizer proposed openstack/nova master: Ignore PCI devs with physical_network tag https://review.opendev.org/c/openstack/nova/+/846219 | |
| 18:29:24 | opendevreview | Balazs Gibizer proposed openstack/nova master: Extend device_spec with resource_class and traits https://review.opendev.org/c/openstack/nova/+/846218 | |
| #openstack-nova - 2022-06-17 | |||
| 03:27:03 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Add encryption support to qemu-img create command https://review.opendev.org/c/openstack/nova/+/826752 | |
| 03:27:04 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Consolidate create_cow_image and create_image https://review.opendev.org/c/openstack/nova/+/846246 | |
| 04:35:55 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Add encryption support to qemu-img create command https://review.opendev.org/c/openstack/nova/+/826752 | |
| 04:35:55 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Consolidate create_cow_image and create_image https://review.opendev.org/c/openstack/nova/+/846246 | |
| 07:44:11 | gibi | bauzas, sean-k-mooney: can we get a second opinion on https://review.opendev.org/c/openstack/nova/+/831507/comment/b9fbbb2c_8e65c98e/ about the possible refactor? | |
| 07:47:55 | gibi | Uggla: I don't want to be a heavy handed core but I really don't like the 14 conditionals in the unshelve code | |
| 08:01:25 | Uggla | gibi, hi | |
| 08:02:39 | gibi | o/ | |
| 08:06:07 | Uggla | gibi, ok i'm gonna change in that way. | |
| 08:06:31 | gibi | Uggla: don't rush, I just asked bauzas and sean-k-mooney oppinion about it above | |
| 08:07:14 | Uggla | gibi, I agree that usually having so many conditions is not great. | |
| 08:18:15 | gibi | cores: easy win for a +A (doc patch) https://review.opendev.org/c/openstack/placement/+/845730 | |
| 08:22:45 | gibi | cores: also an easy +A (unit test stabilization) https://review.opendev.org/c/openstack/nova/+/845922 | |
| 08:35:08 | bauzas | gibi: sorry was afk due to some appointment | |
| 08:35:49 | bauzas | Uggla: will try to do the review today | |
| 08:40:15 | Uggla | bauzas, ok | |
| 08:40:30 | bauzas | finishing up my GPU patch | |
| 08:47:29 | bauzas | gibi: do you think we could workaround https://bugs.launchpad.net/nova/+bug/1874664 with some flavor extraspec ? | |
| 08:48:08 | bauzas | it's a placement issue | |
| 08:48:56 | bauzas | if you look at the issue, we try to use the same RP https://paste.opendev.org/show/b9A9TdUl1dzTJaNtbrCd/ | |
| 08:49:53 | bauzas | for both allocations | |
| 09:01:20 | gibi | bauzas: I will check in a minute | |
| 09:05:39 | gibi | bauzas: have you checked that multicreate without vgpu can boot two instance on the same host? (I'm wondering if we do that) | |
| 09:08:38 | bauzas | gibi: good question, I dunno | |
| 09:09:15 | bauzas | gibi: so, my fix I created was using this bug number but I'll then create a separate other | |
| 09:20:36 | gibi | bauzas: I see the limitation | |
| 09:20:39 | gibi | bauzas: it is not placement | |
| 09:20:47 | gibi | bauzas: in that test placement returns 2 allocation candidate | |
| 09:20:55 | gibi | one for each pGPU | |
| 09:21:12 | gibi | but nova only able to consume the first allocation candidate per host | |
| 09:21:55 | gibi | bauzas: https://github.com/openstack/nova/blob/93a65f06df67ce39d65827692150c78013c7f6d5/nova/scheduler/manager.py#L382-L388 | |
| 09:22:12 | gibi | nova only tries to first a_c, for both instance | |
| 09:22:23 | gibi | *tries to claim the first | |
| 09:22:38 | gibi | it succeeds for the first instance but obviously fails for the second | |
| 09:22:48 | gibi | the TODO from jaypipes is valid | |
| 09:23:24 | gibi | if you resolve that TODO then the test_multiple_instance_create_filling_up_capacity test case will be able to boot both instances on the same host | |
| 09:24:24 | bauzas | gibi: looking | |
| 09:33:09 | bauzas | gibi: thanks for finding the RCA | |
| 09:33:18 | gibi | I think we can implement that TODO | |
| 09:33:34 | gibi | without adding to much complexity with sorting | |
| 09:33:51 | gibi | it would not be optimal but would be better than what we have today | |
| 09:39:14 | gibi | I'm affraid the gate is broken on master | |
| 09:39:16 | gibi | https://zuul.opendev.org/t/openstack/builds?job_name=tempest-integrated-compute-centos-9-stream&skip=0 | |
| 09:40:18 | gibi | libvirt.libvirtError: internal error: unable to execute QEMU command 'netdev_add': File descriptor named '(null)' has not been found | |
| 09:49:07 | bauzas | gibi: what a Friday | |
| 09:49:23 | bauzas | Centos 9 Stream /o\ | |
| 09:49:27 | gibi | bauzas: reported a bug https://bugs.launchpad.net/nova/+bug/1979047 please triage it and set it critical I will drop a mail to the ML | |
| 09:50:49 | bauzas | gibi: we need to make this job non-voting | |
| 09:51:14 | gibi | are we give up on this job? :) | |
| 09:51:49 | bauzas | gibi: the priority is to unblock the gate | |
| 09:52:12 | bauzas | want me to write it ? | |
| 09:53:48 | gibi | I sent a mail to the ML | |
| 09:54:06 | bauzas | I'll propose the patch at least | |
| 09:54:31 | bauzas | we're on a Friday | |
| 09:55:01 | gibi | bauzas: my philosophy: sure unblock the gate in 2-3 days if nobody found a quick solution. but keep the pain a bit around to motivate people to try and find a solution first :) | |
| 09:55:42 | bauzas | gibi: my philosophy: remove support from something which breaks and you'll suddently find people who care | |
| 09:56:31 | bauzas | either way, patch is being forged, we need two cores for approving it, so there is no direct impact *yet* | |
| 09:56:37 | gibi | OK, lets go with yours this time :) | |
| 09:56:50 | gibi | I will +2 the non-voting patch | |
| 10:03:12 | opendevreview | Sylvain Bauza proposed openstack/nova master: zuul: Temporarly put Centos9 Stream job non-voting https://review.opendev.org/c/openstack/nova/+/846292 | |
| 10:03:30 | bauzas | gibi: feel free to add a reply to your email with this patch number | |
| 10:04:57 | opendevreview | Sylvain Bauza proposed openstack/nova master: zuul: Temporarly put Centos9 Stream job non-voting https://review.opendev.org/c/openstack/nova/+/846292 | |
| 10:07:14 | gibi | bauzas: thanks +2. done the ML update | |
| 10:17:53 | gibi | bauzas: the grenade job on master also look baaaad https://zuul.opendev.org/t/openstack/builds?job_name=nova-grenade-multinode&project=openstack%2Fnova&branch=master&skip=0 | |
| 10:18:31 | gibi | that seem to be a different error | |
| 10:18:32 | gibi | Details: {'type': 'SecurityGroupInUse', 'message': 'Security Group 40f249d7-9c4e-432c-91d7-d68ef7ac05b0 in use.', 'detail': ''} | |
| 10:18:57 | bauzas | yup, saw | |
| 10:19:00 | bauzas | damn | |
| 10:23:12 | sean-k-mooney | i can take a look at it too i guess | |
| 10:23:49 | opendevreview | Merged openstack/placement master: Add WA about resource_providers.can_host removal https://review.opendev.org/c/openstack/placement/+/845730 | |
| 10:24:31 | sean-k-mooney | so i guessl libvirt need to be udated for a new qemu release? | |
| 10:24:40 | sean-k-mooney | to fix interface attach | |
| 10:24:54 | sean-k-mooney | or they broke there api compat and we need to adress that | |
| 10:25:33 | gibi | sean-k-mooney: I added the libvirt and qemu versions to the bugreport I don't know if any of those are fresh | |
| 10:26:27 | tkajinam | gibi, the bug title reminds me this https://bugzilla.redhat.com/show_bug.cgi?id=2092856 | |
| 10:27:11 | gibi | tkajinam: good find. That very much like it | |
| 10:27:17 | tkajinam | we pinned libvirt to workaround this in tripleo | |
| 10:28:14 | sean-k-mooney | ya i was about to ping danpb and ask about that bug | |
| 10:29:23 | gibi | bauzas: I'm going to open a bug on tempest about the security group issue | |
| 10:29:49 | bauzas | gibi: cool | |
| 10:33:51 | gibi | https://bugs.launchpad.net/tempest/+bug/1979052 | |
| 10:35:52 | bauzas | I need to go off | |
| 10:36:01 | bauzas | (fortunately, wasn't doing gym today) | |
| 10:36:14 | bauzas | but I have my kid with me so I need to cook :) | |
| 10:36:41 | gibi | let me know what did you cook when you are back | |
| 10:36:46 | gibi | I need inspiration | |
| 10:45:07 | sean-k-mooney | im considerign firing up my new(ish) BBQ this evening and cookig somethign or at least doing that at the weekend | |
| 10:45:47 | gibi | sounds nice :) | |
| 10:47:43 | sean-k-mooney | since i invested in a "kamado joe big joe 3" i might as well make use of it. i had left over chicken form tha tlast time in the freezer which i had last night. it was pretty tasty | |
| 10:48:55 | sean-k-mooney | i did a prime rib roast on it before and it was amazing, also did really nice potatos corn and fish | |