| opendevreview | James E. Blair proposed zuul/zuul master: Add ExecutorApi https://review.opendev.org/c/zuul/zuul/+/770902 | 00:31 |
|---|---|---|
| opendevreview | James E. Blair proposed zuul/zuul master: Change zone handling in ExecutorApi https://review.opendev.org/c/zuul/zuul/+/787833 | 00:31 |
| opendevreview | James E. Blair proposed zuul/zuul master: Switch to string constants in BuildRequest https://review.opendev.org/c/zuul/zuul/+/791849 | 00:31 |
| opendevreview | James E. Blair proposed zuul/zuul master: Clean up Executor API build request locking and add tests https://review.opendev.org/c/zuul/zuul/+/788624 | 00:31 |
| opendevreview | James E. Blair proposed zuul/zuul master: Fix race with watches in ExecutorAPI https://review.opendev.org/c/zuul/zuul/+/792300 | 00:31 |
| opendevreview | James E. Blair proposed zuul/zuul master: Execute builds via ZooKeeper https://review.opendev.org/c/zuul/zuul/+/788988 | 00:31 |
| opendevreview | James E. Blair proposed zuul/zuul master: Move build request cleanup from executor to scheduler https://review.opendev.org/c/zuul/zuul/+/794687 | 00:31 |
| opendevreview | James E. Blair proposed zuul/zuul master: Switch to ZooKeeper backed merge result events https://review.opendev.org/c/zuul/zuul/+/784195 | 00:31 |
| opendevreview | James E. Blair proposed zuul/zuul master: Refactor pipeline processing in run handler https://review.opendev.org/c/zuul/zuul/+/795985 | 00:31 |
| opendevreview | James E. Blair proposed zuul/zuul master: Tenant, pipeline and queue locks in Zookeeper https://review.opendev.org/c/zuul/zuul/+/795993 | 00:31 |
| opendevreview | Merged zuul/nodepool master: Azure: reconcile config objects https://review.opendev.org/c/zuul/nodepool/+/779898 | 01:18 |
| opendevreview | James E. Blair proposed zuul/zuul master: Correct URL in matrix howto https://review.opendev.org/c/zuul/zuul/+/796328 | 01:29 |
| corvus | tristanC, mordred: ^ oops | 01:30 |
| opendevreview | Merged zuul/nodepool master: Azure: Handle IPv6 https://review.opendev.org/c/zuul/nodepool/+/780400 | 02:23 |
| *** marios is now known as marios|ruck | 05:16 | |
| *** iurygregory_ is now known as iurygregory | 06:19 | |
| *** jpena|off is now known as jpena | 06:34 | |
| *** rpittau|afk is now known as rpittau | 07:13 | |
| opendevreview | Tobias Henkel proposed zuul/zuul master: Remove gearman job from build when finished https://review.opendev.org/c/zuul/zuul/+/796389 | 07:49 |
| opendevreview | Tobias Henkel proposed zuul/zuul master: Freeze secrets on job level instead of playbook level https://review.opendev.org/c/zuul/zuul/+/796412 | 09:40 |
| *** sshnaidm|afk is now known as sshnaidm | 11:21 | |
| *** jpena is now known as jpena|lunch | 11:25 | |
| opendevreview | Tobias Henkel proposed zuul/zuul master: Cache frozen secrets https://review.opendev.org/c/zuul/zuul/+/796431 | 12:05 |
| opendevreview | Tobias Henkel proposed zuul/zuul master: Cache frozen secrets https://review.opendev.org/c/zuul/zuul/+/796431 | 12:08 |
| *** jpena|lunch is now known as jpena | 12:17 | |
| zbr | i want to expose the fact that user can trigger extra jobs using "check experimental" inside the success-message posted by zuul on reviews but apparently it does not allow me to customize success-message at project level. | 12:17 |
| zbr | and this needs to be part of check pipeline, so obviously that I cannot update the check pipeline default config as that applies to all projects. | 12:18 |
| avass[m] | no that's a pipeline thing at the moment | 12:18 |
| zbr | so basically is not possible to expose that information from a project. | 12:19 |
| zbr | i wonder if we could use a trick like a project var that would be included inside the message. undefined it would do nothing. | 12:21 |
| avass[m] | maybe exposing that in the project stanza as project.check.start-message or something similar is a reasonable thing to do | 12:22 |
| zbr | start-message disappears from gerrit ui quickly, even if we include extra-info in it, we will still need to include the same text in success-message. | 12:24 |
| zbr | for failure I find it optional (but probably would not hurt to include it for consistency) | 12:24 |
| avass[m] | zbr: yeah, that was part of "or something similar" :9 | 12:24 |
| avass[m] | I mean generally exposing the pipeline messages on start/dequeue/success/fail to the project stanza | 12:25 |
| zbr | if *-message texts would support jinja2 and expose defined vars, we could easily implement that. | 12:26 |
| avass[m] | I think you can also report comments to files, but I don't think zuul_retrun allows you to report change comments | 12:30 |
| avass[m] | but I think it supports warnings | 12:30 |
| avass[m] | so maybe that's easy to extend? | 12:30 |
| opendevreview | Tobias Henkel proposed zuul/zuul master: Re-use empty nodeset in job variants https://review.opendev.org/c/zuul/zuul/+/796458 | 12:32 |
| dmsimard | o/ great matrix docs at https://zuul-ci.org/docs/zuul/howtos/matrix.html but the link at the top doesn't render properly: https://matrix.to/#/#_oftc_#zuul:matrix.org | 13:07 |
| fungi | dmsimard: yeah, https://review.opendev.org/796328 should hopefully fix it | 13:14 |
| * fungi reviews now | 13:14 | |
| fungi | approved just now | 13:14 |
| dmsimard | way ahead of me :) | 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 | 14:13 |
| *** rpittau is now known as rpittau|afk | 14:29 | |
| 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:55 |
| *** jpena is now known as jpena|off | 15:18 | |
| nhicher | hello, I'm testing nodepool driver openshiftpod and there is issue with openshift lib https://softwarefactory-project.io/paste/show/2042/, depends were updated on https://review.opendev.org/c/zuul/nodepool/+/765873 | 15:20 |
| opendevreview | Tobias Henkel proposed zuul/zuul master: Intern strings loaded from zuul config https://review.opendev.org/c/zuul/zuul/+/796520 | 15:56 |
| *** marios|ruck is now known as marios|out | 15:57 | |
| corvus | tristanC, Clark: see msg from nhicher | 16:03 |
| tristanC | looking at the code, the None error should not happen without a config error, nhicher is there any "Couldn't load client from config" message before in the logs? | 16:08 |
| opendevreview | Tristan Cacqueray proposed zuul/nodepool master: kubernetes: refactor client creation to utils_k8s https://review.opendev.org/c/zuul/nodepool/+/777022 | 16:14 |
| nhicher | tristanC: you're right, I have /var/lib/nodepool/.kube/config, but not /root/.kube/config. I've added the file and the issue is fixed, thanks | 16:21 |
| nhicher | tristanC: there is deletion issue https://softwarefactory-project.io/paste/show/2044/ | 16:22 |
| *** ricolin_ is now known as ricolin | 16:26 | |
| opendevreview | Tristan Cacqueray proposed zuul/nodepool master: openshiftpods: fix delete_namespaced_pod arguments https://review.opendev.org/c/zuul/nodepool/+/796524 | 16:43 |
| tristanC | nhicher: that was unexpected, ^ should fix the TypeError | 16:43 |
| *** ricolin_ is now known as ricolin | 17:32 | |
| nhicher | tristanC: thanks | 17:35 |
| *** sshnaidm is now known as sshnaidm|afk | 21:15 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!