| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-03-19 | |||
| 15:12:19 | openstackgerrit | Gorka Eguileor proposed openstack/nova master: Use os-brick locking for volume attach and detach https://review.openstack.org/614190 | |
| 15:15:16 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Address old TODO in claim_resources_on_destination https://review.openstack.org/644596 | |
| 15:39:55 | mdbooth | This is weird, and doesn't reproduce for me locally: http://logs.openstack.org/52/626952/8/check/openstack-tox-docs/0876468/ara-report/result/73a64a42-855e-4818-980a-0a106766e721/ | |
| 15:40:31 | mdbooth | Does anybody know how I can kick off a 'openstack-tox-docs' job in check in isolation? | |
| 15:40:59 | mdbooth | I don't see that job in our .zuul.yaml, so I assume it's defined elsewhere | |
| 15:42:11 | artom | mdbooth, tox -e docs? Or am I being a simpleton? | |
| 15:42:26 | mdbooth | artom: That's how I can run it locally. That works fine. | |
| 15:42:40 | mdbooth | It failed in the check queue, though. I want to see if that happens again without having to wait 2 hours. | |
| 15:42:42 | artom | mdbooth, oh, you mean in CI | |
| 15:42:47 | mdbooth | Indeed. | |
| 15:43:04 | artom | mdbooth, I think if you just touch the docs subtree it'll only run those jobs | |
| 15:43:17 | mdbooth | Do you know where that's defined? | |
| 15:43:23 | artom | ... and you mean on your existing patches, don't you? | |
| 15:43:38 | artom | Without pushing a new docs-only DNM patch or w/e | |
| 15:43:46 | mdbooth | Well if I can find a way to kick it off in isolation I can layer a patch on top or something | |
| 15:43:57 | mdbooth | But I don't know what's running it. It's not in nova's .zuul.yaml | |
| 15:44:27 | mriedem | gibi: question for you in bence's neutron docs patch https://review.openstack.org/#/c/640390/5/doc/source/admin/config-qos-min-bw.rst@98 | |
| 15:44:47 | sean-k-mooney | mriedem: the check queue jsut use ansible to run the same tox env | |
| 15:44:55 | sean-k-mooney | mriedem: sorry mdbooth ^ | |
| 15:45:12 | mdbooth | sean-k-mooney: Where is it defined? | |
| 15:45:34 | sean-k-mooney | mdbooth: ill get the link. its in either zull-jobs or openstack-zull-jobs | |
| 15:45:36 | mdbooth | sean-k-mooney: I want a way to actually run it on CI infrastructure. | |
| 15:46:29 | sean-k-mooney | it should be identically to how you run it locally as its using a tox env but you might need to recreate your tox env with -r to make sure its clean | |
| 15:47:08 | mdbooth | It runs successfully in a local clean tox env | |
| 15:47:16 | mdbooth | I'm looking for a way to run it on CI | |
| 15:47:25 | sean-k-mooney | i think its this job https://github.com/openstack-infra/openstack-zuul-jobs/blob/master/zuul.d/jobs.yaml#L293-L321 | |
| 15:47:35 | mdbooth | It failed there for a reason which looks bizarre. | |
| 15:47:40 | sean-k-mooney | oh wait thats the old one | |
| 15:47:55 | sean-k-mooney | this one https://github.com/openstack-infra/openstack-zuul-jobs/blob/master/zuul.d/jobs.yaml#L618-L632 | |
| 15:48:24 | sean-k-mooney | mdbooth: do you have the failure so i can take a look | |
| 15:48:43 | mdbooth | http://logs.openstack.org/52/626952/8/check/openstack-tox-docs/0876468/ara-report/result/73a64a42-855e-4818-980a-0a106766e721/ | |
| 15:48:51 | mdbooth | AttributeError: module 'select' has no attribute 'poll' | |
| 15:49:13 | mdbooth | Given that this is against a monkey patching change, that's obviously suspect. However, it works locally. | |
| 15:51:39 | sean-k-mooney | well you can see the command at the top it was "tox -edocs -vv" | |
| 15:52:28 | mdbooth | Yep. That's how I know how to run it locally. That works ;) | |
| 15:52:57 | mdbooth | It doesn't work in CI, though. I was looking for a lightweight way to run that in CI to see if it can be reproduced in CI without having to wait 2 hours. | |
| 15:53:17 | sean-k-mooney | mdbooth: https://docs.python.org/3.6/library/select.html#select.poll | |
| 15:53:32 | sean-k-mooney | so apparently that is not suppored on all operating systems... | |
| 15:53:50 | mdbooth | sean-k-mooney: You think I might have been scheduled on a Windows CI worker? | |
| 15:53:53 | sean-k-mooney | we might be missing a depencny on the gate job | |
| 15:54:01 | mdbooth | That seems improbable ;) | |
| 15:54:06 | artom | sean-k-mooney, I'd thunk that meant Linux and Windows, not F29 and Ubuntu | |
| 15:54:12 | sean-k-mooney | it was on ubunut 18.04 | |
| 15:54:46 | artom | mdbooth, actually wait, are *you* running in on Windows? | |
| 15:54:51 | artom | You might be, knowing you | |
| 15:55:01 | mdbooth | artom: Hmm.... | |
| 15:55:09 | mdbooth | Yes I am. | |
| 15:55:15 | sean-k-mooney | mdbooth: can i have the link to the gerrit review | |
| 15:55:20 | sean-k-mooney | ill run it on my laptop | |
| 15:56:01 | mdbooth | artom: Running in WSL. However, I'd expect to see that *I* don't have poll() | |
| 15:56:06 | mdbooth | Linux definitely has poll. | |
| 15:56:12 | artom | mdbooth, true. | |
| 15:56:29 | jangutter | mdbooth: It's not a Windows CI worker - the pre jobs installed some bindep packages in Ubuntu: http://logs.openstack.org/52/626952/8/check/openstack-tox-docs/0876468/ara-report/result/9863769a-0d01-4a5a-b45a-345628504cbf/ | |
| 15:57:05 | mdbooth | jangutter: Surely we don't have any Windows CI workers! ? | |
| 15:57:34 | sean-k-mooney | mdbooth: we proably do for hyperv | |
| 15:57:40 | sean-k-mooney | but not for docs | |
| 15:57:56 | mriedem | lyarwood: dansmith: can one of you fast approve https://review.openstack.org/#/c/643011/ ? | |
| 15:59:03 | mriedem | bauzas: you want to get this rocky backport https://review.openstack.org/#/c/643098/ ? | |
| 15:59:10 | mdbooth | Linux is a bazillion times faster than WSL. | |
| 15:59:21 | bauzas | mriedem: sure, looking | |
| 15:59:33 | sean-k-mooney | mdbooth: cygwin was faster the WSL the last time i tried WSL | |
| 15:59:37 | openstackgerrit | Merged openstack/nova master: Remove unnecessary default provider_tree when getting traits https://review.openstack.org/644452 | |
| 16:00:15 | mriedem | once we get those in i think we can do a rocky release | |
| 16:00:26 | mriedem | melwitt: ^ | |
| 16:02:04 | jangutter | mdbooth, sean-k-mooney: do the tox docs jobs normally run without the "Modules with known eventlet monkey patching issues were imported prior to eventlet monkey patching: oslo_context.context, urllib3." warning? | |
| 16:02:25 | lyarwood | mriedem: done | |
| 16:02:41 | mdbooth | jangutter: I just added that message | |
| 16:02:42 | sean-k-mooney | jangutter: we should not be monkeypatch for docs | |
| 16:02:56 | mdbooth | jangutter: However, we have always monkey patched for docs | |
| 16:03:10 | sean-k-mooney | jangutter: that said we might be | |
| 16:03:17 | mdbooth | That warning message simply points it out | |
| 16:03:42 | mdbooth | Ok, I can't run tox -edocs on Linux with or without my patch | |
| 16:03:44 | sean-k-mooney | mdbooth: so i get the same error locally | |
| 16:03:47 | mdbooth | That's annoying | |
| 16:04:03 | mdbooth | Ah, that will be a pyc thing | |
| 16:04:25 | sean-k-mooney | i had to recreate the env with "tox -e docs -vv -r" | |
| 16:04:38 | sean-k-mooney | but i got the same | |
| 16:04:40 | sean-k-mooney | Exception occurred: | |
| 16:04:42 | sean-k-mooney | File "/usr/lib/python3.6/selectors.py", line 348, in __init__ | |
| 16:04:44 | sean-k-mooney | self._poll = select.poll() | |
| 16:04:46 | sean-k-mooney | AttributeError: module 'select' has no attribute 'poll' | |
| 16:04:48 | sean-k-mooney | error | |
| 16:05:02 | sean-k-mooney | that works too | |
| 16:05:10 | sean-k-mooney | ill try it on a centos vm | |
| 16:05:51 | mdbooth | So, the only thing my patching is changing is where we monkey patch | |
| 16:06:16 | sean-k-mooney | mdbooth: ya so i dont think we used to monkey patch docs | |
| 16:06:34 | mdbooth | sean-k-mooney: Why do you think that? | |
| 16:06:37 | sean-k-mooney | docs only installed stuff for the vrsion number and then config stuff | |
| 16:07:03 | sean-k-mooney | mdbooth: i dont think shinix loaded an entrypoint that incldued a path that invokded monky pataching | |
| 16:07:16 | mdbooth | Do you know what entrypoints it loads? | |
| 16:07:20 | sean-k-mooney | it just opened the nova/conf subtree i think | |
| 16:07:44 | sean-k-mooney | that is a stephenfin question but ill see if i can find out quickly | |
| 16:07:54 | mdbooth | Ok, I can't reproduce locally on F29 | |
| 16:08:08 | mdbooth | sean-k-mooney: What eventlet version do you have? | |
| 16:08:22 | sean-k-mooney | https://github.com/openstack/nova/blob/master/doc/source/conf.py#L22 | |
| 16:08:41 | mdbooth | Although that's tox installed, so should be the same | |
| 16:09:00 | sean-k-mooney | eventlet==0.24.1 | |
| 16:09:06 | mdbooth | Hmm, same | |
| 16:09:37 | sean-k-mooney | am why do we do https://github.com/openstack/nova/blob/master/doc/source/conf.py#L201 | |
| 16:10:30 | mdbooth | I don't know, but as that's not eventlet I assume that's ok | |