| 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:28 |
|---|---|---|
| *** bhagyashris_ has joined #zuul | 00:31 | |
| *** bhagyashris has quit IRC | 00:38 | |
| opendevreview | Ian Wienand proposed zuul/zuul master: github: fallback to api_token when can't find installation https://review.opendev.org/c/zuul/zuul/+/794688 | 01:11 |
| opendevreview | Ian Wienand proposed zuul/zuul master: github: log graphql errors https://review.opendev.org/c/zuul/zuul/+/794371 | 01:34 |
| opendevreview | Ian Wienand proposed zuul/zuul master: github: show query response on missing fields https://review.opendev.org/c/zuul/zuul/+/794372 | 01:34 |
| opendevreview | Ian Wienand proposed zuul/zuul master: github: handle exceptions from _updateCanMergeInfo https://review.opendev.org/c/zuul/zuul/+/794393 | 01:34 |
| opendevreview | Ian Wienand proposed zuul/zuul master: github: fallback to api_token when can't find installation https://review.opendev.org/c/zuul/zuul/+/794688 | 03:18 |
| *** josefwells has quit IRC | 04:13 | |
| opendevreview | Ian Wienand proposed zuul/zuul master: github: fallback to api_token when can't find installation https://review.opendev.org/c/zuul/zuul/+/794688 | 04:59 |
| opendevreview | Ian Wienand proposed zuul/zuul master: github: fallback to api_token when can't find installation https://review.opendev.org/c/zuul/zuul/+/794688 | 05:03 |
| *** thogarre has joined #zuul | 05:10 | |
| *** marios has joined #zuul | 05:18 | |
| opendevreview | Ian Wienand proposed zuul/zuul master: github: fallback to api_token when can't find installation https://review.opendev.org/c/zuul/zuul/+/794688 | 05:25 |
| opendevreview | Ian Wienand proposed zuul/zuul master: github: fallback to api_token when can't find installation https://review.opendev.org/c/zuul/zuul/+/794688 | 05:50 |
| opendevreview | Ian Wienand proposed zuul/zuul master: github: fallback to api_token when can't find installation https://review.opendev.org/c/zuul/zuul/+/794688 | 06:15 |
| *** bhagyashris_ is now known as bhagyashris | 06:33 | |
| *** rpittau|afk is now known as rpittau | 07:05 | |
| *** tosky has joined #zuul | 07:23 | |
| *** hashar has joined #zuul | 07:46 | |
| *** jpena|off is now known as jpena | 07:54 | |
| opendevreview | Albin Vass proposed zuul/zuul master: Gitlab: log and handle all merge request approval errors https://review.opendev.org/c/zuul/zuul/+/794735 | 08:17 |
| avass[m] | guillaumec: you may want to take a look at that ^, we're getting some kind of error that's never logged | 08:21 |
| guillaumec | avass: yes i had seen things like that, I have a local change which setups gitlab during quick-start. I had to write loops to perform+poll to check results | 08:33 |
| guillaumec | getting 502 even if the action was performed then 40X (in a loop where it tried to performed an action until 200/201 ...) | 08:35 |
| *** jpward has quit IRC | 09:11 | |
| *** hashar is now known as Guest833 | 09:16 | |
| *** hashar has joined #zuul | 09:16 | |
| *** Guest833 has quit IRC | 09:21 | |
| *** jangutter has joined #zuul | 09:48 | |
| *** jangutter_ has quit IRC | 09:54 | |
| *** jangutter has quit IRC | 10:01 | |
| *** jangutter has joined #zuul | 10:02 | |
| *** jangutter has quit IRC | 10:14 | |
| *** jangutter has joined #zuul | 10:15 | |
| *** jangutter has quit IRC | 10:23 | |
| *** jangutter has joined #zuul | 10:24 | |
| *** hashar_ has joined #zuul | 10:41 | |
| *** hashar is now known as Guest844 | 10:41 | |
| *** hashar_ is now known as hashar | 10:41 | |
| *** Guest844 has quit IRC | 10:47 | |
| *** hashar has quit IRC | 10:53 | |
| *** hashar has joined #zuul | 10:54 | |
| *** jpena is now known as jpena|lunch | 11:32 | |
| *** josefwells has joined #zuul | 11:49 | |
| *** jpena|lunch is now known as jpena | 12:20 | |
| *** kapoios_allos has joined #zuul | 12:41 | |
| *** kapoios_allos has quit IRC | 12:42 | |
| *** bhagyashris_ has joined #zuul | 12:50 | |
| *** bhagyashris has quit IRC | 12:56 | |
| *** bhagyashris_ is now known as bhagyashris | 12:58 | |
| *** hashar has quit IRC | 13:24 | |
| *** rpittau is now known as rpittau|afk | 13:37 | |
| corvus | i have restarted opendev's zuul on current master (includes the changes to cache unparsed config in zk). as expected, there is an increase in zk usage: about 3+x the node count, and about 2x the data size (40k nodes and 22mib) | 13:52 |
| corvus | so far the zk performance metrics seem more or less unchanged | 13:53 |
| avass[m] | guillaumec: I'm probably gonna take another look at error handling and logging in the gitlab driver later. There were some other issues as well, but it's mostly logging | 14:09 |
| *** gmann is now known as gmann_afk | 14:40 | |
| *** mhu has quit IRC | 14:42 | |
| *** gmann_afk is now known as gmann | 15:13 | |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Test zuul-client console-stream https://review.opendev.org/c/zuul/zuul/+/794854 | 15:44 |
| opendevreview | Clark Boylan proposed zuul/zuul master: Consistently use UTC timestamps in fake_gitlab https://review.opendev.org/c/zuul/zuul/+/794856 | 15:58 |
| clarkb | corvus: ^ I think that allows me to run all the tests locally now and have them succeed. I'm rerunning the full set now to confirm though | 15:58 |
| opendevreview | Monty Taylor proposed zuul/zuul master: Add flag for disabling log streaming on a host https://review.opendev.org/c/zuul/zuul/+/721953 | 15:59 |
| opendevreview | Monty Taylor proposed zuul/zuul master: Use node image to build javascript https://review.opendev.org/c/zuul/zuul/+/728690 | 16:04 |
| *** marios is now known as marios|out | 16:04 | |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Test zuul-client console-stream https://review.opendev.org/c/zuul/zuul/+/794854 | 16:10 |
| opendevreview | Monty Taylor proposed zuul/zuul-jobs master: Add a job for publishing a site to netlify https://review.opendev.org/c/zuul/zuul-jobs/+/739047 | 16:10 |
| *** marios|out has quit IRC | 16:18 | |
| *** jpena is now known as jpena|off | 16:38 | |
| clarkb | Ran: 1125 tests in 2448.1198 sec. and py38: commands succeeded | 16:54 |
| clarkb | now to try it on python39 | 16:56 |
| opendevreview | Monty Taylor proposed zuul/zuul master: DNM test patch https://review.opendev.org/c/zuul/zuul/+/794865 | 16:58 |
| corvus | clarkb: the ssl test cause of the 39 stuff, i exclude it locally | 17:02 |
| clarkb | corvus: ya I was going to take a look at those too to see if I can maybe fix that up. But wanted to see failures first before committing to anything | 17:02 |
| corvus | clarkb: tbh, i think i would prefer to just leave it till we drop gear | 17:03 |
| clarkb | corvus: that works too (I thought gear updated to address this though?) | 17:03 |
| corvus | we're really close -- close enough that it's like, every minute we spend dealing with that is one minute less we spend on dropping it :) | 17:04 |
| clarkb | also there are a couple of places where stdout/stderr leak out to the console even though nothing is broken or having problems that I might look at cleaning up so that the noise isnt there | 17:04 |
| clarkb | corvus: https://review.opendev.org/c/zuul/zuul/+/794856 should fix the gitlab test though and I suspect we don't want to clean those up :) | 17:08 |
| corvus | clarkb: ++ that jives with what i saw | 17:09 |
| clarkb | corvus: fwiw py39 tests passed for me. So I'm happy :) | 17:40 |
| corvus | clarkb: without skipping ssl? | 17:43 |
| clarkb | correct | 17:43 |
| clarkb | just running tox -epy39 | 17:43 |
| clarkb | maybe my suse python ssl implementation is hacked up to make things like that better | 17:43 |
| opendevreview | Monty Taylor proposed zuul/zuul master: DNM test patch https://review.opendev.org/c/zuul/zuul/+/794865 | 17:52 |
| opendevreview | Clark Boylan proposed zuul/zuul master: Cleanup Zuul's stdout/stderr output https://review.opendev.org/c/zuul/zuul/+/794878 | 18:44 |
| clarkb | corvus: ^ tahts the other cleanup I was talking about. Makes it easier to find what you should look at when things break | 18:44 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Test zuul-client console-stream https://review.opendev.org/c/zuul/zuul/+/794854 | 19:06 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Test zuul-client console-stream https://review.opendev.org/c/zuul/zuul/+/794854 | 19:34 |
| opendevreview | Merged zuul/zuul master: Consistently use UTC timestamps in fake_gitlab https://review.opendev.org/c/zuul/zuul/+/794856 | 21:20 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Test zuul-client console-stream https://review.opendev.org/c/zuul/zuul/+/794854 | 21:27 |
| *** gmann is now known as gmann_afk | 21:51 | |
| *** jangutter_ has joined #zuul | 21:59 | |
| *** jangutter_ has joined #zuul | 22:00 | |
| *** jangutte_ has joined #zuul | 22:01 | |
| *** jangutte_ has quit IRC | 22:01 | |
| *** tosky has quit IRC | 22:01 | |
| *** jangutter_ has quit IRC | 22:01 | |
| *** jangutter_ has joined #zuul | 22:02 | |
| *** jangutter_ has quit IRC | 22:02 | |
| *** jangutter_ has joined #zuul | 22:04 | |
| *** jangutter has quit IRC | 22:06 | |
| clarkb | ianw: in https://review.opendev.org/c/zuul/zuul/+/794688 why not error if the app perms don't allow this and get people to fix the app perms? that seems preferable for a happy system than trying to manage two sets of credentials? | 22:15 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Test zuul-client console-stream https://review.opendev.org/c/zuul/zuul/+/794854 | 22:15 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: Test zuul-client console-stream https://review.opendev.org/c/zuul/zuul/+/794854 | 22:50 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!