| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-23 | |||
| 20:24:57 | cburgess | mriedem Thats a dangerous thing to say to me. :P | |
| 20:25:09 | mriedem | cburgess: whatever i said, it was probably a day ago, | |
| 20:25:10 | mriedem | so remind me | |
| 20:25:19 | cburgess | mriedem Getting enough cloud providers to ask for volume type. | |
| 20:25:26 | mriedem | oh right, page 45 | |
| 20:25:34 | cburgess | page 45? | |
| 20:25:58 | mriedem | page 45 of the nova core dungeon master's guide states: "once you get enough people asking for the same feature, regardless of policy, it must be accepted" | |
| 20:26:06 | cburgess | lol | |
| 20:26:09 | penick | Pretty sure you just made that up | |
| 20:26:15 | cburgess | I didn't know such a guide exists. Must be for PTLs only. | |
| 20:26:23 | mriedem | "nova core dungeon master's guide" | |
| 20:26:34 | cburgess | Oh core | |
| 20:26:35 | cburgess | My bad.. | |
| 20:26:43 | mriedem | penick: melwitt's is keeping your dining table steady atm | |
| 20:26:45 | dansmith | I haven't seen this guide | |
| 20:26:49 | cburgess | mriedem You do realize that has to become a think now right? | |
| 20:27:01 | penick | We have a dining table? | |
| 20:27:04 | cburgess | melwitt is doing what now? | |
| 20:27:11 | dansmith | isn't that melwitt's workstation now? | |
| 20:27:15 | melwitt | what is happening | |
| 20:27:28 | mriedem | i know what's not happening | |
| 20:27:43 | mriedem | ... https://review.openstack.org/#/c/446239/ | |
| 20:27:44 | mriedem | :) | |
| 20:28:00 | melwitt | it's happening, at the subaru dealership even! | |
| 20:28:05 | mriedem | oh snap | |
| 20:28:18 | melwitt | about to upload the quotas bomb again | |
| 20:28:25 | penick | her workstation locality depends on three factors. The season, is the cat being annoying, or is she at the dealership | |
| 20:28:28 | mriedem | page 62 has a subaru coupon coincidentally | |
| 20:28:31 | mriedem | on free granola bar | |
| 20:28:33 | mriedem | *one | |
| 20:28:45 | penick | free granola bar, that's a sweet deal | |
| 20:28:53 | melwitt | I ate a free Cliff bar here earlier | |
| 20:28:58 | mriedem | haha | |
| 20:29:26 | mriedem | i always feel great when some joke i make which might be offensive turns out to be actual fact | |
| 20:29:36 | melwitt | lol, right? | |
| 20:29:48 | penick | man that nova dungeon masters guide is handy | |
| 20:29:57 | mriedem | why do you think we don't add more cores? | |
| 20:30:01 | mriedem | limited supply | |
| 20:30:03 | mriedem | of guides | |
| 20:32:02 | penick | Not cheap to print. also the "Worlds best nova core" mugs don't exactly grow on trees. Those legbreakers at Cafepress charge a fortune | |
| 20:32:12 | melwitt | merge conflicts in local delete code ... grrrr | |
| 20:32:24 | mriedem | melwitt: blame dansmith | |
| 20:32:53 | melwitt | >:| angry eyebrows | |
| 20:33:18 | dansmith | go ahead | |
| 20:33:26 | dansmith | I'll flip this table and leave | |
| 20:34:13 | melwitt | the workstation table might be hard to flip. but could be toppled over | |
| 20:34:49 | penick | Cool cores don't watch tables flip. They just walk away. (-_- )ﾉ⌒┫ ┻ ┣ | |
| 20:35:18 | dansmith | well, I'm not going to flip _this_ table since it has all my monitors and laptop on, but I will go find a suitable table with minimal collateral damage and flip that mofo | |
| 20:35:35 | melwitt | :D | |
| 20:36:07 | melwitt | we're all on the same page here | |
| 20:36:40 | cburgess | dansmith pics or it didn't happen | |
| 20:36:45 | mriedem | you know what i think we can all agree on? | |
| 20:36:51 | mriedem | removing the gluster volume driver https://review.openstack.org/#/c/463992/ | |
| 20:36:59 | cburgess | +1 LGTM | |
| 20:37:02 | dansmith | heh | |
| 20:39:52 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanup update_instance cell mapping handling https://review.openstack.org/467380 | |
| 20:39:53 | mriedem | dansmith: i'm also hoping you don't find problems with ^ | |
| 20:40:10 | mriedem | i think that stuff all predates _get_instance storing the targeted cell context on the instance object | |
| 20:40:41 | dansmith | mriedem: can we wait and or depends-on that on the multicell devstack patch? | |
| 20:40:56 | mriedem | i can put a change on top with the depends-on to test it out | |
| 20:41:02 | dansmith | okay | |
| 20:41:19 | dansmith | or just dependson it and remove it if it's good | |
| 20:41:20 | mriedem | https://review.openstack.org/#/c/436094/ right? | |
| 20:41:28 | dansmith | yar | |
| 20:41:38 | mriedem | i'm going to do another one on top of this for something else in that method | |
| 20:41:43 | dansmith | okay | |
| 20:41:44 | mriedem | we have some funny business toward the end | |
| 20:42:17 | efried | mriedem Thanks for the Sphinx thing; that spec change merged. | |
| 20:42:53 | mriedem | \o/ | |
| 20:59:42 | openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Test changes with multiple cells https://review.openstack.org/467383 | |
| 20:59:42 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove cells v2 transition code from update_instance https://review.openstack.org/467382 | |
| 21:00:40 | openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Test changes with multiple cells https://review.openstack.org/467383 | |
| 21:00:41 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove cells v2 transition code from update_instance https://review.openstack.org/467382 | |
| 21:05:41 | mriedem | efried: heh https://review.openstack.org/#/c/465135/ | |
| 21:05:46 | mriedem | this was a known issue awhile ago | |
| 21:06:41 | efried | mriedem Yes, I'm pretty sure I also proposed blocking 1.6.1 last week in some other project that didn't get updated by the bot. | |
| 21:07:14 | efried | mriedem keystone-specs, it was: https://review.openstack.org/#/c/465543/ | |
| 21:08:04 | efried | mriedem I believe it was to get https://review.openstack.org/#/c/460344/ to pass. | |
| 21:36:35 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle uuid in HostAPI._find_service https://review.openstack.org/461519 | |
| 21:36:36 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use uuid for id in Services API https://review.openstack.org/464280 | |
| 21:36:36 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle uuid in HostAPI.compute_node_get https://review.openstack.org/461532 | |
| 21:42:10 | melwitt | I wish I had realized it much earlier | |
| 21:42:10 | melwitt | the target_cell behavior change tripped me up, sigh | |
| 21:44:31 | mriedem | that's why i had to rebase ^ | |
| 21:44:36 | dansmith | you're welcome | |
| 21:45:36 | melwitt | I totes forgot I used target_cell in my patches. go me | |
| 21:46:00 | melwitt | I was like, why are my counts totally whack? | |
| 21:47:07 | dansmith | because you forgot to use target_cel and were in 1998? | |
| 21:47:22 | mriedem | i'm assuming pants were backward | |
| 21:47:25 | mriedem | *weren't | |
| 21:47:49 | melwitt | dansmith: does target_cell magically make objects queried from the db under it have self._context set to the cell db? | |
| 21:48:45 | dansmith | melwitt: if you use the yielded context to initialize them then yes | |
| 21:48:54 | melwitt | like with target_cell(ctxt, mapping): instance = Instance.get() | |
| 21:49:28 | dansmith | target_cell(ctxt, mapping) as cctxt | |
| 21:49:36 | dansmith | instance = Instance.get_by_uuid(cctx) | |
| 21:49:38 | dansmith | will work | |
| 21:50:11 | melwitt | dansmith: like this, the instances are queried inside of group.count_members_by_user() https://review.openstack.org/#/c/457859/7/nova/quota.py@1890 | |
| 21:50:40 | melwitt | and I'm not sure if the instances will have the right context in that case? | |
| 21:51:07 | dansmith | melwitt: no, see that's the problem.. group already has a context, right? | |
| 21:51:15 | dansmith | the target won't do anything it that context anymore | |