| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-08 | |||
| 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 | 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 | |