| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-base-jobs] 821919: Switch docs theme to RTD https://review.opendev.org/c/zuul/zuul-base-jobs/+/821919 | 00:15 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-client] 821883: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-client/+/821883 | 00:21 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-operator] 821911: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-operator/+/821911 | 00:21 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-sphinx] 821888: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-sphinx/+/821888 | 00:21 | |
| @jim:acmegating.com | Is anyone maintaining the artifactory roles? https://zuul.opendev.org/t/zuul/build/a90bbdf16c5e449a81a683d46f56524c has spotted an error in them | 00:26 |
|---|---|---|
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 821918: WIP: Switch docs theme to RTD https://review.opendev.org/c/zuul/zuul-jobs/+/821918 | 00:37 | |
| @jim:acmegating.com | and if anyone understands the pyflakes error that has developed on zuul-base-jobs, i'd appreciate the help: https://zuul.opendev.org/t/zuul/build/69267db524fc4d31a0abc24304781ee5 | 00:39 |
| @clarkb:matrix.org | corvus: I think pycodestyle is calling into pyflakes based on that traceback. I wonder if we've pinned one or the other and now they are not longer compatibile | 00:44 |
| @clarkb:matrix.org | hrm no we just install flake8, maybe its the opposite problem where we need to pin one or the other | 00:45 |
| @clarkb:matrix.org | https://github.com/PyCQA/pyflakes/issues/367 related maybe | 00:46 |
| @clarkb:matrix.org | oh wait hacking might pin flake8 | 00:48 |
| @clarkb:matrix.org | corvus: ya I think that is it, the job is installing old flake8 beacuse of hacking | 00:48 |
| @clarkb:matrix.org | and old flake8 was broken with newer python per that issue | 00:48 |
| @avass:vassast.org | corvus: I was doing that when I was working at Volvo :) | 07:53 |
| -@gerrit:opendev.org- Zuul merged on behalf of Matthieu Huin https://matrix.to/#/@mhuin:matrix.org: [zuul/zuul] 820066: Update patternfly/react-core to 4.175.11 https://review.opendev.org/c/zuul/zuul/+/820066 | 14:18 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Matthieu Huin https://matrix.to/#/@mhuin:matrix.org: | 14:35 | |
| - [zuul/zuul] 808041: [web] Pagination in builds, buildsets search https://review.opendev.org/c/zuul/zuul/+/808041 | ||
| - [zuul/zuul] 820194: REST API: add idx_min, idx_max params to getBuilds, getBuildsets https://review.opendev.org/c/zuul/zuul/+/820194 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 821918: Switch docs theme to RTD https://review.opendev.org/c/zuul/zuul-jobs/+/821918 | 14:49 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-base-jobs] 821919: Switch docs theme to RTD https://review.opendev.org/c/zuul/zuul-base-jobs/+/821919 | 14:50 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-jobs] 821918: Switch docs theme to RTD https://review.opendev.org/c/zuul/zuul-jobs/+/821918 | 15:26 | |
| @jim:acmegating.com | Albin Vass: tristanC Clark thanks! looks like we're all set on those now | 16:29 |
| @jim:acmegating.com | zuul-maint: how does this look for a zuul release? commit 230681eb98ae031d3d3c48575e2ad9e86af58b84 (tag: 4.11.0) | 16:31 |
| @jim:acmegating.com | that should be what opendev is running; it omits the pagination changes since opendev isn't running them yet | 16:32 |
| @jim:acmegating.com | okay, i'll push 4.11.0 now | 17:28 |
| @fungicide:matrix.org | yep, lgtm, sorry.... catching up from morning meetings | 17:34 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 821881: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/nodepool/+/821881 | 17:34 | |
| @fungicide:matrix.org | > <@clarkb:matrix.org> corvus: ya I think that is it, the job is installing old flake8 beacuse of hacking | 17:36 |
| are we actually relying on anything from hacking? would we be better off just dropping it? | ||
| @clarkb:matrix.org | ya might be best to drop hacking | 17:36 |
| @fungicide:matrix.org | seems like it's adding more complexity than it may be worth | 17:37 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 821881: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/nodepool/+/821881 | 17:37 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-client] 821883: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-client/+/821883 | 17:37 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-operator] 821911: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-operator/+/821911 | 17:38 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-sphinx] 821888: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-sphinx/+/821888 | 17:38 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 821917: Simplify doc conf.py https://review.opendev.org/c/zuul/zuul/+/821917 | 17:39 | |
| @jim:acmegating.com | yeah i did that in the latest zuul-base-jobs docs change | 17:39 |
| @jim:acmegating.com | we don't have hacking in zuul-jobs, so i think that's probably the best course | 17:39 |
| @jim:acmegating.com | zuul-base-jobs looks gtg if someone wants to +3 it: https://review.opendev.org/821919 | 17:40 |
| @fungicide:matrix.org | nks, taking a look | 17:40 |
| @fungicide:matrix.org | * thanks, taking a look | 17:41 |
| @jim:acmegating.com | i've also approved the docs changes for zuul itself; the others had an error preventing the version selector from appearing, so i just pushed new patchsets for those | 17:41 |
| @fungicide:matrix.org | yep, looks great! very easy on the eyes in the preview | 17:44 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-website] 822006: Remove all redirects https://review.opendev.org/c/zuul/zuul-website/+/822006 | 17:44 | |
| @jim:acmegating.com | i think we may want to land that one really soon ^ | 17:44 |
| @fungicide:matrix.org | agreed | 17:46 |
| @clarkb:matrix.org | I can help with docs reviews in a bit. Trying to finish up some thoughts on paper really quick (its that time of year) | 17:47 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-base-jobs] 821919: Switch docs theme to RTD https://review.opendev.org/c/zuul/zuul-base-jobs/+/821919 | 17:51 | |
| @clarkb:matrix.org | corvus: thinking about https://review.opendev.org/c/zuul/nodepool/+/821881/4/nodepool/version.py we might need to be careful to avoid the pkg_resources bootstrapping costs in some cases. Though maybe pbr is already hitting that and this doesn't make it worse | 18:19 |
| @clarkb:matrix.org | specifically for command line tools where you get a new process for each invocation the many millsiecond overhead for pkg_resources can be annoying if you incur it | 18:20 |
| @clarkb:matrix.org | looks like zuul-client already does it so maybe a non issue | 18:21 |
| @jim:acmegating.com | Clark: i just tested nodepool list -- .549 seconds for master and .511 seconds with that change. i'm assuming that's all within margin of error | 18:23 |
| @clarkb:matrix.org | ya should be | 18:24 |
| @clarkb:matrix.org | it is a couple hundred ms to bootstrap pkg_resources on slower disks with lots of packges installed. Also zuul doesn't have the problem of millions of deps like other projects do so that helps keep it under control | 18:24 |
| @jim:acmegating.com | the nodepool docs build still seems to be missing the version selector :/ | 18:25 |
| @clarkb:matrix.org | hrm same for zuul-client and zuul-operator? | 18:27 |
| @jim:acmegating.com | no those look good to me | 18:27 |
| @jim:acmegating.com | oh, i also just noticed an issue with the version paths... i think we need to make the paths absolute (they are relative, and that won't work for docs that are in subdirs) | 18:32 |
| @jim:acmegating.com | so 2 things: a) fix the paths, b) figure out why nodepool isn't doing the versions | 18:32 |
| @jim:acmegating.com | i guess we'll have to hard code "/docs/zuul" as the root... but maybe we can make that an env var so if people build downstream docs they can override it | 18:33 |
| @clarkb:matrix.org | docs in subdirs you mean the actual doc page itself? ya I guess that makes sense since the version thing applied to all pages | 18:34 |
| @jim:acmegating.com | yep | 18:34 |
| @jim:acmegating.com | like "howtos/badges.html" | 18:35 |
| @jim:acmegating.com | okay... so i built nodepool docs locally and got the version selector | 18:36 |
| @jim:acmegating.com | oh, ha! the zuul build hasn't returned yet for the latest patchset, i just missed that | 18:37 |
| @jim:acmegating.com | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_f38/821881/4/check/zuul-tox-docs/f382e53/docs/ is the right url, and that looks fine | 18:37 |
| @jim:acmegating.com | so i think we just need the path change | 18:37 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 821881: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/nodepool/+/821881 | 18:41 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-sphinx] 821888: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-sphinx/+/821888 | 18:42 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-client] 821883: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-client/+/821883 | 18:43 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 820585: Reorganize docs https://review.opendev.org/c/zuul/zuul/+/820585 | 18:44 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-operator] 821911: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-operator/+/821911 | 18:44 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 18:46 | |
| - [zuul/zuul] 820593: Add a version selector to docs https://review.opendev.org/c/zuul/zuul/+/820593 | ||
| - [zuul/zuul] 820915: Remove cross-project-gating howto https://review.opendev.org/c/zuul/zuul/+/820915 | ||
| - [zuul/zuul] 820916: Remove zuul-from-scratch howto https://review.opendev.org/c/zuul/zuul/+/820916 | ||
| @jim:acmegating.com | Clark: https://review.opendev.org/822006 is time sensitive | 18:47 |
| @clarkb:matrix.org | Done | 18:47 |
| @jim:acmegating.com | thx. now just waiting on updated builds of the others. i ran the nodepool build locally and the urls looked good to me. i didn't put the hostname in them, so they won't work in the preview, but we can mouseover them and see if they look right. | 18:49 |
| @jim:acmegating.com | (we could put the hostname in if we want that to work -- i was just trying to keep it minimal) | 18:50 |
| @clarkb:matrix.org | its probably fine to keep it minimal | 18:52 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-website] 822006: Remove all redirects https://review.opendev.org/c/zuul/zuul-website/+/822006 | 18:52 | |
| @clarkb:matrix.org | I guess the worst case is we publish something that doesn't work then might have to manually edit the files in afs? | 18:52 |
| @jim:acmegating.com | i think if we mess it up we can just land the patch and update latest | 18:52 |
| @jim:acmegating.com | we only have to think about editing if we tag something with a borked docs build | 18:53 |
| @jim:acmegating.com | so hopefully if there's an issue it's something we resolve almost immediately | 18:53 |
| @michiel:piscaer.com | What is the best way to install Zuul when you want to use gitlab and Openstack as a git and instance provider? I tryed to use docker, but I get some errors about: "Temporary failure in name resolution". I have removed gerrit from the docker-compose and from the zuul config. | 18:55 |
| @michiel:piscaer.com | The other way that I tryed was using the manual https://zuul-ci.org/docs/zuul/howtos/ubuntu_setup.html But this says that I need to install zookeerper but the default version in the apt repo is to old and does not support TLS. | 18:56 |
| @jim:acmegating.com | masterpe: the installation method shouldn't matter. if you aren't comfortable setting up zk, then i'd stick with the docker-compose which does that for you. i'd look into what's causing your dns resolution problems. | 18:58 |
| @jim:acmegating.com | and indeed, the doc you just linked to is mere hours from being removed from the docs since it's out of date | 18:59 |
| @michiel:piscaer.com | The problem is that docker is very new and I also do not feel very comfortable in docker. | 18:59 |
| @michiel:piscaer.com | Why remove the doc, why not bring it up to date? | 19:00 |
| @jim:acmegating.com | masterpe: ah, then if you're more comfortable installing python packages from source, feel free to do that. you just need a zookeeper, however you get it. you could install a newer version, or run only that in docker. | 19:00 |
| @jim:acmegating.com | masterpe: because there are two other places in the documentation describing how to install zuul, and people keep them up to date unlike that one. | 19:02 |
| @michiel:piscaer.com | The only two that I know is https://zuul-ci.org/docs/zuul/howtos/zuul-from-scratch.html and https://zuul-ci.org/docs/zuul/tutorials/quick-start.html | 19:04 |
| @jim:acmegating.com | https://zuul-ci.org/docs/zuul/installation.html is the installation reference | 19:10 |
| @tristanc_:matrix.org | corvus: thank you for updating the doc, I find it looks much better like that! | 19:18 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 820593: Add a version selector to docs https://review.opendev.org/c/zuul/zuul/+/820593 | 19:21 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 19:22 | |
| - [zuul/zuul] 820915: Remove cross-project-gating howto https://review.opendev.org/c/zuul/zuul/+/820915 | ||
| - [zuul/zuul] 820916: Remove zuul-from-scratch howto https://review.opendev.org/c/zuul/zuul/+/820916 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-operator] 821911: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-operator/+/821911 | 19:22 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-client] 821883: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-client/+/821883 | 19:22 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-sphinx] 821888: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-sphinx/+/821888 | 19:23 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 821881: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/nodepool/+/821881 | 19:23 | |
| @jim:acmegating.com | okay, z-o, z-s, and z-c all lgtm now and are ready for +3 | 20:23 |
| @jim:acmegating.com | waiting on the preview builds for zuul and nodepool, but i'm expecting those to be good based on the zuul-*'s | 20:23 |
| @jim:acmegating.com | fungi, tristanC if you're around, the rest of the zuul-docs changes look ready to go: https://review.opendev.org/q/hashtag:%22zuul-docs%22+status:open | 21:23 |
| @tristanc_:matrix.org | I've approved the rest of the zuul-docs changes, thanks. I'll now be afk until next year, so I wish you all a happy new year and see you in 2022! | 22:24 |
| @jim:acmegating.com | tristanC: thanks! you too! | 22:38 |
| @jim:acmegating.com | tristanC: though.. i don't see votes on the zuul-* and nodepool changes... | 22:39 |
| @tristanc_:matrix.org | oops, I may have closed the tabs too fast. it should be good now :-) Thanks again for the doc improvements! | 22:45 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-sphinx] 821888: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-sphinx/+/821888 | 22:49 | |
| @jim:acmegating.com | tristanC: understandable if you're on your way out the door for holdays :) thanks, and enjoy! | 23:01 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-client] 821883: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-client/+/821883 | 23:10 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 821881: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/nodepool/+/821881 | 23:44 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!