| *** Shrews <Shrews!~shrews@2603-6080-660e-4ebc-6951-1fc2-b1a5-39f2.res6.spectrum.com> has joined #zuul | 00:30 | |
| *** Shrews <Shrews!~shrews@2603-6080-660e-4ebc-6951-1fc2-b1a5-39f2.res6.spectrum.com> has quit IRC () | 00:34 | |
| *** bhagyashris__ <bhagyashris__!~bhagyashr@103.98.79.86> has joined #zuul | 01:51 | |
| *** zenkuro <zenkuro!~zenkuro@37.214.28.122> has quit IRC (Read error: Connection reset by peer) | 01:57 | |
| *** bhagyashris_ <bhagyashris_!~bhagyashr@103.98.79.86> has quit IRC (Ping timeout: 480 seconds) | 01:58 | |
| *** marios <marios!~marios@62-65-73.netrun.cytanet.com.cy> has joined #zuul | 05:08 | |
| *** marios is now known as marios|ruck | 05:10 | |
| *** test01002[m] <test01002[m]!~test01002@2001:470:1af1:101::87f5> has joined #zuul | 05:29 | |
| *** frickler <frickler!~jens@0002014f.user.oftc.net> has left #zuul | 05:33 | |
| *** iurygregory_ <iurygregory_!~iurygrego@2a02:8308:a007:3900:93ab:1c8f:2b9e:9349> has joined #zuul | 06:07 | |
| *** iurygregory|afk <iurygregory|afk!~iurygrego@ip-86-49-241-46.net.upcbroadband.cz> has quit IRC (Read error: Connection reset by peer) | 06:07 | |
| *** icey <icey!~icey@0001bbad.user.oftc.net> has quit IRC () | 06:18 | |
| *** icey <icey!~icey@0001bbad.user.oftc.net> has joined #zuul | 06:18 | |
| *** bhagyashris__ is now known as bhagyashris | 06:36 | |
| *** holser <holser!~sgolovat@d167-142.icpnet.pl> has quit IRC (Remote host closed the connection) | 07:00 | |
| *** holser <holser!~sgolovat@d167-142.icpnet.pl> has joined #zuul | 07:01 | |
| *** rpittau|afk is now known as rpittau | 07:07 | |
| *** bhavikdbavishi <bhavikdbavishi!~Thunderbi@74.217.76.101> has joined #zuul | 07:12 | |
| *** hashar <hashar!~hashar@hashar.user.oftc.net> has joined #zuul | 07:27 | |
| *** bhavikdbavishi <bhavikdbavishi!~Thunderbi@74.217.76.101> has quit IRC (Remote host closed the connection) | 07:45 | |
| *** tosky <tosky!~luigi@dynamic-adsl-78-13-253-141.clienti.tiscali.it> has joined #zuul | 07:45 | |
| *** jpena|off is now known as jpena | 08:35 | |
| *** zenkuro <zenkuro!~zenkuro@37.214.28.58> has joined #zuul | 09:51 | |
| avass[m] | mnaser: https://github.com/curl/curl-docker wants to set up zuul too, can you add them to the tenant config? :D | 10:04 |
|---|---|---|
| opendevreview | Matthieu Huin proposed zuul/zuul master: Import collections.Mapping correctly https://review.opendev.org/c/zuul/zuul/+/796254 | 10:20 |
| opendevreview | Merged zuul/nodepool master: Azure: delete disk based on VM reference https://review.opendev.org/c/zuul/nodepool/+/779897 | 10:21 |
| *** mhuin <mhuin!~mhuin@89-159-62-32.rev.numericable.fr> has joined #zuul | 10:38 | |
| *** mhu1 <mhu1!~mhuin@89-159-62-32.rev.numericable.fr> has quit IRC (Ping timeout: 480 seconds) | 10:41 | |
| *** jpena is now known as jpena|lunch | 11:04 | |
| mnaser | avass[m]: oh! https://github.com/vexxhost/zuul-config-curl :) | 11:20 |
| mnaser | also in other awesome news, zuul is running for curl/curl !!’ | 11:24 |
| *** mgoddard- <mgoddard-!~mgoddard@240.240.125.91.dyn.plus.net> has joined #zuul | 11:35 | |
| *** mgoddard <mgoddard!~mgoddard@238.240.125.91.dyn.plus.net> has quit IRC (Ping timeout: 480 seconds) | 11:38 | |
| *** mgoddard- is now known as mgoddard | 11:38 | |
| *** jpena|lunch is now known as jpena | 11:49 | |
| *** josefwells <josefwells!~josefwell@cpe-70-112-68-60.austin.res.rr.com> has joined #zuul | 12:16 | |
| *** erlon <erlon!uid7593@id-7593.brockwell.irccloud.com> has joined #zuul | 12:18 | |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Tests: add non-voting unit testing for Python 3.10 https://review.opendev.org/c/zuul/zuul/+/796270 | 12:19 |
| mhuin | tristanC, regarding https://review.opendev.org/c/zuul/zuul/+/796254 I found this in the doc: | 12:28 |
| mhuin | "Deprecated since version 3.3, will be removed in version 3.10: Moved Collections Abstract Base Classes to the collections.abc module. For backwards compatibility, they continue to be visible in this module through Python 3.9." | 12:29 |
| mhuin | https://docs.python.org/3/library/collections.html | 12:29 |
| tristanC | mhuin: ok thank you for the detail | 12:30 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: [DNM] Tests: add non-voting unit testing for Python 3.10 https://review.opendev.org/c/zuul/zuul/+/796270 | 12:42 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: [DNM] Tests: add non-voting unit testing for Python 3.10 https://review.opendev.org/c/zuul/zuul/+/796270 | 12:45 |
| tobiash[m] | mhuin, tristanC can't we just use the new location of that? | 13:10 |
| tobiash[m] | looking at the docs, the mapping class is already in collections.abc since at least py35 so I don't see the need for a try import dance | 13:11 |
| mhuin | tobiash[m], true, there was already some collections.abc imports in the code as well | 13:11 |
| fungi | yeah, i moved one of my personal projects over to just directly relying on collections.abc | 13:11 |
| fungi | as soon as i dropped support for python<3.5 | 13:12 |
| fungi | though on that project i set envvars to treat deprecation warnings as errors so i catch them early | 13:12 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Import collections.Mapping correctly https://review.opendev.org/c/zuul/zuul/+/796254 | 13:13 |
| fungi | tradeoff is you end up with byzantine warning filters to ignore deprecations which affect your dependencies, so it's not all that easy to set up or maintain, and it breaks often when your dependencies get new versions | 13:13 |
| mhuin | tobiash[m], ^ done | 13:14 |
| mhuin | I'll try and add non-voting testing for python 3.10 as well. At least in Fedora 35, it's going to be the default version | 13:14 |
| fungi | i've been testing that aforementioned personal project with 3.10 as well, since it was in beta, i generally keep python-next updated on my workstation every time they tag a new version | 13:16 |
| mhuin | fungi, how do you install 3.10? do you use a PPA? | 13:17 |
| mhuin | (assuming you're testing on ubuntu) | 13:17 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: [DNM] Tests: add non-voting unit testing for Python 3.10 https://review.opendev.org/c/zuul/zuul/+/796270 | 13:18 |
| fungi | er, since it was in alpha i mean | 13:18 |
| fungi | mhuin: i use debian, yeah, but i compile python locally from git source | 13:18 |
| fungi | i make altinstall into ~/lib/... and then symlink the interpreter from ~/bin/ | 13:19 |
| fungi | so that all of them are in my path | 13:19 |
| avass[m] | there's also pyenv that does that for you pretty much | 13:20 |
| fungi | yeah, i'm doing effectively the same thing, though i like having better control over automake options and compiler optimizations | 13:20 |
| mhuin | avass[m], I tried in a previous iteration of ^ with pyenv but for some reason it pulled python 3.11 instead of 3.10 | 13:21 |
| mhuin | at this point in time they're practically the same version but I wanted 3.10 :) | 13:21 |
| avass[m] | mhuin: I think you should be able to specify what version you want with pyenv | 13:21 |
| avass[m] | "pyenv install 3.10.0" | 13:21 |
| fungi | with that method i'm currently using 3.5.10, 3.6.13, 3.7.10, 3.8.10, 3.9.5 and 3.10.0b2 | 13:22 |
| mhuin | I'm trying with a PPA now, but I do suspect I dodn't set the "python_version" var correctly for the ensure-python pre job | 13:23 |
| zbr | is it possible to have zuul config in-repo project config split in two different files? (and have zuul merge jobs declared in each) - like splitting layout.yaml in two. | 13:23 |
| avass[m] | yeah I found 3.10.0b2 with pyenv | 13:23 |
| avass[m] | zbr: you mean what zuul-jobs is doing? | 13:24 |
| avass[m] | zbr: zuul will merge any config found in zuul.d, .zuul.d or any extra config paths: https://review.opendev.org/plugins/gitiles/zuul/zuul-jobs/+/refs/heads/master/zuul-tests.d/ | 13:25 |
| fungi | git checkout v3.10.0b2 && ./configure --enable-optimizations --with-lto --prefix ~/lib/cpython/3.10.0b2 && make -j32 && make -j32 test && rm -rf ~/lib/cpython/3.10.* && make altinstall && ln -sf ../lib/cpython/3.10.0b2/bin/python3.10 ~/bin/ | 13:25 |
| fungi | that's basically what my 3.10 install command looks like | 13:26 |
| fungi | not much to it | 13:26 |
| fungi | i install all the build deps from distro packages, so i'm only recompiling the interpreter/stdlib | 13:26 |
| zbr | avass[m]: i knew that some merging was possible but I was not sure if "project:" key is effectively merged. | 13:27 |
| avass[m] | zbr: it works with any ites | 13:27 |
| avass[m] | * items | 13:27 |
| avass[m] | mhuin: if you want to make it reproducible you can probably also create an overlay for the nix python package to build 3.10.0b2 instead of 3.9 ;) | 13:29 |
| zbr | super! i can separate the integration jobs from the tox ones. with some luck altering one file will avoid triggering the other jobs | 13:29 |
| tristanC | avass[m]: should we add a ensure-nix role first? :-) | 13:49 |
| avass[m] | tristanC: I'm planning on getting to that once I find some time to move my cloud setup to my own server :) | 13:51 |
| avass[m] | also: https://daniel.haxx.se/blog/2021/06/14/bye-bye-travis-ci/ \o/ | 13:51 |
| mordred | woohoo! | 14:02 |
| corvus | zbr: note that if you're using file matchers, you shouldn't need to include the zuul.yaml files in the matcher, as zuul will auto-detect if a job has changed and run it. | 14:04 |
| zbr | corvus: i know that but i got some cases where all the jobs run, even if we only added a new one. Maybe that was due to other condition that slipped in..? | 14:06 |
| fungi | avass[m]: i'm curious to see how they end up dividing job workload between zuul and circle-ci | 14:06 |
| corvus | avass, mnaser: yay! :) | 14:06 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: [DNM] Tests: add non-voting unit testing for Python 3.10 https://review.opendev.org/c/zuul/zuul/+/796270 | 14:07 |
| avass[m] | fungi: I think you can see it on the graph in that post :) | 14:23 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: [DNM] Tests: add non-voting unit testing for Python 3.10 https://review.opendev.org/c/zuul/zuul/+/796270 | 14:29 |
| fungi | avass[m]: well, i mean assuming the circle-ci work is still in progress | 14:37 |
| avass[m] | fungi: travis was running 33 jobs and zuul took 30. So I guess at most circleCI will run the remaining three | 14:38 |
| avass[m] | the last three were the non amd arch jobs | 14:39 |
| corvus | that appears to match the graph | 14:39 |
| fungi | ahh, okay | 14:39 |
| avass[m] | my next step is to support curl/curl-docker, so zuul will still do a bit more work :) | 14:40 |
| avass[m] | mnaser: any clue why https://github.com/curl/curl-docker/pull/57 doesn't trigger anything? | 15:05 |
| corvus | avass: i don't see a project stanza in https://github.com/vexxhost/zuul-config-curl/tree/main/zuul.d so the project may not be attached to the pipeline? | 15:11 |
| avass[m] | corvus: there's a project stanza in: https://github.com/curl/curl-docker/pull/57/files so I guess there's a config error that's not being reported. but I don't see it | 15:12 |
| mnaser | avass[m]: possible that master is not protected branch? | 15:13 |
| avass[m] | corvus: actually I see it | 15:13 |
| corvus | avass: the zuul ignore thing for playbooks? | 15:13 |
| avass[m] | yeah | 15:13 |
| avass[m] | mnaser: it is indeed not a protected branch | 15:14 |
| opendevreview | Merged zuul/zuul master: Import collections.Mapping correctly https://review.opendev.org/c/zuul/zuul/+/796254 | 15:21 |
| mnaser | avass[m]: could we maybe ask to have the branch protected? | 15:34 |
| avass[m] | mnaser: already did :) | 15:34 |
| mnaser | oh cool! yay | 15:34 |
| avass[m] | https://github.com/curl/curl-docker/pull/57 | 15:35 |
| *** marios|ruck is now known as marios|out | 15:49 | |
| corvus | zuul-maint: how does this look for a release? commit d72ddd01e35fe42f83ed3a580b27858cefa65eb5 (HEAD -> master, tag: 4.5.1, origin/master, refs/changes/54/796254/2) | 16:04 |
| tobiash[m] | lgtm | 16:04 |
| fungi | corvus: is this primarily to get 794027 included ahead of the impending security release? | 16:06 |
| *** rpittau is now known as rpittau|afk | 16:08 | |
| corvus | fungi: yep, tristanC said they'd have to carry that patch; plus it's a regression, so seems like a good thing to do | 16:09 |
| corvus | we made sure there's a reno for it too, so it should be ready to go: https://zuul-ci.org/docs/zuul/reference/releasenotes.html#bug-fixes | 16:09 |
| fungi | yeah, the proposed release looks good to me | 16:11 |
| clarkb | that commit is what I see as the master sha1. And commits between it and 4.5.1 appear to not warrant a 4.6.0 | 16:11 |
| clarkb | lgtm | 16:11 |
| fungi | i guess we just need to be careful with our revised messaging about the upcoming security release | 16:11 |
| mordred | ++ | 16:11 |
| corvus | fungi: revised messaging? | 16:12 |
| fungi | corvus: i assume there will be a release announcement for 4.5.1 at least | 16:13 |
| fungi | do we continue to recommend that deployments upgrade to 4.5.0 now that there's a 4.5.1, or do we want to get new communications circulating recommending 4.5.1 instead of 4.5.0? just wondering what will be least confusing for current deployers | 16:14 |
| fungi | which they install ultimately doesn't matter unless like sf they need 794027, but there exists the opportunity for people to get confused about version numbers there | 16:15 |
| corvus | fungi: i don't think it's a big deal; run 4.5.0 or 4.5.1; it won't matter. let me draft something and see what you think | 16:15 |
| fungi | corvus: cool, yeah mainly just asking that we clarify the situation in the release announcement | 16:16 |
| corvus | fungi: https://etherpad.opendev.org/p/adOVJqVMtW_uEdJD2Gf7 ? | 16:19 |
| * corvus made a ninja edit to the last sentence | 16:20 | |
| fungi | no worries, i hadn't pulled it up yet, looking now | 16:20 |
| fungi | corvus: yeah, looks great. thanks! | 16:21 |
| fungi | and i think other comms can probably just ignore the existence of 4.5.1 for now | 16:22 |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 16:22 | |
| corvus | sounds like a plan then; i'll push the tag | 16:23 |
| *** jpena is now known as jpena|off | 16:43 | |
| mnaser | avass[m]: i like what you mentioned the other day wrt all the different jobs reporting + a 'meta' check one | 17:37 |
| avass[m] | mnaser: I found that azure pipelines does extactly that: https://github.com/curl/curl/pull/7245/checks | 17:39 |
| corvus | yeah, we've discussed that solution a few times in here; it's promising | 17:40 |
| mnaser | avass[m], corvus: i wish i had more time :'( | 17:55 |
| opendevreview | James E. Blair proposed zuul/zuul master: Fix typo in matrix spec https://review.opendev.org/c/zuul/zuul/+/796317 | 20:10 |
| corvus | tristanC: http://lists.zuul-ci.org/pipermail/zuul-announce/2021-June/000095.html | 20:16 |
| tristanC | corvus: thanks! | 20:17 |
| corvus | ty | 20:18 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: [DNM] Tests: add non-voting unit testing for Python 3.10 https://review.opendev.org/c/zuul/zuul/+/796270 | 20:51 |
| mordred | corvus: should I hold off on +A of the zuul matrix docs for more reviews? | 22:29 |
| corvus | mordred: i think it's gtg | 22:30 |
| mnaser | corvus: looks like we got a shout here https://daniel.haxx.se/blog/2021/06/14/bye-bye-travis-ci/ :) | 23:32 |
| fungi | yep, avass[m] linked that back around 13:51 utc but it's great to mention again | 23:34 |
| mnaser | ah woops :) | 23:35 |
| mnaser | i'm behind all day today! | 23:35 |
| fungi | no, it was good to revisit because that gave me an opportunity to read the comments which had been made after i originally looked at it | 23:36 |
| opendevreview | Merged zuul/zuul master: Add Matrix HOWTO https://review.opendev.org/c/zuul/zuul/+/796153 | 23:43 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!