Wednesday, 2020-06-03

*** armstrongs has quit IRC00:27
*** jamesmcarthur has quit IRC00:59
*** wuchunyang has joined #zuul01:18
*** swest has quit IRC01:23
*** ysandeep|away is now known as ysandeep01:38
*** swest has joined #zuul01:38
*** wuchunyang has quit IRC01:45
*** jamesmcarthur has joined #zuul02:00
*** wuchunyang has joined #zuul02:16
*** sshnaidm has quit IRC02:34
*** rlandy has quit IRC02:43
*** jamesmcarthur has quit IRC02:57
*** jamesmcarthur has joined #zuul02:57
*** rfolco|rover has quit IRC02:57
*** jamesmcarthur has quit IRC03:03
*** wuchunyang has quit IRC03:20
*** sshnaidm has joined #zuul03:33
*** jamesmcarthur has joined #zuul03:38
*** jamesmcarthur has quit IRC03:48
*** dmellado has quit IRC04:16
*** jamesmcarthur has joined #zuul04:19
*** jamesmcarthur_ has joined #zuul04:31
*** evrardjp has quit IRC04:33
*** evrardjp has joined #zuul04:33
*** jamesmcarthur has quit IRC04:35
*** wuchunyang has joined #zuul04:39
*** sshnaidm_ has joined #zuul04:43
*** sshnaidm has quit IRC04:45
*** saneax has joined #zuul04:49
*** wuchunyang has quit IRC04:53
*** jamesmcarthur_ has quit IRC04:57
*** jamesmcarthur has joined #zuul04:58
*** sgw has quit IRC05:01
*** jamesmcarthur has quit IRC05:03
*** dmellado has joined #zuul05:31
*** jamesmcarthur has joined #zuul05:38
*** sanjayu_ has joined #zuul05:40
*** saneax has quit IRC05:41
*** bhavikdbavishi has joined #zuul05:49
*** jamesmcarthur has quit IRC05:50
*** bhavikdbavishi1 has joined #zuul06:00
*** bhavikdbavishi has quit IRC06:02
*** bhavikdbavishi1 is now known as bhavikdbavishi06:02
*** jamesmcarthur has joined #zuul06:27
*** jamesmcarthur_ has joined #zuul06:33
*** jamesmcarthur has quit IRC06:37
*** jamesmcarthur_ has quit IRC06:41
*** bhavikdbavishi has quit IRC06:56
*** bolg has joined #zuul06:57
*** bhavikdbavishi has joined #zuul07:08
*** jcapitao has joined #zuul07:20
*** tosky has joined #zuul07:24
*** bhavikdbavishi has quit IRC07:49
*** jpena|off is now known as jpena07:53
*** olaph has joined #zuul07:54
openstackgerritJan Kubovy proposed zuul/zuul master: Scheduler's pause/resume functionality  https://review.opendev.org/70973507:57
openstackgerritJan Kubovy proposed zuul/zuul master: Separate connection registries in tests  https://review.opendev.org/71295807:57
openstackgerritJan Kubovy proposed zuul/zuul master: Prepare Zookeeper for scale-out scheduler  https://review.opendev.org/71726907:57
openstackgerritJan Kubovy proposed zuul/zuul master: Mandatory Zookeeper connection for ZuulWeb in tests  https://review.opendev.org/72125407:57
openstackgerritJan Kubovy proposed zuul/zuul master: Driver event ingestion  https://review.opendev.org/71729907:58
openstackgerritJan Kubovy proposed zuul/zuul master: WIP: Store unparsed branch config in Zookeeper  https://review.opendev.org/70571607:58
*** yolanda has joined #zuul08:02
*** nils has joined #zuul08:04
*** bhavikdbavishi has joined #zuul08:25
openstackgerritGuillaume Chauvel proposed zuul/zuul master: DNM: build tutorial image one time and use registry  https://review.opendev.org/73300508:28
openstackgerritGuillaume Chauvel proposed zuul/zuul master: DNM: build tutorial image one time and use registry  https://review.opendev.org/73300508:36
*** jamesmcarthur has joined #zuul08:38
*** ysandeep is now known as ysandeep|lunch08:39
*** jamesmcarthur has quit IRC08:43
*** themroc has joined #zuul09:22
*** ysandeep|lunch is now known as ysandeep09:33
zbrfungi: clarkb AJaeger: https://review.opendev.org/#/c/723534/ please - adds "more" expansion to errored tasks.09:45
zbrthe current "tail last 42 lines" is not enough for lots of cases09:46
openstackgerritGuillaume Chauvel proposed zuul/zuul master: DNM: build tutorial image one time and use registry  https://review.opendev.org/73300510:00
*** jcapitao is now known as jcapitao_lunch10:35
*** jamesmcarthur has joined #zuul10:39
*** jamesmcarthur has quit IRC10:44
openstackgerritGuillaume Chauvel proposed zuul/zuul master: DNM: build tutorial image one time and use registry  https://review.opendev.org/73300511:08
*** bhavikdbavishi has quit IRC11:09
*** bhavikdbavishi has joined #zuul11:10
openstackgerritGuillaume Chauvel proposed zuul/zuul master: Gerrit: report checks, send zuul status url for started event  https://review.opendev.org/73256911:13
*** hashar has joined #zuul11:26
*** nils has quit IRC11:30
*** jpena is now known as jpena|lunch11:31
*** nils has joined #zuul11:35
*** saneax has joined #zuul11:39
*** sanjayu_ has quit IRC11:40
*** rfolco|rover has joined #zuul11:42
*** jcapitao_lunch is now known as jcapitao11:49
*** rpittau|afk is now known as rpittau11:57
*** sgw has joined #zuul12:01
*** ysandeep is now known as ysandeep|afk12:06
*** hashar has quit IRC12:24
*** hashar has joined #zuul12:24
*** rlandy has joined #zuul12:32
*** bhavikdbavishi has quit IRC12:33
openstackgerritGuillaume Chauvel proposed zuul/zuul master: DNM: build tutorial image one time and use registry  https://review.opendev.org/73300512:34
*** jpena|lunch is now known as jpena12:39
*** jamesmcarthur has joined #zuul12:40
*** avass has joined #zuul12:42
swestcorvus: thanks for your review of the circular dependency change! I've addressed your comments in https://review.opendev.org/#/c/685354/12:43
*** Goneri has joined #zuul12:45
*** jamesmcarthur has quit IRC12:45
avasszbr: sorry, I've had a lot to do the last couple of days and will probably be a bit busy for at least another week. Trying to migrate a lot of on-prem services to cloud :)12:46
avasszbr: I'll see if I can do a round of reviews later12:46
zbravass: sure. i know you mentioned the zj_items rules, but IMHO, one fix could avoid adding extra complexity to the rule. is only one place.12:49
*** ysandeep|afk is now known as ysandeep12:55
swestcorvus: not sure if I understand your proposal in https://review.opendev.org/#/c/728774/ Would the scheduler be the owner of the cached TenantParseContexts? I'm just wondering if it would be better to create a long-lived instance of the ConfigLoader in the scheduler instead of passing some kind of parse context around.13:05
*** harrymichal has joined #zuul13:09
openstackgerritMerged zuul/zuul-jobs master: Add the namespace in wait-for-pods role  https://review.opendev.org/73129213:10
*** harrymichal has quit IRC13:47
*** avass has quit IRC13:54
corvusswest: that's an interesting idea worth exploring, but i think that's more complicated than what i was intending to suggest.  i just meant to suggest a way to keep a ParseContext around for the life of a single configuration parsing.  so we would still be creating ParseContexts and schemas each time we entered the configparser.  but i think my suggestion was incomplete.  instead of creating it in14:00
corvusTenantParser.fromYaml, if we create it in both ConfigLoader.createDynamicLayout and ConfigLoader.loadConfig (those are mutually exclusive -- depending on whether we're creating a static or dynamic config), we should end up creating it once for each tenant for each config loading event, but not keeping it around longer than necessary.14:00
*** jamesmcarthur has joined #zuul14:00
swestcorvus: ah ok, that would at least solve the part of recreating the schema for every config file. Maybe it's better to start with this simple optimization than go overboard with some major refactoring.14:05
swestcorvus: I think I'll try that. thanks!14:06
corvusswest: cool :)14:07
*** harrymichal has joined #zuul14:31
*** hashar has quit IRC14:35
*** hashar has joined #zuul14:35
*** jamesmcarthur has quit IRC15:04
*** dmellado has quit IRC15:18
*** dmellado has joined #zuul15:20
masterpeI have a zuul from scratch and I try to upload the logs, I use the role generate-zuul-manifest. But if I use this role the build fails.15:59
masterpeIt fails with the error: POST_FAILURE15:59
*** Goneri has quit IRC16:00
zbrcorvus: how about https://review.opendev.org/#/c/723534/ ?16:17
*** olaph has quit IRC16:19
*** olaph has joined #zuul16:19
*** rpittau is now known as rpittau|afk16:24
*** ysandeep is now known as ysandeep|away16:33
*** hashar has quit IRC16:34
*** hashar has joined #zuul16:34
*** themroc has quit IRC16:35
corvusmasterpe: to find the error, you may need to either watch the streaming logs for the build, or check the executor debug logs.16:45
masterpeWhere can I find the streaming logs?16:47
corvusmasterpe: you can visit the status page while the build is running and click the link for the build there16:49
masterpeI will try16:50
*** nils has quit IRC16:50
fungiyeah, getting to logs about why logs weren't published can be a bit of a catch-2216:52
*** jpena is now known as jpena|off17:04
masterpethanks, at the task "generate-zuul-manifest : Generate Zuul manifest", I get the error 'localhost |   "msg": "Executing local code is prohibited"'17:07
corvusmasterpe: that role has to run in playbook in a config-project.  the quick-start explains this much better than zuul-from-scratch: https://zuul-ci.org/docs/zuul/tutorials/quick-start.html#configure-a-base-job17:08
*** jcapitao has quit IRC17:18
*** ysandeep|away is now known as ysandeep17:20
*** hashar has quit IRC17:22
*** hashar has joined #zuul17:23
*** hashar has quit IRC17:35
*** ysandeep is now known as ysandeep|away18:01
*** jamesmcarthur has joined #zuul18:05
*** yolanda has quit IRC18:23
openstackgerritMerged zuul/zuul master: Ensure pull request number is int  https://review.opendev.org/73271318:42
openstackgerritSorin Sbarnea (zbr) proposed zuul/zuul-jobs master: test-playbooks: improved syntax  https://review.opendev.org/73159118:48
*** hashar has joined #zuul19:05
*** hashar has quit IRC19:13
*** hashar has joined #zuul19:13
*** sshnaidm_ is now known as sshnaidm19:19
*** hashar_ has joined #zuul19:24
*** hashar_ has quit IRC19:24
*** hashar has quit IRC19:25
*** hashar has joined #zuul19:25
AJaegerhttps://review.opendev.org/731606 is a policy change for zuul-jobs, please review20:06
corvustristanC: ^ you or mhu or someone from sf.io would be good to look at that; but if they don't have time this week due to ptg AJaeger, maybe we can hold that till next week?20:18
AJaegercorvus: sure, we can.20:19
corvusleft a note on the change20:19
corvusbut also seems straightforward and lgtm, so +1 :)20:19
tristanCcorvus: AJaeger: lgtm. not sure to understand why we would be affected in particular though.20:21
corvustristanC: sf.io provides a good diverse viewpoint compared to opendev wrt platforms, and think mhu wrote that section to start with20:23
corvusso it's more about getting a "second opinion" and making sure the folks who originially identified the need for that have a chance to weigh in; not that sf.io would be affected more than anyone else.20:25
tristanCoh i see, then yes waiting for mhu (on https://review.opendev.org/731606) would be ideal. I thought this was already the best practice, or at least that's the one I have already used in my recent ensure-* contributions20:28
*** Defolos has quit IRC20:30
*** Defolos has joined #zuul20:34
*** hashar has quit IRC20:35
*** harrymichal has quit IRC20:39
openstackgerritGuillaume Chauvel proposed zuul/zuul master: tutorial: Rework quick-start and prepare for other tutorials  https://review.opendev.org/73206620:50
openstackgerritGuillaume Chauvel proposed zuul/zuul master: tutorial: Add "gate your first patch"  https://review.opendev.org/73206720:50
*** Defolos_ has joined #zuul21:06
*** Defolos has quit IRC21:07
openstackgerritGuillaume Chauvel proposed zuul/zuul master: tutorial: Add "gate your first patch"  https://review.opendev.org/73206722:12
openstackgerritGuillaume Chauvel proposed zuul/zuul-jobs master: Add python_version for tox27  https://review.opendev.org/73341923:25
openstackgerritGuillaume Chauvel proposed zuul/zuul master: tutorial: Add "Use zuul jobs"  https://review.opendev.org/73206823:28
openstackgerritGuillaume Chauvel proposed zuul/zuul master: tutorial: Add "Use zuul jobs"  https://review.opendev.org/73206823:32
*** jamesmcarthur_ has joined #zuul23:45
*** jamesmcarthur has quit IRC23:48
*** tosky has quit IRC23:50

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!