| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-04-20 | |||
| 12:43:02 | fried_rice | thx | |
| 12:44:05 | giblet | sahid: thanks for the unit test | |
| 12:46:20 | neha_alhat | finucannot: In the above docs it is mentioned that: after making conf parameter mutable, they can be reloaded (usually via SIGHUP) at runtime, without a service restart. | |
| 12:48:53 | neha_alhat | finucannot: Actually from the patch[1], what I understood is: If the conf parameter value is cached outside loop and if it is getting used in loop for long running processes like live migration, if operator changes parameter value then it should imediately get reflected | |
| 12:49:19 | neha_alhat | finucannot: [1]: https://review.openstack.org/#/c/319203/ | |
| 12:50:39 | fried_rice | finucannot: I'm having trouble getting my docs venv to build locally (I killed it after like 20 minutes - should it take that long??) so while I try to wrangle that, do you happen to know off hand what's going on here? http://logs.openstack.org/57/517757/13/check/build-placement-api-ref/b43304e/job-output.txt.gz | |
| 12:51:08 | fried_rice | clearly a bug in the tooling, if only because it's not showing me the actual error... | |
| 12:51:36 | finucannot | neha_alhat: I'm not the person to ask, to be honest. The best thing to do would be build a minimal test case and see what happens. If it works as you expect, a patch to the oslo.config docs would be really helpful | |
| 12:53:18 | neha_alhat | finucannot: thanks for input | |
| 12:53:33 | finucannot | fried_rice: That looks like you're assigning a string to something os-api-ref expects to be a dict | |
| 12:53:49 | fried_rice | in a doc? | |
| 12:53:57 | finucannot | In a YAML file, I'd suspect | |
| 12:54:00 | fried_rice | o | |
| 12:54:06 | finucannot | So...this? https://review.openstack.org/#/c/517757/13/placement-api-ref/source/parameters.yaml | |
| 12:54:06 | fried_rice | ... | |
| 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 | placement-api-ref installdeps: -r/home/efried/Neo/nova/doc/requirements.txt | |
| 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: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 | |