| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-20 | |||
| 20:09:02 | bauzas | I know, I co-authored that fix | |
| 20:09:11 | mriedem | i think you authored it.. | |
| 20:09:29 | bauzas | I just passed a new revision but meh | |
| 20:09:40 | bauzas | and yeah, we could functional test it | |
| 20:09:55 | mriedem | this https://review.openstack.org/#/c/491439/ | |
| 20:10:00 | bauzas | lke a regression chnage | |
| 20:10:43 | bauzas | oh fun, it was co-authored because of a pep8 fix :) | |
| 20:11:41 | bauzas | ah nvm, got it :) | |
| 20:11:53 | bauzas | anyway, yeah I can work on a regression test | |
| 20:12:03 | bauzas | mriedem: or a func test, as you want | |
| 20:12:32 | mriedem | i don't know if it was regressed in pike or not | |
| 20:12:38 | mriedem | or if this was a latent bug before pike | |
| 20:15:01 | bauzas | mriedem: that should have been regression when we merged the claims stuff | |
| 20:15:27 | bauzas | mriedem: because before that, when you were asking for 10 instances, it was possibly returning you 10 times the same host | |
| 20:16:17 | bauzas | mmm, wait | |
| 20:26:44 | bauzas | mriedem: holy fuck, we introduced the problem with https://github.com/openstack/nova/commit/2bd7df84 | |
| 20:26:47 | bauzas | whack-a-mole | |
| 20:27:10 | bauzas | we changed _schedule to return the number of hosts per instances | |
| 20:27:27 | bauzas | so it's now returning 1 host | |
| 20:27:33 | bauzas | for a live-migration | |
| 20:27:47 | bauzas | but we haven't fixed the caller, so it's still awaiting 10 | |
| 20:27:51 | bauzas | so we're fscked | |
| 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 | |