| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-04-20 | |||
| 12:54:36 | finucannot | Lemme pull it down and try | |
| 12:57:01 | fried_rice | finucannot: Yeah, I can't see anything bizarre in there. | |
| 12:57:33 | fried_rice | Should I open a bug about that crazy error message? (Although you seem not to have had trouble deciphering it) | |
| 12:58:06 | finucannot | fried_rice: Yeah, some additional error messages wouldn't be any harm, I imagine | |
| 13:03:41 | finucannot | fried_rice: Yup, you'd a typo. Added comment | |
| 13:03:47 | finucannot | I'll go fix os-api-ref now | |
| 13:03:50 | fried_rice | finucannot: Thanks! | |
| 13:04:41 | finucannot | fried_rice: Did you end up opening a bug? If not, I'll just send this patch as-is | |
| 13:05:05 | fried_rice | finucannot: I'm in the process. Two shakes. | |
| 13:05:53 | fried_rice | finucannot: https://github.com/sphinx-doc/sphinx/issues/4869 | |
| 13:06:13 | finucannot | fried_rice: Ah, that's an os-api-ref bug. Not a Sphinx bug | |
| 13:06:36 | fried_rice | finucannot: I was just following the instructions in the error output. | |
| 13:06:38 | finucannot | fried_rice: You can tell based on this bit --> File "/home/zuul/.venv/local/lib/python2.7/site-packages/os_api_ref/__init__.py", line 261, in _check_yaml_sorting | |
| 13:07:01 | finucannot | Yeah, that error message is dumb. _That's_ probably a good Sphinx bug | |
| 13:08:59 | fried_rice | finucannot: Should I close that one, then? | |
| 13:09:07 | finucannot | Probably best, yeah | |
| 13:09:14 | finucannot | They can't do anything about that | |
| 13:09:17 | fried_rice | finucannot: Done. | |
| 13:13:06 | openstackgerrit | Matt Riedemann proposed openstack/nova master: xenapi: Documents update for XAPI pool shared SR migration https://review.openstack.org/562482 | |
| 13:18:13 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757 | |
| 13:24:33 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add columns for generation to consumers https://review.openstack.org/557958 | |
| 13:24:34 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add the Consumer object https://review.openstack.org/561406 | |
| 13:24:35 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add Consumer objects to Allocations https://review.openstack.org/561407 | |
| 13:24:36 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add a microversion for consumer generation https://review.openstack.org/562000 | |
| 13:24:37 | openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - Change DELETE allocations to PUT [] https://review.openstack.org/563110 | |
| 13:33:49 | openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - Change DELETE allocations to PUT {} https://review.openstack.org/563110 | |
| 13:38:08 | finucannot | Can anyone remind me how you can use mock to wrap a function without replacing it entirely | |
| 13:38:17 | finucannot | i.e. passthrough the call and simply capture the call/return value | |
| 13:40:06 | openstackgerrit | Surya Seetharaman proposed openstack/nova stable/pike: Unmap compute nodes when deleting host mappings in delete cell operation https://review.openstack.org/553829 | |
| 13:43:35 | giblet | leakypipes: hi! I've update the bandwidth spec with a better reasoning why we need certain traits for the networking RPs. I hope we can find a good compromise | |
| 13:44:26 | fried_rice | finucannot: Yeah, you have to save off the original function | |
| 13:44:41 | fried_rice | finucannot: Then write a new function that conditionally calls the old one (presumably that's what you want) | |
| 13:44:48 | fried_rice | finucannot: Then make that new function your mock side effect. | |
| 13:45:00 | leakypipes | giblet: roger | |
| 13:45:06 | fried_rice | finucannot: You can't, like, jump into the middle of a function and start mocking, if that's what you're asking. | |
| 13:46:40 | dansmith | sean-k-mooney[m]: around? | |
| 13:47:01 | finucannot | fried_rice: Ah, fair. I was hoping for a 'passthrough=True' flag. Doesn't seem it does that | |
| 13:47:09 | finucannot | Guess it's kind of the opposite of what you'd want | |
| 13:47:29 | fried_rice | finucannot: yeah, because it would have to pass through conditionally, and you'd need some way to specify that condition. | |
| 13:47:40 | fried_rice | finucannot: Otherwise, if you wanted it to pass through always, you would just... not mock it. | |
| 13:51:55 | fried_rice | finucannot: I'm ready to give up trying to build docs locally. tox -re placement-api-ref is just sitting forever at | |
| 13:51:55 | fried_rice | placement-api-ref installdeps: -r/home/efried/Neo/nova/doc/requirements.txt | |
| 13:52:33 | finucannot | fried_rice: Just use 'tox -e docs'? | |
| 13:52:39 | fried_rice | finucannot: same | |
| 13:52:45 | fried_rice | tried that earlier | |
| 13:52:53 | finucannot | Hmm, that's a problem on your end, I'd imagine | |
| 13:52:57 | finucannot | Working a-ok here | |
| 13:53:07 | fried_rice | it spun, seemingly doing nothing, for a good 20 or 30 minutes before I killed it. | |
| 13:53:17 | finucannot | Tried with -vvv ? | |
| 13:53:23 | finucannot | I assume that will work | |
| 13:53:30 | fried_rice | was about to ask if there was some way to verbosify it. | |
| 13:53:32 | fried_rice | trying... | |
| 13:55:34 | fried_rice | Well, that's getting me more output, anyway. | |
| 13:55:39 | fried_rice | wtf is snowballstemmer?? | |
| 13:58:31 | fried_rice | finucannot: It looks like it's getting stuck really early, here: http://paste.openstack.org/show/719628/ | |
| 13:58:48 | finucannot | What version of pip are you on? | |
| 13:59:27 | fried_rice | finucannot: It looks like it's installing 10.0.1 into the venv | |
| 13:59:33 | melwitt | frickler: ah, sorry :( | |
| 13:59:43 | fried_rice | finucannot: under py27 | |
| 14:00:28 | finucannot | Yeah, that should be fine. I'm guessing this is a manifestation of some pypi.python.org -> pypi.org switchover bug | |
| 14:00:37 | finucannot | Weird that you're seeing it and no one else is though | |
| 14:00:52 | finucannot | Maybe just give up for today and try again Monday? :) | |
| 14:02:21 | fried_rice | finucannot: Well, maybe everybody else already has their venv created for docs. | |
| 14:02:43 | fried_rice | finucannot: I'm also still on trusty, don't know if that makes a difference. | |
| 14:03:10 | melwitt | frickler: and that's a good point about stable, I'll test it out | |
| 14:03:31 | finucannot | It shouldn't. That's why I asked what version of pip you were using but if it's installing everything from scratch, it can't be that | |
| 14:05:59 | finucannot | fried_rice: Yeah, works fine with '--recreate' on my end | |
| 14:06:23 | fried_rice | finucannot: Weird. Well, thanks for checking. | |
| 14:22:02 | melwitt | bauzas, superdan: could one of you please review this pcid cpu feature flags backport? need it to release ocata soon https://review.openstack.org/#/c/561168 | |
| 14:22:33 | superdan | yar | |
| 14:23:12 | melwitt | thx | |
| 14:23:37 | melwitt | also need this report allocated size of preallocated file based disks https://review.openstack.org/#/c/562078 pike change merged recently | |
| 14:23:47 | superdan | melwitt: on that one, he added _LW because backport but didn't line up the string | |
| 14:24:08 | superdan | melwitt: mriedem pointed it out kinda, but not sure if he was saying yeah this is fine or not | |
| 14:24:19 | superdan | melwitt: I got the pike one yesterday and was waiting for it to land | |
| 14:27:12 | melwitt | hm, yeah. "yar", that's usually a good thing | |
| 14:27:41 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757 | |
| 14:30:03 | openstackgerrit | Merged openstack/nova master: Fix link in placement contributor doc https://review.openstack.org/563086 | |
| 14:30:20 | melwitt | superdan: yeah, not sure if he noticed that. we can ask him | |
| 14:31:23 | leakypipes | sahid: I'm really confused by the two patches that set driver name to None for vhost-user VIFs. Is there a bug that those patches are fixing? Also, I'm confused why there are two small patches where the second patch basically undoes part of the previous patch. | |
| 14:32:41 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION for "Rocky" https://review.openstack.org/558783 | |
| 14:32:42 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: Drop MIN_LIBVIRT_SET_ADMIN_PASSWD https://review.openstack.org/562312 | |
| 14:33:48 | superdan | leakypipes: those are supposed to be hygeine patches for things were setting that are basically ignored right now by libvirt, but were creating confusing things underneath | |
| 14:34:07 | superdan | leakypipes: I dunno about the undoing-in-last-patch bit, but I asked those to be broken out because they were even more confusing being in the actual implementation patch | |
| 14:34:31 | superdan | leakypipes: we were setting driver=vhost even when type=vhostuser or something like that, and libvirt was just ignoring us | |
| 14:34:59 | leakypipes | superdan: ok. yes, I get that (as noted in my review comment). can those bottom two patches just be combined? | |
| 14:35:18 | superdan | leakypipes: I dunno, there was only one last I looked | |
| 14:35:36 | superdan | leakypipes: I want to see the vhostuser neutron job run against this if at all possible first though | |
| 14:35:51 | leakypipes | superdan: yes, saw that comment on the last patch in the series. | |
| 14:36:36 | superdan | leakypipes: ack cool | |
| 14:36:47 | superdan | I dunno why the bottom one was split, I haven't gone back to look at that yet today | |
| 14:36:49 | melwitt | lyaaaaaaaaarwood: want to hit this again? pike change merged https://review.openstack.org/#/c/560167 | |
| 14:37:09 | melwitt | and this https://review.openstack.org/#/c/560162 | |
| 14:39:26 | lyaaaaaaaaarwood | melwitt: ack | |
| 14:41:26 | kashyap | melwitt: Hey there, post milestone work busyness, also want to have a gander at this: https://review.openstack.org/#/c/558783/ | |
| 14:41:31 | kashyap | melwitt: You acked the other version bump thing. | |
| 14:41:47 | kashyap | (MattR already gave some feedback, I addressed it. Will let Zuul do its thing first) | |
| 14:41:57 | melwitt | superdan: here's another one for ocata https://review.openstack.org/#/c/545087 if you could pls take a look | |
| 14:43:20 | melwitt | kashyap: k, will put it on my list. still working on getting a couple of stable releases in line | |
| 14:43:43 | kashyap | melwitt: Sure, I just noticed I missed Matt's one more remark, I'll address that too | |