| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-05 | |||
| 17:29:13 | bauzas | jaypipes: found a miss | |
| 17:29:15 | mriedem | bauzas: yo'ure going to need to rebase https://review.openstack.org/#/c/417111/1 on top of a change of mine probably | |
| 17:29:20 | bauzas | jaypipes: just uploading now | |
| 17:29:33 | jaypipes | bauzas: k | |
| 17:29:44 | openstackgerrit | Sylvain Bauza proposed openstack/nova: Put all the magic in a PlacementClient https://review.openstack.org/417111 | |
| 17:29:44 | openstackgerrit | Sylvain Bauza proposed openstack/nova: Expose a REST API for a specific list of RPs https://review.openstack.org/392569 | |
| 17:29:51 | bauzas | jaypipes: that's it ^ | |
| 17:29:55 | bauzas | mriedem: roger. | |
| 17:30:24 | mriedem | bauzas: not sure how though, you could just approve my fix and rebase once it's merged | |
| 17:30:25 | bauzas | mriedem: hell, good point you found | |
| 17:30:31 | bauzas | mriedem: indeed :) | |
| 17:30:42 | bauzas | my stats are so poor this week | |
| 17:30:55 | bauzas | thanks for helping me to resurrect my stats | |
| 17:31:12 | bauzas | note for self : never ever take vacations | |
| 17:31:23 | bauzas | that's loose time | |
| 17:32:04 | mriedem | bauzas: i can feed you plenty of things that are ready to go :) | |
| 17:32:12 | bauzas | mriedem: <3 | |
| 17:32:22 | bauzas | I'm pretty sure I can find those | |
| 17:32:30 | mriedem | your time is probably better served right now on getting the scheduler bits up | |
| 17:32:32 | bauzas | magic of dashboards ! | |
| 17:32:42 | bauzas | anyway, +W'd | |
| 17:32:48 | mriedem | would be awesome to at least have a WIP of that before next week | |
| 17:32:56 | bauzas | mriedem: yeah, my goal | |
| 17:33:13 | bauzas | we're 4 weeks away, and I'm a bit stressed | |
| 17:33:22 | mriedem | me too | |
| 17:33:27 | bauzas | the scheduler patch shouldn't be a big deal tho | |
| 17:33:29 | mriedem | i'm on vacation the week of o-3 bw | |
| 17:33:31 | jaypipes | cdent: did we decide not to add a microversion to the API when exposing member_of param in the GET /r-p API? | |
| 17:33:31 | mriedem | *btw | |
| 17:33:52 | mriedem | bauzas: so you'll have to handle the beta release tag duties | |
| 17:34:02 | bauzas | mriedem: fine with me, I can handle that | |
| 17:34:05 | mriedem | thanks | |
| 17:34:11 | mriedem | i'm out of country 1/21-1/28 | |
| 17:34:28 | jaypipes | I'm on PTO that week as well. | |
| 17:34:31 | cdent | jaypipes: not that I'm aware of, the code under review has a microversion bump right now | |
| 17:34:49 | bauzas | mriedem: good to know | |
| 17:35:00 | cdent | jaypipes: but I'm happy to change that if we don't care | |
| 17:35:10 | bauzas | mriedem, hope you're not visiting Mexico by that time, or you could be trumped | |
| 17:35:15 | jaypipes | cdent: but technically, the handler *already* accounts for member_of (since it looks at the ResourceProviderList.allowed_filters... | |
| 17:35:20 | mriedem | bauzas: i am :) | |
| 17:35:24 | mriedem | shit that's right | |
| 17:35:25 | bauzas | mriedem: oooooh | |
| 17:35:27 | mriedem | i might not get back in | |
| 17:35:41 | jaypipes | cdent: so the API-facing change has already landed :( | |
| 17:35:49 | cdent | ? | |
| 17:35:57 | bauzas | mriedem: I could give you the French nationality but you would have to rant every day | |
| 17:36:15 | jaypipes | cdent: meaning, the code in nova *right now* supports specifying member_of in the GET params. | |
| 17:36:31 | jaypipes | cdent: and will route it to the ResourceProviderList.get_all_by_filters() call. | |
| 17:36:43 | bauzas | jaypipes: good point, we silently accepted a change in the API | |
| 17:36:53 | bauzas | that's what I thought when I rebased | |
| 17:37:05 | openstackgerrit | Gleb Stepanov proposed openstack/python-novaclient: Add evacuate host method https://review.openstack.org/349179 | |
| 17:37:14 | bauzas | since we're contracting on allowed_filters | |
| 17:37:26 | bauzas | but anyway, it's merged | |
| 17:37:27 | cdent | it won't work very well | |
| 17:37:30 | jaypipes | cdent: this was the main issue with having the handler just look at the object allowed_filters key that me switching to JSONSChema representation of the GET request solved. | |
| 17:37:35 | cdent | sure | |
| 17:37:44 | cdent | I was just using what somebody else had already done | |
| 17:37:50 | jaypipes | understood. | |
| 17:37:54 | cdent | the right thing to do at this point is to just merge the next thing and then clean it up | |
| 17:37:59 | jaypipes | cdent: let's get your microversion patch merged ASAP. | |
| 17:38:02 | bauzas | cdent: but name and uuid were supported because we microversioned | |
| 17:38:03 | cdent | jinx | |
| 17:38:03 | jaypipes | jinx :) | |
| 17:38:06 | jaypipes | fuck. | |
| 17:38:26 | bauzas | cdent: jaypipes: which microversion patch ? | |
| 17:38:26 | jaypipes | cdent: link to that patch pls? | |
| 17:38:34 | jaypipes | bauzas: jinx :) | |
| 17:38:39 | cdent | https://review.openstack.org/#/c/407742/ | |
| 17:38:56 | bauzas | oh heh | |
| 17:39:04 | bauzas | nice potential conflict to mine \o/ | |
| 17:39:08 | jaypipes | bauzas: that is the patch I *thought* had already merged. | |
| 17:39:19 | bauzas | jaypipes: no, I checked when rebasing | |
| 17:39:49 | bauzas | I need to bail out for a few hours for social and parental reasons | |
| 17:39:53 | jaypipes | bauzas: no, I mean, when I said cdent's patch had merged, that was the one I thought had merged, not the object change one. | |
| 17:39:56 | bauzas | but I should be back | |
| 17:40:00 | jaypipes | k | |
| 17:40:14 | cdent | I think this sort of thing is bound to happen with the way we've (or maybe just I've) habitually separated object change with http change | |
| 17:40:18 | bauzas | jaypipes: but the change of allowed_filters impacts the API, right? | |
| 17:40:36 | bauzas | jaypipes: because we verify that object field at the API level | |
| 17:40:47 | bauzas | so we changed our contract when we modified that list | |
| 17:40:55 | cdent | we're not really in any current danger from that contract change | |
| 17:40:57 | bauzas | another reason like you said to have proper schemas | |
| 17:41:17 | jaypipes | bauzas: yes, unfortunatgely there'll be one or two commits that will have an API change that is technically different from the published microversion 1.3 change. | |
| 17:41:28 | bauzas | :/ | |
| 17:41:36 | bauzas | either way, /me drops | |
| 17:41:43 | openstackgerrit | Matt Riedemann proposed openstack/nova: Update docstring of _schema_validation_helper https://review.openstack.org/417118 | |
| 17:41:54 | jaypipes | bauzas: for a couple commits, GET /resource_providers will accept a member_of param that doesn't use in: | |
| 17:42:00 | cdent | which is not really a big deal is it? anyoe who is a trunk chaser will chase trunk and never notice. and anyone else will never notice | |
| 17:42:13 | jaypipes | cdent: yes, that is my thought as well. | |
| 17:42:37 | cdent | and above and beyond that: the only real user is the resource tracker at this point and it doesn't send anything we don't control | |
| 17:42:48 | jaypipes | right | |
| 17:42:49 | cdent | so the exposure is conceptually bad and but reality-wise not so uch | |
| 17:42:54 | cdent | s/uch/much/ | |
| 17:42:56 | jaypipes | right | |
| 17:42:59 | cdent | cool | |
| 17:44:05 | mriedem | btw, we have query parameter json schema validation now | |
| 17:44:17 | mriedem | but would have to be duplicated in the placement API i guess to use it? | |
| 17:46:46 | cdent | mriedem: jay's added something very similar in the patch bauzas is working on | |
| 17:48:22 | jaypipes | cdent: reviewed. -1 for wording. | |
| 17:48:44 | cdent | roger than jaypipes, will look | |