| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-27 | |||
| 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 | https://review.openstack.org/#/c/510159/ | |
| 18:57:08 | mriedem | thanks | |
| 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: allow filter providers in tree https://review.openstack.org/377215 | |
| 19:34:55 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | |
| 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: adds REST API for nested providers https://review.openstack.org/384807 | |
| 19:37:58 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | |
| 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 midonet vif plugging to privsep. https://review.openstack.org/515198 | |
| 20:42:37 | openstackgerrit | Michael Still proposed openstack/nova master: Move plumgrid vif plugging to privsep. https://review.openstack.org/515336 | |
| 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 plumgrid vif plugging to privsep. https://review.openstack.org/515336 | |
| 20:44:24 | openstackgerrit | Michael Still proposed openstack/nova master: Move contrail vif plugging to privsep. https://review.openstack.org/515916 | |
| 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 | |
| 20:59:03 | mriedem | close, field goal | |
| 20:59:15 | mriedem | cdent: yes, if you watch the game, it's all people dressed up in different stuff | |
| 20:59:21 | mriedem | because there is no home team, just ex-pats | |
| 20:59:22 | cdent | weird | |
| 20:59:50 | mriedem | i think the official home team is jacksonville or miami, both of whom are pretty bad every year, so you're welcome | |
| 20:59:56 | cdent | is it ex-pats? I was under the impression it was crazy brits deciding they love american football and not knowing how to show it | |
| 20:59:58 | mriedem | could be worse, | |
| 21:00:00 | mriedem | could be the gd browns | |
| 21:00:12 | mriedem | oh shit | |
| 21:00:19 | mriedem | guess who the vikings play on sunday in london? | |
| 21:00:24 | mriedem | the f'ing browns! | |
| 21:03:00 | openstackgerrit | Merged openstack/nova master: Only filter/weigh hosts once if scheduling a single instance https://review.openstack.org/513931 | |
| 21:03:25 | bauwser | mriedem: well, Barca-Juv was in NYC this summer :p | |
| 21:03:40 | mriedem | no one knows what that is | |
| 21:03:48 | bauwser | gdi | |
| 21:03:50 | bauwser | man, soccer! | |
| 21:04:19 | bauwser | you know, the game that US folks try to play and at the end, they loose :p | |
| 21:04:25 | mriedem | unless they are women | |
| 21:04:33 | cdent | us women’s soccer rocs | |
| 21:04:47 | bauwser | that's correct :) | |
| 21:06:12 | cdent | tickets to the london nfl games are a lot cheaper than I would have predicted | |