| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-27 | |||
| 17:32:30 | leakypipes | jianghuaw: np | |
| 17:32:38 | leakypipes | jianghuaw: now go to bed! :) | |
| 17:33:04 | jianghuaw | :) | |
| 17:45:31 | openstackgerrit | Merged openstack/nova master: Include /resource_providers/uuid/allocations link https://review.openstack.org/499826 | |
| 17:50:43 | openstackgerrit | Eric Fried proposed openstack/nova master: Granularize resources_from_{flavor|request_spec} https://review.openstack.org/515223 | |
| 18:35:30 | mriedem | figleaf: superdan: leakypipes: cdent: how would you guys all love to have a thrilling hangout to make sure we're on the same page about https://review.openstack.org/#/c/510159/ ? | |
| 18:35:36 | mriedem | assuming cdent not so much given friday night fun time | |
| 18:35:54 | superdan | you mean now? | |
| 18:36:05 | cdent | i’m around actually, but haven’t been following along today | |
| 18:36:20 | mriedem | superdan: i'm game, i don't think this would be a long call | |
| 18:36:22 | mriedem | else it waits | |
| 18:37:40 | mriedem | i think in at least one retrospective we said, 'when in doubt, hangout' | |
| 18:37:55 | mriedem | and, 'if it's brown, flush it down' | |
| 18:38:12 | figleaf | mriedem: I can do it in the next 15 min; then I have to run out for a bit | |
| 18:39:23 | mriedem | figleaf: as in you're free at 2pm? | |
| 18:39:38 | mriedem | or out at 2pm? | |
| 18:39:42 | figleaf | mriedem: no, I'm free now. Gotta be somewhere at 2 | |
| 18:40:44 | mriedem | https://hangouts.google.com/call/M3gJOjmdi0q0Ayha5KkYAAEE | |
| 18:41:49 | superdan | cdent: you're breathing.. heavily | |
| 18:48:50 | leakypipes | mriedem: cool with me. | |
| 18:55:53 | figleaf | superdan: you have that effect on people | |
| 18:56:11 | superdan | I'm going to pretend you didn't say that | |
| 18:57:05 | mriedem | figleaf: ok left a summary in the patch | |
| 18:57:08 | mriedem | thanks | |
| 18:57:08 | mriedem | https://review.openstack.org/#/c/510159/ | |
| 19:34:54 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138 | |
| 19:34:55 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | |
| 19:34:55 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | |
| 19:34:56 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | |
| 19:37:05 | openstackgerrit | Eric Fried proposed openstack/nova master: ResourceRequest.to_querystring() https://review.openstack.org/515811 | |
| 19:37:58 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | |
| 19:37:58 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | |
| 19:45:16 | fried_rice | leakypipes Would you mind skimming the next couple patches in the series? I can dig having a RequestGroup class, but I'd like to keep the encompassing thing as a class (rather than a plain list of RequestGroup) so it do things like merge all the resources and produce a querystring. | |
| 19:45:36 | fried_rice | s/so it do/so it can do/ | |
| 19:47:37 | leakypipes | fried_rice: yup, I definitely will go through the remainder of the series. Probably won't be today, though... | |
| 19:48:10 | leakypipes | fried_rice: you're high in my queue, though. | |
| 19:48:37 | fried_rice | leakypipes Cool man. | |
| 20:04:26 | bauwser | leakypipes: around ? | |
| 20:04:34 | leakypipes | bauwser: yup! | |
| 20:04:44 | bauwser | leakypipes: not sure you saw the discussion we had with jianghuaw | |
| 20:05:06 | bauwser | leakypipes: about https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L807 | |
| 20:05:23 | leakypipes | bauwser: no, I didn't | |
| 20:05:49 | bauwser | leakypipes: the problem is that if the compute node resources are not changing, then we don't call the inventory | |
| 20:06:18 | leakypipes | bauwser: yes? | |
| 20:06:23 | bauwser | leakypipes: which is a problem with the VGPU resources | |
| 20:06:27 | leakypipes | bauwser: why? | |
| 20:06:30 | bauwser | since they're not in the compute node | |
| 20:07:05 | leakypipes | bauwser: ah, yeah... | |
| 20:07:17 | bauwser | leakypipes: I thought about that | |
| 20:07:30 | leakypipes | bauwser: that line can be moved to after the call to get_inventory() eventually | |
| 20:07:36 | bauwser | leakypipes: that's my point :p | |
| 20:07:43 | leakypipes | yup | |
| 20:07:52 | bauwser | I was asking you if you were okay with that :) | |
| 20:08:38 | bauwser | leakypipes: we could also persist the inventory | |
| 20:08:58 | bauwser | leakypipes: so passing it to the _resource_change method too | |
| 20:09:18 | bauwser | and so, not setting the inventory if that's not changing too :) | |
| 20:09:26 | bauwser | anyway, lemme provide the chnage | |
| 20:10:11 | leakypipes | bauwser: that's kind of what my nested resource provider series is doing. | |
| 20:12:40 | leakypipes | bauwser: https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L609 | |
| 20:12:56 | leakypipes | bauwser: basically, that needs to come before the _resource_changed() thing in the RT. | |
| 20:13:09 | leakypipes | bauwser: or we just get rid of the _resource_changed() thing entirely... | |
| 20:13:17 | bauwser | mmmm | |
| 20:14:53 | bauwser | leakypipes: what I honestly need is to look at your series :p | |
| 20:26:42 | fried_rice | leakypipes Sure makes it trickier to test... | |
| 20:27:18 | leakypipes | fried_rice: makes what trickier to test? | |
| 20:28:05 | fried_rice | leakypipes The resource groups being objects instead of a big dict I can just build up and compare. It's not a big deal. Just prolly going to write some test-only methods on the objects. | |
| 20:28:18 | leakypipes | fried_rice: ack. | |
| 20:28:58 | leakypipes | fried_rice: well, I prefer if you don't add methods to the objects themselves. rather, add a comparator method to the test class. | |
| 20:29:11 | leakypipes | assertRequestGroupsSame() or something like that... | |
| 20:29:27 | fried_rice | leakypipes Yuh. | |
| 20:31:26 | fried_rice | leakypipes But then I have to build up RequestGroups for the 'expected' side. | |
| 20:31:37 | leakypipes | fried_rice: correct. | |
| 20:31:45 | leakypipes | fried_rice: which is explicit. and good. | |
| 20:32:04 | fried_rice | leakypipes But reduces the test coverage, cause I'm using Thing to test Thing. | |
| 20:32:20 | fried_rice | leakypipes Again, not a big deal, can do. | |
| 20:32:29 | leakypipes | fried_rice: if Thing is just a plain-old-data object, that's good. | |
| 20:36:49 | openstackgerrit | Merged openstack/nova master: Update placement api-ref: allocations link in 1.11 https://review.openstack.org/515748 | |
| 20:42:36 | openstackgerrit | Michael Still proposed openstack/nova master: Move infiniband vif plugging to privsep. https://review.openstack.org/515197 | |
| 20:42:37 | openstackgerrit | Michael Still proposed openstack/nova master: Move plumgrid vif plugging to privsep. https://review.openstack.org/515336 | |
| 20:42:37 | openstackgerrit | Michael Still proposed openstack/nova master: Move midonet vif plugging to privsep. https://review.openstack.org/515198 | |
| 20:42:38 | openstackgerrit | Michael Still proposed openstack/nova master: Move control vif plugging to privsep. https://review.openstack.org/515916 | |
| 20:44:24 | openstackgerrit | Michael Still proposed openstack/nova master: Move contrail vif plugging to privsep. https://review.openstack.org/515916 | |
| 20:44:24 | openstackgerrit | Michael Still proposed openstack/nova master: Move plumgrid vif plugging to privsep. https://review.openstack.org/515336 | |
| 20:53:02 | openstackgerrit | Merged openstack/nova master: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051 | |
| 20:55:14 | mriedem | fried_rice: comments in 16 and 18 https://review.openstack.org/#/c/492247/ | |
| 20:55:21 | fried_rice | ... | |
| 20:56:43 | fried_rice | mriedem Roger wilco, will update. Thanks. | |
| 20:56:58 | mriedem | |_o_| | |
| 20:57:10 | mriedem | 3 points if you do | |
| 20:57:22 | mriedem | superdan knows what that 3 points is for | |
| 20:57:29 | cdent | been a lot of football today | |
| 20:57:34 | mriedem | heh | |
| 20:57:46 | mriedem | that reminds me | |
| 20:57:51 | mriedem | vikings in london on sunday | |
| 20:58:06 | mriedem | that means i get football at 8:30, noon, 3:30 and 7:30 | |
| 20:58:30 | cdent | apparently when the nfl go to london _loads_ of people go nutbar and dress up for the game | |
| 20:58:32 | superdan | obviously three points for a three pointer shot from midfield | |
| 20:58:42 | cdent | which when I heard about it, completely surprised me | |
| 20:58:48 | superdan | swish | |
| 20:58:51 | superdan | goooooooal | |