| *** Goneri has joined #zuul | 00:05 | |
| *** mattw4 has quit IRC | 00:06 | |
| *** jamesmcarthur has joined #zuul | 00:45 | |
| *** jamesmcarthur has quit IRC | 01:00 | |
| *** jamesmcarthur has joined #zuul | 01:10 | |
| tristanC | corvus: that is indeed a useful feature, allowed-labels is hard to use to prevent all-but-one tenant from using a label | 01:11 |
|---|---|---|
| *** michael-beaver has quit IRC | 01:13 | |
| *** jamesmcarthur has quit IRC | 01:22 | |
| *** jamesmcarthur has joined #zuul | 01:45 | |
| *** Goneri has quit IRC | 01:47 | |
| *** igordc has joined #zuul | 01:52 | |
| *** jamesmcarthur has quit IRC | 02:22 | |
| *** bhavikdbavishi has joined #zuul | 02:56 | |
| openstackgerrit | Merged zuul/project-config master: Migrate to upload-afs-roots role https://review.opendev.org/705372 | 02:59 |
| *** bhavikdbavishi1 has joined #zuul | 02:59 | |
| *** bhavikdbavishi has quit IRC | 03:00 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 03:00 | |
| *** rlandy has quit IRC | 03:03 | |
| *** raukadah is now known as chkumar|rover | 04:22 | |
| *** igordc has quit IRC | 04:59 | |
| ianw | fungi: so we don't lose track of the zuul release tarball issues you raised, i've filed https://storyboard.openstack.org/#!/story/2007261 with the updates discussed in #openstack-infra | 05:06 |
| *** bolg has joined #zuul | 05:06 | |
| ianw | i've assigned it to you for lack of anyone else | 05:07 |
| ianw | mordred/corvus : ^ you will probably have some thoughts on the issues in there too. basically release job only releases to pypi; it's unclear if that is the overall intention | 05:08 |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #zuul | 05:34 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Use unique loop vars to avoid conflicts https://review.opendev.org/705337 | 05:43 |
| openstackgerrit | Merged zuul/zuul-jobs master: Link the policy to the CONTRIBUTING.rst file https://review.opendev.org/704484 | 05:51 |
| *** felixedel has joined #zuul | 06:24 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: WIP: Store unparsed branch config in Zookeeper https://review.opendev.org/705716 | 06:36 |
| *** bolg has quit IRC | 07:34 | |
| openstackgerrit | Felix Schmidt proposed zuul/zuul master: Report retried builds via sql reporter. https://review.opendev.org/633501 | 07:52 |
| openstackgerrit | Felix Schmidt proposed zuul/zuul master: Store information about gate resets in MQTT and SQL reporter https://review.opendev.org/696670 | 07:52 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Evaluate CODEOWNERS settings during canMerge check https://review.opendev.org/644557 | 08:12 |
| *** sugaar has joined #zuul | 08:14 | |
| *** hashar has joined #zuul | 08:27 | |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Spec for allowing circular dependencies https://review.opendev.org/643309 | 08:36 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Add optional support for circular dependencies https://review.opendev.org/685354 | 08:36 |
| *** jpena|off is now known as jpena | 08:53 | |
| *** zxiiro has quit IRC | 09:06 | |
| *** bhavikdbavishi has quit IRC | 09:19 | |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Spec for allowing circular dependencies https://review.opendev.org/643309 | 10:04 |
| *** bolg has joined #zuul | 10:14 | |
| openstackgerrit | Chandan Kumar (raukadah) proposed zuul/zuul-jobs master: Added bzip2 package for CentOS 8 https://review.opendev.org/705911 | 10:18 |
| *** hashar has quit IRC | 10:41 | |
| *** felixedel has quit IRC | 11:21 | |
| *** felixedel has joined #zuul | 11:24 | |
| *** wxy-xiyuan has quit IRC | 11:36 | |
| *** pcaruana has quit IRC | 11:37 | |
| *** pcaruana has joined #zuul | 11:50 | |
| *** tosky has joined #zuul | 12:12 | |
| *** jpena is now known as jpena|lunch | 12:33 | |
| fungi | thanks ianw! i may not be able to get to it this week, but suspect we can copy logic from existing jobs for it | 12:49 |
| *** bhavikdbavishi has joined #zuul | 12:52 | |
| *** bhavikdbavishi1 has joined #zuul | 12:55 | |
| *** bhavikdbavishi has quit IRC | 12:57 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 12:57 | |
| *** rlandy has joined #zuul | 13:01 | |
| *** nhicher has quit IRC | 13:05 | |
| *** nhicher has joined #zuul | 13:06 | |
| *** smcginnis|FOSDEM is now known as smcginnis | 13:19 | |
| *** jamesmcarthur has joined #zuul | 13:20 | |
| mordred | ianw: I don't think it was intentional, other than we didn't have an appropriate destination at the time IIRC. | 13:23 |
| *** bhavikdbavishi has quit IRC | 13:29 | |
| *** electrofelix has joined #zuul | 13:29 | |
| *** bhavikdbavishi has joined #zuul | 13:29 | |
| *** jpena|lunch is now known as jpena | 13:35 | |
| *** jamesmcarthur has quit IRC | 13:37 | |
| fungi | yep, and then some months later we forgot the job was incomplete and put it into production | 13:44 |
| *** jamesmcarthur has joined #zuul | 13:47 | |
| *** saneax has quit IRC | 13:49 | |
| *** zxiiro has joined #zuul | 13:53 | |
| openstackgerrit | Felix Schmidt proposed zuul/zuul master: Implement github checks API https://review.opendev.org/705168 | 13:55 |
| *** pcaruana has quit IRC | 14:17 | |
| *** felixedel has quit IRC | 14:20 | |
| openstackgerrit | Alfredo Moralejo proposed zuul/zuul-jobs master: Add CentOS8 High Availability repository https://review.opendev.org/705973 | 14:45 |
| openstackgerrit | Alfredo Moralejo proposed zuul/zuul-jobs master: Add CentOS8 High Availability repository https://review.opendev.org/705973 | 14:47 |
| *** swest has quit IRC | 14:47 | |
| *** swest has joined #zuul | 14:48 | |
| *** bhavikdbavishi1 has joined #zuul | 14:48 | |
| *** swest has quit IRC | 14:48 | |
| *** bhavikdbavishi has quit IRC | 14:49 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 14:49 | |
| *** swest has joined #zuul | 14:50 | |
| *** sshnaidm|afk is now known as sshnaidm | 14:51 | |
| *** swest has quit IRC | 14:55 | |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: Add index_links option to zuul manifest https://review.opendev.org/705580 | 15:19 |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Google logs: Link to index.html files https://review.opendev.org/705586 | 15:21 |
| mordred | corvus: ^^ rebased the followup change so I could +A it | 15:22 |
| *** bolg has quit IRC | 15:23 | |
| corvus | mordred: thanks! | 15:24 |
| corvus | i think i'll carry over ianw and tobiash's +2 since the fix wasn't substantial | 15:24 |
| mordred | ++ | 15:25 |
| *** chkumar|rover is now known as raukadah | 15:27 | |
| corvus | adding in the slowest test output has made the failing test fall outside the display error in the job summary window (see https://zuul.opendev.org/t/zuul/build/2a1e75da64804bf8a7e728837f952a47 ) | 15:27 |
| corvus | should we expand that window to copy over a few more lines? | 15:27 |
| clarkb | corvus: maybe we can do a grep -i failure|error -A 30 -B 30 type of thing? | 15:28 |
| mordred | corvus: yeah- I think so - I frequently find that it's just slightly too short - which may also be a result of having slowest on in other openstack repos | 15:28 |
| clarkb | to try and get it closer to the relevant data | 15:28 |
| corvus | clarkb: oh yeah -- maybe do that backwards from the end and show the first hit | 15:28 |
| clarkb | corvus: ++ | 15:29 |
| mordred | ++ | 15:29 |
| openstackgerrit | Merged zuul/zuul-jobs master: Add upload-logs-gcs role https://review.opendev.org/703711 | 15:34 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Add disallowed-labels tenant option https://review.opendev.org/705856 | 15:36 |
| *** ianychoi_ is now known as ianychoi | 15:40 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Add index_links option to zuul manifest https://review.opendev.org/705580 | 15:40 |
| openstackgerrit | Merged zuul/zuul-jobs master: Google logs: Link to index.html files https://review.opendev.org/705586 | 15:44 |
| *** pcaruana has joined #zuul | 15:49 | |
| corvus | the new pipeline proposed by ttx for openstack may be interesting to folks: https://review.opendev.org/705990 (it basically will be used for quick jobs on the zuul executor to do a process approval check and vote in a dedicated gerrit review label) | 15:59 |
| *** Goneri has joined #zuul | 16:02 | |
| mordred | corvus: that's cool | 16:03 |
| ttx | kinda curious to see if that will work in practice :) | 16:04 |
| mordred | ttx: I like it as a concept! | 16:04 |
| *** openstackstatus has joined #zuul | 16:06 | |
| *** ChanServ sets mode: +v openstackstatus | 16:06 | |
| fungi | if nothing else, it'll be an interesting experiment | 16:07 |
| ttx | We might extend that job so that it would subscribe the PTL/liaison to the change if they are missing | 16:07 |
| corvus | yeah, that should work; you might need an extra gerrit account credential to authonticate to do that | 16:09 |
| ttx | does comment-added also trigger on patchset-added? | 16:09 |
| ttx | like when a patchset is added, I /think/ it also posts a comment so one should include the other | 16:09 |
| ttx | (in order to catch that the request is actually coming from the PTL so does not need further approval in comment) | 16:10 |
| fungi | ttx: opendev's check pipeline trigger lists them out separately at least: https://opendev.org/openstack/project-config/src/branch/master/zuul.d/pipelines.yaml#L19-L30 | 16:20 |
| ttx | ok, we'll see if there are corner cases to catch. | 16:21 |
| fungi | so we enqueue on patchset-created, change-restored, comment-added with a recheck pattern, and comment-added with an approval vote | 16:21 |
| fungi | for the openstack tenant anyway | 16:21 |
| fungi | the zuul tenant is slightly simpler what with lacking clean-check enforcement: https://opendev.org/zuul/project-config/src/branch/master/zuul.d/pipelines.yaml#L16-L21 | 16:23 |
| *** jamesmcarthur has quit IRC | 16:24 | |
| *** jamesmcarthur has joined #zuul | 16:38 | |
| *** jamesmcarthur_ has joined #zuul | 16:41 | |
| *** jamesmcarthur has quit IRC | 16:44 | |
| *** jamesmcarthur_ has quit IRC | 16:55 | |
| *** jamesmcarthur has joined #zuul | 16:58 | |
| tobiash | corvus: I added a question on 705856, what do you think? | 17:03 |
| corvus | tobiash: ++. followup ok? | 17:04 |
| tobiash | sure :) | 17:04 |
| *** jamesmcarthur has quit IRC | 17:05 | |
| *** zxiiro has quit IRC | 17:07 | |
| *** zxiiro has joined #zuul | 17:08 | |
| *** jamesmcarthur has joined #zuul | 17:27 | |
| *** evrardjp has quit IRC | 17:33 | |
| *** evrardjp has joined #zuul | 17:34 | |
| *** igordc has joined #zuul | 17:34 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Add spec for scale out scheduler https://review.opendev.org/621479 | 17:49 |
| sshnaidm | is there a good way to apply "irrelevant_files" to whole repo, no matter what jobs run there and which project templates are used? | 17:51 |
| dmsimard | sshnaidm: https://opendev.org/openstack/project-config/src/branch/master/zuul.d/projects.yaml#L13-L18 | 17:52 |
| dmsimard | ah, I guess irrelevant_files isn't used at the project level | 17:53 |
| tristanC | dmsimard: iiuc, sshnaidm wants to apply an attribute to all the jobs except one of a pipeline | 17:53 |
| *** jpena is now known as jpena|off | 17:53 | |
| sshnaidm | right ^ | 17:53 |
| pabelanger | you could write a base job for that project, and parent everything in the project to that. And base has irrelevant_files | 17:55 |
| dmsimard | sshnaidm: ^ what he said but it looks like you're already doing that: https://github.com/openstack/tripleo-ci/blob/master/zuul.d/base.yaml#L6-L65 | 17:55 |
| sshnaidm | pabelanger, I use mostly jobs and project templates from other repos | 17:55 |
| *** jamesmcarthur has quit IRC | 17:56 | |
| sshnaidm | dmsimard, hmm.. actually this is the way, thanks | 17:56 |
| sshnaidm | if zuul supported two parents, I could create a "mixin" and attach to specific base job :) | 17:58 |
| dmsimard | sshnaidm: closest I could think of would be to have a job which has the base job as parent, define some other things in that child and then have jobs which parent to the child | 17:59 |
| dmsimard | base -> base_with_more_stuff <- actual-job | 17:59 |
| sshnaidm | dmsimard, yeah | 17:59 |
| tristanC | sshnaidm: actually zuul support multi parenting using job variant, but iirc all variants needs to be defined in a single repo, you can't mix in outside of the repo defining the job | 18:00 |
| tristanC | iirc, that results in a job using 2 parents: job: { name: "my-job", parent: "parent1" } and job: { name: "my-job", parent: "parent2" } | 18:02 |
| *** johnsom has quit IRC | 18:03 | |
| sshnaidm | tristanC, and second won't just overwrite the first one? | 18:03 |
| *** johnsom has joined #zuul | 18:03 | |
| tristanC | sshnaidm: i don't think, let me test this out in your collect-logs template | 18:03 |
| *** jamesmcarthur has joined #zuul | 18:03 | |
| sshnaidm | tristanC, interesting.. does somebody use it? wondering if there are real-life examples | 18:04 |
| tristanC | corvus: is using variant to use multiple parents still used in opendev somewhere? | 18:05 |
| corvus | tristanC: i'm not sure; i think we used it in system-config then refactored something and stopped. | 18:06 |
| tristanC | sshnaidm: iirc, that should work: https://review.opendev.org/706071 DNM: test parent mixin | 18:07 |
| corvus | note that yaml anchors may also be useful here (they let you repeat data blocks) | 18:08 |
| *** mattw4 has joined #zuul | 18:09 | |
| tristanC | corvus: iirc, sshnaidm would like this rule to be also applied in the ppc | 18:11 |
| mordred | corvus: do we not still use multi-parent for container jobs? | 18:12 |
| tristanC | e.g. where extra jobs can be added locally, and iiuc yaml anchor, their scope stops at a single file | 18:12 |
| corvus | yeah, you could add "irrelevant-files: *my_files" to every item in the ppc | 18:12 |
| * mordred also can't remember | 18:12 | |
| corvus | mordred: i have a vague memory of changing the inheritance hierarchy in such a way it didn't make sense any more. i think as part of the promote work. | 18:13 |
| tristanC | corvus: could *my_files be available when the ppc is defined in another repository from the anchor? | 18:13 |
| mordred | corvus: oh yeah - I think you're right | 18:13 |
| corvus | tristanC: no, anchors are only scoped to a file | 18:13 |
| sshnaidm | tristanC, couldn't you do just that? http://paste.openstack.org/show/789173/ | 18:13 |
| sshnaidm | tristanC, yeah, yaml anchors works only in the file they're defined.. | 18:14 |
| corvus | sshnaidm: i don't think you will convince the maintainers of the openstack-tox-linters job to merge that change | 18:14 |
| sshnaidm | corvus, I mean in different repo | 18:15 |
| tristanC | sshnaidm: iirc job variant can only be defined in a single project, e.g. you can't declare a job `openstack-tox-linters` in both ansible-role-collect-jobs and openstack-zuul-jobs | 18:15 |
| sshnaidm | tristanC, oh, I see | 18:15 |
| sshnaidm | now it makes sense | 18:15 |
| corvus | sshnaidm: that would change the job for everyone which you aren't allowed to do. but you are allowed to change it for your project, just by adding irrelevant-files to the job in the ppc | 18:16 |
| sshnaidm | so variants are scoped to one repo | 18:16 |
| *** sugaar has quit IRC | 18:17 | |
| corvus | you can do this: http://paste.openstack.org/show/789174/ | 18:17 |
| corvus | and you can repeat that for every job you want to apply it to. and you can use yaml anchors to make the repitition simpler. | 18:17 |
| corvus | theoretically i think maybe you could do that with a parent mixin too. but i've never tried it. | 18:19 |
| corvus | (basically, a job in a ppc is an anonymous project-local variant, so generally anything you can do in a variant you can do in the ppc) | 18:21 |
| corvus | we use that heavily in the zuul project; here's an example with irrelevant-files: https://opendev.org/zuul/zuul/src/branch/master/.zuul.yaml#L252-L262 | 18:23 |
| *** kmalloc has quit IRC | 18:27 | |
| *** kmalloc has joined #zuul | 18:27 | |
| sshnaidm | corvus, yeah, but we use templates mostly and would be nice if I had option to apply this to template (which would apply it for ALL jobs in template) | 18:33 |
| corvus | sshnaidm: i hear you. that doesn't exist right now, so i'm just trying to help you find something that will work | 18:38 |
| sshnaidm | corvus, of course, interesting if there is an option to have it in roadmap, maybe it would be useful for others as well | 18:41 |
| *** bhavikdbavishi has quit IRC | 18:41 | |
| fungi | if you can figure out what that could look like, starting a zuul-discuss ml thread on it and/or submitting a spec could be a good way forward | 18:45 |
| *** mattw4 has quit IRC | 18:48 | |
| *** jamesmcarthur has quit IRC | 18:54 | |
| *** mattw4 has joined #zuul | 18:56 | |
| *** electrofelix has quit IRC | 18:59 | |
| *** mattw4 has quit IRC | 19:06 | |
| *** jamesmcarthur has joined #zuul | 19:15 | |
| *** mattw4 has joined #zuul | 19:19 | |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Extract the watcher from git driver https://review.opendev.org/706092 | 19:21 |
| *** maxamillion has quit IRC | 19:27 | |
| *** maxamillion has joined #zuul | 19:27 | |
| mordred | corvus: git add | 19:37 |
| *** maxamillion has quit IRC | 19:37 | |
| *** kmalloc has quit IRC | 19:39 | |
| *** zxiiro has quit IRC | 19:39 | |
| *** johnsom has quit IRC | 19:40 | |
| corvus | mordred: details | 20:00 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Extract the watcher from git driver https://review.opendev.org/706092 | 20:00 |
| mordred | corvus: the computer should know what you mean | 20:01 |
| corvus | mordred: i read in the tech press that development is all done by ai at this point, so... yeah | 20:02 |
| fungi | my development sure is | 20:02 |
| fungi | there is no intelligence more artificial | 20:03 |
| fungi | or was that alcohol intelligence? | 20:03 |
| *** jamesmcarthur has quit IRC | 20:21 | |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Revert "Add index_links option to zuul manifest" https://review.opendev.org/706106 | 20:41 |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: Handle index_links in manifest https://review.opendev.org/706115 | 21:03 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: Move index_links to correct task in generate_zuul_manifest https://review.opendev.org/706116 | 21:07 |
| *** tosky has quit IRC | 21:18 | |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Extract the watcher from git driver https://review.opendev.org/706092 | 21:21 |
| openstackgerrit | Merged zuul/zuul-jobs master: Move index_links to correct task in generate_zuul_manifest https://review.opendev.org/706116 | 21:31 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: add_host: enable using ansible_fqdn and ansible_private_key_file https://review.opendev.org/706124 | 21:52 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Gerrit: add polling support for refs https://review.opendev.org/706138 | 22:29 |
| *** yolanda has joined #zuul | 22:35 | |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Extract allow/disallow filter into util function https://review.opendev.org/706144 | 22:44 |
| openstackgerrit | Merged zuul/zuul-jobs master: Add CentOS8 High Availability repository https://review.opendev.org/705973 | 22:50 |
| *** maxamillion has joined #zuul | 23:01 | |
| *** johnsom has joined #zuul | 23:02 | |
| *** kmalloc has joined #zuul | 23:02 | |
| *** zxiiro has joined #zuul | 23:03 | |
| *** jamesmcarthur has joined #zuul | 23:03 | |
| *** jamesmcarthur has quit IRC | 23:06 | |
| *** rlandy is now known as rlandy|bbl | 23:25 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!