| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-10-02 | |||
| 19:55:43 | mriedem | :P | |
| 19:55:44 | artom | dansmith, "sufficiently-endowed", seriously? | |
| 19:55:53 | artom | mriedem, dammit | |
| 19:55:53 | mriedem | now don't go removing it yet, i'm commenting | |
| 19:56:05 | dansmith | you two being the only two reviewers, I had to bait you good | |
| 19:56:56 | artom | Careful how you spell bait | |
| 19:57:44 | dansmith | you guys are too easy | |
| 19:57:56 | artom | It's more fun that way. | |
| 19:58:10 | dansmith | endowed with permission is a totally legit phrasing | |
| 19:59:06 | sean-k-mooney | dansmith: you have a lot of references in that spec. i added it to my list of things to revew | |
| 19:59:32 | dansmith | sean-k-mooney: I listed a bunch of them so that reviewers would be afraid to not approve it | |
| 20:00:15 | sean-k-mooney | it looks relitivly short for a spec but im to tired to review stuff a tthe momemnt so ill go through it tomorrow. | |
| 20:00:53 | sean-k-mooney | hehe i see you are going for the over load them with example of why this is need so you cant question it approch | |
| 20:01:18 | sean-k-mooney | it fairly effective | |
| 20:02:30 | artom | dansmith, I don't suppose setting up glance -> computes multicast is part of your MVP spec ;) | |
| 20:03:04 | artom | But it'd be a way to get both the ease of fire-and-forget RPC cast while not killing the glance server | |
| 20:03:11 | artom | Not that it's, you know, realistic in any way | |
| 20:03:46 | dansmith | artom: the right solution, IMHO, is bittorrent | |
| 20:03:52 | dansmith | which was proposed and implemented in RAX at one point | |
| 20:04:11 | dansmith | basically, glance becomes a torrent seeder and only has to dish out the image once | |
| 20:04:12 | artom | I genuinely thought you were kidding at first | |
| 20:04:25 | dansmith | or however many workers you have, that's how many seeds you have | |
| 20:04:44 | artom | But yeah, that's actually smart | |
| 20:04:53 | dansmith | imagine how good that would be for an edge site where you have four computes.. right now they all pull the image over the WAN... but that makes no sense, they should share once one of them has it | |
| 20:04:54 | dansmith | right | |
| 20:05:38 | dansmith | and if one got it yesterday and still has it, the peer compute can pull it from it instead of over the way | |
| 20:05:39 | dansmith | *wan | |
| 20:06:31 | dansmith | also imagine how in the future iteration where we need to report which computes have that image cached, if we could just ask the torrent magnet which computes are now seeds... :D | |
| 20:07:42 | artom | *stunned Pikachu face* | |
| 20:08:04 | openstackgerrit | melanie witt proposed openstack/nova master: Restore console proxy deployment info to cells v2 layout doc https://review.opendev.org/686271 | |
| 20:08:29 | sean-k-mooney | we talkabout using bittorrent like 2 years ago | |
| 20:08:42 | dansmith | we talked about it in 2012 | |
| 20:08:47 | melwitt | efried, mriedem: added ^ to https://etherpad.openstack.org/p/nova-train-release-todo as a possible thing for RC2 | |
| 20:08:49 | dansmith | but probably two years ago as well | |
| 20:08:49 | sean-k-mooney | was it that long | |
| 20:09:14 | sean-k-mooney | anyway im glad its now a thing | |
| 20:09:24 | sean-k-mooney | because ya for edge it toptally makes sense | |
| 20:10:17 | sean-k-mooney | its resilent, spreads the load across multiple peers and works over low bandwitd links | |
| 20:10:24 | dansmith | I'm not really proposing that in the short term, but it's always been something I wanted us to do | |
| 20:11:18 | sean-k-mooney | well part of why i like it is its also not us e.g. openstack comping up with our own solution for a proablem that has and existing soltion already | |
| 20:11:33 | dansmith | yup | |
| 20:14:05 | mriedem | more comments | |
| 20:14:50 | artom | dansmith, brainstorming here - but does oslo.concurrency give us a distributed semaphore? | |
| 20:15:07 | dansmith | artom: not without some external service | |
| 20:15:11 | artom | I'm thinking - cast to the computes, but have the grab the semaphore when downloading | |
| 20:15:51 | bnemec | Sounds like a use case for tooz. | |
| 20:15:58 | mriedem | i was just going to make an etcd joke | |
| 20:16:08 | bnemec | oslo.concurrency only works within a single machine. | |
| 20:16:24 | artom | That way the concurrency becomes configurable (just set the semaphore's... err, cardinality?) and you don't ddos the glance server | |
| 20:16:26 | dansmith | artom: yeah, it's just not really necessary when we can threadpool it from the conductor | |
| 20:16:49 | artom | dansmith, for an RPC call though, right? | |
| 20:17:07 | dansmith | artom: yup | |
| 20:17:23 | artom | Yeah, I may be overthinking this | |
| 20:17:26 | mriedem | melwitt: ack i'm cool with that, docs only and regression in train | |
| 20:20:26 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Noop CantStartEngineError in targets_cell if API DB not configured https://review.opendev.org/686276 | |
| 20:26:02 | openstackgerrit | Merged openstack/nova master: Use os-brick locking for volume attach and detach https://review.opendev.org/614190 | |
| 20:26:02 | dansmith | mriedem: yawna read my response to the parallelism thing and if you're cool with it I can squash those todos in my next rev: https://review.opendev.org/#/c/686232/4/specs/ussuri/approved/image-precache-support.rst@123 | |
| 20:26:12 | melwitt | mriedem: coolio | |
| 20:28:36 | mriedem | dansmith: replied | |
| 20:29:04 | dansmith | mriedem: thanks | |
| 20:30:52 | openstackgerrit | melanie witt proposed openstack/nova master: Restore console proxy deployment info to cells v2 layout doc https://review.opendev.org/686271 | |
| 20:35:06 | openstackgerrit | Dan Smith proposed openstack/nova-specs master: Add image-precache-support spec https://review.opendev.org/686232 | |
| 20:36:50 | efried | melwitt, mriedem: +2, good catch | |
| 20:38:58 | efried | aspiers: how's that SEV fixup looking? Would be nice to produce RC2 tomorrow (and not need RC3). Your fix is the last one I know of that's not well on the way to merging. | |
| 20:42:12 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Handle get_host_availability_zone error during reschedule https://review.opendev.org/686292 | |
| 20:44:11 | mriedem | efried: deadline for RCs is oct 10 right? | |
| 20:44:19 | efried | yes | |
| 20:44:27 | mriedem | i mean, we want time for rc3 for sev issue #12 but sure | |
| 20:44:34 | efried | smy point | |
| 20:45:36 | efried | Oh, I misread, you're actually picking on SEV. I don't think we've had especially more issues for SEV than for other blueprints. Have there been more than one for any? | |
| 20:46:08 | efried | Seems like one for SEV, one for VPMEM, one for console tokens, one (?) for cpu-resources... | |
| 20:46:30 | efried | which is fine, exactly the reason FF happens before the release. | |
| 20:51:18 | sean-k-mooney | efried: i dont think we had any for cpu-resouce but we had/have one for numa migration | |
| 20:51:50 | sean-k-mooney | artom: has your fix for the RT cacheing issue landed? | |
| 20:51:58 | mriedem | yes | |
| 20:52:06 | sean-k-mooney | ok cool | |
| 20:52:12 | mriedem | console proxy was a docs thing | |
| 20:52:50 | mriedem | my point is sev landed late with a ton of churn over 2 releases (speaking as an outside observer to that series), so i wouldn't be surprised if there aren't more hairy issues | |
| 20:52:59 | mriedem | seems like every issue in that series is very hard to resolve | |
| 20:53:34 | mriedem | i also won't be surprised when there are issues with PCPU - we haven't seen issues because we're not using it anywhere yet | |
| 20:53:50 | mriedem | but in 18 months...watch out! | |
| 20:54:24 | sean-k-mooney | well there kind of is one but it was an issue we defeind as out of scope | |
| 20:54:31 | sean-k-mooney | it related to https://review.opendev.org/#/c/686079/ | |
| 20:55:52 | sean-k-mooney | you can only have flaoting instnace and pinned instance on the same host and not have mempage issue if you enable the numa aware memory tracking using hw:mem_page_size=<any valid value> | |
| 20:56:01 | sean-k-mooney | but we know it would not address that | |
| 20:58:55 | efried | mriedem: the console token thing we just merged? That wasn't a docs issue. | |
| 20:59:12 | efried | it may not have been related to an actual blueprint either though, not sure. | |
| 21:00:02 | sean-k-mooney | the console issue was passing the path argument in case it was not required right? | |
| 21:00:10 | dansmith | I think he means the cells console proxy doc thing | |
| 21:00:16 | efried | no | |
| 21:00:36 | efried | https://review.opendev.org/#/c/686066/ Reduce scope of 'path' query parameter to noVNC consoles | |
| 21:00:50 | dansmith | this: https://review.opendev.org/#/c/686271/ | |
| 21:01:14 | efried | no | |
| 21:01:16 | efried | -----^ | |
| 21:01:25 | dansmith | efried: I mean he==mriedem, not you | |
| 21:01:30 | efried | right | |
| 21:01:43 | efried | and confirmed it wasn't a blueprint, it was a bug | |
| 21:01:54 | efried | anywho | |
| 21:02:26 | sean-k-mooney | im going to call it a night o/ | |
| 21:05:49 | mriedem | efried: oh i wasn't thinking of the serial console thing | |