Thursday, 2023-08-31

-@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 893180: Connection health: add gerrit info https://review.opendev.org/c/zuul/zuul/+/89318009:51
-@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 893180: Connection health: add gerrit info https://review.opendev.org/c/zuul/zuul/+/89318013:19
-@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/nodepool] 893375: Kubernetes driver: allow setting dynamic k8s labels https://review.opendev.org/c/zuul/nodepool/+/89337513:57
@mhuin:matrix.orgAnybody got an idea about failing zuul-nox-docs tests since today? this change https://review.opendev.org/c/zuul/zuul/+/892985 failed in gate because of it. Seems to affect master as well14:58
@fungicide:matrix.org`autodoc: failed to import class 'web.LogStreamHandler' from module 'zuul'; the following exception was raised: PyO3 modules may only be initialized once per interpreter process`15:12
@clarkb:matrix.orgI suspect ^ needs to be fixed first, but afterwards if we land https://review.opendev.org/c/zuul/zuul/+/892696 and https://review.opendev.org/c/zuul/nodepool/+/892697 we can get zuul and nodepool running on bookworm in OpenDev over the weekend (which then serves as a form of beta test for everyone downstream)15:30
@clarkb:matrix.orghttps://pyo3.rs/main/migration.html seems related to the pyo3 thing15:31
@clarkb:matrix.orgdid something add a new rust dependency maybe?15:33
@clarkb:matrix.orghttps://pyo3.rs/main/migration.html#from-016-to-017 is the specific migration that raises that error and that is older15:33
@jim:acmegating.comthe crypto stuff was rusting, right?15:33
@clarkb:matrix.orgyes, but it has for a very long time I think. But maybe they changed their linkage library15:34
@clarkb:matrix.orgLogStreamHandler is a subclass of ws4py.websocket.WebSocket. I bet that is where the rust linkage is15:37
@clarkb:matrix.orghrm maybe not ws4py hasn't had a release in quite some time15:38
@fungicide:matrix.orgsetuptools-rust latest release was tuesday of last week (2023-08-22)15:39
@fungicide:matrix.orgbut zuul-nox-docs only broke between 13:38z (last success) and 23:03z (first failure) yesterday15:41
@fungicide:matrix.orgso roughly a 9.5-hour window there15:41
@clarkb:matrix.orgcan compare package differences between those15:42
@fungicide:matrix.orgmost recent pyo3 release was a month ago too: https://github.com/PyO3/pyo3/tags15:43
@clarkb:matrix.orgI wonder if dropping eithe rthe pyclass or autodoc call against the class will workaround it. Currently trying ot test that theory locally15:45
@clarkb:matrix.orgI want to reproduce locally first though15:45
@jim:acmegating.comsphinx 7.2.4 -> 7.2.5 is the culprit15:48
@jim:acmegating.com#11645: Fix a regression preventing autodoc from importing modules within packages that make use of if typing.TYPE_CHECKING: to guard circular imports needed by type checkers. Patch by Matt Wozniski.15:49
@jim:acmegating.comlooks suspciously related to the py03 thing15:49
@clarkb:matrix.orgif I remove the logstreamhandler autodoc the error moves to zuulweb. So cherrypy must be doing rust things?15:49
@jim:acmegating.comClark: https://paste.opendev.org/show/bffjJPhzXpkbth4pKurA/15:50
@fungicide:matrix.orgin the initial pip install, the other thing i see changing is virtualenv 20.24.3->20.24.415:50
@clarkb:matrix.orgcorvus: oh how do you get the full list of warnings? Because ya I'm finding each one I remove leads to another popping up15:51
@clarkb:matrix.orgthat is unlikely to be a viable short term workaround then15:51
@jim:acmegating.com`sphinx-build -E -d doc/build/doctrees -b html doc/source/ doc/build/html`15:51
@jim:acmegating.comhttps://github.com/sphinx-doc/sphinx/issues/1166215:53
@fungicide:matrix.orgsmoking gun15:53
@jim:acmegating.combased on the state of discussion in that bug, i think we should pin sphinx with a link to it.  i'll do that.15:54
@fungicide:matrix.orgsgtm, thanks!15:54
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 893411: Avoid sphinx 7.2.5 https://review.opendev.org/c/zuul/zuul/+/89341115:56
@jim:acmegating.comdid that as a != in the hope they'll magically fix it and release soon :)15:56
@clarkb:matrix.orgI'm surprised that removing a module from sys.modules is ever sufficient to actually unload it...15:57
@jim:acmegating.comyeah... like in the python dictionary sense, yes... in the sense of having run machine code... not so much, eh? :)15:58
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 893411: Avoid sphinx 7.2.5 https://review.opendev.org/c/zuul/zuul/+/89341117:01
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 892985: Add failure-output option to zuul-nox-py311 job https://review.opendev.org/c/zuul/zuul/+/89298517:20
@jpew:matrix.orgWould someone mind a peek at https://review.opendev.org/q/topic:decline-from-quota ?18:40
@jpew:matrix.orgAlso we found https://review.opendev.org/c/zuul/nodepool/+/892805 useful for debugging purposes18:41

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!