Posted	Nick	Remark
#openstack-nova - 2019-09-14
05:09:19	openstackgerrit	Merged openstack/nova master: Make SRIOV computes non symmetric in func test https://review.opendev.org/681667
05:09:57	openstackgerrit	Merged openstack/nova master: Support migrating SRIOV port with bandwidth https://review.opendev.org/676980
05:10:06	openstackgerrit	Merged openstack/nova master: Allow migrating server with port resource request https://review.opendev.org/671497
05:10:16	openstackgerrit	Merged openstack/nova master: Allow resizing server with port resource request https://review.opendev.org/679019
05:10:25	openstackgerrit	Merged openstack/nova master: Extract pf$N literals as constants from func test https://review.opendev.org/680991
05:10:35	openstackgerrit	Merged openstack/nova master: Improve dest service level func tests https://review.opendev.org/680998
05:27:15	openstackgerrit	Merged openstack/nova master: Follow up for Ib50b6b02208f5bd2972de8a6f8f685c19745514c https://review.opendev.org/681490
05:27:23	openstackgerrit	Merged openstack/nova master: Skip querying resource request in revert_resize if no qos port https://review.opendev.org/681513
05:27:37	openstackgerrit	Merged openstack/nova master: Follow up for I220fa02ee916728e241503084b14984bab4b0c3b https://review.opendev.org/680810
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?

Earlier   Later