| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-04 | |||
| 17:13:07 | dansmith | jaypipes: in SQLandia, if I to a SELECT * WHERE longstring="uuidthing", | |
| 17:13:21 | dansmith | is that much more efficient than doing an IN check for a list of one? | |
| 17:13:23 | openstackgerrit | Ed Leafe proposed openstack/nova master: WIP: Don't delete allocation if instance being scheduled https://review.openstack.org/460177 | |
| 17:13:23 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add a convenience attribute for reportclient https://review.openstack.org/462679 | |
| 17:13:28 | edleafe | dansmith: ^^ | |
| 17:13:35 | dansmith | meaning SELECT * WHERE longstring IN ("uuidthing") or whatever the syntax is | |
| 17:13:43 | dansmith | edleafe: woot | |
| 17:13:48 | edleafe | dansmith: just addressed the comments. Will work on the rest now | |
| 17:13:50 | mriedem | on an indexed column | |
| 17:13:59 | dansmith | mriedem: ah, yeah, good point the index | |
| 17:14:13 | mriedem | dansmith: well, after https://review.openstack.org/#/c/454887/ that is | |
| 17:14:29 | jaypipes | dansmith: all optimizers I know of will rewrite the latter to the former. | |
| 17:14:44 | dansmith | jaypipes: ah, okay | |
| 17:14:57 | dansmith | actually, I think I remember seeing warnings about doing that | |
| 17:18:31 | mriedem | dansmith: if you see the patch after it, i could have ComputeNode.get_by_uuid call ComputeNodeList.get_all_by_uuids and internally handle the [uuid] and [0], but i just wondered about the IN performance | |
| 17:19:06 | dansmith | mriedem: yeah | |
| 17:20:08 | mriedem | but maybe this is a small enough difference that it doesn't really matter and we could just squash those two changes, and just hide it all in the object code | |
| 17:20:46 | dansmith | yeah, seems like it might be | |
| 17:21:09 | mriedem | http://stackoverflow.com/questions/9926433/mysql-performance-in-clause-vs-equals-for-a-single-value | |
| 17:21:23 | mriedem | "There is no difference between the MySQL statements, and the MySQL optimiser will transform the IN to the = when IN is just one element. Don't bother." | |
| 17:21:30 | mriedem | so what jaypipes said | |
| 17:22:33 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: De-complicate some of the instance delete path https://review.openstack.org/458636 | |
| 17:22:34 | openstackgerrit | Dan Smith proposed openstack/nova master: Move to proper target_cell calling convention https://review.openstack.org/458634 | |
| 17:22:34 | openstackgerrit | Dan Smith proposed openstack/nova master: Make target_cell() yield a new context https://review.openstack.org/458635 | |
| 17:22:35 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack https://review.openstack.org/458537 | |
| 17:25:58 | peter-hamilton | mriedem: sorry, i was out - not a problem, i just wanted to put it in the queue :) | |
| 17:55:09 | openstackgerrit | Matt Riedemann proposed openstack/nova master: db api: add service_get_by_uuid https://review.openstack.org/461428 | |
| 17:55:09 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle uuid in HostAPI._find_service https://review.openstack.org/461519 | |
| 17:55:10 | openstackgerrit | Matt Riedemann proposed openstack/nova master: objects: add Service.get_by_uuid method https://review.openstack.org/461429 | |
| 17:55:10 | openstackgerrit | Matt Riedemann proposed openstack/nova master: objects: add ComputeNode.get_by_uuid method https://review.openstack.org/461448 | |
| 17:55:11 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle uuid in HostAPI.compute_node_get https://review.openstack.org/461532 | |
| 17:55:11 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add online data migration for populating services.uuid https://review.openstack.org/454899 | |
| 17:55:12 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add uuid to service.update notification payload https://review.openstack.org/461473 | |
| 17:57:27 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle uuid in HostAPI._find_service https://review.openstack.org/461519 | |
| 17:57:27 | openstackgerrit | Matt Riedemann proposed openstack/nova master: objects: add ComputeNode.get_by_uuid method https://review.openstack.org/461448 | |
| 17:57:28 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle uuid in HostAPI.compute_node_get https://review.openstack.org/461532 | |
| 17:57:29 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add uuid to service.update notification payload https://review.openstack.org/461473 | |
| 17:58:18 | dansmith | mriedem: pushing the same set three times in a row is fooling nobody | |
| 17:58:29 | mriedem | 2 times in a row | |
| 17:58:44 | dansmith | N times in a row | |
| 18:04:58 | mriedem | gah, plus i screwed up the topic branch | |
| 18:05:32 | efried | mriedem mordred jaypipes [service catalog bp] I'd like to settle this issue of determining interface if not otherwise specified. | |
| 18:05:43 | mriedem | pistols at dawn it is | |
| 18:05:56 | efried | At some point in the early (at least in my involvement) discussions, there was talk of trying the known ones in order. | |
| 18:06:02 | efried | This is a dict lookup, so that's cheap. | |
| 18:06:14 | mriedem | maybe the idea with starting using internal first is to avoid defaulting to public in ksa? | |
| 18:06:16 | efried | But on the other side, there's "use the default from ksa" | |
| 18:06:30 | efried | ...which is "public", for whatever that's worth. | |
| 18:06:44 | efried | Can any of you remember the rationale behind the "loop through 'em" suggestion? | |
| 18:07:04 | mriedem | efried: that was mordred or jaypipes' idea i thought, but presumably to hit the internal/admin urls before public | |
| 18:07:09 | efried | mriedem Yeah, starting with "internal" seemed like a more... uh... "secure" idea. | |
| 18:07:15 | mriedem | since today ops are probably using internal/admin | |
| 18:08:08 | efried | Okay, right now I've got internal->public->admin; that's an easy swap. But question remains: do we want to do the loop thing at all? | |
| 18:09:10 | mriedem | i don't really feel strongly either way, looping over a hard-coded list which might not match what's actually in the service catalog seems weird to me | |
| 18:09:52 | efried | mriedem Yeah, that makes me a tad uncomfortable too. Maybe mordred can point us to a way to get that list from ksa. | |
| 18:10:51 | mriedem | https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/identity/base.py#L173 | |
| 18:11:01 | mriedem | the docs say one thing but i'm pretty sure interface is open ended | |
| 18:11:23 | mriedem | although at one point i thought i found json schema in the keystone api for v3 that restricted the interfaces | |
| 18:11:24 | mriedem | to a known set | |
| 18:13:06 | efried | One approach is to say, we try these three, and if it ain't one of those, they better specify it. | |
| 18:13:10 | efried | They can always specify it. | |
| 18:14:14 | mriedem | efried: https://github.com/openstack/keystone/blob/master/keystone/catalog/schema.py#L94 | |
| 18:14:47 | efried | noyce. | |
| 18:14:47 | mriedem | so the interface types are validated via jsonschema in the keystone api, at least for the v3 api i think | |
| 18:15:02 | mriedem | so iterating through admin/internal/public probably seems fine | |
| 18:15:45 | efried | What order would you like? | |
| 18:16:06 | mriedem | don't ask me :) | |
| 18:16:08 | mriedem | ask operators | |
| 18:16:13 | mriedem | #openstack-operators | |
| 18:17:01 | abhi89 | dansmith, jaypipes: please review https://review.openstack.org/#/c/462561/.. its for stable/ocata merge.. | |
| 18:43:48 | mriedem | melwitt: looks like we have a KeyError when the tests run now because of ^ https://review.openstack.org/#/c/460305/6/nova/tests/unit/compute/test_compute.py | |
| 18:47:07 | mriedem | https://bugs.launchpad.net/nova/+bug/1688366 | |
| 18:47:07 | openstack | Launchpad bug 1688366 in OpenStack Compute (nova) "KeyError: u'overs' in unit test runs" [Low,Triaged] | |
| 18:47:28 | melwitt | mriedem: sigh ... I miss that config option that used to make format errors fatal for tests | |
| 18:47:42 | mriedem | i don't know why that was removed | |
| 18:47:50 | mriedem | maybe it was too useful | |
| 18:48:38 | melwitt | yeah, really. it was removed as part of the centralize-config-options work, and I tried to recreate the behavior with wonky monkey patching and wasn't successful https://review.openstack.org/#/c/411615 | |
| 18:52:44 | dansmith | 184489 | |
| 18:52:50 | dansmith | heh sorry | |
| 18:53:10 | melwitt | there's been something in the unit tests spewing garbled text into the output for months now, I assume that's another KeyError some place in a test exception | |
| 18:56:37 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Updates for [glance]api_servers and ksa https://review.openstack.org/461481 | |
| 18:56:47 | efried | mriedem ^^ | |
| 19:05:29 | openstackgerrit | Steve Noyes proposed openstack/nova master: Add Cinder v3 detach to shutdown_instance https://review.openstack.org/456877 | |
| 19:08:20 | dansmith | melwitt: I wonder if I'm failing this test because of legit lack of cells support in consoleauth? http://logs.openstack.org/37/458537/9/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/fe108a8/logs/testr_results.html.gz | |
| 19:12:45 | melwitt | dansmith: I would guess so. looking to see if we do a target cell for getting the vnc console | |
| 19:13:19 | melwitt | I can't remember what the state of the compute rpcapi mq switching is | |
| 19:13:44 | dansmith | yeah, I think it might actually be in the proxy thing | |
| 19:16:40 | melwitt | in nova/api/openstack/compute/remote_consoles.py it gets an instance and passes it down to get_vnc_console, so if the context has already been targeted by the get_instance, it should work | |
| 19:17:52 | dansmith | melwitt: right, but that's to get the url, right? then we actually connect through the novncproxy to get to the rfb data, | |
| 19:18:01 | dansmith | and it has to know where to proxy to, which is what is failing I think | |
| 19:18:20 | dansmith | it makes a consoleauth rpc call, which may be going to the wrong place, I'm guessing | |
| 19:18:21 | melwitt | oh, yeah. must be | |
| 19:18:44 | dansmith | although I don't see anything barfing about an rpc timeout or anything | |
| 19:19:29 | melwitt | yeah, dtp's change made the consoleauth calls stay at the API and that call shouldn't try to mq switch | |
| 19:20:58 | melwitt | I can never remember how the consoles work. get the url first, now looking for where it does the consoleauth part | |
| 19:21:08 | dansmith | yeah I have to re-learn each time | |
| 19:22:18 | melwitt | okay, the consoleauth is in compute/api, in get_vnc_console, so that should be fine I think | |
| 19:23:04 | dansmith | that's the thing that authorizes, not what checks right? | |
| 19:23:22 | dansmith | yeah | |
| 19:23:31 | dansmith | so we go there, create a token, return a url | |