| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-04-24 | |||
| 22:34:35 | aspiers | I guess 1615 is kinda early to start ;-) | |
| 22:34:54 | efried | it'll be 2315 in your time zone | |
| 22:35:00 | aspiers | true | |
| 22:35:01 | efried | pubs already closed | |
| 22:35:13 | efried | (do they still close at 11? It was so when I was growing up) | |
| 22:35:21 | aspiers | It varies | |
| 22:35:26 | aspiers | You used to live here? | |
| 22:35:49 | aspiers | or just remember the tourist pub run | |
| 22:36:35 | efried | I used to live in Buckinghamshire | |
| 22:36:40 | aspiers | no way | |
| 22:36:46 | aspiers | for how long? | |
| 22:36:58 | efried | 1984-7 and 89-94 | |
| 22:37:11 | aspiers | that's a decent length of time | |
| 22:37:19 | efried | Most of my formative years, I would say. | |
| 22:37:21 | aspiers | whereabouts? | |
| 22:37:30 | efried | Gerrard's Cross and Beaconsfield, respectively. | |
| 22:37:46 | aspiers | funny old world | |
| 22:37:50 | aspiers | where are you now? | |
| 22:37:52 | efried | you know Bucks? | |
| 22:37:58 | efried | Central Texas, Austin area | |
| 22:38:07 | aspiers | not really, but I know of both those places | |
| 22:38:29 | aspiers | most notably, Dominic Grieve represents the latter in Parliament IIRC | |
| 22:38:44 | aspiers | that's quite a change | |
| 22:38:49 | efried | heh, never paid any attention to the politics. Still don't. | |
| 22:38:52 | aspiers | I like Austin, don't know the surrounding area | |
| 22:38:58 | efried | I used to like Austin | |
| 22:39:02 | aspiers | haha | |
| 22:39:07 | efried | Now it's overrun with hipster douchebags | |
| 22:39:20 | efried | and getting too snobby for its britches | |
| 22:39:30 | aspiers | :) | |
| 22:39:32 | aspiers | happens to all the best places | |
| 22:41:20 | efried | if you like it they will come | |
| 22:41:40 | efried | case in point, the Trocadero | |
| 22:41:57 | efried | Carnaby Street | |
| 22:42:03 | efried | Camden Town | |
| 22:42:23 | aspiers | :) | |
| 22:43:38 | aspiers | it's a bit like the Ballmer Peak | |
| 22:44:10 | aspiers | places are shit for ages, then suddenly get cool, then everyone realises and floods the trendy cool place and ruins it again | |
| 22:45:06 | efried | 1630-beer: Disabled compute service request filter (mriedem) | |
| 22:45:06 | efried | 1615-1630: Compute capabilities traits placement request filter (mriedem) | |
| 22:45:06 | efried | Friday | |
| 22:45:06 | efried | Thursday 1330-1415: Let's plan the next steps of the bandwidth support feature (gibi) | |
| 22:45:06 | efried | aspiers, mriedem, gibi: I swapped both the traits filter topics with the bandwidth continuation. So now we have | |
| 22:45:14 | efried | um, jaypipes, did you want to be around for the latter? | |
| 22:45:21 | aspiers | awesome thanks | |
| 22:49:32 | mriedem | efried: ok, although i didn't think the bw spec was that controversial | |
| 22:49:35 | mriedem | just needs to be updated | |
| 22:50:56 | efried | mriedem: Sorry, lack of context, I just realized that aspiers responded to me and not the list - he wanted to be around for the traits filter topics (which also increases our chances of convincing him to own them :) but is unavailable Thursday. | |
| 22:51:39 | efried | mriedem, gibi: And definitely if we don't need as much (or any) time to talk about the bandwidth stuff, let me know so I can shuffle some more stuff in. | |
| 22:52:24 | efried | mriedem: I got the impression that gibi wanted to have a kind of live design session for some of the issues. | |
| 22:52:35 | mriedem | i'll let gibi answer that, i think the biggest hangups in the bw migration spec was (1) do we use a microversion (sounds like yeah probably) and (2) upgrade issues which i think he understands now | |
| 22:55:21 | efried | neat, I'll fup with gibi | |
| 22:56:55 | mriedem | dansmith: re https://review.opendev.org/#/c/649197/2/nova/compute/rpcapi.py@381 i think that's as easy as just resetting the compute.rpcapi._ROUTER global on ComputeManager.reset() or am i missing some nasty detail? | |
| 22:57:11 | dansmith | mriedem: nope, I think that'll do it | |
| 22:57:52 | mriedem | what do you think about just throwing reset_globals() from https://review.opendev.org/#/c/649197/2/nova/tests/unit/compute/test_rpcapi.py@37 into the rpcapi module itself? then i can hit it from multiple locations | |
| 23:01:14 | dansmith | sec, phone | |
| 23:03:30 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Always pass HostAPI to get_availability_zones https://review.opendev.org/655583 | |
| 23:04:45 | mriedem | heh "Problme: Even though splited, still long and boring. Need fully review." | |
| 23:04:54 | mriedem | L224 in the ptg etherpad | |
| 23:06:14 | mriedem | efried: L223 is likely not something we really need for the ptg https://etherpad.openstack.org/p/nova-ptg-train it's just a request for review | |
| 23:06:25 | mriedem | which i've done once before, just - it's a lot of code and i forgot about it | |
| 23:07:05 | eandersson | What is this fun error | |
| 23:07:06 | eandersson | > ValueError: unmatched '{' in format | |
| 23:07:10 | eandersson | When running cell mapping | |
| 23:07:26 | efried | mriedem: ack | |
| 23:07:27 | mriedem | are you using templates for the transport_url or db connection? | |
| 23:07:40 | eandersson | no template | |
| 23:08:17 | eandersson | my best guess is a special char in the password is causing some fun bug | |
| 23:08:21 | mriedem | do you have a stacktrace you can paste? | |
| 23:08:24 | efried | mriedem: a little background, I've scheduled that there because of the stx association, and dtroyer will still be in the room. If we don't need to discuss it, we can take a coffee break at that time :) | |
| 23:08:25 | mriedem | maybe | |
| 23:08:34 | mriedem | i suppose you don't want to paste the trace b/c of said password... | |
| 23:09:27 | mriedem | efried: ok, whatever, i think it's just a request for review | |
| 23:09:35 | efried | ack | |
| 23:09:45 | mriedem | melwitt and i have both reviewed it before | |
| 23:10:02 | efried | well cool, if y'all merge it before the PTG I'll definitely strike it :) | |
| 23:10:12 | mriedem | it's like 800 LOC though | |
| 23:10:15 | mriedem | heh | |
| 23:10:22 | efried | and in merge conflict | |
| 23:10:30 | mriedem | if it were a blueprint (which it arguably could be) then it could sit in a runway | |
| 23:10:35 | mriedem | the bottom patch isn't | |
| 23:10:51 | ccstone | (I'm also working with eandersson) He already gave away the special character which is { :) | |
| 23:10:55 | mriedem | then the reviews would floweth like fine wine | |
| 23:11:04 | efried | sure they would | |
| 23:11:36 | efried | lemme see what a rebase would look like. | |
| 23:12:02 | mriedem | ccstone: which release of rocky? | |
| 23:12:05 | mriedem | latest patch? | |
| 23:12:26 | mriedem | 18.2.0? | |
| 23:12:42 | mriedem | i was thinking maybe https://review.opendev.org/#/c/633086/ but that might not hit you if you're not using templated urls | |
| 23:14:16 | mriedem | ccstone: can you provide a bug or trace without the password in it? i don't know which command is being referenced | |
| 23:14:27 | ccstone | Yeah, we're on 18.2.0, looks like we do have that patch. | |
| 23:15:57 | eandersson | http://paste.openstack.org/show/749729/ | |
| 23:16:09 | openstackgerrit | Eric Fried proposed openstack/nova master: Clean up orphan instances https://review.opendev.org/627765 | |
| 23:16:09 | openstackgerrit | Eric Fried proposed openstack/nova master: Clean up orphan instances virt driver https://review.opendev.org/648912 | |
| 23:16:17 | efried | mriedem, melwitt: fyi, rebase was easy ^ | |
| 23:18:08 | mriedem | eandersson: ccstone: please report a bug, i want to say we talked about special characters in passwords when adding that code and there were some assertions made, but i can't remember now, would definitely want dansmith to look | |
| 23:18:10 | mriedem | but he's busy | |
| 23:20:31 | dansmith | mriedem: sure, that seems okay I think (re reset_globals) | |
| 23:21:12 | mriedem | eandersson: ccstone: looks like we'll catch and trace that ValueError but things don't fail right? we return the URL as-is | |
| 23:21:35 | dansmith | mriedem: eandersson ccstone: passwords should be url-encoded | |