| *** armstrongs has joined #zuul | 00:02 | |
| *** jamesmcarthur has quit IRC | 00:02 | |
| *** jamesmcarthur has joined #zuul | 00:03 | |
| *** jamesmcarthur has quit IRC | 00:04 | |
| *** jamesmcarthur_ has joined #zuul | 00:04 | |
| *** armstrongs has quit IRC | 00:11 | |
| *** jamesmcarthur_ has quit IRC | 00:52 | |
| *** jamesmcarthur has joined #zuul | 00:52 | |
| *** jamesmcarthur has quit IRC | 01:08 | |
| *** jamesmcarthur has joined #zuul | 01:09 | |
| *** jamesmcarthur has quit IRC | 01:14 | |
| *** saneax has quit IRC | 01:27 | |
| *** swest has quit IRC | 01:34 | |
| *** cloudnull has quit IRC | 01:39 | |
| *** jamesmcarthur has joined #zuul | 01:41 | |
| *** swest has joined #zuul | 01:48 | |
| *** jamesmcarthur has quit IRC | 02:03 | |
| *** jamesmcarthur has joined #zuul | 02:04 | |
| *** jamesmcarthur has quit IRC | 02:09 | |
| *** jamesmcarthur has joined #zuul | 02:11 | |
| *** jamesmcarthur has quit IRC | 02:14 | |
| *** jamesmcarthur has joined #zuul | 02:15 | |
| *** jamesmcarthur has quit IRC | 02:20 | |
| *** bhavikdbavishi has joined #zuul | 02:40 | |
| *** jamesmcarthur has joined #zuul | 02:44 | |
| *** jamesmcarthur has quit IRC | 02:51 | |
| *** jamesmcarthur has joined #zuul | 02:52 | |
| *** jamesmcarthur has quit IRC | 02:52 | |
| *** jamesmcarthur has joined #zuul | 02:52 | |
| *** jamesmcarthur has quit IRC | 02:58 | |
| *** jamesmcarthur has joined #zuul | 03:05 | |
| *** jamesmcarthur has quit IRC | 03:09 | |
| *** jamesmcarthur has joined #zuul | 03:10 | |
| *** jamesmcarthur has quit IRC | 03:13 | |
| *** jamesmcarthur has joined #zuul | 03:13 | |
| *** bhavikdbavishi has quit IRC | 03:14 | |
| *** bhavikdbavishi has joined #zuul | 03:40 | |
| *** saneax has joined #zuul | 04:02 | |
| *** bhavikdbavishi has quit IRC | 04:07 | |
| *** jamesmcarthur has quit IRC | 04:07 | |
| *** jamesmcarthur has joined #zuul | 04:08 | |
| *** jamesmcarthur has quit IRC | 04:09 | |
| *** jamesmcarthur has joined #zuul | 04:10 | |
| *** jamesmcarthur has quit IRC | 04:18 | |
| *** jamesmcarthur has joined #zuul | 04:18 | |
| *** jamesmcarthur has quit IRC | 04:22 | |
| *** jamesmcarthur_ has joined #zuul | 04:22 | |
| *** bhavikdbavishi has joined #zuul | 04:27 | |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #zuul | 04:33 | |
| *** threestrands has joined #zuul | 04:38 | |
| *** tosky has joined #zuul | 06:30 | |
| *** jamesmcarthur_ has quit IRC | 06:48 | |
| *** jamesmcarthur has joined #zuul | 06:49 | |
| *** jamesmcarthur has quit IRC | 06:54 | |
| *** jcapitao has joined #zuul | 06:55 | |
| *** jhesketh has quit IRC | 07:03 | |
| *** bhavikdbavishi has quit IRC | 07:04 | |
| *** jhesketh has joined #zuul | 07:05 | |
| *** jamesmcarthur has joined #zuul | 07:21 | |
| *** rpittau|afk is now known as rpittau | 07:21 | |
| *** jhesketh has quit IRC | 07:25 | |
| *** jhesketh has joined #zuul | 07:25 | |
| *** sshnaidm|off is now known as sshnaidm | 07:37 | |
| *** arxcruz|off is now known as arxcruzz | 07:37 | |
| *** arxcruzz is now known as arxcruz | 07:37 | |
| *** jamesmcarthur has quit IRC | 07:38 | |
| *** jamesmcarthur has joined #zuul | 07:38 | |
| *** jamesmcarthur has quit IRC | 07:40 | |
| *** jamesmcarthur has joined #zuul | 07:41 | |
| *** nils has joined #zuul | 07:43 | |
| *** jamesmcarthur has quit IRC | 07:46 | |
| *** yolanda has joined #zuul | 07:49 | |
| *** jpena|off is now known as jpena | 07:58 | |
| avass | if anyones interested, this allows you to run jobs locally with some minimal added configuration: https://review.opendev.org/#/c/728684/23 I'm not sure if it's wanted in zuul-jobs but I had fun creating it :) | 08:01 |
|---|---|---|
| avass | it still needs some polish and I'm thinking about extending the usecase to handle VMs with Vagrant or something similar as well to be able to run things that requires access to the kernel | 08:02 |
| *** jamesmcarthur has joined #zuul | 08:14 | |
| *** asaleh_ has joined #zuul | 08:27 | |
| *** jamesmcarthur has quit IRC | 08:30 | |
| *** bogdando has joined #zuul | 08:54 | |
| bogdando | hi folks. Could someone please point me out on misconfiguration I did for package_mirrors? I explained details in comments of https://review.opendev.org/#/c/730013 | 08:55 |
| bogdando | why did it not take effect and I ended up with mirror_fqdn?.. | 08:55 |
| bogdando | the task "configure-mirrors : Install CentOS 8 repository files" | 08:56 |
| bogdando | I'm using a static lbvirt-based centos8 subnode | 08:56 |
| bogdando | hm, that is probably because the "role vars" wins inventory host_vars/* | 09:00 |
| bogdando | so how could that package_mirror be overrided properly? | 09:00 |
| openstackgerrit | Bogdan Dobrelya (bogdando) proposed zuul/zuul-jobs master: Support overrideable package_mirror https://review.opendev.org/730602 | 09:37 |
| avass | bogdando: oh, you could also set the variables when you include the role, that should override 'vars' in the role. But moving that to defaults seems like a good idea | 09:41 |
| bogdando | avass: thanks for you support! I'm not sure I know a lot of zuul-vars override magic | 09:42 |
| bogdando | meaning in end CI jobs | 09:43 |
| bogdando | in CI reproducer, those we config, end up in host_group scope | 09:43 |
| avass | bogdando: yeah | 09:43 |
| avass | bogdando: I believe vars set in a zuul job end up in the inventory, so role vars win over those: https://docs.ansible.com/ansible/latest/user_guide/playbooks_variables.html#variable-precedence-where-should-i-put-a-variable | 09:45 |
| bogdando | yea | 09:45 |
| avass | bogdando: so moving what should be possible to ovveride to defaults is probably what we want | 09:45 |
| bogdando | hope so, thanks! :) | 09:45 |
| avass | bogdando: no problem :) | 09:46 |
| *** jhesketh has quit IRC | 09:48 | |
| *** jhesketh has joined #zuul | 09:49 | |
| openstackgerrit | Bogdan Dobrelya (bogdando) proposed zuul/zuul-jobs master: Support overrideable package_mirror https://review.opendev.org/730602 | 10:06 |
| *** rpittau is now known as rpittau|bbl | 10:12 | |
| openstackgerrit | Bogdan Dobrelya (bogdando) proposed zuul/zuul-jobs master: Support overrideable package_mirror https://review.opendev.org/730602 | 10:24 |
| *** jamesmcarthur has joined #zuul | 10:27 | |
| *** jamesmcarthur has quit IRC | 10:31 | |
| *** panda is now known as panda|off | 10:51 | |
| *** tumble has joined #zuul | 11:05 | |
| *** jcapitao is now known as jcapitao_lunch | 11:06 | |
| *** threestrands has quit IRC | 11:11 | |
| *** jpena is now known as jpena|lunch | 11:31 | |
| *** armstrongs has joined #zuul | 11:38 | |
| *** hashar has joined #zuul | 11:45 | |
| *** armstrongs has quit IRC | 11:47 | |
| *** ianychoi has joined #zuul | 11:50 | |
| *** rfolco has joined #zuul | 11:51 | |
| *** rfolco is now known as rfolco|rover | 12:02 | |
| *** mwhahaha has quit IRC | 12:07 | |
| *** mwhahaha has joined #zuul | 12:10 | |
| *** rpittau|bbl is now known as rpittau | 12:11 | |
| *** jcapitao_lunch is now known as jcapitao | 12:17 | |
| *** rfolco|rover has quit IRC | 12:19 | |
| *** rfolco|rover has joined #zuul | 12:19 | |
| *** jamesmcarthur has joined #zuul | 12:28 | |
| *** jamesmcarthur has quit IRC | 12:32 | |
| *** jpena|lunch is now known as jpena | 12:33 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Builds: add "held" attribute https://review.opendev.org/728994 | 12:44 |
| *** bhavikdbavishi has joined #zuul | 12:54 | |
| tobiash | corvus: I just noticed that a merge conflict reduces the gate window which I didn't expect. I think we may want to keep the gate window in this case, what do you think? | 13:04 |
| *** nhicher has joined #zuul | 13:07 | |
| fungi | that's an interesting observation... going back to the point of shrinking the active queue window, it's a defense against changes failing in a long queue and causing changes behind them to reset testing. if you approve a bunch of potentially conflicting changes i guess the changes behind them don't get reset because they'll never be able to all be sequenced anyway and each of the subsequent conflicts will be | 13:10 |
| fungi | pulled aside immediately, so don't actually impact resource consumption the same way build failures do | 13:10 |
| fungi | so it seems reasonable to me to exclude merge conflicts from the failures which shrink the window, yes | 13:11 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Don't decrease window size on merge failure https://review.opendev.org/730624 | 13:14 |
| mordred | fungi, tobiash: ++ I agree | 13:40 |
| *** iamweswilson has quit IRC | 13:50 | |
| *** iamweswilson has joined #zuul | 13:50 | |
| *** bhavikdbavishi has quit IRC | 13:52 | |
| *** rpittau is now known as rpittau|brb | 13:56 | |
| *** yolanda has quit IRC | 14:05 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Builds: add "held" attribute https://review.opendev.org/728994 | 14:19 |
| *** yolanda has joined #zuul | 14:19 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: CLI: Fix errors with the REST client https://review.opendev.org/728061 | 14:21 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: How-to: using the REST API with cURL https://review.opendev.org/727785 | 14:26 |
| *** jamesmcarthur has joined #zuul | 14:28 | |
| *** rpittau|brb is now known as rpittau | 14:30 | |
| *** jamesmcarthur has quit IRC | 14:33 | |
| *** yolanda has quit IRC | 14:36 | |
| *** yolanda has joined #zuul | 14:37 | |
| *** bogdando has left #zuul | 14:43 | |
| *** yolanda has quit IRC | 14:53 | |
| *** yolanda has joined #zuul | 15:06 | |
| *** Goneri has joined #zuul | 15:12 | |
| *** y2kenny has joined #zuul | 15:15 | |
| y2kenny | I just added Zuul's logo to this diagramming tool. I hope it's ok: https://github.com/mingrammer/diagrams/commit/c17bf1e2d292faa3891238eb371e43c13af1c852 (I am using the tool to outline my CI infrastructure.) | 15:18 |
| *** yolanda has quit IRC | 15:19 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: CLI: Fix errors with the REST client https://review.opendev.org/728061 | 15:28 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: fix discrepancies between RPC and REST outputs for autohold https://review.opendev.org/728073 | 15:28 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Add simple testing for Zuul CLI & REST API https://review.opendev.org/728098 | 15:28 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: improve tenant scoping of autohold, authorizations https://review.opendev.org/728118 | 15:28 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: CLI: add autohold-info, autohold-delete via REST https://review.opendev.org/728410 | 15:35 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: CLI: add autohold-info, autohold-delete via REST https://review.opendev.org/728410 | 15:36 |
| fungi | i've commented before, we probably need to add proper license and copyright information to image metadata for assets like that... the repository doesn't even seem to include information on the copyright holders for each of the images in there though so it seems a bit sketchy to me | 15:37 |
| *** yolanda has joined #zuul | 15:38 | |
| *** bhavikdbavishi has joined #zuul | 15:43 | |
| *** jamesmcarthur has joined #zuul | 15:45 | |
| *** cloudnull has joined #zuul | 16:01 | |
| *** hashar has quit IRC | 16:03 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Builds: add "held" attribute https://review.opendev.org/728994 | 16:24 |
| *** jcapitao has quit IRC | 16:32 | |
| *** bhavikdbavishi has quit IRC | 16:43 | |
| *** rpittau is now known as rpittau|afk | 16:48 | |
| *** jamesmcarthur has quit IRC | 16:49 | |
| *** jpena is now known as jpena|off | 17:01 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: requirements: add explicit reference to dateutil https://review.opendev.org/730654 | 17:41 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: gerrit: disable poller when no http authentication is defined https://review.opendev.org/730655 | 17:42 |
| *** fbo is now known as fbo|off | 17:48 | |
| *** asaleh_ has quit IRC | 17:52 | |
| *** guillaumec has joined #zuul | 17:56 | |
| avass | AJaeger: can you take a look at: https://review.opendev.org/#/c/730602/3 ? | 18:09 |
| AJaeger | avass: I prefer if corvus or clarkb review that since it has implications for our mirror setup ^ | 18:10 |
| avass | sure | 18:12 |
| AJaeger | avass: just gave a -1 - we don't want this without a README update. | 18:13 |
| AJaeger | avass: and then I hope that others can say whether this is fine. | 18:13 |
| avass | ah, that's true | 18:14 |
| AJaeger | avass: just remembered something about mirrors - added another comment (copy & paste from https://review.opendev.org/#/c/693887/ ) | 18:15 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add tutorial tests https://review.opendev.org/728194 | 18:16 |
| avass | AJaeger: ah, yep, might be worth getting that working | 18:19 |
| AJaeger | avass: yep, let's see what others say | 18:22 |
| *** nils has quit IRC | 18:24 | |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: bindep: update include to import_tasks https://review.opendev.org/730660 | 18:34 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: ensure-pip: update include to import_tasks https://review.opendev.org/730661 | 18:36 |
| *** EmilienM has quit IRC | 18:39 | |
| *** EmilienM has joined #zuul | 18:40 | |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: multi-node-bridge: update include to import_tasks https://review.opendev.org/730662 | 18:40 |
| *** jamesmcarthur has joined #zuul | 18:46 | |
| *** yolanda has quit IRC | 18:49 | |
| avass | AJaeger: are the gentoo tests flaky? | 18:50 |
| *** jamesmcarthur has quit IRC | 18:51 | |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: bindep: update include to import_tasks https://review.opendev.org/730660 | 18:52 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: ensure-pip: update include to import_tasks https://review.opendev.org/730661 | 18:52 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Fix deprecation warning from multinode tests https://review.opendev.org/730479 | 18:52 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: multi-node-bridge: update include to import_tasks https://review.opendev.org/730662 | 18:52 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: use-docker-mirror: update include to import_tasks https://review.opendev.org/730664 | 18:52 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: persistent-firewall: update include to import_tasks https://review.opendev.org/730665 | 18:52 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: configure-mirrors: update include to import_tasks https://review.opendev.org/730666 | 18:52 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: ensure-podman: update include to import_tasks https://review.opendev.org/730667 | 18:52 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: fetch-subunit-output: update include to import_tasks https://review.opendev.org/730668 | 18:52 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: ensure-virtualenv: update include to import_tasks https://review.opendev.org/730669 | 18:52 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: add-build-sshkey: update include to import_tasks https://review.opendev.org/730670 | 18:52 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: ensure-package-repositories: update include to import_tasks https://review.opendev.org/730671 | 18:52 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: ensure-bazel: update include to import_tasks https://review.opendev.org/730672 | 18:52 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: tox: update include to import_tasks https://review.opendev.org/730673 | 18:52 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: test-base-roles: update include to import_tasks https://review.opendev.org/730674 | 18:52 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: test-base-roles: update include to import_playbook https://review.opendev.org/730674 | 18:54 |
| AJaeger | avass: the multinode test seems broken | 18:59 |
| AJaeger | avass: you're sending way too many changes ;) Thanks! | 18:59 |
| AJaeger | avass: could you update later this week https://review.opendev.org/#/c/719322/ , so that we can get rid of these, please? | 19:01 |
| AJaeger | zuul-jobs maintainer, please review https://review.opendev.org/729407 https://review.opendev.org/730500 https://review.opendev.org/730477 https://review.opendev.org/729520 https://review.opendev.org/728708 | 19:02 |
| avass | AJaeger: I was using ansible2.9 when testing the local runner and got a lot of deprecation warnings, so I might as well fix it :) | 19:03 |
| avass | AJaeger: yep, I'll update 719322 tomorrow so we can merge it :) | 19:03 |
| AJaeger | avass: ah, thanks! | 19:04 |
| AJaeger | avass: https://review.opendev.org/727642 might also be nice to get updated to finish that stack | 19:07 |
| avass | AJaeger: working on it right now :) | 19:08 |
| AJaeger | thanks - have a good evening! | 19:09 |
| *** jamesmcarthur has joined #zuul | 19:26 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add tutorial tests https://review.opendev.org/728194 | 19:37 |
| avass | hopefully that's correct but I'll need to add tests tomorrow | 19:58 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add linting rule to enforce no-same-owner policy https://review.opendev.org/727642 | 19:58 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add linting rule to enforce no-same-owner policy https://review.opendev.org/727642 | 19:58 |
| *** y2kenny has quit IRC | 20:35 | |
| mordred | avass: don't hate me - but I just -1'd most of your update include stack | 20:38 |
| mordred | avass, AJaeger: tl;dr - include is runtime, import is compile time - so if somethign is conditional it's better to use include_ - that way the include doesn't happen if the when doesn't work- as opposed to the when beinga pplied to each task in the file using import | 20:39 |
| mordred | seems like the sort of thing a linter rule could potentially find - "this task has a when on it, use include_tasks" | 20:40 |
| *** rfolco|rover has quit IRC | 20:42 | |
| *** sgw has joined #zuul | 21:56 | |
| *** marvs has quit IRC | 22:07 | |
| *** tumble has quit IRC | 22:24 | |
| *** jamesmcarthur has quit IRC | 22:24 | |
| *** jamesmcarthur has joined #zuul | 22:25 | |
| *** jamesmcarthur has quit IRC | 22:32 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add tutorial tests https://review.opendev.org/728194 | 22:36 |
| *** jamesmcarthur has joined #zuul | 22:38 | |
| *** jamesmcarthur has quit IRC | 22:42 | |
| *** tosky has quit IRC | 23:18 | |
| *** jamesmcarthur has joined #zuul | 23:41 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!