| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-05 | |||
| 17:20:18 | mriedem | https://www.youtube.com/watch?v=dsx2vdn7gpY | |
| 17:20:22 | mriedem | what a whiner | |
| 17:20:58 | mriedem | i literally had a nightmare the other nighth that the aliens from aliens were chasing me through a space station, was terrible | |
| 17:21:35 | dtp | true dansmith | |
| 17:21:36 | dansmith | but it turns out they all had "cinder" on their shirts and they were screaming "multiattach" right? | |
| 17:21:49 | mriedem | heh | |
| 17:22:00 | mriedem | multiattach is one thing i haven't lost sleep over this week | |
| 17:22:15 | dansmith | heh, well, congrats then.. aliens seem not so bad | |
| 17:25:11 | openstackgerrit | Sylvain Bauza proposed openstack/nova: Expose a REST API for a specific list of RPs https://review.openstack.org/392569 | |
| 17:25:11 | openstackgerrit | Sylvain Bauza proposed openstack/nova: Put all the magic in a PlacementClient https://review.openstack.org/417111 | |
| 17:25:21 | bauzas | jaypipes: cdent: your eyes will be helpful | |
| 17:25:36 | bauzas | at least to make sure I haven't trampled the member_of lofic | |
| 17:25:40 | bauzas | logic | |
| 17:25:44 | jaypipes | yup, on it. | |
| 17:28:41 | cdent | bauzas: will look after dinner | |
| 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 | |