| *** armstrongs has quit IRC | 00:08 | |
| *** jamesmcarthur has joined #zuul | 00:49 | |
| *** jamesmcarthur has quit IRC | 01:17 | |
| *** jamesmcarthur has joined #zuul | 01:17 | |
| *** jamesmcarthur has quit IRC | 01:22 | |
| *** Goneri has quit IRC | 01:27 | |
| *** zenkuro has quit IRC | 02:00 | |
| *** jamesmcarthur has joined #zuul | 02:14 | |
| *** jamesmcarthur has quit IRC | 02:29 | |
| openstackgerrit | Pierre-Louis Bonicoli proposed zuul/zuul master: gitlab: handle protected branches https://review.opendev.org/756900 | 02:33 |
|---|---|---|
| *** hamalq_ has quit IRC | 02:45 | |
| *** bhavikdbavishi has joined #zuul | 03:08 | |
| *** bhavikdbavishi1 has joined #zuul | 03:11 | |
| *** bhavikdbavishi has quit IRC | 03:12 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 03:12 | |
| *** jamesmcarthur has joined #zuul | 03:21 | |
| *** jamesmcarthur has quit IRC | 03:26 | |
| *** wuchunyang has joined #zuul | 04:05 | |
| *** wuchunyang has quit IRC | 04:14 | |
| *** jamesmcarthur has joined #zuul | 04:15 | |
| *** pmannidi has joined #zuul | 04:26 | |
| *** jamesmcarthur has quit IRC | 04:26 | |
| *** sai438 has quit IRC | 04:28 | |
| *** evrardjp has quit IRC | 04:34 | |
| *** evrardjp has joined #zuul | 04:34 | |
| *** bhagyashris is now known as bhagyashris|sick | 05:09 | |
| *** mattd01 has quit IRC | 05:32 | |
| *** jamesmcarthur has joined #zuul | 05:38 | |
| *** jamesmcarthur has quit IRC | 05:43 | |
| *** vishalmanchanda has joined #zuul | 05:58 | |
| *** yolanda has quit IRC | 06:14 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Lock nodes in executor instead of scheduler https://review.opendev.org/756304 | 06:14 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Switch to using zookeeper instead of gearman for mergers https://review.opendev.org/759263 | 06:14 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Switch to using zookeeper instead of gearman for jobs https://review.opendev.org/744416 | 06:18 |
| *** rpittau|afk is now known as rpittau | 06:21 | |
| *** jfoufas1 has joined #zuul | 06:27 | |
| *** jamesmcarthur has joined #zuul | 06:54 | |
| *** jamesmcarthur has quit IRC | 06:58 | |
| *** jcapitao has joined #zuul | 07:03 | |
| *** jamesmcarthur has joined #zuul | 07:24 | |
| *** frenzyfriday has joined #zuul | 07:27 | |
| *** bhavikdbavishi has quit IRC | 07:27 | |
| *** jamesmcarthur has quit IRC | 07:28 | |
| *** tosky has joined #zuul | 07:36 | |
| *** jpena|off is now known as jpena | 07:56 | |
| *** bhavikdbavishi has joined #zuul | 07:57 | |
| *** holser has quit IRC | 07:59 | |
| *** holser has joined #zuul | 08:00 | |
| *** saneax has joined #zuul | 08:25 | |
| *** nils has joined #zuul | 08:34 | |
| *** vorotech has joined #zuul | 08:39 | |
| *** vorotech has quit IRC | 08:40 | |
| *** ekwall has joined #zuul | 08:44 | |
| *** zenkuro has joined #zuul | 08:44 | |
| *** zenkuro has quit IRC | 09:01 | |
| *** vorotech has joined #zuul | 09:01 | |
| *** zenkuro has joined #zuul | 09:01 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Component Registry in ZooKeeper https://review.opendev.org/759187 | 09:12 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: WIP: ZooKeeper Work abstraction as a replacement for Gearman Jobs https://review.opendev.org/759192 | 09:12 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Switch to using zookeeper instead of gearman for jobs https://review.opendev.org/744416 | 09:12 |
| *** vorotech has quit IRC | 09:28 | |
| *** vorotech has joined #zuul | 09:31 | |
| *** vorotech has joined #zuul | 09:32 | |
| *** Eighth_Doctor has quit IRC | 10:24 | |
| *** mordred has quit IRC | 10:24 | |
| *** masterpe has quit IRC | 10:24 | |
| *** bhavikdbavishi has quit IRC | 10:30 | |
| *** masterpe has joined #zuul | 10:33 | |
| *** jcapitao is now known as jcapitao_lunch | 10:34 | |
| *** zenkuro has quit IRC | 10:44 | |
| *** Eighth_Doctor has joined #zuul | 10:50 | |
| *** mordred has joined #zuul | 10:50 | |
| *** yolanda has joined #zuul | 10:57 | |
| avass | I think I found the problem. The branch 'patch-9' exists and I believe that's okay. But someone else created a branch called 'refs/heads/patch-9' and that's causing the merger to crash at: https://opendev.org/zuul/zuul/src/branch/master/zuul/merger/merger.py#L620 | 11:01 |
| *** bhavikdbavishi has joined #zuul | 11:21 | |
| *** jpena is now known as jpena|lunch | 11:31 | |
| *** rfolco has joined #zuul | 11:53 | |
| *** vorotech has quit IRC | 11:54 | |
| *** jcapitao_lunch is now known as jcapitao | 11:59 | |
| *** jamesmcarthur has joined #zuul | 12:01 | |
| *** mattd01 has joined #zuul | 12:04 | |
| *** mach1na has joined #zuul | 12:05 | |
| *** mattd01 has left #zuul | 12:05 | |
| *** jamesmcarthur has quit IRC | 12:06 | |
| *** wuchunyang has joined #zuul | 12:06 | |
| *** mach1na has quit IRC | 12:09 | |
| *** rlandy has joined #zuul | 12:09 | |
| *** wuchunyang has quit IRC | 12:11 | |
| *** vorotech has joined #zuul | 12:19 | |
| openstackgerrit | Albin Vass proposed zuul/zuul master: WIP: bugfix: branches with pattern 'refs/heads/br' fails merge jobs https://review.opendev.org/759412 | 12:19 |
| openstackgerrit | Albin Vass proposed zuul/zuul master: WIP: bugfix: branches with pattern 'refs/heads/br' fails merge jobs https://review.opendev.org/759412 | 12:23 |
| openstackgerrit | Albin Vass proposed zuul/zuul master: WIP: bugfix: branches with pattern 'refs/heads/br' fails merge jobs https://review.opendev.org/759412 | 12:26 |
| *** jpena|lunch is now known as jpena | 12:33 | |
| tobiash | avass: this is fixing a similar issue btw: https://review.opendev.org/757481 | 12:42 |
| openstackgerrit | Albin Vass proposed zuul/zuul master: WIP: bugfix: branches with pattern 'refs/heads/br' fails merge jobs https://review.opendev.org/759412 | 12:42 |
| avass | tobiash: oh | 12:45 |
| *** jamesmcarthur has joined #zuul | 12:47 | |
| *** zenkuro has joined #zuul | 12:52 | |
| *** jamesmcarthur has quit IRC | 12:53 | |
| openstackgerrit | Albin Vass proposed zuul/zuul master: bugfix: branches with pattern 'refs/heads/..' fails merge jobs https://review.opendev.org/759412 | 13:07 |
| openstackgerrit | Albin Vass proposed zuul/zuul master: bugfix: branches with pattern 'refs/heads/..' fails merge jobs https://review.opendev.org/759412 | 13:08 |
| *** zenkuro has quit IRC | 13:11 | |
| *** ianychoi_ has joined #zuul | 13:13 | |
| *** ianychoi has quit IRC | 13:16 | |
| *** vishalmanchanda has quit IRC | 13:16 | |
| *** vorotech has quit IRC | 13:18 | |
| *** vorotech has joined #zuul | 13:19 | |
| *** bhavikdbavishi has quit IRC | 13:26 | |
| *** rlandy_ has joined #zuul | 13:27 | |
| *** rlandy has quit IRC | 13:28 | |
| *** saneax has quit IRC | 13:28 | |
| *** saneax has joined #zuul | 13:29 | |
| *** smyers_ has joined #zuul | 13:29 | |
| *** smyers has quit IRC | 13:30 | |
| *** smyers_ is now known as smyers | 13:30 | |
| *** vorotech has quit IRC | 13:30 | |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: Allow test-setup to perform a connection reset https://review.opendev.org/759424 | 13:33 |
| *** vorotech has joined #zuul | 13:40 | |
| *** rlandy_ is now known as rlandy | 13:49 | |
| *** saneax has quit IRC | 13:54 | |
| *** rpittau is now known as rpittau|afk | 14:06 | |
| *** saneax has joined #zuul | 14:08 | |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: Add test_setup_reset_connection setting https://review.opendev.org/653130 | 14:15 |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: Add test_setup_reset_connection setting https://review.opendev.org/653130 | 14:19 |
| *** jfoufas1 has quit IRC | 14:24 | |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: Add test_setup_reset_connection setting https://review.opendev.org/653130 | 14:26 |
| *** Goneri has joined #zuul | 14:42 | |
| openstackgerrit | Albin Vass proposed zuul/zuul master: bugfix: branches with pattern 'refs/heads/..' fails merge jobs https://review.opendev.org/759412 | 14:47 |
| avass | tobiash: should be ready ^ | 14:48 |
| *** jcapitao has quit IRC | 15:02 | |
| *** ekwall has quit IRC | 15:09 | |
| openstackgerrit | Pierre-Louis Bonicoli proposed zuul/zuul master: Gerrit & Pagure: reuse CachedBranchConnection https://review.opendev.org/758386 | 15:12 |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: Add test_setup_reset_connection setting https://review.opendev.org/653130 | 15:15 |
| openstackgerrit | Pierre-Louis Bonicoli proposed zuul/zuul master: Allow to reuse the code handling the branch cache https://review.opendev.org/756725 | 15:17 |
| openstackgerrit | Pierre-Louis Bonicoli proposed zuul/zuul master: Gerrit & Pagure: reuse CachedBranchConnection https://review.opendev.org/758386 | 15:18 |
| openstackgerrit | Pierre-Louis Bonicoli proposed zuul/zuul master: gitlab: handle protected branches https://review.opendev.org/756900 | 15:19 |
| *** vorotech has quit IRC | 15:22 | |
| *** vorotech has joined #zuul | 15:24 | |
| *** vorotech has quit IRC | 15:33 | |
| *** bhavikdbavishi has joined #zuul | 15:45 | |
| *** wuchunyang has joined #zuul | 16:07 | |
| *** wuchunyang has quit IRC | 16:12 | |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: Add test_setup_reset_connection setting https://review.opendev.org/653130 | 16:28 |
| *** stevthedev_ has joined #zuul | 16:50 | |
| *** gundalow_ has joined #zuul | 16:50 | |
| *** ianw has quit IRC | 16:51 | |
| *** persia_ has joined #zuul | 16:51 | |
| *** bhavikdbavishi has quit IRC | 16:52 | |
| *** mordred has quit IRC | 16:57 | |
| *** stevthedev has quit IRC | 16:57 | |
| *** persia has quit IRC | 16:57 | |
| *** gundalow has quit IRC | 16:57 | |
| *** stevthedev_ is now known as stevthedev | 16:57 | |
| *** gundalow_ is now known as gundalow | 16:57 | |
| *** jpena is now known as jpena|off | 16:58 | |
| *** mordred has joined #zuul | 17:04 | |
| *** zenkuro has joined #zuul | 17:07 | |
| *** sanjayu_ has joined #zuul | 17:10 | |
| *** saneax has quit IRC | 17:13 | |
| *** hamalq has joined #zuul | 17:17 | |
| *** hamalq_ has joined #zuul | 17:20 | |
| *** hamalq has quit IRC | 17:23 | |
| *** ianw has joined #zuul | 17:23 | |
| *** sanjayu_ has quit IRC | 17:37 | |
| *** frenzyfriday has quit IRC | 17:37 | |
| *** nils has quit IRC | 18:20 | |
| *** zenkuro has quit IRC | 19:28 | |
| *** zenkuro has joined #zuul | 19:29 | |
| *** cloudnull has quit IRC | 19:47 | |
| *** cloudnull has joined #zuul | 19:47 | |
| *** cloudnull has quit IRC | 19:49 | |
| *** cloudnull has joined #zuul | 19:58 | |
| *** cloudnull has quit IRC | 20:04 | |
| *** cloudnull has joined #zuul | 20:05 | |
| *** holser has quit IRC | 20:22 | |
| *** holser has joined #zuul | 20:33 | |
| *** sassyn has joined #zuul | 20:42 | |
| sassyn | hi group | 20:43 |
| sassyn | quick question | 20:43 |
| sassyn | I have a clone of the linux kernel repo | 20:44 |
| sassyn | i have added it to my zuul | 20:44 |
| sassyn | and I see in the scheduler this error | 20:44 |
| sassyn | File "/usr/local/lib/python3.6/dist-packages/zuul/rpclistener.py", line 384, in handle_job_list | 20:44 |
| sassyn | 501eb376c4d44801aa3596c2b9a7f717> complete, merged: False, updated: False, commit: None, branches: [] | 20:44 |
| sassyn | any idea? AttributeError: 'NoneType' object has no attribute 'layout' | 20:44 |
| sassyn | what this means? | 20:44 |
| *** bhagyashris has joined #zuul | 20:47 | |
| *** ianw has quit IRC | 20:50 | |
| *** Goneri has quit IRC | 20:50 | |
| *** bhagyashris|sick has quit IRC | 20:50 | |
| *** flaper87 has quit IRC | 20:50 | |
| *** SotK has quit IRC | 20:50 | |
| *** ianw has joined #zuul | 20:50 | |
| *** Goneri has joined #zuul | 20:50 | |
| *** SotK has joined #zuul | 20:50 | |
| clarkb | sassyn: using a paste service to share a complete traceback would be helpful | 20:50 |
| clarkb | I think that is saying the merger failed to merge a commit so you may also want to look at the merger log | 20:51 |
| sassyn | hi clarkb | 20:51 |
| sassyn | hope u are doing good | 20:51 |
| sassyn | here it is: https://pastebin.pl/view/e2e8b9dc | 20:51 |
| *** rf0lc0 has joined #zuul | 20:52 | |
| clarkb | thanks give me a few minuted | 20:52 |
| sassyn | thank u | 20:52 |
| *** rfolco has quit IRC | 20:53 | |
| *** flaper87 has joined #zuul | 20:56 | |
| *** rf0lc0 has quit IRC | 20:56 | |
| clarkb | yes it seems at least related to merging on the merger. Can you check those logs too | 21:05 |
| clarkb | depending on how you deployed zuul that may be a dedicated merger process or the merger bundled in an executor | 21:06 |
| sassyn | so it seems that the issue is with the timeout | 21:14 |
| sassyn | https://pastebin.pl/view/938a162c | 21:14 |
| sassyn | the merger.git_timeout | 21:15 |
| sassyn | cause it takes time to clone the repos | 21:15 |
| sassyn | they are huge | 21:15 |
| sassyn | but why the executor also clone the repo? it is not only the merger? | 21:15 |
| clarkb | that is a configurable option. under [merger] git_timeout | 21:15 |
| clarkb | executors also run a merger by default | 21:16 |
| sassyn | so I can setup this as well on the exexcutor? | 21:16 |
| clarkb | yes it should be the same setting I think | 21:16 |
| sassyn | OK | 21:16 |
| sassyn | let me try | 21:16 |
| clarkb | another option is to preclone the repos for zuul | 21:16 |
| clarkb | then it will only need to update them as it goes | 21:17 |
| fungi | technically you don't need any separate merger, but it's recommended especially to speed up scheduler startup when you have lots of repos/branches | 21:17 |
| sassyn | https://zuul-ci.org/docs/zuul/discussion/components.html#attr-executor.merge_jobs | 21:17 |
| sassyn | after this line there is a merger section | 21:17 |
| fungi | and to handle reconfigurations more quickly | 21:17 |
| sassyn | it is not clear why it is there | 21:17 |
| sassyn | hi fungi :-) | 21:17 |
| sassyn | I added 24 merger with 3 machine - each running 8 time the services | 21:18 |
| clarkb | sassyn: that may be a bug in the docs? I think the executor merger and the merger merger share the asme base class and should take teh same configs set under [merger] | 21:19 |
| sassyn | OK | 21:19 |
| clarkb | ya BaseMergeServer seems to handle a bunch of those otpions | 21:20 |
| sassyn | so I need to put the setting in the [executor] ? like git_user_name & git_user_email & git_timeout? | 21:21 |
| sassyn | or it should be under the [merger] | 21:21 |
| sassyn | in the executor machine? | 21:21 |
| clarkb | it should be under merger for both | 21:21 |
| clarkb | the shared code does self.git_timeout = get_default(config, 'merger', 'git_timeout', 300) | 21:22 |
| sassyn | thank u | 21:23 |
| sassyn | again both of u | 21:23 |
| sassyn | I hope all good with you 2 | 21:24 |
| sassyn | Zuul is super cool! | 21:24 |
| sassyn | have a nice weekend | 21:24 |
| clarkb | you too | 21:25 |
| clarkb | I wonder if we should have error logs for timeouts point to the config options to change the values | 21:27 |
| clarkb | "This hit a timeout, if the value is too small for you it can be changed via: $configoption" | 21:27 |
| fungi | sassyn: thanks, all's fine with me. hope you have a good weekend too! | 21:35 |
| *** rlandy has quit IRC | 21:38 | |
| *** ikhan has quit IRC | 21:56 | |
| *** ikhan has joined #zuul | 22:10 | |
| *** wuchunyang has joined #zuul | 22:10 | |
| *** wuchunyang has quit IRC | 22:15 | |
| avass | if anyone wants to do a quick review: https://review.opendev.org/#/c/759412/ :) | 22:34 |
| fungi | how did you guess? ;) | 22:35 |
| *** ikhan has quit IRC | 22:41 | |
| *** ikhan has joined #zuul | 22:48 | |
| *** zenkuro has quit IRC | 23:06 | |
| *** zenkuro has joined #zuul | 23:06 | |
| *** sean-k-mooney has quit IRC | 23:20 | |
| *** sean-k-mooney has joined #zuul | 23:21 | |
| *** hamalq_ has quit IRC | 23:43 | |
| openstackgerrit | Merged zuul/zuul master: bugfix: branches with pattern 'refs/heads/..' fails merge jobs https://review.opendev.org/759412 | 23:43 |
| *** tosky has quit IRC | 23:46 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!