Posted	Nick	Remark
#openstack-nova - 2017-11-15
23:53:38	efried	jaypipes I find it very confusing that _Provider represents a node in a tree of providers, but ProviderTree actuall represents any number of such trees.
23:54:12	efried	Rename ProviderTree to ProviderTrees. Or ProviderBush.
23:56:38	edleafe	efried: ProviderVine?
23:57:06	efried	edleafe ProvideMeVino
23:57:24	edleafe	I like the way you think!
23:57:38	eandersson	btw does placement take actual resources into account when scheduling vms on overcommited hosts?
23:59:04	eandersson	In Mitaka we are hitting this when the compute is out of memory, but the scheduler still thinks there is room available
23:59:05	eandersson	> qemu-kvm: cannot set up guest memory 'pc.ram': Cannot allocate memory
23:59:08	openstackgerrit	Eric Fried proposed openstack/nova master: Iterator for a provider tree https://review.openstack.org/520243
23:59:28	efried	jaypipes We're gonna need that ^
23:59:42	mriedem	eandersson: are you setting the reserved space config?
23:59:51	eandersson	Yea
#openstack-nova - 2017-11-16
00:00:00	eandersson	This is with overcommit 1.5 on memory, which is fine in most cases for us
00:00:02	mriedem	reserved_host_memory_mb
00:00:04	mriedem	^
00:00:15	eandersson	The problem is that if one or two VMs are using all their memory
00:00:24	eandersson	the overcommit ratio won
00:00:26	eandersson	won't work
00:00:44	mriedem	the reserved_host_memory_mb config option is used to tell placement that there is reserved space on a compute that can't be claimed by nova
00:00:49	mriedem	and the scheduler will take that into account
00:01:30	eandersson	Lets say you have 10GB memory, and 4 VMs with 5GB each and overcommit set to 2x (total 20GB)
00:02:00	eandersson	or even better 12GB memory, 4 VMs with 5GB etc and reserved_host_memory_mb set to 2GB
00:02:29	eandersson	If three of those VMs are actually using 10GB in total
00:02:37	eandersson	The forth VM will fail to be created
00:03:32	eandersson	but the scheduler will think that there is room on the VM, as it thinks the compute has 20+2GB memory
00:04:23	mriedem	have you tried to recreate that in ocata?
00:04:36	mriedem	otherwise i'm going to have to phone a friend (jaypipes)
00:04:53	eandersson	I haven't yet
00:05:02	eandersson	tbh it's not a big deal, but it's the issue causing the duplicate ips
00:05:13	eandersson	(so fixing the dup ips and worst case it will reschedule)
00:05:15	mriedem	ok, well, reschedules is a big deal we're trying to kill off
00:05:28	mriedem	and still support reschedules with cells v2
00:05:37	mriedem	so we're assuming that reschedules shouldn't happen due to failed claims
00:05:58	eandersson	I'll see if I can test it in Ocata or Pike.
00:05:59	mriedem	at least now that in pike with the filter scheduler we 'claim' in the scheduler rather than the compute
00:06:03	mriedem	try it with pike
00:06:11	mriedem	pike is the scheduler change to allocate resources in the scheduler rather than compute
00:06:15	mriedem	using placement with the filter scheduler
00:06:18	eandersson	Only tested it in Mitaka so far, but would motivate us to upgrade sooner :D
00:06:26	mriedem	if things still don't work and yo'ure hitting reschedules, that would be very good to know
00:10:25	openstackgerrit	Eric Fried proposed openstack/nova master: WIP: Use update_inventory from the resource tracker https://review.openstack.org/520246
00:10:44	efried	jaypipes And here's where I'm going with it ^
00:12:21	openstackgerrit	Hongbin Lu proposed openstack/nova-specs master: [WIP] Choose default network on ambiguity https://review.openstack.org/520247
00:26:11	openstackgerrit	Matt Riedemann proposed openstack/nova master: Always deallocate networking before reschedule if using Neutron https://review.openstack.org/520248
00:26:12	mriedem	eandersson: here you go ^
00:26:42	eandersson	<3
00:26:51	openstackgerrit	Takashi NATSUME proposed openstack/python-novaclient master: Fix missing metavar in aggregate-update https://review.openstack.org/512485
00:27:04	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Set the 'Upgrade impact' subsection as optional https://review.openstack.org/517223
00:27:19	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Create specs directory for Rocky https://review.openstack.org/514101
00:27:29	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix missing marker functions https://review.openstack.org/514579
00:27:49	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514
00:28:03	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Update document related to host aggregate https://review.openstack.org/514499
01:09:45	alex_xu	gmann: cleanup few things https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:cleanup_extension
01:09:58	alex_xu	gmann: I rememer we have a blueprint for those cleanup, do you have the link?
01:21:49	gmann	to track the work
01:21:49	gmann	alex_xu: https://etherpad.openstack.org/p/api-extensions-merge-queens
01:21:49	gmann	let me search
01:21:49	gmann	alex_xu: i think i created etherpad also for that, may be forgot to ping you.
01:21:49	alex_xu	gmann: thanks
01:21:49	gmann	alex_xu: yea, you can use this https://blueprints.launchpad.net/nova/+spec/api-extensions-merge-queens
01:36:10	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach interface https://review.openstack.org/512190
02:16:10	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach/swap volumes https://review.openstack.org/517205
02:26:17	openstackgerrit	Merged openstack/nova stable/newton: Set group_members when converting to legacy request spec https://review.openstack.org/517868
02:51:09	openstackgerrit	Matt Riedemann proposed openstack/nova stable/newton: Pass requested_destination in filter_properties https://review.openstack.org/516119
02:51:10	openstackgerrit	Matt Riedemann proposed openstack/nova stable/newton: Validate new image via scheduler during rebuild https://review.openstack.org/519684
02:51:10	openstackgerrit	Matt Riedemann proposed openstack/nova stable/newton: Fix non-parameterized service id in hypervisors sample tests https://review.openstack.org/519763
02:51:11	openstackgerrit	Matt Riedemann proposed openstack/nova stable/newton: Add security release note for OSSA-2017-005 https://review.openstack.org/519755
02:51:23	mriedem	tonyb: if you're around i just had to rebase those ^
02:51:28	mriedem	just need a re-approve on the stack
02:57:41	openstackgerrit	Tony Breeds proposed openstack/nova stable/newton: Pass requested_destination in filter_properties https://review.openstack.org/516119
02:57:42	openstackgerrit	Tony Breeds proposed openstack/nova stable/newton: Validate new image via scheduler during rebuild https://review.openstack.org/519684
02:57:42	openstackgerrit	Tony Breeds proposed openstack/nova stable/newton: Fix non-parameterized service id in hypervisors sample tests https://review.openstack.org/519763
02:57:43	openstackgerrit	Tony Breeds proposed openstack/nova stable/newton: Add security release note for OSSA-2017-005 https://review.openstack.org/519755
02:58:30	tonyb	mriedem: It seems I was off doign the same thing :(
02:59:38	mriedem	:( https://review.openstack.org/#/c/516119/5..6/nova/tests/unit/objects/test_request_spec.py
02:59:43	mriedem	plus you didn't drop the depends-on
02:59:47	mriedem	should i just push mine again?
03:00:52	mriedem	looks like i can't, no new changes
03:01:55	openstackgerrit	Tony Breeds proposed openstack/nova stable/newton: Fix non-parameterized service id in hypervisors sample tests https://review.openstack.org/519763
03:01:55	openstackgerrit	Tony Breeds proposed openstack/nova stable/newton: Pass requested_destination in filter_properties https://review.openstack.org/516119
03:01:56	openstackgerrit	Tony Breeds proposed openstack/nova stable/newton: Add security release note for OSSA-2017-005 https://review.openstack.org/519755
03:01:56	openstackgerrit	Tony Breeds proposed openstack/nova stable/newton: Validate new image via scheduler during rebuild https://review.openstack.org/519684
03:02:37	tonyb	mriedem: Fixed my messup and I switch the Depends-On to the merged review which should be harmless
03:03:05	mriedem	yeah ok
03:03:09	mriedem	you going to re-approve or should i?
03:03:25	tonyb	either or
03:03:35	mriedem	i will do it
03:03:59	tonyb	I was worried about it given I resolved a merge conflict but given yours was the same I'm happy now
03:04:03	tonyb	cool.
03:05:07	mriedem	done
03:05:14	tonyb	\/o
03:05:24	mriedem	and now i must go - laura is getting irritated because this is cutting into our 'stranger things' time
03:05:26	tonyb	Oh I seem to have a broken neck
03:05:27	tonyb	:(
03:05:41	tonyb	hehe okay enjoy
03:07:14	openstackgerrit	Huan Xie proposed openstack/nova master: XenAPI: Don't use nicira-iface-id for XenServer VIF https://review.openstack.org/451641
03:07:53	takashin	mriedem: Hi
03:08:23	takashin	mriedem: The condition under which py27 job is executed is wrong in nova-specs.
03:09:17	takashin	mriedem: https://review.openstack.org/#/c/508164/ has been merged.
03:10:10	takashin	mriedem: But it causes the following error.

Earlier   Later