| *** timburke has quit IRC | 00:00 | |
| *** timburke has joined #zuul | 00:00 | |
| *** timburke has quit IRC | 00:26 | |
| *** zenkuro has quit IRC | 01:37 | |
| *** Clark[m] has quit IRC | 01:40 | |
| *** josefwells_ has joined #zuul | 02:53 | |
| *** josefwells has quit IRC | 03:00 | |
| *** josefwells_ has quit IRC | 03:10 | |
| *** mgoddard has quit IRC | 03:33 | |
| *** mgoddard has joined #zuul | 03:33 | |
| *** marios has joined #zuul | 05:28 | |
| *** rpittau|afk is now known as rpittau | 07:26 | |
| *** jpena|off is now known as jpena | 07:37 | |
| *** hashar has joined #zuul | 08:29 | |
| *** sshnaidm|afk is now known as sshnaidm | 08:51 | |
| *** sshnaidm has quit IRC | 08:58 | |
| *** sshnaidm has joined #zuul | 08:59 | |
| *** mhu has joined #zuul | 09:03 | |
| *** bhagyashris__ has joined #zuul | 09:06 | |
| *** bhagyashris_ has quit IRC | 09:07 | |
| *** zenkuro has joined #zuul | 10:38 | |
| *** bhagyashris__ is now known as bhagyashris | 10:54 | |
| tobiash[m] | ianw: | 10:59 |
|---|---|---|
| tobiash[m] | ianw: commented on ^, otherwise lgtm | 11:00 |
| *** holser has joined #zuul | 11:24 | |
| *** jpena is now known as jpena|lunch | 11:34 | |
| *** eliadcohen has quit IRC | 11:47 | |
| *** eliadcohen has joined #zuul | 11:50 | |
| *** tosky has joined #zuul | 11:59 | |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Test zuul-client console-stream https://review.opendev.org/c/zuul/zuul/+/794854 | 12:15 |
| *** josefwells_ has joined #zuul | 12:19 | |
| *** marios is now known as marios|call | 12:21 | |
| *** jpena|lunch is now known as jpena | 12:21 | |
| *** marios|call is now known as marios | 12:30 | |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Test zuul-client console-stream https://review.opendev.org/c/zuul/zuul/+/794854 | 12:58 |
| opendevreview | Felix Edel proposed zuul/zuul master: WIP: Lock/unlock nodes on executor server https://review.opendev.org/c/zuul/zuul/+/774610 | 13:13 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Test zuul-client console-stream https://review.opendev.org/c/zuul/zuul/+/794854 | 13:18 |
| *** zenkuro has quit IRC | 13:26 | |
| *** zenkuro has joined #zuul | 13:26 | |
| *** Clark[m] has joined #zuul | 13:32 | |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Test zuul-client console-stream https://review.opendev.org/c/zuul/zuul/+/794854 | 13:59 |
| opendevreview | Mohammed Naser proposed zuul/zuul-helm master: Allow overriding image names, tags and pull policy https://review.opendev.org/c/zuul/zuul-helm/+/795112 | 14:04 |
| *** JohannesFoufas[m] has joined #zuul | 14:43 | |
| JohannesFoufas[m] | woah, finally found my way in again | 14:44 |
| JohannesFoufas[m] | anyone knows here we can find Emit stats for semaphore holders? | 14:45 |
| corvus | Johannes Foufas: yep, they should be in gauges.zuul.tenant.<tenant>.semaphore.<semaphore name>.holders | 14:46 |
| *** zenkuro has quit IRC | 14:49 | |
| *** zenkuro has joined #zuul | 14:49 | |
| JohannesFoufas[m] | thank you, I think @vas | 14:50 |
| JohannesFoufas[m] | will debugg, sicne I can not find anything there | 14:50 |
| opendevreview | Merged zuul/zuul-website master: Use matrix for webchat links https://review.opendev.org/c/zuul/zuul-website/+/793668 | 14:53 |
| *** timburke has joined #zuul | 15:00 | |
| zenkuro | hi, is there a way to set a limit to a particular job? | 15:19 |
| clarkb | zenkuro: there are timeouts and you can use labels to limit where jobs run. What sort of limits are you looking for? | 15:22 |
| fungi | also you can limit concurrency with semaphores and/or the serial pipeline manager | 15:23 |
| zenkuro | clarkb: I want to make sure that my job that is running on static machine will not multiple times... since Im planing to use another machine to 'restore from snapshot' the main one | 15:24 |
| zenkuro | fungi: thanks, I defiantly need to get familiar with this thing. | 15:24 |
| corvus | zenkuro: sounds like semaphores are what you need | 15:25 |
| clarkb | by default each label should only be used by one job at a time. You could then have a cleanup phase playbook do the restore. Or do that in a prerun | 15:25 |
| fungi | zenkuro: zuul will only run one job at a time on each job node anyway, unless you've done something to make it look like multiple nodes | 15:25 |
| clarkb | but ya also semaphores are useful if you want to break that up across jobs | 15:25 |
| fungi | s/node/nodeset/ i mean | 15:26 |
| corvus | https://zuul-ci.org/docs/zuul/reference/job_def.html#attr-job.semaphores and https://zuul-ci.org/docs/zuul/reference/semaphore_def.html | 15:26 |
| *** timburke has quit IRC | 15:39 | |
| *** timburke has joined #zuul | 15:40 | |
| mnaser | corvus: do i remember that you shared helm values/etc for tls zk not-so-long ago? | 15:45 |
| opendevreview | Matthieu Huin proposed zuul/zuul-client master: Add console-stream subcommand https://review.opendev.org/c/zuul/zuul-client/+/751238 | 15:46 |
| opendevreview | Matthieu Huin proposed zuul/zuul-client master: Add console-stream subcommand https://review.opendev.org/c/zuul/zuul-client/+/751238 | 15:49 |
| *** timburke has quit IRC | 15:53 | |
| *** timburke has joined #zuul | 16:00 | |
| *** rpittau is now known as rpittau|afk | 16:01 | |
| *** marios is now known as marios|out | 16:05 | |
| *** zenkuro has quit IRC | 16:14 | |
| corvus | mnaser: i switched gerrit's zuul to just plan k8s yaml (because it was easier for a singleton instance to do that); but they're based on the helm charts, so should be easy to incorporate: https://gerrit.googlesource.com/zuul/ops/+/refs/heads/master/k8s/zuul.yaml | 16:15 |
| mnaser | corvus: ok awesome, i'll hack away.. finally doing the tls thing today ;< | 16:16 |
| corvus | mnaser: ++ | 16:16 |
| corvus | mnaser: there's also tls support in both the current operator (using zk-ca.sh), and the kopf update (stack ends here: https://review.opendev.org/786349 if you want to look at it, i recommend just checking that out locally); the kopf version uses cert-manager (it's about the same thing as what i did for gerrit). | 16:18 |
| *** marios|out has quit IRC | 16:19 | |
| avass[m] | mnaser: i have a change in zuul-helm that sets up TLS for zookeeper. I don't remember what state it is in though | 16:20 |
| avass[m] | But maybe that helps | 16:20 |
| mnaser | avass[m]: neat, zuul-helm is failing tests so i think i need to unbreak it | 16:20 |
| avass[m] | I think i just did a code dump from our charts :) | 16:21 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Test zuul-client console-stream https://review.opendev.org/c/zuul/zuul/+/794854 | 16:30 |
| *** jpena is now known as jpena|off | 16:40 | |
| opendevreview | Ade Lee proposed zuul/zuul-jobs master: Add role to enable FIPS on a node https://review.opendev.org/c/zuul/zuul-jobs/+/788778 | 17:47 |
| *** timburke_ has joined #zuul | 17:50 | |
| *** timburke has quit IRC | 17:56 | |
| clarkb | if anyone has time for a review https://review.opendev.org/c/zuul/zuul/+/794878 is a small cleanup that I did after getting local test runs going again | 17:56 |
| *** timburke_ has quit IRC | 18:17 | |
| *** timburke_ has joined #zuul | 18:28 | |
| *** spamaps[m] has joined #zuul | 19:24 | |
| *** spamaps[m] has left #zuul | 19:25 | |
| *** jangutter_ has quit IRC | 19:33 | |
| clarkb | corvus tobiash[m] I don't see benjamin in here but https://review.opendev.org/c/zuul/zuul/+/788481 lgtm. Idid leave a couple of thoughts though | 20:39 |
| clarkb | similar situation on https://review.opendev.org/c/zuul/zuul/+/663413. Both lgtm but I had a couple of ideas | 20:53 |
| corvus | clarkb: i replied on the first one | 20:53 |
| clarkb | thanks, and ya I agree re picking a behavior. | 21:00 |
| clarkb | It wasn't clear to me from the spec which behaviors we wanted though | 21:00 |
| corvus | clarkb: ++ and also replied on the 2nd | 21:01 |
| clarkb | wfm thanks | 21:02 |
| clarkb | ianw: I responded to your comments on https://review.opendev.org/c/zuul/zuul/+/794371 can you take a look and check that I'm making sense? | 21:10 |
| opendevreview | Merged zuul/zuul master: Cleanup Zuul's stdout/stderr output https://review.opendev.org/c/zuul/zuul/+/794878 | 21:18 |
| *** hashar has quit IRC | 21:40 | |
| *** zenkuro has joined #zuul | 21:41 | |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Test zuul-client console-stream https://review.opendev.org/c/zuul/zuul/+/794854 | 21:50 |
| ianw | clarkb: i can put a >200 <300 in if you feel strongly. I can't find any documentation on the return codes, I suspect because we're doing queries here we shouldn't get things like created etc. | 21:59 |
| clarkb | ianw: ya that is probably the case. In my head I guess it was to help avoid problems like the one we have today where it mysteriously doesn't work ebcause we're either filtering too little or too much | 21:59 |
| clarkb | if you want to just go with 200 for now and address and api weirdness later should it show up I'm ok with that too I guess | 22:00 |
| *** tosky has quit IRC | 22:04 | |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Test zuul-client console-stream https://review.opendev.org/c/zuul/zuul/+/794854 | 22:34 |
| opendevreview | James E. Blair proposed zuul/zuul master: Fix layout cache and adding a project to a pipeline https://review.opendev.org/c/zuul/zuul/+/795189 | 23:07 |
| opendevreview | James E. Blair proposed zuul/zuul master: Fix reporting merge failure behind change adding ppc https://review.opendev.org/c/zuul/zuul/+/795190 | 23:07 |
| mhu | corvus, guillaumec I'm having a bit of trouble testing the console stream with zuul-client in https://review.opendev.org/794854 | 23:10 |
| mhu | in the latest build I can see that zuul-web can't maintain the websocket connection with zuul-client apparently, but no idea why https://d29e095404ca86d71246-7daaa7daeb1030fb2122101d5d422feb.ssl.cf2.rackcdn.com/794854/14/check/zuul-tox-zuul-client/9da7d02/testr_results.html | 23:11 |
| mhu | Any ideas why? | 23:12 |
| clarkb | corvus: why does the test job run by the new test in https://review.opendev.org/c/zuul/zuul/+/795189/1/tests/unit/test_v3.py fail? | 23:36 |
| clarkb | looks like its playbook just runs an empty task list | 23:36 |
| corvus | clarkb: search for "failJob" | 23:37 |
| corvus | (artificially induced failure) | 23:38 |
| clarkb | oh that is an externally induced side effect | 23:38 |
| corvus | clarkb: yep; that test class doesn't actually run ansible | 23:38 |
| corvus | (still needs a playbook to make zuul happy though; but contents don't matter) | 23:39 |
| clarkb | gotcha | 23:40 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!