Posted	Nick	Remark
#openstack-nova - 2017-09-20
20:31:30	dansmith	I could do some dansplaining
20:31:35	dansmith	see what I did there?
20:31:57	mriedem	how could i not
20:32:05	melwitt	heh
20:33:19	dansmith	mriedem: so, hangout? or have you seen enough of me for six months?
20:36:52	mriedem	i'm gearing up
20:37:22	melwitt	for a hangout?
20:37:44	mriedem	and the apocalypse
20:37:50	mriedem	but more a hangout right now yes
20:37:53	melwitt	:)
20:38:15	mriedem	https://hangouts.google.com/call/QDaiUUdRHaNIQmbJi5NVAAkE
20:39:09	dansmith	oh now?
20:39:16	mriedem	yeah
20:39:25	mriedem	my kid will be home in 20 making a bunch of noise
20:43:40	mriedem	process_sort_params in the db api
20:43:46	mriedem	default_keys=['created_at', 'id'],
20:58:57	bauzas	dansmith: mriedem: oh fun, rediscovered https://review.openstack.org/#/c/446446/5/specs/pike/approved/az-block-name-update.rst
20:59:09	bauzas	I should copyright that :)
21:20:16	mriedem	dansmith: talking about this https://github.com/openstack/nova/commit/c4820305d2f9ee8d62bcc708baf3fa6dfe7ca960
21:42:16	efried	stephenfin Ic05c2c8364e015f6878b0bc25449216624568ad5 ouch. This means folks who paid attention to the deprecation and moved to [vnc]vncserver_proxyclient_address are now busted, without a deprecation period on the old-name-in-the-new-group.
21:44:28	efried	Arguably the rename should have been done as part of the move. But it warn't. mriedem Can I get a ruling ^ ? (https://review.openstack.org/#/c/498387/)
21:49:50	mriedem	wuh
21:51:16	mriedem	(1) vncserver_listen was in the DEFAULT group, and moved to the [vnc] group, (2) vncserver_listen was in the [vnc] group and renamed to server_listen in the [vnc] group
21:51:38	mriedem	so now [DEFAULT]vncserver_listen just won't work, right?
21:51:48	mriedem	or will it?
21:52:36	melwitt	did anyone figure out how to get gerrit email notifications again?
21:53:13	mriedem	efried: i'm having a hard time following, do you have a bug report or a test to recreate the issue?
21:53:41	efried	mriedem Yeah, I'm writing up the bug report, and I have the fix.
21:58:18	efried	mriedem stephenfin https://bugs.launchpad.net/nova/+bug/1718545
21:58:20	openstack	Launchpad bug 1718545 in OpenStack Compute (nova) "[vnc]vncserver_proxyclient_address and [vnc]vncserver_listen removed without deprecation" [Undecided,New]
22:09:05	mriedem	thanks
22:09:47	mriedem	i'd be fine with a revert honestly
22:10:06	mriedem	the constant random config option changes annoy me abit
22:10:33	openstackgerrit	Merged openstack/nova master: Move the dac_admin privsep code to a new location. https://review.openstack.org/504194
22:12:13	openstackgerrit	Eric Fried proposed openstack/nova master: Restore [vnc]vnc*; remove [DEFAULT] equivalents https://review.openstack.org/505831
22:12:26	efried	mriedem stephenfin ^
22:14:30	openstackgerrit	Merged openstack/nova master: Squash dac_admin privsep context. https://review.openstack.org/504195
22:14:55	efried	cdent too, as a reviewer of the original.
22:15:02	openstackgerrit	Merged openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI https://review.openstack.org/478437
22:15:40	cdent	efried: I’m too asleep to give that a good review now, and I guess must have been too asleep when I reviewed it the first time
22:15:50	efried	You're not alone :)
22:16:11	mriedem	efried: commented
22:16:38	cdent	I’ve put it on my list for tomorrow
22:17:35	mriedem	dansmith: you hate methods with 100 kwargs?!
22:17:45	dansmith	mriedem: yes, yes I do
22:17:52	dansmith	mriedem: especially when they're ALL optional
22:17:55	mriedem	like _provision_instances, build_instances, build_and_run_instance, rebuild_instance
22:17:56	mriedem	ETC?!
22:18:03	dansmith	yeah, all of ours
22:18:36	mriedem	does java allow default values yet?
22:19:15	dansmith	heh idk
22:19:18	mriedem	guess not
22:19:45	mriedem	i remember missing that when going from c++ to java, and then being pleasantly surprised to see it again when going from java to python
22:20:07	mriedem	now, what would go do
22:20:44	tasker	require an interface.
22:21:22	openstackgerrit	Merged openstack/nova master: Add ComputeNodeList.get_by_hypervisor_type() https://review.openstack.org/501343
22:21:53	dansmith	tasker: I dunno if you're making fun of python or go with that statement, but... don't tell me :)
22:21:56	openstackgerrit	Merged openstack/nova master: Add nova-manage db command for ironic flavor migrations https://review.openstack.org/501025
22:22:15	tasker	. D
22:22:30	openstackgerrit	Merged openstack/nova master: Squash dacnet_admin privsep context. https://review.openstack.org/504805
22:22:46	mriedem	now someone needs to backport all of these https://review.openstack.org/#/q/topic:ironic-offline-migration+(status:open+OR+status:merged)
22:25:54	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/505839
22:26:08	efried	mriedem Replied :(letting you know since we don't seem to be getting emails):
22:28:01	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646
22:28:04	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708
22:49:44	mriedem	i haven't read my gerrit email in a looong time
22:49:49	mriedem	couldn't keep up with it
22:57:39	dansmith	yeah, gerrit emails are borked today, known issue
23:03:27	dansmith	here comes
23:03:42	openstackgerrit	Dan Smith proposed openstack/nova master: Make instance_list honor global query limit https://review.openstack.org/504984
23:03:42	openstackgerrit	Dan Smith proposed openstack/nova master: Add base implementation for efficient cross-cell instance listing https://review.openstack.org/504983
23:03:43	openstackgerrit	Dan Smith proposed openstack/nova master: Support pagination in instance_list https://review.openstack.org/504986
23:03:43	openstackgerrit	Dan Smith proposed openstack/nova master: Add db.instance_get_by_sort_filters() https://review.openstack.org/504985
23:03:44	openstackgerrit	Dan Smith proposed openstack/nova master: Add tests to validate instance_list handles faults correctly https://review.openstack.org/505392
23:03:44	openstackgerrit	Dan Smith proposed openstack/nova master: Add fault-filling into instance_get_all_by_filters_sort() https://review.openstack.org/505391
23:03:45	openstackgerrit	Dan Smith proposed openstack/nova master: Copy some tests to a cellsv1 mixin https://review.openstack.org/505442
23:03:45	openstackgerrit	Dan Smith proposed openstack/nova master: Add get_instance_objects_sorted() https://review.openstack.org/505417
23:03:46	openstackgerrit	Dan Smith proposed openstack/nova master: Use improved instance_list module in compute API https://review.openstack.org/505418
23:03:46	openstackgerrit	Dan Smith proposed openstack/nova master: Fix a pagination logic bug in test_bug_1689692 https://review.openstack.org/505661
23:03:47	openstackgerrit	Dan Smith proposed openstack/nova master: Remove legacy fault-loading routines https://review.openstack.org/505456
23:12:02	cfriesen	from what I understand nova-api handles both the normal API endpoint and the metadata endpoint...how does the multiple calls to launcher.launch_service() end up using the same process(es)?
23:31:15	tasker	did review.openstack just break?
23:34:00	tasker	nevermind - i can get to it again.
23:56:51	openstackgerrit	Chris Dent proposed openstack/nova master: Move project_id and user_id to Allocation object https://review.openstack.org/500410
23:56:52	openstackgerrit	Chris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073
23:56:52	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051
#openstack-nova - 2017-09-21
00:39:31	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/505839
00:41:48	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646
00:41:52	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708
01:04:58	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements https://review.openstack.org/493146
01:06:41	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-novaclient stable/pike: Updated from global requirements https://review.openstack.org/493187
01:23:37	mriedem	owalsh: whenever you get a sec, if you know how to configure nova and cells v2 for clustered rabbit with the cell transport_url, it would be cool if you could help sort this out https://bugs.launchpad.net/nova/+bug/1717915
01:23:38	openstack	Launchpad bug 1717915 in oslo.messaging "nova services and transport_url, cannot connect to vhost if specified" [Undecided,New]
01:30:01	openstackgerrit	jichenjc proposed openstack/nova master: Add quiesce and unquiesce in support matrix https://review.openstack.org/482390
01:33:08	openstackgerrit	jichenjc proposed openstack/nova master: check query param for service's index function https://review.openstack.org/489492
01:36:21	openstackgerrit	Matt Riedemann proposed openstack/nova stable/pike: Split out the core of the ironic flavor migration https://review.openstack.org/505901
01:36:22	openstackgerrit	Matt Riedemann proposed openstack/nova stable/pike: Add nova-manage db command for ironic flavor migrations https://review.openstack.org/505903
01:36:22	openstackgerrit	Matt Riedemann proposed openstack/nova stable/pike: Add ComputeNodeList.get_by_hypervisor_type() https://review.openstack.org/505902
01:45:44	openstackgerrit	jichenjc proposed openstack/nova master: Remove 400 as expected error https://review.openstack.org/505485
02:01:43	openstackgerrit	Merged openstack/nova master: Optimize MiniDNS for fewer syscalls https://review.openstack.org/486829

Earlier   Later