| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/nodepool] 847698: dib: Bump to 3.22.0 https://review.opendev.org/c/zuul/nodepool/+/847698 | 00:39 | |
| -@gerrit:opendev.org- Kenny Ho marked as active: [zuul/zuul] 837063: Add build set URL to reporter https://review.opendev.org/c/zuul/zuul/+/837063 | 01:15 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Szymon Datko: [zuul/zuul-jobs] 847656: [tox] Update readme regarding default values https://review.opendev.org/c/zuul/zuul-jobs/+/847656 | 08:59 | |
| -@gerrit:opendev.org- Jonathan Rosser proposed: [zuul/zuul-jobs] 847775: Allow upload-logs-swift files list to be overriden https://review.opendev.org/c/zuul/zuul-jobs/+/847775 | 12:20 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: | 13:42 | |
| - [zuul/zuul-operator] 847789: Setup cmd direct entrypoint https://review.opendev.org/c/zuul/zuul-operator/+/847789 | ||
| - [zuul/zuul-operator] 847790: Add support for namespaced deployment https://review.opendev.org/c/zuul/zuul-operator/+/847790 | ||
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 847809: Remove CONTRIBUTE.md documentation https://review.opendev.org/c/zuul/zuul-operator/+/847809 | 16:09 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 847830: Update to the latest version of kopf https://review.opendev.org/c/zuul/zuul-operator/+/847830 | 16:11 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 847830: Update to the latest version of kopf https://review.opendev.org/c/zuul/zuul-operator/+/847830 | 16:26 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 847830: Update to the latest version of kopf https://review.opendev.org/c/zuul/zuul-operator/+/847830 | 17:20 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 847646: Fix merging with submitWholeTopic https://review.opendev.org/c/zuul/zuul/+/847646 | 17:29 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 847842: Test increasing the timeout https://review.opendev.org/c/zuul/zuul-operator/+/847842 | 18:21 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 847106: Add nodepool.image_build_requests metric https://review.opendev.org/c/zuul/nodepool/+/847106 | 18:24 | |
| @tristanc_:matrix.org | corvus: I think there is an issue with the pykube library in the operator, it does not seems like it can find the in-cluster kube config. Do you remember why it is used instead of the official client? | 18:44 |
|---|---|---|
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 847845: Try loading the config earlier https://review.opendev.org/c/zuul/zuul-operator/+/847845 | 19:09 | |
| @jim:acmegating.com | tristanC: istr something about pykube-ng being easier to use when dealing with custom resources, but i don't remember details. i do see that they just made a release: https://pypi.org/project/pykube-ng/#history | 19:13 |
| @jim:acmegating.com | might be worth seeing if an older version works | 19:13 |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 847846: Pin pykube-ng https://review.opendev.org/c/zuul/zuul-operator/+/847846 | 19:15 | |
| @tristanc_:matrix.org | corvus: ok let's see. | 19:19 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 846448: Add --wait-for-init scheduler option https://review.opendev.org/c/zuul/zuul/+/846448 | 19:20 | |
| @tristanc_:matrix.org | I tried to remove pykube, but the official python client does seem more complicated. It's too bad because the golang kube-client is quite nice actually. | 19:21 |
| @jim:acmegating.com | yeah, pykube is nice to work with; hopefully it's just a small bug | 19:25 |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 845851: Update apiVersion of the CRD https://review.opendev.org/c/zuul/zuul-operator/+/845851 | 19:38 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: | 19:51 | |
| - [zuul/zuul-operator] 845851: Update apiVersion of the CRD https://review.opendev.org/c/zuul/zuul-operator/+/845851 | ||
| - [zuul/zuul-operator] 847854: Add timeout for wsdump test https://review.opendev.org/c/zuul/zuul-operator/+/847854 | ||
| @blaisep-sureify:matrix.org | I just want to bask in that delicious sense of accomplishment, having finally succeeded in my first `/config/resources/resources.yaml` change. | 19:52 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 846806: Support deleting DIB images while builder is offline https://review.opendev.org/c/zuul/nodepool/+/846806 | 20:00 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 846806: Support deleting DIB images while builder is offline https://review.opendev.org/c/zuul/nodepool/+/846806 | 20:04 | |
| @tristanc_:matrix.org | corvus: pinning pykube seems to help, but the job now fails on the console stream test | 20:25 |
| @jim:acmegating.com | tristanC: i'm running `wsdump -r --eof-wait 5 -t '{"uuid":"1ef64fea79ce40ff803e0ef321980396","logfile":"console.log"}' wss://zuul.opendev.org/api/tenant/local/console-stream` to see what it does | 20:34 |
| @jim:acmegating.com | oh wait, it looks like the wsdump worked... | 20:34 |
| @jim:acmegating.com | was it the test on the static host and not the pod that failed? | 20:35 |
| @jim:acmegating.com | sorry, i deleted some message for clarity since i was confused | 20:36 |
| @jim:acmegating.com | it looks like "connect to console-stream" may be the task that did not finish, even though it appears to have printed all of the output. so perhaps it didn't get an EOF? that's what my test above should help with | 20:36 |
| @jim:acmegating.com | tristanC: okay, the job finished, and my wsdump command is still running. so it looks like zuul itself is really not sending an EOF at the end of a websocket console stream. | 20:38 |
| @jim:acmegating.com | (or possibly it is, but wsdump isn't handling that the same way anymore; that seems less likely though) | 20:39 |
| @jim:acmegating.com | tristanC: actually -- according to lsof i don't have a connection open anymore, but wsdump still hasn't exited. i think we should consider the possibility that "--eof-wait" doesn't work as we expect (anymore) | 20:46 |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 847854: Add timeout for wsdump test https://review.opendev.org/c/zuul/zuul-operator/+/847854 | 21:02 | |
| @tristanc_:matrix.org | corvus: thank you for checking, let me see if i can find a solution for that | 21:04 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 847856: Use attributes instead of a nested dict for web cache https://review.opendev.org/c/zuul/zuul/+/847856 | 21:17 | |
| @jim:acmegating.com | mhu: ^ is a followup to https://review.opendev.org/832293 which i approved | 21:18 |
| @mhuin:matrix.org | +1'ed for great justice | 21:20 |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 847854: Provide input to the wsdump test https://review.opendev.org/c/zuul/zuul-operator/+/847854 | 21:30 | |
| @tristanc_:matrix.org | corvus: i think i got it, it seems like `wsdump` is waiting for something to happen on stdin. I'll squash 847854 and 847846, then we should have a working ci again for the zuul-operator | 22:02 |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 847846: Pin pykube-ng and fix the CI https://review.opendev.org/c/zuul/zuul-operator/+/847846 | 22:04 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 845851: Update apiVersion of the CRD https://review.opendev.org/c/zuul/zuul-operator/+/845851 | 22:05 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 845851: Update CRD apiVersion to v1 (from v1beta) https://review.opendev.org/c/zuul/zuul-operator/+/845851 | 22:06 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 847830: Update to the latest version of kopf https://review.opendev.org/c/zuul/zuul-operator/+/847830 | 22:07 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 847809: Remove CONTRIBUTE.md documentation https://review.opendev.org/c/zuul/zuul-operator/+/847809 | 22:08 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-operator] 785985: Move some config to configmaps https://review.opendev.org/c/zuul/zuul-operator/+/785985 | 22:37 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed on behalf of Albin Vass: [zuul/zuul-operator] 800302: WIP: Add structural schema to crd https://review.opendev.org/c/zuul/zuul-operator/+/800302 | 22:39 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed on behalf of Albin Vass: [zuul/zuul-operator] 800265: Gracefully shutdown executor on SIGTERM https://review.opendev.org/c/zuul/zuul-operator/+/800265 | 22:41 | |
| @tristanc_:matrix.org | Here is the fix to unlock the zuul-operator CI: https://review.opendev.org/c/zuul/zuul-operator/+/847846 | 23:15 |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 845851: Update CRD apiVersion to v1 (from v1beta) https://review.opendev.org/c/zuul/zuul-operator/+/845851 | 23:38 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!