*** gmann_afk is now known as gmann | 00:19 | |
*** shanemcd has quit IRC | 00:40 | |
*** rlandy has quit IRC | 02:18 | |
*** saneax has quit IRC | 04:25 | |
*** ykarel has joined #zuul | 04:38 | |
*** masterpe has quit IRC | 04:45 | |
*** Eighth_Doctor has quit IRC | 04:47 | |
*** mordred has quit IRC | 04:50 | |
*** openstackstatus has quit IRC | 04:58 | |
*** openstack has joined #zuul | 05:01 | |
*** ChanServ sets mode: +o openstack | 05:01 | |
*** mordred has joined #zuul | 05:19 | |
*** Eighth_Doctor has joined #zuul | 05:23 | |
*** evrardjp has quit IRC | 05:33 | |
*** evrardjp has joined #zuul | 05:33 | |
*** saneax has joined #zuul | 05:36 | |
*** jfoufas1 has joined #zuul | 05:51 | |
*** ykarel has quit IRC | 05:55 | |
*** vishalmanchanda has joined #zuul | 06:12 | |
*** ykarel has joined #zuul | 06:12 | |
*** piotrowskim has joined #zuul | 06:56 | |
*** jcapitao has joined #zuul | 07:41 | |
*** rpittau|afk is now known as rpittau | 07:51 | |
*** hashar has joined #zuul | 07:54 | |
*** zbr|pto is now known as zbr | 08:35 | |
*** jpena|off is now known as jpena | 08:57 | |
*** tosky has joined #zuul | 09:12 | |
*** ykarel is now known as ykarel|lunch | 09:34 | |
*** hashar is now known as hasharLunch | 10:45 | |
*** ykarel|lunch is now known as ykarel | 10:53 | |
*** nils has joined #zuul | 11:18 | |
*** hasharLunch has quit IRC | 11:19 | |
*** ianychoi__ is now known as ianychoi | 11:30 | |
*** hasharLunch has joined #zuul | 11:42 | |
openstackgerrit | Oleksandr Kozachenko proposed zuul/zuul-jobs master: Update upload-logs-swift and upload-logs-gcs https://review.opendev.org/c/zuul/zuul-jobs/+/774650 | 11:44 |
---|---|---|
*** cloudnull has quit IRC | 12:02 | |
*** jcapitao is now known as jcapitao_lunch | 12:04 | |
*** cloudnull has joined #zuul | 12:05 | |
*** rlandy has joined #zuul | 12:28 | |
*** hasharLunch is now known as hashar | 12:29 | |
*** harrymichal has joined #zuul | 12:29 | |
*** jpena is now known as jpena|lunch | 12:36 | |
harrymichal | Hey folks! I have, yet again, question about the "badge" API endpoint. On softwarefactory the endpoint is still not available (despite running an update version of zuul with the supposed fix for this). Could somebody, please, look into this? | 12:39 |
*** hashar is now known as hasharAway | 12:39 | |
harrymichal | When I try to access https://softwarefactory-project.io/zuul/api/tenant/local/badge?project=containers/toolbox&pipeline=periodic&branch=master, I only get 404 response from the server. | 12:40 |
harrymichal | But when I try to access e.g., https://softwarefactory-project.io/zuul/api/tenant/local/builds, I get a valid response, therefore the URL should not be bad... | 12:41 |
avass | harrymichal: could it be that you need to urlencode the slash in 'containers/toolbox' ? | 12:49 |
*** iurygregory has quit IRC | 12:51 | |
*** jcapitao_lunch is now known as jcapitao | 13:11 | |
tristanC | harrymichal: we haven't backported the badge api fix to the stable branch, it's only available in the master version (the change is https://softwarefactory-project.io/r/#/c/20718/4/zuul.spec ) | 13:17 |
tristanC | harrymichal: we are waiting for a new zuul release to update sf-project.io services | 13:18 |
*** ykarel_ has joined #zuul | 13:24 | |
tristanC | harrymichal: it should work now | 13:27 |
harrymichal | avass: You have a point there but that did not solve it. | 13:27 |
harrymichal | tristanC: Oh! Let me check then. | 13:27 |
*** ykarel has quit IRC | 13:27 | |
harrymichal | tristanC: It works! Thank you very much! | 13:28 |
*** jpena|lunch is now known as jpena | 13:33 | |
*** ykarel_ is now known as ykarel | 13:40 | |
*** newbie2020 has joined #zuul | 13:41 | |
newbie2020 | Hi guys, | 13:46 |
newbie2020 | I am having an issue with a Zuul intallation I do not have direct control, | 13:46 |
newbie2020 | and the logs I have access to are of limited use. | 13:46 |
avass | harrymichal: no I checked in our setup and it wasn't needed | 13:46 |
newbie2020 | From the Web UI Notification I get the following error: | 13:46 |
newbie2020 | foo - undefined (undefined) | 13:46 |
newbie2020 | Zuul encountered an error while accessing the repo foo. The | 13:46 |
newbie2020 | error was: | 13:46 |
newbie2020 | string indices must be integers | 13:46 |
newbie2020 | where foo is a project apparently correctly part of the tenant. | 13:46 |
newbie2020 | The error seems your typical Python error, | 13:46 |
newbie2020 | but just looking at the logs or at a copy of the | 13:46 |
newbie2020 | tenant configuration I received I cannot tell the problem | 13:46 |
newbie2020 | Do you have by any change an hunch on what could be the issue here? | 13:46 |
tristanC | newbie2020: perhaps an issue with the foo repo configuration, could you check its definition the tenant config and in zuul.yaml? | 13:51 |
mhu | yep, this error message is returned when there's a configuration exception | 13:51 |
tristanC | `undefined (undefined)` seems to indicate the source_context.path and source_context.branch are not defined | 13:53 |
fungi | newbie2020: i want to say i've seen that problem more generally when someone incorrectly puts a string in yaml where something like an associative array is expected | 13:53 |
fungi | maybe left off a : | 13:54 |
*** cloudnull has quit IRC | 13:56 | |
newbie2020 | Thank you guys! Those are good hints that I can start looking at | 13:59 |
*** cloudnull has joined #zuul | 13:59 | |
*** ianychoi has quit IRC | 14:09 | |
*** newbie2020 has quit IRC | 14:10 | |
*** iurygregory has joined #zuul | 14:13 | |
*** hasharAway has quit IRC | 14:15 | |
*** harrymichal has quit IRC | 14:38 | |
*** hasharAway has joined #zuul | 14:46 | |
openstackgerrit | Gomathi Selvi Srinivasan proposed zuul/zuul-jobs master: Create a template for ssh-key and size https://review.opendev.org/c/zuul/zuul-jobs/+/773474 | 14:49 |
*** hasharAway is now known as hashar | 14:54 | |
openstackgerrit | Gomathi Selvi Srinivasan proposed zuul/zuul-jobs master: Create a template for ssh-key and size https://review.opendev.org/c/zuul/zuul-jobs/+/773474 | 15:08 |
openstackgerrit | Sorin Sbârnea proposed zuul/zuul-jobs master: Upgrade ansible-lint to 5.0 https://review.opendev.org/c/zuul/zuul-jobs/+/773245 | 15:38 |
*** hashar is now known as hasharAway | 15:42 | |
*** saneax has quit IRC | 15:43 | |
*** ykarel is now known as ykarel|away | 15:54 | |
*** harrymichal_ has joined #zuul | 16:05 | |
*** harrymichal_ is now known as harrymichal | 16:07 | |
openstackgerrit | Oleksandr Kozachenko proposed zuul/zuul-jobs master: Update upload-logs-swift and upload-logs-gcs https://review.opendev.org/c/zuul/zuul-jobs/+/774650 | 16:08 |
*** harrymichal_ has joined #zuul | 16:10 | |
*** harrymichal has quit IRC | 16:12 | |
*** harrymichal_ is now known as harrymichal | 16:12 | |
*** harrymichal has quit IRC | 16:15 | |
*** ykarel|away has quit IRC | 16:17 | |
*** ianw has quit IRC | 16:39 | |
*** ianw has joined #zuul | 16:39 | |
*** hasharAway has quit IRC | 16:47 | |
*** hasharAway has joined #zuul | 16:49 | |
openstackgerrit | Oleksandr Kozachenko proposed zuul/zuul-jobs master: Update upload-logs-swift and upload-logs-gcs https://review.opendev.org/c/zuul/zuul-jobs/+/774650 | 17:15 |
openstackgerrit | Oleksandr Kozachenko proposed zuul/zuul-jobs master: Update upload-logs-swift and upload-logs-gcs https://review.opendev.org/c/zuul/zuul-jobs/+/774650 | 17:35 |
*** nils has quit IRC | 17:50 | |
*** piotrowskim has quit IRC | 17:54 | |
*** hasharAway is now known as hashar | 18:00 | |
*** jpena is now known as jpena|off | 18:01 | |
*** saneax has joined #zuul | 18:13 | |
*** jfoufas1 has quit IRC | 18:15 | |
*** rpittau is now known as rpittau|afk | 18:23 | |
*** jcapitao has quit IRC | 18:39 | |
Open10K8S | Hi tobiash. Now the zuul-storage-proxy repo created https://opendev.org/zuul/zuul-storage-proxy. Is that ok to push your swift-proxy code in this repo? | 18:54 |
Open10K8S | corvus: tobiash: can you check https://review.opendev.org/c/zuul/zuul-jobs/+/774650 also? I changed according to your comments. | 18:55 |
openstackgerrit | Tobias Henkel proposed zuul/zuul-storage-proxy master: Initial import of zuul storage proxy https://review.opendev.org/c/zuul/zuul-storage-proxy/+/774942 | 19:02 |
tobiash | Open10K8S: ^ | 19:02 |
openstackgerrit | Tobias Henkel proposed zuul/zuul-storage-proxy master: Initial import of zuul storage proxy https://review.opendev.org/c/zuul/zuul-storage-proxy/+/774942 | 19:03 |
Open10K8S | tobiash: thank you | 19:08 |
openstackgerrit | Tobias Henkel proposed zuul/zuul-storage-proxy master: Initial import of zuul storage proxy https://review.opendev.org/c/zuul/zuul-storage-proxy/+/774942 | 19:09 |
tobiash | corvus: is something left to do to make zuul act on this repo? ^ | 19:11 |
corvus | checking | 19:13 |
fungi | add job configuration, yes | 19:13 |
fungi | right now it's only been added to the tenant | 19:13 |
fungi | oh, or maybe your change already does that, looking | 19:14 |
tobiash | fungi: the initial change adds that | 19:14 |
fungi | yeah, i need to restart gertty ;) | 19:14 |
tobiash | unless I don't have a typo in zuul.yaml | 19:14 |
fungi | oh! | 19:15 |
fungi | "Last reconfigured: Wed, Feb 3, 2021 11:02 PM" | 19:15 |
fungi | i think our reconfig may not be triggering, or might just not have fired yet? | 19:15 |
tobiash | ah ok | 19:15 |
fungi | that's a far more likely explanation anyway | 19:15 |
corvus | jobs failed on https://review.opendev.org/772364 | 19:15 |
tobiash | oh, timeout in post jobs | 19:16 |
tobiash | that explains it | 19:16 |
fungi | oh, interesting, it was a timeout | 19:17 |
corvus | looking at log now | 19:18 |
corvus | -> #opendev | 19:18 |
fungi | Playbook run took 0 days, 1 hours, 26 minutes, 14 seconds | 19:18 |
fungi | omw ;) | 19:19 |
openstackgerrit | Monty Taylor proposed zuul/zuul-storage-proxy master: Use opendev base docker image and add jobs https://review.opendev.org/c/zuul/zuul-storage-proxy/+/774998 | 19:24 |
* mordred makes drive-by patch while eating lunch | 19:25 | |
corvus | fungi, tobiash, Open10K8S: i think manage-projects will eventually run in a cron, or at least for some other change; it might be best to just check back in later and see if it has updated | 19:26 |
corvus | mordred: ++ thanks! | 19:27 |
fungi | yes, we're under a bit of an incident in opendev (thanks for spotting this and raising it) but can do one of several things to make the zuul deployment/reconfig happen once it settles out | 19:28 |
openstackgerrit | Tobias Henkel proposed zuul/zuul-storage-proxy master: Initial import of zuul storage proxy https://review.opendev.org/c/zuul/zuul-storage-proxy/+/774942 | 19:29 |
tobiash | mordred: awesome :) | 19:31 |
openstackgerrit | James E. Blair proposed zuul/zuul master: Use a full regex match on job implied branches https://review.opendev.org/c/zuul/zuul/+/774566 | 19:32 |
*** hashar has quit IRC | 19:33 | |
tobiash | corvus: I have a (maybe) overlapping change in review: https://review.opendev.org/c/zuul/zuul/+/640272 | 19:33 |
corvus | looking | 19:33 |
avass | yep :) | 19:34 |
tobiash | looks like we even used the same variable names :) | 19:37 |
corvus | yeah, only the test is different | 19:38 |
corvus | hah, there is a one-line ordering difference in model.py | 19:39 |
corvus | tobiash: i agree that your test validates the behavior, though it does it in a test for project-template branch matchers, which confuses the situation a little bit (mostly the test is focused on project-template implied branch matchers). i could go either way on this -- we could merge my change, or i'd be happy to merge yours (which has the advantage of not adding a new test), but i think we should add some | 19:42 |
corvus | comments explaining clearly we're testing a second behavior | 19:42 |
corvus | i'd be happy to write that comment if that's what we want to do | 19:42 |
tobiash | corvus: since yours is already in gate I'll abandon mine | 19:43 |
corvus | tobiash: you sure? i'm totally able to be swayed by "have one less test" :) | 19:43 |
corvus | i just failed to find the test that already had the substring issue in it when i wrote my change :) | 19:43 |
tobiash | corvus: I leave the decision to you which test you like better, I'm glad that this is finally fixed | 19:44 |
openstackgerrit | James E. Blair proposed zuul/zuul master: Use implied branch matcher for implied branches https://review.opendev.org/c/zuul/zuul/+/640272 | 19:50 |
corvus | tobiash: ^ there's a comment that makes me happy. i dug into the test fixtures to make sure it's testing the same thing, and i'm convinced it is. i think we should go with that. | 19:50 |
tobiash | thanks | 19:51 |
corvus | tristanC, mordred: ^ can you review 640272 which is nearly identical to https://review.opendev.org/774566 which you previously +2d ? | 19:51 |
corvus | (and sorry, that's a rebase + comment change in one patchset; i cherry-picked it to run the tests locally) | 19:52 |
tobiash | zuul reconfig happened and storage proxy jobs run now | 20:19 |
openstackgerrit | Tobias Henkel proposed zuul/zuul-storage-proxy master: Use opendev base docker image and add jobs https://review.opendev.org/c/zuul/zuul-storage-proxy/+/774998 | 20:20 |
openstackgerrit | Tobias Henkel proposed zuul/zuul-storage-proxy master: Use opendev base docker image and add jobs https://review.opendev.org/c/zuul/zuul-storage-proxy/+/774998 | 20:22 |
tobiash | mordred: fixed small issue in your drive-by patch ^ | 20:30 |
tobiash | corvus: there qre several retry limits in zuul's jobs: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_e8f/772602/3/gate/zuul-stream-functional-2.8/e8f735e/job-output.txt | 20:33 |
tobiash | we had the same issue installing cryptogrphy in our system | 20:34 |
tobiash | older pip don't use the abi3 wheels and use source installation which now needs a rust compiler | 20:34 |
openstackgerrit | Tobias Henkel proposed zuul/zuul master: Upgrade pip before installing zuul https://review.opendev.org/c/zuul/zuul/+/775013 | 20:39 |
tobiash | let's see if this fixes the zuul-stream-functional jobs | 20:39 |
openstackgerrit | Tobias Henkel proposed zuul/nodepool master: Upgrade pip before installing zuul https://review.opendev.org/c/zuul/nodepool/+/775017 | 20:44 |
tobiash | corvus: those two should fix the zuul gate, live logs show successful installation of cryptography ^ | 20:55 |
tristanC | tobiash: i guess we are now ok to depends on rust dependencies for zuul? | 21:11 |
tobiash | tristanC: we can use the precompiled wheels | 21:12 |
tobiash | tristanC: which depebdency do you have in mind? | 21:13 |
tobiash | (despite cryptography) | 21:13 |
tristanC | tobiash: the orjson library for example | 21:14 |
tobiash | tristanC: if that has precompiled wheels I guess yes | 21:14 |
corvus | looks good to me; i think the main thing is it will preclude pypy, but i don't think there's a lot of interest in trying to get zuul to run in pypy | 21:15 |
tobiash | corvus: well, I spent some time since that promised to improve our main loop performance | 21:16 |
corvus | could really come in handy for the zk work, where we're (de)serializing json a lot | 21:16 |
tobiash | but pypy needs a reimplementation of the re2 wrapper using cffi | 21:17 |
tobiash | but ++ for a faster json lib, based on profiling a while ago nodepool could also be improved by a faster json lib | 21:18 |
corvus | tobiash: do you think pypy is worth keeping in mind? i mean, we can try to minimize native dependencies; or we could look at making orjson optional in the same way that the c yaml lib is optional | 21:18 |
tobiash | corvus: I think currently the only blocker for pypy is re2 | 21:19 |
corvus | (and maybe now cryptography?) | 21:19 |
tobiash | so if we could get rid of this blocker at some time pypy still would be an option we could explore | 21:19 |
tobiash | need to check if/how cryptography is compatible with pypy | 21:20 |
corvus | okay, so it's probably worth keeping that in mind -- if we can just use orjson as a drop-in replacement that keeps our options open | 21:20 |
corvus | i think that should be easy, because our usage of json is pretty simple. | 21:21 |
tobiash | there is an open issue for cryptography on pypy: https://github.com/pyca/cryptography/issues/5806 | 21:22 |
tobiash | hrm, according to that issue pypy works when having a rust compiler | 21:23 |
tobiash | so that doesn't seem to be a problem other than needing the rust compiler at installation time | 21:23 |
tobiash | but orjson explicitly states that it's nit compatible to pypy | 21:24 |
tristanC | tobiash: wheels are convenients when available, but it seems important to be able to build from source too | 21:25 |
tobiash | tristanC: I guess build from source will always be possible giben a rust compiler, however I think we should require wheels to be available when adding new dependencies | 21:27 |
clarkb | yes I think cryptography is expected to work with pypi you just need the build deps | 21:28 |
clarkb | (no pypy wheels) | 21:28 |
tristanC | tobiash: fair enough, it doesn't seem like we have much of a choice with cryptography, and i'm happy to see such language being used, but that is quite a change | 21:32 |
tobiash | corvus, tristanC: I just found a new re2 wrapper using pybind11 under the hood instead of native cpython bindings: https://pypi.org/project/google-re2/ | 21:33 |
tobiash | so that could enable exploration of pypy | 21:34 |
tobiash | at least pybind11 claims compatibility with pypy | 21:34 |
corvus | tobiash: nice. btw, i like pypy, but i'm worried that it could be risky and uncover new edge cases or otherwise not behave as well. just a word of caution. :) | 21:35 |
tobiash | corvus: I won't throw it straight into production :) | 21:35 |
tobiash | but I guess I'll see if it works in principal and if it does try it out in our staging env | 21:36 |
*** andy-ladjadj has joined #zuul | 21:39 | |
*** andy-ladjadj has quit IRC | 21:40 | |
*** andy-ladjadj has joined #zuul | 21:40 | |
openstackgerrit | Tobias Henkel proposed zuul/zuul master: DNM: Try out google-re2 instead of fb-re2 https://review.opendev.org/c/zuul/zuul/+/775028 | 21:41 |
corvus | google vs fb: fight! | 21:41 |
tobiash | :) | 21:41 |
tobiash | the package name seems to be the same so it might be just a matter of the dependency (and successful installation of it) | 21:42 |
openstackgerrit | Tobias Henkel proposed zuul/zuul master: DNM: Try pypy https://review.opendev.org/c/zuul/zuul/+/775029 | 21:48 |
*** andy-ladjadj has quit IRC | 21:48 | |
*** andy-ladjadj has joined #zuul | 21:48 | |
*** andy-ladjadj has quit IRC | 21:50 | |
*** jamesmcarthur has joined #zuul | 21:51 | |
openstackgerrit | Tobias Henkel proposed zuul/zuul master: DNM: Try out google-re2 instead of fb-re2 https://review.opendev.org/c/zuul/zuul/+/775028 | 21:55 |
openstackgerrit | Tobias Henkel proposed zuul/zuul master: DNM: Try pypy https://review.opendev.org/c/zuul/zuul/+/775029 | 21:55 |
tobiash | missing dep, fb won, next round | 21:55 |
*** andy-ladjadj has joined #zuul | 22:12 | |
openstackgerrit | Merged zuul/zuul master: Upgrade pip before installing zuul https://review.opendev.org/c/zuul/zuul/+/775013 | 22:15 |
openstackgerrit | Merged zuul/zuul master: Use implied branch matcher for implied branches https://review.opendev.org/c/zuul/zuul/+/640272 | 22:21 |
corvus | guillaumec: re https://review.opendev.org/771733 discussion from #openstack-infra -- we probably should have the version check for http then, but since zuul added support for http after that landed, it's unlikely anyone ran into a problem with it. | 22:22 |
corvus | guillaumec: but it's still the case that if someone is running 2.12 with ssh, this could break them. | 22:23 |
corvus | guillaumec: so in addition to the 2 options i left in review, how about a third option: we ask if anyone is using <2.13 and if no one replies, we merge it as is? basically, find out what our minimum gerrit version is | 22:24 |
clarkb | qualcomm has stated they are on 2.7 on the gerrit mailing list | 22:25 |
clarkb | I have no idea if they use zuul, but there are definitely some old gerrit still out there | 22:25 |
corvus | neat | 22:25 |
corvus | well, i'd like it if they did use zuul | 22:25 |
avass | corvus: I happen to know of people (not volvo) using gerrit <2.13 who has shown interest in zuul. but no clue if they will actually use it :) | 22:26 |
corvus | guillaumec: how about we just add the version check in both places? | 22:26 |
guillaumec | corvus, was already doing it :) | 22:26 |
corvus | cool :) | 22:26 |
*** andy-ladjadj has quit IRC | 22:36 | |
*** jamesmcarthur has quit IRC | 22:41 | |
*** jamesmcarthur has joined #zuul | 22:42 | |
*** andy-ladjadj has joined #zuul | 22:43 | |
*** andy-ladjadj has quit IRC | 22:44 | |
*** jamesmcarthur has quit IRC | 22:44 | |
*** jamesmcarthur has joined #zuul | 22:45 | |
*** vishalmanchanda has quit IRC | 22:51 | |
openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: gerrit: Add gerrit version check for HTTP review tag https://review.opendev.org/c/zuul/zuul/+/775037 | 22:53 |
openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Gerrit: Add SSH review tags https://review.opendev.org/c/zuul/zuul/+/771733 | 22:54 |
openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Gerrit: Add SSH review tag https://review.opendev.org/c/zuul/zuul/+/771733 | 23:01 |
openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Gerrit: Add gerrit version check for HTTP review tag https://review.opendev.org/c/zuul/zuul/+/775037 | 23:01 |
*** andy-ladjadj has joined #zuul | 23:04 | |
*** andy-ladjadj has quit IRC | 23:07 | |
corvus | guillaumec: thanks! | 23:11 |
openstackgerrit | James E. Blair proposed zuul/zuul master: Add spec for building Nodepool images in Zuul https://review.opendev.org/c/zuul/zuul/+/775042 | 23:12 |
*** rlandy has quit IRC | 23:30 | |
openstackgerrit | Merged zuul/zuul master: Add waiting cause to status API https://review.opendev.org/c/zuul/zuul/+/773218 | 23:43 |
clarkb | corvus: iurygregory you approved the ssh change any reason to not approve https://review.opendev.org/c/zuul/zuul/+/775037 this http one? | 23:53 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!