| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-05 | |||
| 17:15:30 | mriedem | rechecking | |
| 17:15:31 | jaypipes | mriedem: something being worked on, or should I look into it? | |
| 17:15:36 | jaypipes | ah, k | |
| 17:15:37 | mriedem | should be fine | |
| 17:15:55 | bauzas | oh snap | |
| 17:16:21 | openstackgerrit | Jay Jahns proposed openstack/nova: Add connection pool size to vSphere settings https://review.openstack.org/414085 | |
| 17:16:33 | bauzas | jaypipes: cdent: so we merged something using allowed_filters :( | |
| 17:17:10 | jaypipes | bauzas: yeah. will need to be modded to use JSONSchema. | |
| 17:17:38 | bauzas | jaypipes: I need to look at the change | |
| 17:17:51 | jaypipes | bauzas: whichever one of those landed first, the other would need modding. | |
| 17:18:02 | jaypipes | bauzas: just a fact of life in this code at this moment. | |
| 17:18:17 | bauzas | sure, I just need to understand the logic of the merged patch | |
| 17:18:18 | dansmith | dtp: yep, just like that :) | |
| 17:18:29 | dtp | cool | |
| 17:18:32 | jaypipes | bauzas: shouldn't be much needed other than adding 'member_of' into the JSONSChema for get. | |
| 17:18:46 | bauzas | jaypipes: no worries, just looking at the merged change | |
| 17:18:50 | dtp | thanks. also, why is no one fighting over gary sinise? | |
| 17:18:51 | mriedem | sdague: i meant to ask you, is there a hook in grenade such that we can do some stuff AFTER ocata code is installed but BEFORE the services are restarted to use that code? | |
| 17:18:59 | mriedem | dtp: i couldn't remember his last name :) | |
| 17:19:03 | mriedem | i knew it was gary though | |
| 17:19:08 | openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform missing delete notifications https://review.openstack.org/410297 | |
| 17:19:14 | mriedem | 'the other guy from forest gump' | |
| 17:19:19 | dtp | oof | |
| 17:19:28 | dtp | lt. dan | |
| 17:19:48 | dansmith | dtp: um, pretty sure he never said that without screaming, right? | |
| 17:19:53 | mriedem | or bill paxton | |
| 17:19:55 | dansmith | LT DAAYUNN! | |
| 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. | |