Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-10-31
19:18:21 mriedem edleafe: because PUT /allocations/{consumer_id} format is going to change
19:18:26 mriedem per https://review.openstack.org/#/c/508164/5
19:18:39 cdent as it currently has code which modifies the allocation data structures
19:19:11 edleafe So requesting microversion 1.14 will return an alloc_cand that can't be PUT with 1.14?
19:19:12 cdent until we change the report client code that cooks allocations client side, we can’t up the microversion at which allocation_candidates is requested
19:19:16 mriedem cdent: die because the scheduler is assuming a certain format you mean right?
19:19:21 cdent yes
19:19:22 mriedem edleafe: correct
19:19:34 edleafe that's completely messed up
19:19:46 cdent both of your are a bit wrong
19:19:46 edleafe the whole point of allocation_candidates is defeated
19:19:54 mriedem cdent: you have a fair point
19:19:59 mriedem so i can concede that on the scheduler side,
19:20:08 cdent if we bump the microverson on allocation_candidates we _have_ to change the format of the allocations
19:20:14 mriedem but in general i think we need allocation_requests in GET /allocation_candidates to match PUT /allocations at the same microversion
19:20:25 cdent by which I mean “at which allocation_candidates is requested"
19:20:41 cdent yes, I agree, but it wasn’t specificied in that spec because I was trying to be limited
19:20:56 mriedem yeah i understand, i think it will save us some headache later though
19:20:59 mriedem to just do it now
19:21:04 edleafe the only reason allocation_candidates exist is to be immediately PUT back in a claim
19:21:12 mriedem edleafe: yes, that's my point in why we should be consistent
19:21:20 edleafe those processses should always be in sync
19:22:09 mriedem ok so i think we're all basically agreeing
19:22:14 cdent yes
19:22:17 mriedem cool
19:22:17 cdent one missing point
19:23:01 cdent do you think we need to update the client side too, or can that wait (that is can we pin report client to not go past 1.10 when get /allocation_cand for now)?
19:23:24 mriedem that can wait for now as it's opt-in
19:23:24 cdent we probably do
19:23:33 cdent since we want to get rid of the migration uuid races
19:23:42 cdent although, again, we don’t _have_ to
19:23:53 mriedem well,
19:24:02 mriedem conductor is going to be calling POST /allocations for the swap,
19:24:14 mriedem scheduler will still call GET /allocation_candidates at 1.10 for now
19:24:45 cdent right, but when calling POST /allocations for the swap, we are using allocations that are dict-ish
19:24:55 cdent and the current cooking code is list ish
19:25:01 jaypipes efried: ok, almost ready to push my "refactor AllocationCandidates._get_by_filters() mega-method" series. I think you're gonna dig it.
19:25:02 cdent many cascades
19:25:45 mriedem when the conductor code that starts calling POST /allocations does that thing, it will have to know the microversoin and request format to use for that thing,
19:25:56 mriedem but i don't think that's tied to GET /allocation_candidates,
19:25:58 cdent yes
19:26:01 mriedem it's maybe for GET /allocations
19:26:11 mriedem but still, gonna have to know what we're requesting when we do POST /allocations
19:26:20 efried jaypipes Looking forward to it. For my part, I'm about to push a new rev of the placement side of the granular parsing. Much improved, with y'all's suggestions.
19:26:28 mriedem i think the conductor code will be building that request itself, not taking it from GET /allocation_candidates
19:26:48 cdent if that’s the case then cool
19:27:10 mriedem cdent: yeah i think we'll be ok with the migration allocation swap stuff
19:27:20 mriedem it's the scheduler -> cell via reschedule stuff that is hairy
19:27:56 mriedem cdent: ok so you're going to tweak the spec and such? i've got to run for the elementary school halloween parade of joyfulness
19:28:52 cdent mriedem: not tonight, but should be able to squeeze something out before I catch a plane tomorrow. if I get lost and can’t decide what to do, I’ll respond to your comments with a “huh?”
19:30:38 mriedem cdent: sure, wfm
19:30:39 mriedem thanks
19:30:57 cdent u r welcome
19:36:50 openstackgerrit Eric Fried proposed openstack/nova master: placement: Parse granular resources & traits https://review.openstack.org/514091
19:36:51 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Numbered groupings to GET /allocation_candidates https://review.openstack.org/514092
19:36:56 efried jaypipes cdent alex_xu ^
19:37:31 efried and gibi too ^
19:37:43 edleafe mriedem: bouncing between meeting and IRC. What did you mean by "i think the conductor code will be building that request itself, not taking it from GET /allocation_candidates"
19:43:05 cdent edleafe: that was when managing the doubling of allocations for migrations etc
19:45:17 edleafe cdent: ah, that bit
20:03:29 efried What be wrong with the zuul?
20:07:03 openstackgerrit Jay Pipes proposed openstack/nova master: begin refactor AllocCandidates._get_by_filters() https://review.openstack.org/516778
20:07:04 openstackgerrit Jay Pipes proposed openstack/nova master: build alloc request resources for shared resources https://review.openstack.org/516780
20:07:04 openstackgerrit Jay Pipes proposed openstack/nova master: build ProviderSummary objects in sep function https://review.openstack.org/516779
20:07:05 openstackgerrit Jay Pipes proposed openstack/nova master: finish refactor AllocCandidates._get_by_filters() https://review.openstack.org/516782
20:07:05 openstackgerrit Jay Pipes proposed openstack/nova master: create allocation request for single provider https://review.openstack.org/516781
20:07:14 jaypipes efried, alex_xu: ^^
20:07:32 efried jaypipes Sweet. Trade ya.
20:07:53 jaypipes efried: on it.
20:13:00 openstackgerrit Eric Fried proposed openstack/nova master: placement: Contributor doc microversion checklist https://review.openstack.org/516783
20:13:11 efried mriedem Updates we talked about the other day ^
20:15:56 efried mriedem Is a similar update needed at https://docs.openstack.org/nova/pike/contributor/microversions.html#other-necessary-changes ?
20:34:39 efried jaypipes Is this series going to have any additional UT?
20:34:51 jaypipes efried: no sir.
20:34:58 efried or functional, more to the point.
20:35:02 jaypipes efried: none of this is unit-tested.
20:35:15 jaypipes efried: nope. true refactoring. no functional changes at all.
20:36:12 efried jaypipes I haven't gone through the whole thing, but if that first thing is truly finding only RPs where *all* resources are satisfied, it's not going to be used yet (not until use_same_provider=True groups come in)
20:36:43 efried unless I'm totally missing something.
20:38:03 jaypipes efried: yes, you're missing something :) that first one is an optimized code path for when the *deployment has no sharing providers*, not for when the *request is for same provider*. :)
20:38:28 efried jaypipes So it'll need to be reworked for nested.
20:38:45 jaypipes efried: it's basically "hey, do we have any sharing providers? no? great, let's not fuck around with non-shared, shared mixology and just do this."
20:39:15 jaypipes efried: well, nested and sharing providers are different complexities, but yes.
20:39:33 efried jaypipes Cause with nested in play, but without shared, the "unnumbered" group ought to be able to get its resources from anywhere in the non-sharing tree.
20:39:58 jaypipes efried: sure, but this patch doesn't touch any of that.
20:40:36 efried jaypipes Cool, that's what I needed to understand. So we're currently assuming no nested, single compute node RP; and this method also assumes no shared.
20:40:58 jaypipes efried: correcto.
20:41:34 efried jaypipes And actually, when that stuff does come into play, we should leave this method alone, precisely for the use_same_provider=True case, and create different methods that handle nested and/or shared.
20:42:18 jaypipes efried: k, Parse granular resources & traits patch reviewed. great work on that. much improved.
20:42:26 mriedem efried: yes probably also need to say something in https://docs.openstack.org/nova/pike/contributor/microversions.html#other-necessary-changes about api-ref
20:42:32 jaypipes efried: you hit the nail on the head :)
20:42:47 efried jaypipes Rockin.
20:43:19 jaypipes efried: hit the nail on the head w.r.t. your description of the "leave this method alone" above.
20:43:38 jaypipes efried: and hit the naail on the head with your patch, too... but still needs a couple minor fixups.
20:43:46 efried jaypipes Onnit.
21:07:04 openstackgerrit Eric Fried proposed openstack/nova master: placement: Parse granular resources & traits https://review.openstack.org/514091
21:07:05 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Numbered groupings to GET /allocation_candidates https://review.openstack.org/514092
21:07:16 efried jaypipes Updated ^
21:07:28 efried (Left the WIP one alone for now)
21:08:30 efried jaypipes The other side still on your radar? https://review.openstack.org/#/c/515151/

Earlier   Later