Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-09-20
20:27:59 bauzas definitely a pike regression then
20:28:16 bauzas but we could write a func test anyway
20:29:07 mriedem yeah so just write a functional regression test like we have for others
20:29:44 mriedem should be pretty simple, create 2 computes and 2 instances forced to 1 compute, then live migrate one of the instances and it should fail with novalidhost
20:30:43 mriedem melwitt: dansmith: cells meeting rodeo in 30 minutes
20:30:53 dansmith yup
20:31:03 mriedem trying to wrap my head around this heapq craziness
20:31:25 dansmith do you want to do this one as a hangout?
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: Add base implementation for efficient cross-cell instance listing https://review.openstack.org/504983
23:03:42 openstackgerrit Dan Smith proposed openstack/nova master: Make instance_list honor global query limit https://review.openstack.org/504984
23:03:43 openstackgerrit Dan Smith proposed openstack/nova master: Add db.instance_get_by_sort_filters() https://review.openstack.org/504985
23:03:43 openstackgerrit Dan Smith proposed openstack/nova master: Support pagination in instance_list https://review.openstack.org/504986
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:44 openstackgerrit Dan Smith proposed openstack/nova master: Add tests to validate instance_list handles faults correctly https://review.openstack.org/505392
23:03:45 openstackgerrit Dan Smith proposed openstack/nova master: Add get_instance_objects_sorted() https://review.openstack.org/505417
23:03:45 openstackgerrit Dan Smith proposed openstack/nova master: Copy some tests to a cellsv1 mixin https://review.openstack.org/505442
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:46 openstackgerrit Dan Smith proposed openstack/nova master: Use improved instance_list module in compute API https://review.openstack.org/505418
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: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051
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
#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

Earlier   Later