Posted	Nick	Remark
#openstack-nova - 2017-08-31
21:22:38	efried	I think if you give me nested resource providers, with some careful modeling I can do everything in the virt driver. The rest of nova doesn't need to know about a device as being any different from any other resource.
21:27:48	efried	edleafe You mentioned earlier that "we still have some work to do with traits and nested providers before any of this will actually work" - somewhere I can find a summary of what's still on the table?
21:33:29	cfriesen_	efried: I think the PTG etherpad has some links
21:33:39	efried	ah, cool
21:33:42	efried	thanks cfriesen_
21:37:29	cfriesen	efried: I think this is the nested providers work: https://review.openstack.org/#/q/topic:bp/nested-resource-providers,n,z
21:38:06	edleafe	efried: yeah. There's the series from alex_xu starting with https://review.openstack.org/#/c/489206/, as well as Jay's nested RP series starting with https://review.openstack.org/#/c/470575/
21:38:17	efried	edleafe cfriesen Thanks. Looks like not toooo much; hopefully containable in early q?
21:38:48	edleafe	efried: only when you look at the code with light-red-tinted lenses
21:38:59	cfriesen	efried: edleafe: I think there's also a missing piece to wire it up to live migration
21:39:02	efried	Sigh.
21:39:17	efried	I just thought you were blushing.
21:40:04	mriedem	there are bugs that still need to be fixed from claims in the scheduler before we can add nested and shared RPs
21:40:16	mriedem	the move operations are all over the floor
21:40:28	openstackgerrit	Matt Riedemann proposed openstack/nova master: Modernize set_vm_state_and_notify https://review.openstack.org/499799
21:41:13	mriedem	for anyone that cares, the main ones i'm tracking for backports to pike are
21:41:15	mriedem	1. https://bugs.launchpad.net/nova/+bug/1713786
21:41:16	openstack	Launchpad bug 1713786 in OpenStack Compute (nova) "Allocations are not managed properly in all evacuate scenarios" [High,In progress] - Assigned to Matt Riedemann (mriedem)
21:41:26	mriedem	2. https://bugs.launchpad.net/nova/+bug/1713796
21:41:27	openstack	Launchpad bug 1713796 in OpenStack Compute (nova) "Failed unshelve does not remove allocations from destination node" [High,Triaged]
21:41:47	mriedem	i think that's it for right now
21:41:57	mriedem	once those are fixed and backported to pike and merged, we can do a pike patch release
21:42:17	mriedem	i've got the fix for half of the first bug up for review
21:42:27	bauzas	roger.
21:42:28	mriedem	starting here https://review.openstack.org/#/c/499678/2
21:43:08	mriedem	tomorrow i'm going to work on the 2nd half, which is if evacuate fails on the compute, we have to cleanup the allocations created by the scheduler for the dest node
21:43:30	mriedem	like, if the rebuild claim fials
21:43:32	mriedem	*fails
21:46:38	cfriesen	question about flavors...if I create a flavor that isn't public, does that automatically give my tenant access to that flavor? if so, where is that code?
21:47:05	mriedem	stephenfin: gibi: sdague: fyi, new docs theme breaks the hide/show ability in the versioned notifications samples https://bugs.launchpad.net/nova/+bug/1714363
21:47:06	openstack	Launchpad bug 1714363 in OpenStack Compute (nova) "docs: versioned notification samples are always shown now and you can't hide them" [Medium,Confirmed]
21:48:12	mriedem	cfriesen: don't think so
21:48:21	mriedem	looking at nova.compute.flavors.create
21:51:19	cfriesen	mriedem: good, I'm not blind. :)
22:07:29	openstackgerrit	Eric Fried proposed openstack/nova master: Include /resource_providers/uuid/allocations link https://review.openstack.org/499826
22:32:21	openstackgerrit	Merged openstack/nova master: api-ref: add warnings about forcing the host for live migrate/evacuate https://review.openstack.org/499769
#openstack-nova - 2017-09-01
00:21:29	openstackgerrit	Merged openstack/nova master: Clean variable names and docs around neutron allocate_for_instance https://review.openstack.org/489267
02:49:27	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add recreate test for evacuate claim failure https://review.openstack.org/499874
03:02:08	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add recreate test for evacuate claim failure https://review.openstack.org/499874
03:02:09	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add a test to make sure failed evacuate cleans up dest allocation https://review.openstack.org/499877
03:21:22	openstackgerrit	Matt Riedemann proposed openstack/nova master: Remove dest node allocation if evacuate MoveClaim fails https://review.openstack.org/499878
05:55:42	openstackgerrit	Merged openstack/nova master: De-duplicate two delete_allocation_for_* methods https://review.openstack.org/496936
06:07:47	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954
06:24:37	openstackgerrit	Lei Zhang proposed openstack/nova master: placement: extract traits from flavor extra spec https://review.openstack.org/492026
06:28:21	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954
06:52:58	openstackgerrit	LiChunlin proposed openstack/nova master: update the description of hypervisor statistics response https://review.openstack.org/462355
07:35:06	openstackgerrit	David Rabel proposed openstack/nova master: Adds support for graceful shutdown for VMware instances https://review.openstack.org/494169
07:36:00	gibi	dansmith: you are right that https://review.openstack.org/#/c/498950 was failed notification tests due to the service version bump
07:44:51	openstackgerrit	David Rabel proposed openstack/nova master: VMware: add support for graceful shutdown of instances https://review.openstack.org/494169
08:40:04	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451
09:08:08	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add VersionedObjectPrintable mixin https://review.openstack.org/493082
09:24:07	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/nova master: [WIP][POC] Host plugin information from os-vif https://review.openstack.org/499941
09:24:33	ygl_	hi all
09:24:39	ygl_	my instance creaiton is failing
09:24:59	ygl_	its is saying "Connection to glance host failed:"
09:25:05	ygl_	but i can list the images
10:16:23	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Default the service version in the notification tests https://review.openstack.org/499957
10:40:07	openstackgerrit	Balazs Gibizer proposed openstack/nova master: doc: fix show-hide sample in notification devref https://review.openstack.org/499963
11:33:30	openstackgerrit	Markus Zoeller (markus_z) proposed openstack/nova master: docs: Explain the flow of the "serial console" feature https://review.openstack.org/476188
12:15:54	wlfightup	https://blueprints.launchpad.net/nova/+spec/instance-mem-and-cpu-hot-plugging
12:36:19	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add VersionedObjectPrintable mixin https://review.openstack.org/493082
12:41:42	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add '_has_dns_extension' function https://review.openstack.org/480675
12:41:43	openstackgerrit	Stephen Finucane proposed openstack/nova master: Always refresh cache when checking extensions https://review.openstack.org/480674
12:41:43	openstackgerrit	Stephen Finucane proposed openstack/nova master: metadata: Don't use nova-net DNS values with neutron https://review.openstack.org/480616
12:41:44	openstackgerrit	Stephen Finucane proposed openstack/nova master: Store dns_domain value from neutron https://review.openstack.org/480676
12:41:44	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add '_has_qos_queue_extension' function https://review.openstack.org/500010
12:42:18	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/500011
12:47:01	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/500028
13:23:50	openstackgerrit	Eric Fried proposed openstack/nova master: Include /resource_providers/uuid/allocations link https://review.openstack.org/499826
13:23:53	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Make 'get_domain' private https://review.openstack.org/417378
13:29:59	cdent	fried_rice: I’m fairly in favor of making that links change, but I dunno if it will pass the rules
13:30:22	fried_rice	cdent Yuh; I'm happy to take my guidance from whoever - but who is that?
13:30:36	cdent	I would guess a final arbiter is someone with +2, which is not me, nor figleaf
13:30:52	cdent	I’ll go ahead and +1 it to express my opinion
13:31:30	fried_rice	Ah, gotcha. Someone like dansmith or leakypipes or mriedem
13:31:50	fried_rice	or sdague
13:32:27	figleaf	cdent: it's not fixing an error in the sense we intended. It's correcting an oversight
13:32:43	mriedem	is it changing the response?
13:32:52	figleaf	yes
13:32:52	mriedem	if it's changing the response then it's a new microversion
13:32:56	cdent	sdague is probably the main arbiter on this kind of thing
13:33:12	cdent	mriedem: the squiggle is that we changed the content of that section before without a microversion
13:33:22	cdent	thus the lawyering going on now
13:33:36	figleaf	two clouds requesting the same microversion would get different data
13:33:42	mriedem	past performance is not an indicator of future earnings
13:34:13	mriedem	what was the previous non-versioned change?
13:34:41	cdent	Yeah, the rules are it must be a new microversion. The reason I’m being squishy about it is just to reinforce that just because I wrote the latest version of the rules doesn’t mean I’m in love with microversions and want to marry them.
13:34:52	fried_rice	mriedem https://review.openstack.org/#/c/468923/1/nova/api/openstack/placement/handlers/resource_provider.py
13:34:53	cdent	mriedem: it’s linked in fried_rice’s comments
13:34:54	cdent	jinx
13:35:15	mriedem	let me note that it's too early in my day to have to deal with the age old, "does this require a microversion?" question
13:35:49	figleaf	it's the usual problem: a seemingly backwards-compatible change
13:36:13	mriedem	so alex_xu and sdague approved https://review.openstack.org/#/c/468923/
13:36:20	mriedem	with no comment on the microversion part of this
13:36:29	fried_rice	right
13:38:10	cdent	fried_rice: did you find this because you are actually using the links, or just because you were having a poke around. If someone’s using genuine hypertextuality, i’m going to need to sit down.
13:39:10	cdent	(somewhat related, I’ve started using genuine hypertextuality in my gabbi tests of placement, just for the sake of it)
13:39:20	fried_rice	cdent Poking around. Larnin' up the placement API and resource providers and whatnot.
13:39:32	fried_rice	Working towards being able to do generic device management via placement.
13:39:46	cdent	Larnin’!!!??? Ain’t no body got time for that.

Earlier   Later