| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-06 | |||
| 19:06:35 | openstackgerrit | Rick Bartra proposed openstack/nova master: Add policy granularity to the Flavors API https://review.openstack.org/449288 | |
| 19:06:46 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add ability to signal and perform online volume size change https://review.openstack.org/454322 | |
| 19:16:09 | melwitt | dansmith: I've mostly removed the instance uuid from the access url for the console stuff, but met a confusing point regarding the transition period between consoleauth/memcached and the database backend, | |
| 19:16:32 | melwitt | the code in the console proxy was using the presence of the instance uuid to determine whether to go to the database backend for validation vs going to the consoleauth service, | |
| 19:16:45 | melwitt | without the instance uuid, we don't really know if the access url is old or new, memcached or database. I'm not sure what to do about that | |
| 19:16:57 | melwitt | I thought, could try first for the database, if that fails, try consoleauth, if that fails too then it's really invalid. but that assumes the operator is running consoleauth still. which might be okay | |
| 19:17:18 | melwitt | my concern there is after all of the outstanding consoles have moved to the database backend, it's safe to stop running consoleauth, but then the fallback behavior should also be turned off. | |
| 19:17:31 | melwitt | so I was thinking we might need a config option | |
| 19:17:49 | dansmith | I don't have this cached in my head.. if hitting the DB is the way forward, | |
| 19:17:51 | dansmith | why not just always do that and stop looking at the auth cache? | |
| 19:18:03 | dansmith | could have a workaround conf for it I guess, but if database always works... | |
| 19:18:17 | melwitt | because of existing urls that people might be using, that had been generated pre-database | |
| 19:18:21 | mriedem | jaypipes: ok comments inline https://review.openstack.org/#/c/476631/ | |
| 19:18:43 | dansmith | melwitt: aren't those expired in like an hour or something? | |
| 19:19:32 | melwitt | dansmith: maybe. I was thinking that, the easy way would be just invalidate them all and make people get new urls. but I dunno how accommodating we need to be about the transition | |
| 19:19:53 | dansmith | melwitt: if it's a short timeout, I don't think we should need to jump through crazy hoops for that | |
| 19:20:06 | melwitt | k. I'll double check what it is | |
| 19:20:20 | dansmith | I wish we could talk about this and fix it in like one week.. I keep paging this data in and out of my brain | |
| 19:29:14 | mriedem | give your mind a break, | |
| 19:29:19 | mriedem | with christopher cross https://www.youtube.com/watch?v=VMkIuKXwmlU | |
| 19:29:59 | mriedem | he's like the fat older brother of that guy from the guardians of the galaxy movies | |
| 19:30:13 | melwitt | hah, I was just thinking, he looks like somebody else | |
| 19:30:17 | mriedem | chris pratt | |
| 19:30:41 | mriedem | http://i.huffpost.com/gen/1939272/images/o-CHRIS-PRATT-facebook.jpg | |
| 19:30:44 | mriedem | same fing person | |
| 19:31:00 | melwitt | hahaha | |
| 19:32:05 | mriedem | he's got 18 strings but only needs like 6 | |
| 19:32:09 | mriedem | maybe 4 | |
| 19:32:32 | melwitt | there's a lot about this scene I didn't expect considering how the song sounds | |
| 19:32:38 | mriedem | the jersey | |
| 19:32:56 | mriedem | laid | |
| 19:32:57 | mriedem | back | |
| 19:35:44 | melwitt | this consoles code doesn't make any sense to me. says "Remove expired tokens from the cache." but I only see a "get" no delete of them | |
| 19:38:23 | mriedem | shouldn't the cache auto remove things based on the timer? | |
| 19:38:36 | mriedem | using oslo.cache i assume | |
| 19:38:50 | mriedem | it might just be an old comment from when we switched over to oslo.cache, | |
| 19:38:54 | mriedem | that was never cleaned up | |
| 19:39:01 | melwitt | the token cache is a memcached thing. and as far as I can tell, it doesn't expire anything. I'm sure I'm just not finding where it is maybe | |
| 19:39:01 | mriedem | i.e. maybe before that refactor we were manually removing things? | |
| 19:39:10 | melwitt | oh, hm | |
| 19:39:13 | mriedem | i think there is a default timeout in oslo.cache | |
| 19:39:49 | mriedem | https://docs.openstack.org/oslo.cache/latest/opts.html | |
| 19:40:29 | melwitt | I see. thanks | |
| 19:40:50 | mriedem | i'm only guessing, i do remember going over the patch in nova that converted to use oslo.cache though and had questions like this | |
| 19:42:02 | melwitt | yeah, I think you're right though. the consoleauth stuff just uses nova/cache_utils and in cache_utils I see there's oslo_cache.dict and dogpile.cache.memcached as options. I'm not sure what/how we default to | |
| 19:45:27 | melwitt | default TTL is 600, so 10 minutes. a lot shorter than I thought | |
| 19:47:34 | mriedem | ildikov: i see the live migration change isn't in this series https://review.openstack.org/#/c/330285/ | |
| 19:47:54 | mriedem | ildikov: so, i guess to test the new api flow plus live migration, we can just do a DNM nova change that depends on both | |
| 19:47:56 | mriedem | yeah? | |
| 19:50:10 | ildikov | mriedem: I can add it to the chain or just do what you say | |
| 19:50:30 | mriedem | ildikov: let's test it out first, i'll push a patch | |
| 19:51:08 | ildikov | mriedem: sounds good, thanks | |
| 19:51:29 | melwitt | okay, so if they configured oslo.cache themselves we use that, if they didn't, we make it a oslo_cache.dict | |
| 19:52:06 | mriedem | from what i remember, if it's not configured for some other backend, we use an in-memory dogpile cache dict cache yeah | |
| 19:52:21 | melwitt | your memory is excellent | |
| 19:53:11 | openstackgerrit | Moshe Levi proposed openstack/nova master: hardware offload support for openvswitch https://review.openstack.org/398265 | |
| 19:53:31 | mriedem | all that clean livin' | |
| 19:55:42 | melwitt | dansmith: so the default TTL is 10 minutes but they could configure it for a lot longer | |
| 19:55:56 | dansmith | melwitt: yeah | |
| 19:56:59 | dansmith | melwitt: I dunno, I think it's fine to say "db is the way forward, memcache is deprecated, if you need, make sure you've configured for db and given enough time for things to expire before rolling | |
| 19:57:17 | melwitt | I dunno how big of a deal it would be. if we don't try to fall back on consoleauth, then at worst they get a InvalidToken and have to re-get a new access url and do again | |
| 19:57:34 | dansmith | right | |
| 19:58:01 | dansmith | there's some config thing now to choose whether we use memcache for that or not, right? | |
| 19:58:44 | melwitt | dansmith: not yet. there was described in the spec, but I think it wasn't added bc instance uuid presence could be used to detect old vs new (that's my speculation, I don't see it documented why they didn't go ahead with the config option) | |
| 19:58:53 | dansmith | oh I see | |
| 19:59:06 | dansmith | but | |
| 19:59:16 | dansmith | and so the code to always do it in the DB is unmerged? | |
| 19:59:45 | melwitt | I think the original design didn't have instance uuid (that was only added bc cells v2 + one proxy at top) and so they had the idea of the config option | |
| 19:59:56 | melwitt | yeah, nothing is happening in the DB yet | |
| 20:00:01 | dansmith | ah, okay I see | |
| 20:00:15 | dansmith | so | |
| 20:00:24 | dansmith | how about: | |
| 20:00:28 | dansmith | 1. Default to only db | |
| 20:00:36 | dansmith | 2. Workaround to check memcache first, fall back to db | |
| 20:00:49 | dansmith | 3. Expect that going forward for Queens it'll be db only | |
| 20:01:05 | dansmith | and a reno to say if you really really care about not resetting consoles, do the workaorund | |
| 20:01:15 | dansmith | I think we discussed forcing the console reset at some point | |
| 20:02:11 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: update live migration to use v3 cinder api https://review.openstack.org/463987 | |
| 20:02:45 | melwitt | dansmith: you mean add the config option for that right? where the choices are "DB only" and "consoleauth + DB" | |
| 20:03:00 | dansmith | melwitt: well, make it a workarounds config | |
| 20:03:08 | melwitt | oh, workarounds config. okay | |
| 20:03:11 | melwitt | I forgot about those | |
| 20:03:21 | dansmith | melwitt: call it [workarounds]/my_users_are_very_delicate_and_need_handholding=True | |
| 20:03:33 | melwitt | :D sure | |
| 20:03:52 | dansmith | I think that messages it as "this is a transitional helper thingy" | |
| 20:04:01 | melwitt | yeah, makes sense | |
| 20:04:10 | dansmith | could default it to true I guess, but I would say default to false and note it in the reno | |
| 20:04:19 | dansmith | because I expect very few people will need it | |
| 20:05:10 | openstackgerrit | Moshe Levi proposed openstack/os-vif master: unplug_vf_passthrough: don't try to delete representor netdev https://review.openstack.org/478820 | |
| 20:05:34 | openstackgerrit | Moshe Levi proposed openstack/os-vif master: fix read the representor phys_port_name https://review.openstack.org/480213 | |
| 20:38:18 | dims | mriedem : jaypipes : do you remember "why" there was pushback on the ConfKeyManager being in castellan? can't seem to find the email thread mentioned in the spec | |
| 20:38:58 | mriedem | ConfKeyManager? | |
| 20:39:26 | dims | mriedem : https://github.com/openstack/nova/search?utf8=%E2%9C%93&q=ConfKeyManager&type= | |
| 20:40:11 | dims | mriedem : was reading "Move ConfKeyManager elsewhere" section in https://specs.openstack.org/openstack/nova-specs/specs/newton/implemented/use-castellan-key-manager.html | |
| 20:40:54 | mriedem | i think the spec was updated here https://review.openstack.org/#/c/338518/ | |
| 20:41:37 | dims | yep, reading the updated text | |
| 20:41:38 | mriedem | i don't think it was in the ML | |
| 20:41:49 | mriedem | it was discussion in a patch or irc between kfarr, sdague and myself | |
| 20:42:03 | dims | mriedem : "Might be good to add a link to a ML discussion about the expressed concern?" was a note jay added to the review | |
| 20:42:10 | mriedem | yeah, it wasn't in the ML | |