Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-10-27
17:24:42 openstackgerrit Ed Leafe proposed openstack/nova master: Only filter/weigh hosts once if scheduling a single instance https://review.openstack.org/513931
17:24:42 openstackgerrit Ed Leafe proposed openstack/nova master: Add Selection objects https://review.openstack.org/499239
17:24:43 openstackgerrit Ed Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver https://review.openstack.org/495854
17:24:43 openstackgerrit Ed Leafe proposed openstack/nova master: Change RPC for select_destinations() https://review.openstack.org/510159
17:24:44 openstackgerrit Ed Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils https://review.openstack.org/511357
17:24:44 openstackgerrit Ed Leafe proposed openstack/nova master: Make conductor pass and use host_lists https://review.openstack.org/511358
17:31:39 jianghuaw leakypipes, thanks for the +2.
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 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

Earlier   Later