| *** jamesmcarthur has joined #zuul | 00:03 | |
| *** jamesmcarthur has quit IRC | 00:08 | |
| *** mattw4 has quit IRC | 00:12 | |
| *** jamesmcarthur has joined #zuul | 00:37 | |
| clarkb | pabelanger: can you see my comment on https://review.opendev.org/#/c/694390/2 I wonder if the thing I've identified contributes to the too greedy github searching | 00:42 |
|---|---|---|
| *** jamesmcarthur has quit IRC | 00:42 | |
| *** sshnaidm has quit IRC | 00:45 | |
| pabelanger | clarkb: ack, will look more tomorrow | 00:49 |
| *** sgw has quit IRC | 00:56 | |
| *** sshnaidm has joined #zuul | 01:00 | |
| *** Goneri has quit IRC | 01:00 | |
| openstackgerrit | Clark Boylan proposed zuul/zuul master: Improve functionality and docs around ansible installation https://review.opendev.org/675403 | 01:06 |
| clarkb | tobiash_: corvus ^ addressed your recentish comments on that one. Thanks | 01:06 |
| *** igordc has quit IRC | 01:13 | |
| *** michael-beaver has quit IRC | 01:18 | |
| *** sgw has joined #zuul | 01:21 | |
| *** rlandy|bbl is now known as rlandy | 01:36 | |
| *** jamesmcarthur has joined #zuul | 01:39 | |
| *** jamesmcarthur has quit IRC | 01:43 | |
| *** jamesmcarthur has joined #zuul | 02:10 | |
| *** jamesmcarthur has quit IRC | 02:15 | |
| *** rlandy has quit IRC | 02:22 | |
| *** EmilienM|PTO is now known as EmilienM | 02:42 | |
| *** jamesmcarthur has joined #zuul | 02:52 | |
| *** jamesmcarthur has quit IRC | 02:57 | |
| *** bhavikdbavishi has joined #zuul | 03:14 | |
| *** bhavikdbavishi1 has joined #zuul | 03:17 | |
| *** bhavikdbavishi has quit IRC | 03:18 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 03:18 | |
| openstackgerrit | Paul Belanger proposed zuul/zuul-jobs master: WIP: Print extra debug info https://review.opendev.org/696013 | 03:21 |
| *** jamesmcarthur has joined #zuul | 03:29 | |
| *** jamesmcarthur has quit IRC | 03:34 | |
| *** sshnaidm_ has joined #zuul | 03:53 | |
| *** sshnaidm has quit IRC | 03:57 | |
| *** saneax has quit IRC | 04:09 | |
| *** sshnaidm__ has joined #zuul | 04:25 | |
| *** sshnaidm_ has quit IRC | 04:26 | |
| *** jamesmcarthur has joined #zuul | 04:31 | |
| *** jamesmcarthur has quit IRC | 04:35 | |
| *** saneax has joined #zuul | 05:05 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Use RDO trunk repos work for openvswitch on centos8 https://review.opendev.org/695833 | 05:40 |
| openstackgerrit | Merged zuul/zuul-jobs master: update-test-platforms.py : handle non-voting jobs https://review.opendev.org/695830 | 05:40 |
| openstackgerrit | Merged zuul/zuul-jobs master: Make opensuse-15 job voting again https://review.opendev.org/695831 | 05:41 |
| *** raukadah is now known as chkumar|rover | 05:41 | |
| *** adam_g has quit IRC | 05:55 | |
| *** adam_g has joined #zuul | 05:55 | |
| *** jamesmcarthur has joined #zuul | 06:07 | |
| *** jamesmcarthur has quit IRC | 06:12 | |
| *** threestrands has quit IRC | 06:29 | |
| *** pcaruana has joined #zuul | 06:32 | |
| *** tosky has joined #zuul | 08:25 | |
| *** bolg has joined #zuul | 08:27 | |
| *** saneax has quit IRC | 08:36 | |
| *** saneax has joined #zuul | 08:37 | |
| *** themroc has joined #zuul | 08:39 | |
| *** jpena|off is now known as jpena | 08:44 | |
| *** hashar has joined #zuul | 09:27 | |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Add optional support for circular dependencies https://review.opendev.org/685354 | 09:31 |
| *** SotK has quit IRC | 09:37 | |
| *** SotK has joined #zuul | 09:46 | |
| *** mhu has joined #zuul | 09:51 | |
| *** bhavikdbavishi has quit IRC | 10:00 | |
| *** electrofelix has joined #zuul | 10:09 | |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Keep task stdout/stderr separate in result object https://review.opendev.org/650276 | 10:11 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Align template formating for reporters https://review.opendev.org/643306 | 10:18 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: authentication config: add optional token_expiry https://review.opendev.org/642408 | 11:15 |
| *** sshnaidm__ is now known as sshnaidm | 11:15 | |
| *** electrofelix has quit IRC | 11:43 | |
| *** electrofelix has joined #zuul | 11:43 | |
| *** bolg has quit IRC | 11:49 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Authorization rules: support YAML nested dictionaries https://review.opendev.org/684790 | 12:05 |
| *** rfolco has joined #zuul | 12:05 | |
| *** bhavikdbavishi has joined #zuul | 12:33 | |
| *** Goneri has joined #zuul | 12:53 | |
| zbr_ | can someone look at this bug https://storyboard.openstack.org/#!/story/2006830 ? | 12:57 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: zuul_stream: handle module that emit msg as a list https://review.opendev.org/696081 | 13:05 |
| tristanC | zbr_: ^ may fixes that story | 13:06 |
| *** rlandy has joined #zuul | 13:11 | |
| zbr_ | tributarian: haha! thanks. I was working with ykarel to write a fix for this but you are faster | 13:11 |
| zbr_ | tristanC: ^ | 13:11 |
| zbr_ | tristanC: what do you think about alternative https://stackoverflow.com/a/11301392/99834 ? | 13:16 |
| zbr_ | list is only one of the iterables that are not strings | 13:16 |
| zbr_ | probably it would fail even with a tuple | 13:17 |
| tristanC | zbr_: because of json, i think list is enough. | 13:18 |
| *** bhavikdbavishi1 has joined #zuul | 13:19 | |
| zbr_ | tristanC: yeah. if it passed CI, we should merge it. i want it fixed asap, faced it too many times in the last weeks. | 13:20 |
| zbr_ | i hate having to go into JSON to see the error. | 13:20 |
| tristanC | zbr_: do you know if we could reproduce the issue in zuul tests? | 13:20 |
| zbr_ | package module trying to install missing rpm on ansible 2.8 with python3. | 13:21 |
| *** bhavikdbavishi has quit IRC | 13:21 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 13:21 | |
| zbr_ | tristanC: look at https://logs.rdoproject.org/26/23826/4/check/tripleo-podman-integration-master-centos-8/bcdc131/job-output.json -- search for "Failed to install" | 13:23 |
| zbr_ | this should be a test case. | 13:23 |
| zbr_ | not sure if this is a bug in ansible, but either way we need to cope with it. | 13:23 |
| zbr_ | i am looking now at tests but I am not familiar with zuul codebase. it would be very good idea to add a unit test for this result, to avoid regressions. | 13:26 |
| *** jpena is now known as jpena|lunch | 13:26 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: zuul_stream: handle module that emit non str msg https://review.opendev.org/696081 | 13:28 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Keep task stdout/stderr separate in result object https://review.opendev.org/650276 | 13:30 |
| *** rfolco has quit IRC | 13:47 | |
| *** rfolco has joined #zuul | 13:49 | |
| *** rfolco has quit IRC | 13:50 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: enqueue: make trigger optional https://review.opendev.org/695446 | 14:14 |
| *** jpena|lunch is now known as jpena | 14:18 | |
| *** rfolco has joined #zuul | 14:28 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: wip: add cleanup-timeout job attribute https://review.opendev.org/696098 | 14:34 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Spec for allowing circular dependencies https://review.opendev.org/643309 | 14:35 |
| *** Goneri has quit IRC | 14:41 | |
| openstackgerrit | Paul Belanger proposed zuul/zuul-jobs master: Update download-artifact to use zuul.artifacts https://review.opendev.org/696013 | 14:47 |
| ttx | Hi Zuul-ites... I got a cryptic error trying to define a new experimental job for openstack/releases: <<unhashable type: 'list'>> without any line number on https://review.opendev.org/#/c/696095/ | 14:51 |
| *** swest has quit IRC | 14:51 | |
| ttx | The only borderline thing I'm doing there is asking it to run locally on the executor... so is this error Zuul's way of telling me doing that in a project-local configuration is not allowed? Documentation around that is pretty scarce | 14:52 |
| *** swest has joined #zuul | 14:52 | |
| *** Goneri has joined #zuul | 14:53 | |
| ttx | That nodeset stanza is coherent with what's done at https://opendev.org/openstack/project-config/src/branch/master/zuul.d/jobs.yaml#L1055-L1056 for example | 14:53 |
| openstackgerrit | Felix Schmidt proposed zuul/zuul master: Make reporting asynchronous https://review.opendev.org/691253 | 14:54 |
| openstackgerrit | Felix Schmidt proposed zuul/zuul master: Make direct-push configurable on project-level https://review.opendev.org/677109 | 14:54 |
| openstackgerrit | Felix Schmidt proposed zuul/zuul master: Implement push job in merger https://review.opendev.org/677110 | 14:54 |
| openstackgerrit | Felix Schmidt proposed zuul/zuul master: Push changes in GerritReporter if direct-push is enabled https://review.opendev.org/677111 | 14:54 |
| pabelanger | ttx: try using "" around {{ zuul.change }} | 14:55 |
| pabelanger | zuul cannot parse jinja2 | 14:56 |
| pabelanger | wonder if that needs to be string | 14:56 |
| ttx | ok let's try that | 14:57 |
| pabelanger | if that doesn't work, you could ask for more debug info using https://zuul-ci.org/docs/zuul/user/config.html#attr-project.%3Cpipeline%3E.debug | 14:58 |
| ttx | It did not immediately barf on me so I suspect that was it | 14:58 |
| pabelanger | yay | 14:58 |
| *** swest has quit IRC | 15:01 | |
| ttx | that is what I get for starting to code it as an Ansible role before realizing I could do it purely in Zuul | 15:01 |
| openstackgerrit | Paul Belanger proposed zuul/zuul-jobs master: Update download-artifact to use zuul.artifacts https://review.opendev.org/696013 | 15:04 |
| *** saneax has quit IRC | 15:04 | |
| *** chkumar|rover is now known as raukadah | 15:17 | |
| *** yoctozepto has quit IRC | 15:24 | |
| *** yoctozepto has joined #zuul | 15:25 | |
| pabelanger | AJaeger: care to look at https://review.opendev.org/696013/ I know you've been using it recently (download-artifact). I'd like to update it to be able to use for child_jobs | 15:25 |
| *** bhavikdbavishi has quit IRC | 15:38 | |
| *** michael-beaver has joined #zuul | 15:54 | |
| pabelanger | okay, I think I have an initial ansible collection artifacts job working in zuul! | 16:05 |
| pabelanger | \o/ | 16:05 |
| *** hashar has quit IRC | 16:19 | |
| *** jamesmcarthur has joined #zuul | 16:19 | |
| *** jangutter has quit IRC | 16:22 | |
| *** jpena is now known as jpena|brb | 16:45 | |
| zbr_ | pabelanger: plese check https://bugs.launchpad.net/tripleo/+bug/1854073 | 16:48 |
| openstack | Launchpad bug 1854073 in tripleo "periodic rhel-8-ovb-3ctlr_1comp-featureset001-master times out during overcloud-prep-images" [Critical,Triaged] | 16:48 |
| clarkb | tobiash_: did you see my comment on https://review.opendev.org/#/c/684414/ I wanted to make sure I'm not wrong about that one | 16:49 |
| pabelanger | zbr_: sorry, not following | 16:50 |
| zbr_ | pabelanger: that is fixing a problem with zuul streamer and ansible 2.8, where package module failure gets .... silenced. | 16:51 |
| zbr_ | msg returned is a list instead of string, messing the code. the sooner we merge into zuul the better because "failure to un package" is quite common on CI, especially with efforths to enable new platforms, like centos-8 | 16:52 |
| pabelanger | zbr_: what you linked was for a tripleo period job (wrong url?) | 16:52 |
| zbr_ | pabelanger: right, sorry https://review.opendev.org/#/c/696081/ | 16:53 |
| pabelanger | kk, will review shortly | 16:53 |
| *** sshnaidm is now known as sshnaidm|afk | 17:05 | |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: WIP pagure: remove connectors burden and simplify code https://review.opendev.org/696134 | 17:07 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: WIP pagure: remove connectors burden and simplify code https://review.opendev.org/696134 | 17:10 |
| tobiash_ | clarkb: thanks for spotting, I haven't yet time to think through it | 17:19 |
| *** igordc has joined #zuul | 17:20 | |
| *** jpena|brb is now known as jpena | 17:21 | |
| *** jamesmcarthur has quit IRC | 17:22 | |
| AJaeger | the Zuul tenant throws some errors, here's a first fix for review, please: https://review.opendev.org/695843 | 17:26 |
| clarkb | AJaeger: do you know why we are trying to run jobs like the dns job in zuul? | 17:28 |
| AJaeger | clarkb: https://review.opendev.org/#/c/694178/ is the change that broke it, I guess | 17:30 |
| clarkb | hrm | 17:30 |
| AJaeger | but some might also be related to system-config | 17:31 |
| clarkb | tobiash_: also addressed your comment in https://review.opendev.org/#/c/675403/ thanks! | 17:42 |
| tobiash_ | +2 | 17:47 |
| *** tobiash_ is now known as tobiash | 17:47 | |
| *** openstackgerrit has quit IRC | 17:49 | |
| clarkb | tobiash: I think really old python is the reason for writing the reversal that way fwiw (reversed wasn't added until 2.4 :) ) | 17:49 |
| tobiash | clarkb: thanks, now I feel young again ;) | 17:50 |
| *** jpena is now known as jpena|off | 18:31 | |
| *** electrofelix has quit IRC | 18:50 | |
| *** jamesmcarthur has joined #zuul | 19:36 | |
| *** mhu has quit IRC | 19:55 | |
| *** rlandy is now known as rlandy|brb | 19:56 | |
| *** mhu has joined #zuul | 20:06 | |
| *** rlandy|brb is now known as rlandy | 20:10 | |
| *** nhicher has quit IRC | 20:33 | |
| *** nhicher has joined #zuul | 20:33 | |
| *** nhicher has quit IRC | 20:35 | |
| *** nhicher has joined #zuul | 20:39 | |
| *** tosky has quit IRC | 20:46 | |
| *** rfolco has quit IRC | 21:14 | |
| *** threestrands has joined #zuul | 21:27 | |
| *** igordc has quit IRC | 21:33 | |
| *** plaurin has joined #zuul | 21:36 | |
| plaurin | Hello, I have an unexpected behaviour from zuul when I try to set an override-branch to a job. | 21:37 |
| plaurin | Using override-checkout: V13 but job still runs on master | 21:37 |
| plaurin | this job has a 'parent' job that adds a post-run playbook. that one does run on the V13 branch | 21:38 |
| plaurin | basically, RUN-START uses master and POST-RUN uses V13 | 21:38 |
| plaurin | but I need both to use V13 | 21:39 |
| clarkb | plaurin: is the job defined on multiple branches? | 21:39 |
| *** Tahvok has joined #zuul | 21:39 | |
| *** igordc has joined #zuul | 21:39 | |
| plaurin | yes it is. | 21:39 |
| Tahvok | Hey guys! I'm interested in using zuul as alternative to jenkins pipeline as a solution to our problem of dependencies across projects | 21:40 |
| plaurin | I have quite a huge .zuul.yaml and it's a bit hard to manage the relevant jobs across multiple branches when I do a release. (I have some jobs for master and some for the releases) | 21:40 |
| Tahvok | The abilities so far look great, and looks like it could greatly fit our needs. However, I was sad to find out that it works mainly with dockerhub to push images to. | 21:41 |
| clarkb | plaurin: we (opendev) use a config repo to keep track of things that are a bit less branch specific | 21:41 |
| Tahvok | Is it possible to use gcr.io? If not, how hard do you think it would be implement another repository solution into zuul? | 21:41 |
| clarkb | plaurin: this way we avoid branch specific config applying in the wrong case (which may be happening here). The way to confirm that is to check the zuul inventory for the job that is behaving improperly. In that file is a job inheritance path which should show you where it is loading configs from | 21:41 |
| pabelanger | Tahvok: The jobs are written in ansible, so you can modify it to push to other places if needed | 21:42 |
| clarkb | Tahvok: what pabelanger said, but also corvus has been working recently to make it work against arbitrary docker image repos too | 21:42 |
| plaurin | clarkb hum okay let me check that out | 21:43 |
| Tahvok | Oh, great then! I will be looking into testing it soon | 21:43 |
| Tahvok | Thanks a lot! | 21:43 |
| Tahvok | I remember I really wanted something like zuul when I was working on openstack-ansible project in my previous company. And just recently found out that you made it available for github - which fits greatly for the current company I'm in | 21:44 |
| clarkb | plaurin: let me find an example from one of our jobs so that you know what you are looking for | 21:45 |
| plaurin | okay | 21:45 |
| clarkb | plaurin: https://zuul.opendev.org/t/openstack/build/fff672698562463fb30ac21654e454e9/log/zuul-info/inventory.yaml#25 | 21:45 |
| pabelanger | Tahvok: yup, it works well with github, we in ansible have been using it for sometime now | 21:45 |
| clarkb | pabelanger: have you had a chance to look at my comments on the github driver change you wrote? | 21:46 |
| clarkb | I think it is possible that the search query itself is too greedy which contributes to what you observed (details in the chagne) | 21:46 |
| pabelanger | clarkb: not yet, been head down trying to get this galaxy artifacts stuff working properly | 21:46 |
| pabelanger | hopefully later this week I can loop back to it | 21:46 |
| plaurin | I'm not sure I can get this /log/zuul-info path. | 21:49 |
| plaurin | I have a job-output.txt and a /console section | 21:49 |
| plaurin | that would be quite helpful for me to get all that info | 21:51 |
| clarkb | plaurin: I think there is a role you can add to your base job to collect this data. Let me find it | 21:54 |
| plaurin | great, yea I figure out I am probably missing one of those. checking too | 21:54 |
| clarkb | plaurin: https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/log-inventory and this is where we apply it to our base job https://opendev.org/opendev/base-jobs/src/branch/master/playbooks/base/pre.yaml#L5 | 21:55 |
| plaurin | Thank you very much I will install that tomorrow morning on my setup | 21:55 |
| plaurin | quick question I have to leave soon | 21:56 |
| plaurin | How can I safely delete branches? | 21:56 |
| plaurin | I use gerrit, and zuul returns error when I delete a branch | 21:56 |
| clarkb | is the error related to invalid configs? | 21:56 |
| plaurin | if I remember it complains for 'missing' data from those branches | 21:57 |
| plaurin | or invalod config, been a while since I tried it | 21:57 |
| clarkb | I wonder too if it is possible that the mergers don't properly apply branch removal updates? though I would have expected to see that | 21:57 |
| plaurin | that's managed by zuul-merger? okay | 21:58 |
| clarkb | plaurin: if you can provide the specific errors we can help work through it. My hunch is that if you have that branch in some places but have removed it in others that the remaining instances of the branches depend on things that were in the deleted branches? | 21:58 |
| clarkb | but ya double check that the zuul-merger managed git repos show the branch has properly been cleaned up too (if not that would eb a zuul bug) | 21:58 |
| plaurin | okay will run some tests and will come back here with more details. Thanks a million. | 21:59 |
| plaurin | dinner time here in eastern time. catch you later | 21:59 |
| *** openstackgerrit has joined #zuul | 22:01 | |
| openstackgerrit | Vitaliy Lotorev proposed zuul/zuul master: doc: Clarify that some regexp has restricted syntax https://review.opendev.org/695991 | 22:01 |
| openstackgerrit | Vitaliy Lotorev proposed zuul/zuul master: doc: Document regexp usage https://review.opendev.org/695991 | 22:05 |
| pabelanger | plaurin: you need to have https://zuul-ci.org/docs/zuul-jobs/general-roles.html#role-log-inventory in your base job | 22:05 |
| pabelanger | plaurin: https://zuul-ci.org/docs/zuul-jobs/general-roles.html#role-validate-host is another good one | 22:06 |
| *** pcaruana has quit IRC | 22:16 | |
| SpamapS | Tahvok: I use zuul with ECR, which is similar to gcr.io | 22:55 |
| SpamapS | a simple matter of ansible | 22:55 |
| SpamapS | I should probably submit my roles to zuul-jobs actually | 22:56 |
| * fungi would review | 23:00 | |
| SpamapS | The problem is I made them before all the registry stuff. | 23:00 |
| SpamapS | I kind of want them to be drop-ins. | 23:00 |
| fungi | makes sense | 23:00 |
| SpamapS | Yeah, but then that's a fallacy, and I should just publish and the community can push with me. | 23:01 |
| SpamapS | I always forget that part where sometimes if you release imperfect open source, the community will perfect it for you. :) | 23:01 |
| fungi | or you release imperfect open source and they don't, but at least it's out there. no judgement | 23:03 |
| SpamapS | so true | 23:03 |
| fungi | we've all published worse trash than whatever you're going to push up, so glass houses, stones, merge | 23:04 |
| *** rfolco has joined #zuul | 23:47 | |
| *** rfolco has quit IRC | 23:51 | |
| *** rfolco has joined #zuul | 23:51 | |
| *** rfolco has quit IRC | 23:55 | |
| *** rfolco has joined #zuul | 23:55 | |
| *** rlandy has quit IRC | 23:58 | |
| *** rlandy has joined #zuul | 23:58 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!