Posted	Nick	Remark
#openstack-nova - 2019-03-13
20:40:20	dansmith	pfft, he'll say that about anything
20:40:45	edleafe	(uses my jaypipes imitation) Availability zones suck! Let's just get rid of them!
20:41:10	mriedem	* suck! Let's just get rid of them!
20:41:11	mriedem	volumes
20:41:12	mriedem	quotas
20:41:14	mriedem	everything
20:41:23	mriedem	*root volumes
20:41:34	dansmith	instances kinda suck too
20:41:37	dansmith	we should nix those
20:48:08	openstackgerrit	Merged openstack/nova master: Trivialfix for help description of images_type https://review.openstack.org/642942
20:48:18	openstackgerrit	Merged openstack/nova stable/rocky: Fix an error when generating a host ID https://review.openstack.org/641173
20:48:27	openstackgerrit	Merged openstack/nova stable/rocky: Move legacy-grenade-dsvm-neutron-multinode-live-migration in-tree https://review.openstack.org/640191
20:48:35	openstackgerrit	Merged openstack/nova stable/rocky: Fix resource tracker updates during instance evacuation https://review.openstack.org/640982
21:00:08	openstackgerrit	Merged openstack/nova master: Add missing libvirt exception during device detach https://review.openstack.org/641480
21:03:18	openstackgerrit	Merged openstack/nova stable/queens: Make service all-cells min version helper use scatter-gather https://review.openstack.org/637594
21:03:24	openstackgerrit	Merged openstack/nova master: api-ref: Add description for BDM volume_size https://review.openstack.org/641255
21:03:34	openstackgerrit	Merged openstack/nova master: docs: Fix nits in remote console guide https://review.openstack.org/641098
22:11:00	gmann	legacy base jobs to Bionic are merged and nova-next job failing, hold the recheck until this workaround is merged (in gate) - https://review.openstack.org/#/c/643129/3
22:11:08	gmann	sent on ML too.
22:17:29	openstackgerrit	Matt Riedemann proposed openstack/nova stable/rocky: Update instance.availability_zone during live migration https://review.openstack.org/643173
22:45:31	fungi	melwitt: oops, i missed that https://review.openstack.org/642940 had a depends-on to the demonstration patch. that commit message needs to be amended to remove the depends-on line
22:45:51	fungi	mriedem_afk: dansmith: ^
22:48:51	efried	fungi, melwitt, mriedem_afk, dansmith: I'll take care of it, stand by...
22:49:07	openstackgerrit	Eric Fried proposed openstack/nova master: Update nova-tox-functional-py35 parent job https://review.openstack.org/642940
22:49:07	fungi	thanks!!!
22:49:14	fungi	and sorry for missing that in review
22:49:35	efried	fungi: done
22:49:43	efried	I noticed that, but I thought it was intentional
22:49:54	efried	like, I thought the other patch was going to be non-DNM'd soon
22:50:02	efried	or something
22:50:10	efried	I didn't read the backstory closely at all
22:50:19	fungi	nope, it was entirely a demonstration. apologies for the confusion
22:50:51	fungi	we couldn't actually show the effect of the change without a depends-on chain to an alternate universe base job
22:51:06	fungi	at least not until the actual base job change merged
22:52:07	efried	ack
22:52:07	efried	and with that, I'm outta here o/
23:25:13	openstackgerrit	Merged openstack/nova master: Update instance.availability_zone during live migration https://review.openstack.org/567701
23:25:24	openstackgerrit	Merged openstack/nova master: Skip the ceph based live migration testing https://review.openstack.org/643122
23:40:27	gmann	fungi: efried we have cross-dependency to unblock the gate. 643129 cannot merge which need 642940 and vice versa.
23:41:17	fungi	possible we'll need to squash those
23:41:26	gmann	fungi: can we get this forcefully in - 642940? or i can make temp patch to unblock either one
23:43:13	fungi	they'll be properly tested if we squash the two commits together, but would need a nova-core on hand to approve the resulting revision of whichever change we squash into
23:43:54	gmann	ok, let me squash first then i can catch someone alex_xu might be coming online soon
23:47:18	fungi	i'm just veeery hesitant to bypass ci for a change where it's not strictly necessary
23:47:36	fungi	pretty much every time i do, i break something a job would have caught
23:48:50	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Disable the tls-proxy in nova-next & fix nova-tox-functional-py35 parent https://review.openstack.org/643129
23:48:55	gmann	done ^^
23:49:33	gmann	efried: melwitt dansmith alex_xu bauzas stephenfin mriedem_afk ^^ if you are around.
23:50:02	openstackgerrit	Merged openstack/nova master: Trivial: fix typo in reno https://review.openstack.org/643137
23:54:39	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Migrate legacy jobs to Ubuntu Bionic https://review.openstack.org/639017
#openstack-nova - 2019-03-14
00:03:58	melwitt	gmann: I'll get it. thank you for all your work on this
00:04:49	gmann	thanks.
00:13:18	fungi	thanks melwitt!
00:19:51	openstackgerrit	melanie witt proposed openstack/nova master: Re-enable Ceph in live migration testing https://review.openstack.org/643150
01:12:46	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add functional recreate test for bug 1819963 https://review.openstack.org/643151
01:12:46	openstack	bug 1819963 in OpenStack Compute (nova) "Reverting a resize does not update the instance.availability_zone value to the source az" [Medium,In progress] https://launchpad.net/bugs/1819963 - Assigned to Matt Riedemann (mriedem)
01:12:46	openstackgerrit	Matt Riedemann proposed openstack/nova master: Update instance.availability_zone on revertResize https://review.openstack.org/643155
02:14:20	openstackgerrit	Takashi NATSUME proposed openstack/nova stable/rocky: Fix resetting non-persistent fields when saving obj https://review.openstack.org/643215
02:31:54	openstackgerrit	huanhongda proposed openstack/nova stable/queens: Fix resource tracker updates during instance evacuation https://review.openstack.org/643219
02:44:00	openstackgerrit	Boxiang Zhu proposed openstack/nova master: Raise error when force_raw_images and images_type are incompatible https://review.openstack.org/640271
04:06:33	openstackgerrit	Merged openstack/nova master: Disable the tls-proxy in nova-next & fix nova-tox-functional-py35 parent https://review.openstack.org/643129
04:14:40	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Override the 'get' method in DriverBlockDevice class https://review.openstack.org/638821
05:20:32	openstackgerrit	Luyao Zhong proposed openstack/nova master: object: Add pmem_namespaces field to the NUMACell obj https://review.openstack.org/634547
05:20:33	openstackgerrit	Luyao Zhong proposed openstack/nova master: object: Add virtual_pmems fields to the InstanceNUMACell obj https://review.openstack.org/634548
05:20:33	openstackgerrit	Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configures PMEM namespace when initiating libvirt driver https://review.openstack.org/634549
05:20:34	openstackgerrit	Luyao Zhong proposed openstack/nova master: libvirt: Update PMEM namespaces info and usage https://review.openstack.org/634550
05:20:34	openstackgerrit	Luyao Zhong proposed openstack/nova master: libvirt: report pmem namespaces resources by provider tree https://review.openstack.org/634551
05:20:35	openstackgerrit	Luyao Zhong proposed openstack/nova master: API: parse pmem related flavor extra spec https://review.openstack.org/634552
05:20:35	openstackgerrit	Luyao Zhong proposed openstack/nova master: scheduler: translate virtual pmems request to placement request group https://review.openstack.org/634553
05:20:36	openstackgerrit	Luyao Zhong proposed openstack/nova master: update _numa_fit_instance_cell for the support of virtual_pmems https://review.openstack.org/634554
05:20:36	openstackgerrit	Luyao Zhong proposed openstack/nova master: libvirt: boot VM with vpmems and cleanup vpmems when destroying https://review.openstack.org/634555
05:20:37	openstackgerrit	Luyao Zhong proposed openstack/nova master: support VM resizing with vpmem data migration https://review.openstack.org/634556
07:35:49	openstackgerrit	zhaixiaojun proposed openstack/nova master: Fix bug: non-admin user can not filter instance by instance-uuid https://review.openstack.org/642403
07:47:49	openstackgerrit	Boxiang Zhu proposed openstack/nova master: Raise error when force_raw_images and images_type are incompatible https://review.openstack.org/640271
08:34:56	openstackgerrit	Yongli He proposed openstack/nova master: Clean up orphan instances https://review.openstack.org/627765
08:44:01	openstackgerrit	Merged openstack/nova master: Update compute rpc version alias for stein https://review.openstack.org/642599
09:04:57	openstackgerrit	Merged openstack/nova master: Migrate legacy jobs to Ubuntu Bionic https://review.openstack.org/639017
09:05:56	openstackgerrit	Boxiang Zhu proposed openstack/nova master: Remove unsafe convert of lvm imagebackend https://review.openstack.org/642667
09:08:44	kashyap	gibi: Morning, mind having a quick gander at this (simple change, but brings non-trivial perf benefit): https://review.openstack.org/#/c/640781/
09:12:04	openstackgerrit	zhaixiaojun proposed openstack/nova master: Fix bug: non-admin user can not filter instance by instance-uuid https://review.openstack.org/642403
10:49:41	jangutter	kashyap: 15x --- there's that story that nearly every programming task reduces to either a caching or string manipulation problem.
10:49:57	kashyap	jangutter: Yeah...
10:50:26	kashyap	It was a "one-worder" change, but I guess you read the full commit message
10:50:37	kashyap	It took 2 days to dig into QEMU and talk to Block Layer folks
10:51:08	jangutter	and the persistent nagging feeling that it can't be this simple.
11:22:18	kashyap	jangutter: Yeah, need to consider impact on multiple use cases
11:23:20	kashyap	mdbooth: Morning, if you get a moment: https://review.openstack.org/#/c/641981/ and https://review.openstack.org/#/c/640781/
11:23:26	kashyap	(You were already +1 on those before.)
11:24:08	mdbooth	"But that's not correct, because of our misunderstanding of how cache modes work."
11:24:23	mdbooth	I'll say again: we never understood how cache modes work, only what they're called ;)
11:24:34	mdbooth	...misunderstood...
11:24:47	mdbooth	Don't change that, though.
11:25:07	kashyap	Hehe, noted.
11:25:38	kashyap	Now I think not in terms of just named cache modes, but the three booleans each one impacts
11:27:22	kashyap	mdbooth: FWIW, the little blob of text written in driver.py -- I got it reviewed by Kevin (of QEMU) over a few emails. So it's not me just riffing on it :-)
11:37:16	mdbooth	ack
11:52:11	openstackgerrit	Fan Zhang proposed openstack/nova master: Retry after hitting libvirt error VIR_ERR_OPERATION_INVALID in live migration. https://review.openstack.org/612272
12:10:17	awalende	I'am trying to use hugepages with nova and qemu/libvirt. Hugepages is enabled and I have reserved a bunch of hugepages in standard 2048kb size. I added flavor properties with "flavor set hugePagesTest --property hw:mem_page_size=2Mb". However starting instances fails with "Exceeded maximum number of retries". Nova-compute log does not show any exceptions. Any Idea how to debug this?
12:34:57	efried	awalende: Have you looked at the scheduler log?
12:36:04	awalende	Yes, I could narrow it down. It has something to do with NUMA-Topology:

Earlier   Later