| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-14 | |||
| 07:05:21 | openstackgerrit | Brin Zhang proposed openstack/nova master: Fix legacy issues in Filter migrations by user_id/project_id https://review.opendev.org/682198 | |
| 07:25:17 | brinzhang_ | The failure rate of Zuul in these two days is low | |
| 07:25:30 | brinzhang_ | so many ptach were failed | |
| 13:40:44 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/stein: libvirt: stub logging of host capabilities https://review.opendev.org/682210 | |
| 15:37:42 | donnyd | is there a method in nova to tell it to boot an instance when the hypervisor comes up? So for instance if I needed to do maintenance, but wanted specific instances to come up automatically | |
| 15:37:50 | donnyd | Is there a way to do that in Nova | |
| 16:37:58 | openstackgerrit | Merged openstack/nova master: PDF documentation build https://review.opendev.org/676730 | |
| 17:29:28 | openstackgerrit | Merged openstack/nova master: Use SpawnIsSynchronousFixture in reschedule functional tests https://review.opendev.org/682140 | |
| 22:43:57 | sean-k-mooney | donnyd: i dont know if you found it our not but you can use https://docs.openstack.org/nova/rocky/configuration/config.html#DEFAULT.resume_guests_state_on_host_boot | |
| #openstack-nova - 2019-09-15 | |||
| 01:34:30 | openstackgerrit | Merged openstack/nova master: DB API changes to get non-matching aggregates from metadata https://review.opendev.org/671074 | |
| 02:57:30 | openstackgerrit | Merged openstack/nova master: New objects for NUMA live migration https://review.opendev.org/634827 | |
| 02:57:48 | openstackgerrit | Merged openstack/nova master: LM: Use Claims to update numa-related XML on the source https://review.opendev.org/635229 | |
| 05:34:15 | openstackgerrit | Merged openstack/nova master: Note about Destination.forbidden_aggregates https://review.opendev.org/680945 | |
| 05:36:49 | openstackgerrit | Merged openstack/nova master: Add a new request filter to isolate aggregates https://review.opendev.org/671075 | |
| 06:12:43 | openstackgerrit | Merged openstack/nova master: db: Add resources column in instance_extra table https://review.opendev.org/678447 | |
| 06:33:08 | openstackgerrit | Merged openstack/nova master: object: Introduce Resource and ResourceList objs https://review.opendev.org/678448 | |
| 06:33:16 | openstackgerrit | Merged openstack/nova master: Add resources dict into _Provider https://review.opendev.org/678449 | |
| 06:33:43 | openstackgerrit | Merged openstack/nova master: Set user_id/project_id from context when creating a Migration https://review.opendev.org/679413 | |
| 06:33:53 | openstackgerrit | Merged openstack/nova master: Retrieve the allocations early https://review.opendev.org/678450 | |
| 09:20:05 | openstackgerrit | Merged openstack/nova master: Docs for isolated aggregates request filter https://review.opendev.org/667952 | |
| 11:12:30 | openstackgerrit | Merged openstack/nova master: NUMA live migration support https://review.opendev.org/634606 | |
| 11:23:00 | openstackgerrit | garyk proposed openstack/nova master: Deconstruct the mother of all locks https://review.opendev.org/682242 | |
| 14:48:54 | MarkMielke | oslo_utils doesn't seem to have unit tests, but nova does... if oslo_utils needs a fix, is there any special downstream nova process that should be followed to pick it up, and test with it? | |
| 14:49:42 | MarkMielke | specifically, I'm talking about https://review.opendev.org/682247 which I just submitted | |
| 15:05:29 | sean-k-mooney | oh your fixing that issue | |
| 15:05:36 | sean-k-mooney | that is on my todo list | |
| 15:05:46 | sean-k-mooney | i personally would add tests to oslo utils | |
| 15:07:02 | sean-k-mooney | thats breaking fedora with the virt preview repo enabeld currently | |
| 15:07:20 | sean-k-mooney | alshtough im not sure it affct any distros default packages yet? | |
| 15:07:46 | MarkMielke | right | |
| 15:08:25 | MarkMielke | qemu 4.1 provides important nVMX patches | |
| 15:08:51 | MarkMielke | so not necessarily in distro | |
| 15:09:00 | MarkMielke | can use qemu 4.1 on RHEL 7 if need be | |
| 15:09:40 | MarkMielke | all of the tests for outputs seem to be in nova | |
| 15:09:50 | MarkMielke | seems like unit test confusion :-) | |
| 15:09:52 | sean-k-mooney | non of them shoudl be in nova | |
| 15:10:02 | sean-k-mooney | they should all be in oslo_untils | |
| 15:10:18 | sean-k-mooney | but i think that code was original ported from nova | |
| 15:10:25 | sean-k-mooney | so i guess they did not mvoe the tests | |
| 15:11:15 | MarkMielke | I want to add a test, but I'm not familiar with setting it up from scratch, and I don't want to take on doing it all :-) | |
| 15:11:16 | sean-k-mooney | nova really should not be asserting the parseing behavior of oslo code so i sustpect the nova tests are just legacy test we did not remove | |
| 15:11:46 | MarkMielke | two character fix :-( hehe | |
| 15:11:49 | sean-k-mooney | you could add more test in nova but personally i think those are techdebt | |
| 15:12:11 | MarkMielke | i agree from observing | |
| 15:12:23 | sean-k-mooney | well you could file a bug for the lack of test in oslo_utils | |
| 15:12:44 | MarkMielke | good idea | |
| 15:12:47 | sean-k-mooney | and submit the ptach without tests. you would have to use a depend on to test the feature in nova | |
| 15:13:15 | sean-k-mooney | but we are pass the non client lib freeze so i dont exepct there to be another oslo release in Train | |
| 15:13:31 | MarkMielke | hmm - i wondered about that | |
| 15:13:45 | MarkMielke | how important patches get in to dependency libraries | |
| 15:14:02 | sean-k-mooney | form a rhel perspective we can always backport this downstream | |
| 15:14:16 | MarkMielke | patches to the packages, you mean? | |
| 15:14:35 | sean-k-mooney | but i dont really consider this an important patch unless your saying the advance virt strema now ships this by default on rhel 7 | |
| 15:15:03 | MarkMielke | well i wondered in general - importance depends upon perspective, so i agree others might not consider it important | |
| 15:15:12 | MarkMielke | qemu 4.1 is important for nested virtualization use case | |
| 15:15:23 | MarkMielke | it's the first release that officially supports nested virtualization live migration | |
| 15:15:24 | sean-k-mooney | nested virt work without it | |
| 15:15:36 | MarkMielke | nah, they've eliminated it in 3.1 and 4.0 | |
| 15:15:40 | sean-k-mooney | nested virt live migration also work without it | |
| 15:15:41 | MarkMielke | because it was fundamentally broken | |
| 15:15:52 | MarkMielke | they have a live migration blocker implemented | |
| 15:15:56 | MarkMielke | that many distros disable | |
| 15:16:00 | sean-k-mooney | you mean live migat of th l1 guest | |
| 15:16:01 | MarkMielke | it's basically unsafe | |
| 15:16:03 | sean-k-mooney | not the l2 guest | |
| 15:16:30 | MarkMielke | there is special KVM state in the L0 hypervisor that isn't transferred prior to qemu 4.1 | |
| 15:16:38 | MarkMielke | basically results in undefined memory at the target | |
| 15:17:12 | MarkMielke | undefined behaviour is the result ... better not actually be doing nested virtualization previously :-) | |
| 15:17:13 | sean-k-mooney | right but are you saying that qemu 4.1 fix live migration of l1 vms that contin l2 vm or live migration of l2 vms | |
| 15:17:18 | MarkMielke | yes | |
| 15:17:26 | sean-k-mooney | ok that is different | |
| 15:17:38 | MarkMielke | the patches were added - and it passes for Oracle and others | |
| 15:17:40 | sean-k-mooney | you could previosly safely migrate the l2 guests | |
| 15:17:45 | MarkMielke | unfortunately i have a use case that it doesn't work for | |
| 15:17:55 | MarkMielke | but they've tried to solve all the edge cases | |
| 15:18:29 | MarkMielke | it uses several new kernel features only in Linux 4.19 and later | |
| 15:18:37 | MarkMielke | so RHEL 8 would be a candidate for some of it | |
| 15:18:50 | sean-k-mooney | rhel 8 is based on 4.18 | |
| 15:18:59 | sean-k-mooney | but ya it could be backported | |
| 15:19:11 | MarkMielke | yuck i thought it used 4.19 | |
| 15:19:22 | sean-k-mooney | nope which is annoying | |
| 15:19:34 | sean-k-mooney | but rhel kernel version dont really mena anything | |
| 15:19:37 | MarkMielke | Red Hat will just backport most of the patches anyways :-) | |
| 15:19:51 | sean-k-mooney | rhel 7 was 3.x and has patch from 4.15 | |
| 15:21:02 | sean-k-mooney | i work for redhat by the way which is why i said we could backport the oslo utils change if it was need becaue we stated shiping qemu 4.1 by default on rhel 7 with the advance vert stream | |
| 15:21:12 | MarkMielke | https://github.com/qemu/qemu/commit/ebbfef2f34cfc749c045a4569dedb4f748ec024a | |
| 15:21:15 | sean-k-mooney | but i dont think we have or we would have got a bug report | |
| 15:21:21 | MarkMielke | there are a few patches in the set... but that's one of the important ones | |
| 15:24:26 | MarkMielke | I've been tracking these changes and the need for them since 2018, so it being almost usable is great. :-) | |
| 15:25:14 | MarkMielke | lots of nVMX patches in Linux 5.0, 5.1, and 5.2 though... so 4.18 will just be annoying to patch to for you | |
| 15:26:06 | sean-k-mooney | out of interest why to you need to migrate the l1 guests? are the l1 guest your tenant guest and they are using nested virt on your cloud? | |
| 15:26:40 | MarkMielke | in my particular use case - the users have "simulations" of network equipment running in guests | |
| 15:26:44 | MarkMielke | previously they were on physical workstations | |
| 15:26:53 | MarkMielke | now they are in virtual machines | |
| 15:27:10 | MarkMielke | could be an embedded Linux using KVM, or could be Vxworks | |
| 15:27:24 | MarkMielke | so those machines are basically not safe to live migration until now | |
| 15:27:37 | MarkMielke | i've found the only safe recipe is one by one: | |
| 15:27:52 | sean-k-mooney | yes but live migration is really only for mantaince | |
| 15:27:56 | MarkMielke | 1) confirm no L2 guests running in the L1 hypervisor | |
| 15:27:57 | sean-k-mooney | is that your usecase | |