Posted	Nick	Remark
#openstack-nova - 2017-01-09
10:53:44	openstackgerrit	sahid proposed openstack/nova: numa: update numa usage incording the cpu reserved https://review.openstack.org/400877
10:56:28	openstackgerrit	Huan Xie proposed openstack/nova: xenapi: support the hotplug of a neutron port https://review.openstack.org/365505
11:15:09	openstackgerrit	Huan Xie proposed openstack/nova: xenapi: support the hotplug of a neutron port https://review.openstack.org/365505
11:24:51	huanxie	@johnthetubaguy Could you help review my patch for using os-xenapi lib https://review.openstack.org/#/c/406059/ https://review.openstack.org/#/c/411241/, thanks a lot
11:44:05	openstackgerrit	sahid proposed openstack/nova: libvirt: update the active migrations DS to support migration result https://review.openstack.org/409260
11:44:06	openstackgerrit	sahid proposed openstack/nova: libvirt: ensure live-migration success https://review.openstack.org/409261
11:47:47	openstackgerrit	Alex Xu proposed openstack/nova: [POC][placement] Caches and synchronizes the traits from os_traits https://review.openstack.org/416007
11:47:48	openstackgerrit	Alex Xu proposed openstack/nova: [POC][placement] Add Trait and TraitList objects https://review.openstack.org/376199
11:48:17	openstackgerrit	Marco Voelz proposed openstack/nova-specs: Add port deletion on termination of server https://review.openstack.org/416997
11:51:42	openstackgerrit	Jeremy Liu proposed openstack/nova: [3/3]Replace six.iteritems() with .items() https://review.openstack.org/415080
11:54:11	openstackgerrit	Alex Xu proposed openstack/nova: [POC][placement] Caches and synchronizes the traits from os_traits https://review.openstack.org/416007
11:54:12	openstackgerrit	Alex Xu proposed openstack/nova: [POC][placement] Import os_traits library https://review.openstack.org/377381
11:54:12	openstackgerrit	Alex Xu proposed openstack/nova: [POC][placement] Add Trait and TraitList objects https://review.openstack.org/376199
11:54:13	openstackgerrit	Alex Xu proposed openstack/nova: [POC][placement] Add Traits related table to the api database https://review.openstack.org/376198
12:04:07	openstackgerrit	melanie witt proposed openstack/nova: WTFWIP: Multicell support for instance listing https://review.openstack.org/396775
12:17:02	melwitt	superdan: FYI this change is needed in order to get use_neutron to default to False in devstack when using n-net for a local cells v1 environment https://review.openstack.org/#/c/416401/
12:18:03	melwitt	so I had to rebase your devstack change locally to get a cells v1 env running
12:21:30	openstackgerrit	Marco Voelz proposed openstack/nova-specs: Add port deletion on termination of server https://review.openstack.org/416997
12:33:16	openstackgerrit	Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142
12:56:45	openstackgerrit	Li Yingjun proposed openstack/nova: Notifications for flavor operations https://review.openstack.org/398171
13:00:50	openstackgerrit	Li Yingjun proposed openstack/nova: Update notification for flavor https://review.openstack.org/413846
13:19:49	openstackgerrit	Pawel Koniszewski proposed openstack/nova: Add new configuration option live_migration_scheme https://review.openstack.org/410817
13:49:18	openstackgerrit	Alex Szarka proposed openstack/nova: Transform instance.resize.error notifications https://review.openstack.org/417882
13:49:52	openstackgerrit	Alex Szarka proposed openstack/nova: Transform instance.resize.error notifications https://review.openstack.org/417882
13:52:33	edleafe	Scheduler subteam meeting in 7 minutes in #openstack-meeting-alt
13:58:38	openstackgerrit	Alex Szarka proposed openstack/nova: Transform instance.resize.error notifications https://review.openstack.org/417882
14:28:01	superdan	melwitt: ah cool, I just ended hacking around it, but that's better
14:30:31	diana_clarke	superdan is going to be a sad panda when he realizes it's Monday, not Friday.
14:31:25	dims	diana_clarke : LOL :)
14:33:24	kashyap	diana_clarke: We should consider the corner case that it _is_ a "Yay! Monday" case for Dan?
14:33:55	diana_clarke	kashyap: Only parents look forward to Mondays :)
14:34:28	kashyap	:-)
14:44:57	openstackgerrit	Gábor Antal proposed openstack/nova: Removed unnecessary parantheses and fixed formation https://review.openstack.org/370549
14:46:57	mtreinish	mriedem: yeah, the logic for networks normally passes a specific network when creating a server
14:47:05	mtreinish	we had to do that for the case with more than network being present
14:50:20	mtreinish	mriedem: most of the tests call https://github.com/openstack/tempest/blob/master/tempest/test.py#L609 and use the network returned as the tenant_network arg in the create server call
14:54:36	mtreinish	mriedem: there are tests that don't set that don't do that though. Like the auto allocate network test
15:03:58	lyarwood_	does anyone have an example stable/* local.conf to hand for devstack? It appears that the CI jobs on stable are using Zuul globals and not local.conf to pick this up.
15:21:57	openstackgerrit	Christopher Martin proposed openstack/nova: Stop using mox in test_compute_cells.py https://review.openstack.org/389750
15:39:40	rfolco	cdent, here?
15:40:13	cdent	rfolco: am now, but seem to be having some network difficulty, so if I disappear, that's why
15:40:21	rfolco	ack
15:41:00	rfolco	cdent, do you think I should replace decorator with a custom version exception handler inside the new method ?
15:41:11	rfolco	cdent, or returning 404 would be acceptable
15:41:27	cdent	Yeah, I would just check the version and if it too low, raise a 405
15:41:57	cdent	there's a few example of how to do that in other areas of the code
15:42:08	cdent	if you can't find it let me now, I'll find a link
15:42:45	rfolco	cdent, ok cool. Will remove the decorator then. Thanks for the tip.
15:43:42	openstackgerrit	Radoslav Gerganov proposed openstack/nova: Add connection pool size to vSphere settings https://review.openstack.org/414085
15:44:14	cdent	rfolco: sure, sorry for misleading you before. I was optimistic about being able to make the decorator more intelligent
15:46:57	openstackgerrit	Julian proposed openstack/nova: Transform keypair.create notification https://review.openstack.org/408689
15:47:13	cdent	but in the end too optimistic
15:47:27	bauzas	gmrlrlbl, we don't have a ComputeNode.get_by_uuid() method...
15:47:42	bauzas	and we don't have an host field for the ResourceProvider
15:47:45	bauzas	graaaaah
15:48:06	rfolco	cdent, well, I was able to hack how the decorator would return non-404, but that's ugly
15:48:20	bauzas	so, no way to update the list of HostStates by keying (host, node) unless adding a new method :(
15:48:50	bauzas	cdent: dansmith: for all the drivers but Ironic, host == node
15:49:37	bauzas	cdent: dansmith: should I add a new DB call for getting the host name just because we have Ironic ? meh
15:51:53	cdent	bauzas: I'm not understanding what you're trying to do? Are you saying: you have a list of compute nodes from placement api, now you want to make a collection of host states?
15:52:03	bauzas	cdent: exactly
15:52:11	bauzas	cdent: but we're keying per a tuple
15:52:18	bauzas	cdent: which (host, node)
15:52:24	bauzas	cdent: so I need to know the host value
15:52:47	cdent	I believe jay has made some adjustment on that, at least in the resource tracker, but I'm not sure if it impacts where you're at
15:52:51	bauzas	cdent: for 90% of our deployments, host == hypervisor_name
15:53:39	bauzas	but given Ironic has host != hypervisor_hostname, I probably need to add a helper method in the ComputeNode object like compute_node.get_by_resource_provider(uuid)
15:53:56	bauzas	or I could just get_by_uuid() because fortunately, cn.uuid == rp.uuid
15:54:15	bauzas	but that sounds silly to DB lookup just because of Ironic
15:54:21	bauzas	graaaah
15:55:25	bauzas	cdent: and yeah, I know jay's cleanups but AFAICT, we're not changing yet that
15:56:24	cdent	presumably he'll show up at some point today and we can get his input too.
16:24:33	cdent	mriedem: could you remind me where you left the ptg planning etherpad?
16:24:45	mriedem	cdent: https://etherpad.openstack.org/p/nova-ptg-pike
16:24:53	cdent	thanks
16:27:42	openstackgerrit	Christopher Martin proposed openstack/nova: Change soft delete resources to prevent DOS https://review.openstack.org/386756
16:30:43	mriedem	ildikov: i won't be at the nova/cinder meeting probably, i'll be around but have about an hour today to get some stuff done before i have to catch a flight
16:31:00	mriedem	ildikov: i'd say priority this week should be figuring out what's up with the endpoint 404 for using cinder v3
16:31:38	ildikov	mriedem: tnx for the notice, safe travels!
16:31:47	ildikov	mriedem: I didn;t have much time to play more with that
16:32:36	ildikov	mriedem: I fixed the issue locally with hard coding the endpoint_template, but for instance the new attach calls in Cinder still acted weird while testing
16:32:55	smcginnis	ildikov: Acted weird?
16:32:56	ildikov	mriedem: like on call worked from the CLI, but not from Nova
16:33:02	ildikov	*one
16:33:03	smcginnis	Ah
16:33:17	mriedem	well, which cli?
16:33:29	mriedem	remember that latest novaclient doesn't proxy through to cinder anymore
16:33:34	ildikov	I called attachment-create and I looked into the logs, the exact same looking call was 202 from the CLI and got 404 when Nova called it...
16:33:36	mriedem	so things like listing attachments in novaclient aren't using cinder
16:33:41	ildikov	Cinder CLI, sorry
16:33:55	ildikov	the old Nova attach call worked fine
16:34:11	ildikov	but the new call failed for at least seemingly no good reason
16:34:12	smcginnis	ildikov: Well that's good. At least the existing stuff didn't break with it.
16:34:15	openstackgerrit	Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961
16:34:15	openstackgerrit	Sylvain Bauza proposed openstack/nova: WIP/DNM : Put all the magic in a PlacementClient https://review.openstack.org/417111
16:34:23	openstackgerrit	Dan Smith proposed openstack/nova: WIP: Make metadata server know about cell mappings https://review.openstack.org/417962
16:34:23	openstackgerrit	Dan Smith proposed openstack/nova: WTFWIP: Multicell support for instance listing https://review.openstack.org/396775
16:35:03	ildikov	mriedem: smcginnis: in Devstack with admin cred I got 'Volume not found' and with demo cred I got 'Malformed request' error
16:35:44	ildikov	it can still be some misconfig on my side, not saying it's not, but still looks odd
16:36:06	ildikov	smcginnis: no, it seemed to work, but I didn't do any thorough testing on it either

Earlier   Later