| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-08 | |||
| 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 | mriedem | what are they called now? | |
| 18:36:32 | dansmith | but they did around the seagulls timeframe | |
| 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: Sort CellMappingList.get_all() for safety https://review.openstack.org/443174 | |
| 19:20:49 | openstackgerrit | Dan Smith proposed openstack/nova master: Address minor feedback in superconductor targeting https://review.openstack.org/443175 | |
| 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:50 | openstackgerrit | Dan Smith proposed openstack/nova master: Add get_by_instance_uuids() to InstanceMappingList https://review.openstack.org/443292 | |
| 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: Sort CellMappingList.get_all() for safety https://review.openstack.org/443174 | |
| 20:04:09 | openstackgerrit | Dan Smith proposed openstack/nova master: Add get_by_instance_uuids() to InstanceMappingList https://review.openstack.org/443292 | |
| 20:04:10 | openstackgerrit | Dan Smith proposed openstack/nova master: Address minor feedback in superconductor targeting https://review.openstack.org/443175 | |
| 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: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, | |
| 20:37:07 | dansmith | which might not be n-cpu | |
| 20:37:33 | dansmith | dangit | |
| 20:38:03 | mriedem | yeah service delete calls service_get_by_id | |
| 20:38:12 | mriedem | and we never test service delete | |
| 20:38:20 | mriedem | not in tempest anyway | |
| 20:38:41 | mriedem | could have a simple functional test with multiple nova-compute services in 2 cells and you delete one in the 2nd cell | |
| 20:38:52 | mriedem | that's also going to be jacked... | |
| 20:39:08 | mriedem | b/c 2 nova-computes in 2 cells are going to have ids 1 and 2 | |
| 20:39:18 | mriedem | so you'll delete the first one found in the first cell | |
| 20:39:18 | mriedem | shit | |
| 20:39:28 | mriedem | which might not be the one you wanted to delete | |
| 20:39:47 | mriedem | i want to delete nova-compute 2 in cell 2, but it depends the service with id=2 in cell1 | |
| 20:40:54 | dansmith | yeah, this api is fairly retarded | |
| 20:42:17 | mriedem | wasn't there a similar one for getting a compute node by id | |
| 20:42:32 | mriedem | yeah compute_node_get | |
| 20:42:40 | dansmith | the api does that by name though right? | |
| 20:43:02 | mriedem | no | |
| 20:43:16 | mriedem | your comment even says "Unfortunately this API exposes database identifiers | |
| 20:43:16 | mriedem | # which means we really can't do something efficient here" | |
| 20:43:27 | mriedem | so the os-hypervisors uptime and show APIs pass in the compute node id | |
| 20:43:40 | dansmith | mriedem: that comment is about service | |
| 20:43:43 | mriedem | so we'll be showing details and uptime info for the first match on that id in the first cell we hit | |
| 20:43:49 | mriedem | dansmith: it's in there twice | |
| 20:43:55 | mriedem | as i said, this is a hairy change :) | |
| 20:44:00 | mriedem | lots of moving parts | |
| 20:44:11 | dansmith | okay, I remember seeing name in os-hypervisors multiple times.. maybe there's a search or something? | |
| 20:44:21 | mriedem | https://developer.openstack.org/api-ref/compute/?expanded=show-hypervisor-details-detail#show-hypervisor-details | |
| 20:44:24 | mriedem | there is the search one | |
| 20:44:41 | dansmith | okay maybe that's what I'm thinking of | |
| 20:44:51 | mriedem | dansmith: that would be this one https://developer.openstack.org/api-ref/compute/?expanded=search-hypervisor-detail#search-hypervisor | |
| 20:45:12 | mriedem | yeah so service delete, and compute node show and uptime are going to be wrong | |
| 20:45:15 | mriedem | in multi-cell | |