| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-08 | |||
| 18:17:24 | sfinucan | However, I'm not sure what the plan is for weighers and what the migration path is when we move to resource providers, so I'd definitely like jaypipes' input | |
| 18:17:42 | sfinucan | For now though, it's late and the Lego Batman movie isn't going to see itself :) | |
| 18:18:16 | sean-k-mooney | sfinucan: nice i must go see that soon. | |
| 18:18:28 | sean-k-mooney | sfinucan: let me know if its any good :) | |
| 18:21:10 | bauzas | mriedem: I +W'd https://review.openstack.org/#/c/442723/1 but I think we could possibly have follow-ups | |
| 18:21:14 | bauzas | see commentqs | |
| 18:21:25 | openstackgerrit | Dan Peschman proposed openstack/nova master: No API cell up-call to delete consoleauth tokens https://review.openstack.org/415922 | |
| 18:21:58 | openstackgerrit | Dan Smith proposed openstack/nova master: Add functools.wraps to wrapper https://review.openstack.org/443175 | |
| 18:22:08 | mriedem | dansmith: no ice_t? | |
| 18:22:28 | dansmith | mriedem: if I can't have my fun you can't either | |
| 18:22:45 | mriedem | works for me | |
| 18:22:52 | bauzas | dansmith: wow, chilling at the series https://review.openstack.org/#/c/441461/8 | |
| 18:23:04 | dansmith | the first time it came up, I was going to rename it to gangsta_rapper | |
| 18:23:06 | sean-k-mooney | inc0: what is the actully error that ye are seeing with libvirt 2.5? | |
| 18:23:20 | dansmith | bauzas: all but one tempest test is passing with multiple cells with that series :) | |
| 18:23:33 | bauzas | \o/ | |
| 18:23:37 | bauzas | heh for https://review.openstack.org/#/c/443175/3/nova/conductor/manager.py | |
| 18:23:41 | dansmith | and I had that working before, but broke it in cleanup | |
| 18:23:46 | bauzas | that remembers me some comment I made :) | |
| 18:25:09 | bauzas | dansmith: any reason why you added https://review.openstack.org/#/c/443175/3/nova/conductor/manager.py as a top patch and not just modified https://review.openstack.org/#/c/438022/15 ? | |
| 18:25:21 | sean-k-mooney | inc0: oh appolgies my libvert is 1.3.1 for some reason i guess i dont have the cloud archive enabled | |
| 18:25:22 | dansmith | bauzas: yes, see the comments | |
| 18:25:26 | bauzas | oh ok | |
| 18:25:30 | openstackgerrit | Andreas Jaeger proposed openstack/nova master: Replace obsolete vanity openstack.org URLs https://review.openstack.org/443266 | |
| 18:25:32 | dansmith | bauzas: I would eat all infra re-pushing for that nit | |
| 18:26:20 | bauzas | dansmith: um ok | |
| 18:26:23 | mriedem | dansmith: comments inline https://review.openstack.org/#/c/438022/15 | |
| 18:29:36 | dansmith | mriedem: replies inline.. I'd tack the log fix into the rapper fix if it were me, but your call | |
| 18:30:09 | dansmith | if you want it fixed, I'll just squash that nit patch into this one | |
| 18:30:20 | dansmith | fixed pre-merge I mean | |
| 18:31:13 | mriedem | dansmith: question about the decorator on build_instances - that's not just cellsv1, it's also reschedules from the compute manager - but at that point we're already targeted right? | |
| 18:31:40 | dansmith | mriedem: right, and so we can't actually look up there, so the target would have to be graceful and fail, yet it'd be ugly | |
| 18:31:49 | dansmith | mriedem: that goes back to the cell conductor instead of super | |
| 18:32:13 | mriedem | oh yeah, need that diagram | |
| 18:32:44 | mriedem | so i'm ok with fixing the log thing in the rapper patch, but can you also put a comment in that patch for both the build_instances thing and mdbooth's question that had a long explanation? | |
| 18:32:55 | dansmith | yep | |
| 18:33:00 | mriedem | word homey | |
| 18:33:31 | mriedem | also, | |
| 18:33:38 | mriedem | no one mentioned your flock of seagulls reference | |
| 18:33:41 | mriedem | pick a genre | |
| 18:33:52 | dansmith | mriedem: I know, I thought that was amusing :) | |
| 18:34:00 | mriedem | i got it though | |
| 18:34:59 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #4: full flavor https://review.openstack.org/391288 | |
| 18:35:14 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: console https://review.openstack.org/409402 | |
| 18:35:50 | openstackgerrit | Dan Smith proposed openstack/nova master: Address minor feedback in superconductor targeting https://review.openstack.org/443175 | |
| 18:36:17 | dansmith | mriedem: at least I was decade consistent | |
| 18:36:23 | dansmith | afaik, nobody calls them "rappers" anymore | |
| 18:36:32 | dansmith | but they did around the seagulls timeframe | |
| 18:36:32 | mriedem | what are they called now? | |
| 18:36:54 | dansmith | um, I dunno, "Hip Hop Artists" ? | |
| 18:37:12 | mriedem | yeah maybe | |
| 18:37:14 | mriedem | gdi kids | |
| 18:37:16 | mriedem | *gd | |
| 18:37:19 | dansmith | I feel like if someone said "who's your favorite rapper" nowadays you'd get laughed at, but in the 80s that was a serious question | |
| 18:37:34 | mriedem | "umm, vanilla ice, duh!" | |
| 18:37:44 | dansmith | I'm sure that was the right answer in heartland america :) | |
| 18:38:00 | mriedem | oh psh | |
| 18:38:25 | mriedem | odb | |
| 18:38:27 | mriedem | snoop?! | |
| 18:38:54 | dansmith | snoop was on the trailing end of calling them rappers I think | |
| 18:38:57 | dansmith | I'm thinking more like "sir mix a lot" or "ice cube" | |
| 18:39:04 | mriedem | oh yeah, too $short | |
| 18:39:06 | dansmith | definitely ice-t, he was an early mover | |
| 18:39:07 | mriedem | oops | |
| 18:39:09 | mriedem | Too $hort | |
| 18:39:22 | mriedem | filthy, filthy lyrics | |
| 18:40:20 | JayF | I think I found your favorite rapper: http://www.famousbirthdays.com/people/nova.html | |
| 18:41:17 | mriedem | heh | |
| 18:41:29 | artom | The shame of being the *fifth* most poplar person named Nova | |
| 18:41:53 | JayF | lol | |
| 18:42:12 | artom | Being sixth and last isn't too bad, it means nobody knows or cares about you | |
| 18:42:20 | edleafe | This is more mriedem's style: https://www.youtube.com/watch?v=3yerCiByca4 | |
| 18:42:36 | artom | But second to last... Someone cares just enough to give you hope, and then it's all dashed when you find out it's meaningless | |
| 18:44:42 | mriedem | edleafe: need i remind you https://www.youtube.com/watch?v=w5Fgp-KihIA | |
| 18:45:11 | edleafe | mriedem: ARRRGGGH! Not twice in two days! | |
| 18:45:17 | mriedem | my wife accused me of being angry and dissatisfied with everything all the time last night, and i said, oh yeah, would an angry guy have this song within an arms reach?! https://www.youtube.com/watch?v=w5Fgp-KihIA | |
| 18:45:47 | dansmith | heh | |
| 18:46:34 | edleafe | mriedem: of course he would - to torture his victims with! | |
| 18:50:39 | mriedem | alright back to work | |
| 19:20:49 | openstackgerrit | Dan Smith proposed openstack/nova master: Address minor feedback in superconductor targeting https://review.openstack.org/443175 | |
| 19:20:49 | openstackgerrit | Dan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety https://review.openstack.org/443174 | |
| 19:20:50 | openstackgerrit | Dan Smith proposed openstack/nova master: Add get_by_instance_uuids() to InstanceMappingList https://review.openstack.org/443292 | |
| 19:20:50 | openstackgerrit | Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736 | |
| 19:20:51 | openstackgerrit | Dan Smith proposed openstack/nova master: Make server_groups determine deleted-ness from InstanceMappingList https://review.openstack.org/443293 | |
| 20:04:09 | openstackgerrit | Dan Smith proposed openstack/nova master: Add get_by_instance_uuids() to InstanceMappingList https://review.openstack.org/443292 | |
| 20:04:09 | openstackgerrit | Dan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety https://review.openstack.org/443174 | |
| 20:04:10 | openstackgerrit | Dan Smith proposed openstack/nova master: Make server_groups determine deleted-ness from InstanceMappingList https://review.openstack.org/443293 | |
| 20:04:10 | openstackgerrit | Dan Smith proposed openstack/nova master: Address minor feedback in superconductor targeting https://review.openstack.org/443175 | |
| 20:04:11 | openstackgerrit | Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736 | |
| 20:24:22 | jaypipes | mriedem: sorry, been out for a few hours dealing with a couple things. yes, ComputeNode.stats is where that stuff lives, IIRC. | |
| 20:26:38 | mriedem | dansmith: this one is a bit hairy https://review.openstack.org/#/c/442162/ | |
| 20:26:42 | mriedem | but pretty sure _find_service is busted | |
| 20:27:08 | mriedem | jaypipes: np. want to get this in? https://review.openstack.org/#/c/415885 | |
| 20:30:54 | dansmith | mriedem: there's a reason I did it that way, give me a sec to remember | |
| 20:30:58 | dansmith | mriedem: but I commented on the others | |
| 20:34:03 | jaypipes | mriedem: sure, gimme a sec | |
| 20:35:19 | dansmith | mriedem: I'm not sure, it works locally and in the gate in the not-first cell looking for n-cpu | |
| 20:35:29 | dansmith | but I'm having trouble fetching the context from my brain | |
| 20:36:54 | dansmith | oh, no, wait | |
| 20:36:55 | dansmith | I bet that is only in the path of service delete, | |