Thursday, 2020-10-15

*** rf0lc0 has joined #zuul00:18
*** jamesmcarthur has quit IRC00:36
*** jamesmcarthur has joined #zuul00:36
*** zenkuro has quit IRC00:38
*** jamesmcarthur has quit IRC00:41
*** hamalq has quit IRC00:43
*** jamesmcarthur has joined #zuul00:49
*** rf0lc0 has quit IRC01:06
*** jamesmcarthur has quit IRC01:19
*** jamesmcarthur has joined #zuul01:20
*** jamesmcarthur has quit IRC01:24
*** jamesmcarthur has joined #zuul01:43
*** mach1na has joined #zuul01:58
*** mach1na has quit IRC02:03
*** bhavikdbavishi has joined #zuul02:27
*** bhavikdbavishi1 has joined #zuul02:34
*** bhavikdbavishi has quit IRC02:35
*** bhavikdbavishi1 is now known as bhavikdbavishi02:35
*** jamesmcarthur has quit IRC02:45
*** jamesmcarthur has joined #zuul02:46
*** jamesmcarthur has quit IRC02:51
*** jamesmcarthur has joined #zuul03:14
*** bhavikdbavishi has quit IRC03:17
*** bhavikdbavishi has joined #zuul03:24
openstackgerritIan Wienand proposed zuul/zuul-jobs master: fetch-sphinx-tarball: explain what is happening  https://review.opendev.org/75832303:49
openstackgerritIan Wienand proposed zuul/zuul-jobs master: fetch-sphinx-tarball: don't run merge-output-to-logs  https://review.opendev.org/75832403:49
openstackgerritIan Wienand proposed zuul/zuul-jobs master: Revert "Revert "Refactor fetch-sphinx-tarball to be executor safe""  https://review.opendev.org/75832503:49
openstackgerritIan Wienand proposed zuul/zuul-jobs master: Revert "Revert "Refactor fetch-sphinx-tarball to be executor safe""  https://review.opendev.org/75832503:56
*** bhavikdbavishi has quit IRC04:27
*** bhavikdbavishi has joined #zuul04:27
*** evrardjp has quit IRC04:33
*** evrardjp has joined #zuul04:33
*** jamesmcarthur has quit IRC04:33
*** jamesmcarthur has joined #zuul04:34
*** jamesmcarthur has quit IRC04:37
*** jamesmcarthur has joined #zuul04:37
*** smyers_ has joined #zuul04:50
*** smyers has quit IRC04:51
*** smyers_ is now known as smyers04:51
*** ikhan has quit IRC04:53
*** jamesmcarthur has quit IRC05:12
*** jamesmcarthur has joined #zuul05:12
*** jamesmcarthur has quit IRC05:16
openstackgerritIan Wienand proposed zuul/zuul-jobs master: merge-output-to-logs: convert to untrusted executor safe code  https://review.opendev.org/75832505:39
*** reiterative has quit IRC05:42
*** reiterative has joined #zuul05:42
*** jamesmcarthur has joined #zuul05:47
openstackgerritIan Wienand proposed zuul/zuul master: web: add title bar to tenants page  https://review.opendev.org/75694105:56
*** saneax has joined #zuul05:58
*** jamesmcarthur has quit IRC05:59
openstackgerritIan Wienand proposed zuul/zuul-jobs master: merge-output-to-logs: convert to untrusted executor safe code  https://review.opendev.org/75832506:01
*** mach1na has joined #zuul06:03
*** bhavikdbavishi1 has joined #zuul06:06
*** bhavikdbavishi has quit IRC06:07
*** bhavikdbavishi1 is now known as bhavikdbavishi06:07
*** vishalmanchanda has joined #zuul06:09
*** bhavikdbavishi has quit IRC06:22
*** bhavikdbavishi has joined #zuul06:23
*** jfoufas1 has joined #zuul06:23
*** SpamapS has quit IRC06:25
*** SpamapS has joined #zuul06:26
*** bhavikdbavishi has quit IRC06:31
*** bhavikdbavishi has joined #zuul06:32
openstackgerritIan Wienand proposed zuul/zuul-jobs master: merge-output-to-logs: convert to untrusted executor safe code  https://review.opendev.org/75832506:34
*** bhavikdbavishi has quit IRC06:39
openstackgerritIan Wienand proposed zuul/zuul-jobs master: merge-output-to-logs: convert to untrusted executor safe code  https://review.opendev.org/75832506:44
openstackgerritIan Wienand proposed zuul/zuul-jobs master: merge-output-to-logs: convert to untrusted executor safe code  https://review.opendev.org/75832506:53
*** mach1na has quit IRC07:04
openstackgerritIan Wienand proposed zuul/zuul-jobs master: merge-output-to-logs: convert to untrusted executor safe code  https://review.opendev.org/75832507:05
*** hashar has joined #zuul07:05
*** freenzyfriday has joined #zuul07:13
*** jpena|off is now known as jpena07:19
*** rpittau|afk is now known as rpittau07:22
openstackgerritDaniel Pawlik proposed zuul/nodepool master: WIP Add more node deletion metrics  https://review.opendev.org/75835507:25
*** yolanda__ has joined #zuul07:27
*** cloudnull has quit IRC07:33
*** cloudnull has joined #zuul07:34
*** holser has joined #zuul07:38
*** jamesmcarthur has joined #zuul07:38
*** jamesmcarthur has quit IRC07:41
*** tosky has joined #zuul07:42
*** mach1na has joined #zuul07:46
*** jamesmcarthur has joined #zuul07:49
openstackgerritPierre-Louis Bonicoli proposed zuul/zuul master: Allow to reuse the code handling the branch cache  https://review.opendev.org/75672507:56
*** bhavikdbavishi has joined #zuul08:01
*** nils has joined #zuul08:12
*** freenzyfriday has quit IRC08:22
*** yolanda__ has quit IRC08:24
*** yolanda__ has joined #zuul08:24
*** bhavikdbavishi has quit IRC08:29
openstackgerritMatthieu Huin proposed zuul/zuul master: [DNM] Test zuul-client builds subcommand  https://review.opendev.org/75814108:31
*** bhavikdbavishi has joined #zuul08:33
*** jamesmcarthur has quit IRC08:43
*** jamesmcarthur has joined #zuul08:44
*** vorotech has joined #zuul08:44
*** jamesmcarthur has quit IRC08:49
openstackgerritJan Kubovy proposed zuul/zuul master: Prepare Zookeeper for scale-out scheduler  https://review.opendev.org/71726908:50
openstackgerritGuillaume Chauvel proposed zuul/zuul master: Update tests/base.py to use proper git data  https://review.opendev.org/74274608:54
openstackgerritGuillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Add tests  https://review.opendev.org/74274708:54
openstackgerritGuillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Reorg scheduler event process loop  https://review.opendev.org/74274808:54
openstackgerritGuillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Scheduler ref-updated create/delete  https://review.opendev.org/73919808:54
openstackgerritGuillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Allow ref-updated newrev+oldrev reconfiguration  https://review.opendev.org/74274908:54
openstackgerritGuillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Get ref-updated modified files  https://review.opendev.org/73907808:54
openstackgerritPierre-Louis Bonicoli proposed zuul/zuul master: Allow to reuse the code handling the branch cache  https://review.opendev.org/75672508:59
openstackgerritPierre-Louis Bonicoli proposed zuul/zuul master: gitlab: handle protected branches  https://review.opendev.org/75690008:59
*** jamesmcarthur has joined #zuul09:07
openstackgerritMatthieu Huin proposed zuul/zuul master: [DNM] Test zuul-client builds subcommand  https://review.opendev.org/75814109:09
vorotechhi here, at our company we are utilising zuul gating a lot, I'm trying to investigate the issue we faced when trying to execute bubble wrapped ansible modules (e.g. copy) against remote container (eks pod). I hasn't identified the root cause yet, but maybe community can point me into the right direction.  Not sure where to raise the question with details yet, so I have prepared a gist. https://gist.github.com/vorotech/f7b5f63b461944f0fb1b1eee7861bac609:11
*** jamesmcarthur has quit IRC09:12
*** mach1na has quit IRC09:19
*** mach1na has joined #zuul09:23
*** bhavikdbavishi has quit IRC09:24
*** bhavikdbavishi has joined #zuul09:25
hasharvorotech: hello, not sure whether there are much people active during european morning09:44
hasharvorotech: you might want to try the mailing list ( http://lists.zuul-ci.org/cgi-bin/mailman/listinfo/zuul-discuss )09:44
hasharthis way American folks have a better chance to notice it  later today :]09:44
vorotechthanks :)09:44
hasharvorotech: note that the email you will use will be made public09:45
hasharcause the list archive is public, so theorically some friendly spammer can harvest you to deliver random bulk emails :-\09:45
hasharoff for lunch09:46
*** hashar has quit IRC09:46
openstackgerritDaniel Pawlik proposed zuul/nodepool master: WIP Add more node deletion metrics  https://review.opendev.org/75835509:48
openstackgerritTobias Henkel proposed zuul/zuul-jobs master: WIP: Create upload-logs-with-failover role  https://review.opendev.org/75838009:49
openstackgerritMatthieu Huin proposed zuul/zuul master: [DNM] Test zuul-client builds subcommand  https://review.opendev.org/75814110:18
*** wuchunyang has joined #zuul10:20
*** jamesmcarthur has joined #zuul10:20
openstackgerritGuillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Add tests  https://review.opendev.org/74274710:22
openstackgerritGuillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Reorg scheduler event process loop  https://review.opendev.org/74274810:22
openstackgerritGuillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Scheduler ref-updated create/delete  https://review.opendev.org/73919810:22
openstackgerritGuillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Allow ref-updated newrev+oldrev reconfiguration  https://review.opendev.org/74274910:22
openstackgerritGuillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Get ref-updated modified files  https://review.opendev.org/73907810:22
*** mach1na has quit IRC10:23
*** mach1na has joined #zuul10:24
*** wuchunyang has quit IRC10:24
*** jamesmcarthur has quit IRC10:25
*** zenkuro has joined #zuul10:31
openstackgerritPierre-Louis Bonicoli proposed zuul/zuul master: Gerrit & Pagure: reuse CachedBranchConnection  https://review.opendev.org/75838610:34
*** jcapitao is now known as jcapitao_lunch10:37
*** bhavikdbavishi has quit IRC10:38
openstackgerritMatthieu Huin proposed zuul/zuul master: [DNM] Test zuul-client builds subcommand  https://review.opendev.org/75814110:51
*** yolanda__ is now known as yolanda10:52
*** zenkuro has quit IRC11:01
*** zenkuro has joined #zuul11:01
*** vorotech has quit IRC11:02
*** wuchunyang has joined #zuul11:06
openstackgerritAndy Ladjadj proposed zuul/nodepool master: [driver][aws] propagate the instance_id in the node host_id field  https://review.opendev.org/75839311:09
*** mach1na has quit IRC11:11
*** wuchunyang has quit IRC11:11
*** bhavikdbavishi has joined #zuul11:27
avassvorotech: If you're checking this later, I believe it could have something to do with how ansible copies files when kubectl is used: https://github.com/ansible/ansible/blob/stable-2.9/lib/ansible/plugins/connection/kubectl.py#L32111:31
*** jpena is now known as jpena|lunch11:32
avassvorotech: since they're using 'kubectl exec' without passing the command through a shell so '~' is never expanded11:33
*** hashar has joined #zuul11:45
*** mach1na has joined #zuul11:47
*** CraigR has joined #zuul11:48
*** mach1na has quit IRC11:48
*** vorotech has joined #zuul11:49
avassvorotech: http://eavesdrop.openstack.org/irclogs/%23zuul/%23zuul.2020-10-15.log.html#t2020-10-15T11:31:21 :)11:51
avassvorotech: I could be wrong since it looks like they might be passing it through some shell with '"play_context.executable", "-c".11:52
avassvorotech: but maybe it's the error message that isn't formatted correctly?11:53
vorotechavass thanks for pointing to kubectl command, I will review the implementation. not likely the formatting issue since only set of modules not work (those which interact with tmp folder first)11:56
avassvorotech: ah no stderr show the same path11:57
*** rlandy has joined #zuul12:01
*** rlandy is now known as rlandy|rover12:02
*** jcapitao_lunch is now known as jcapitao12:05
mhuhello, it looks like ABORTED builds are not reported by the sql reporter, whereas SKIPPED ones are. I think I get we'd want to do that, but I just wanted to make sure this is intended behavior?12:08
mhuI get *why* we'd want to do that12:08
avassvorotech: ah, they're actually prefixing the path with '/' and then quoting the string to make it safe to pass to a shell, not sure where the '~zuul' part comes from but that's why you get the '/' and why the tilde is not expanded :)12:13
*** rf0lc0 has joined #zuul12:15
vorotechavass that's right! thanks12:17
vorotechdon't think it is something to do on zuul side, probably need to patch the prefix_login_path somehow https://github.com/ansible/ansible/blob/stable-2.9/lib/ansible/plugins/connection/kubectl.py#L29612:19
avassvorotech: agreed12:21
*** CraigR has quit IRC12:25
*** vorotech has quit IRC12:25
*** mach1na has joined #zuul12:26
*** vorotech has joined #zuul12:27
*** jpena|lunch is now known as jpena12:32
openstackgerritMatthieu Huin proposed zuul/zuul master: [DNM] Test zuul-client builds subcommand  https://review.opendev.org/75814112:34
*** jamesmcarthur has joined #zuul12:45
openstackgerritTobias Henkel proposed zuul/zuul master: Fix possible race in _getChange  https://review.opendev.org/75842413:02
tobiashmhu: aborted builds are part of a buildset that is not current (e.g. due to change abandon or a gate reset) and that whole buildset is not reported whereas skipped builds are part of reported finished buildsets13:20
*** bhavikdbavishi has quit IRC13:21
mhutobiash, right, just wanted to make sure it was intended13:21
tobiashyes, that's intended If I didn't overlook some aspect :)13:22
fungito include aborted builds, we'd need to report cancelled buildsets, which we currently don't do as far as i know13:33
fungii suppose if we did, we'd only report them to the database as they would be very noisy elsewhere13:34
fungitaking opendev's openstack tenant as an example, i would not be surprised if at times 90% of the builds occurring in the dependent gate pipeline are aborted due to the churn from gate resets near the front of very long dependent queues13:35
fungitraditionally we viewed those as not particularly valuable to track, since they don't complete and therefore have no real information to convey about the builds13:36
fungiat most, knowing how long they ran prior to aborting has been of interest in calculating aggregate resource utilization at a per-project or per-job level, but we've mined that out of scheduler logs for the most part when needed13:37
openstackgerritMatthieu Huin proposed zuul/zuul master: [DNM] Test zuul-client builds subcommand  https://review.opendev.org/75814113:37
fungigranted, we've also been discussing the possibility of recording builds to the database via some mechanism other than reporting, which might make tracking aborted ones there easier13:38
*** jamesmcarthur has quit IRC13:49
*** sshnaidm has quit IRC13:54
*** sshnaidm has joined #zuul14:00
*** vishalmanchanda has quit IRC14:01
openstackgerritJan Kubovy proposed zuul/zuul master: Prepare Zookeeper for scale-out scheduler  https://review.opendev.org/71726914:03
*** wuchunyang has joined #zuul14:04
openstackgerritJan Kubovy proposed zuul/zuul master: Mandatory Zookeeper connection for ZuulWeb in tests  https://review.opendev.org/72125414:05
*** jamesmcarthur has joined #zuul14:06
openstackgerritJan Kubovy proposed zuul/zuul master: Merge Zookeeper connection methods  https://review.opendev.org/75436014:07
openstackgerritJan Kubovy proposed zuul/zuul master: Connect merger to Zookeeper  https://review.opendev.org/71622114:08
*** wuchunyang has quit IRC14:08
openstackgerritJan Kubovy proposed zuul/zuul master: Connect fingergw to Zookeeper  https://review.opendev.org/71687514:10
openstackgerritJan Kubovy proposed zuul/zuul master: Connect executor to Zookeeper  https://review.opendev.org/71626214:10
openstackgerritJan Kubovy proposed zuul/zuul master: Improve typings in context of 744416  https://review.opendev.org/75357814:18
*** hashar has quit IRC14:25
openstackgerritJan Kubovy proposed zuul/zuul master: Make Zookeeper mandatory for Scheduler  https://review.opendev.org/75671614:25
openstackgerritJan Kubovy proposed zuul/zuul master: Make ConnectionRegistry mandatory for Scheduler  https://review.opendev.org/75709514:25
openstackgerritJan Kubovy proposed zuul/zuul master: Improve typings in context of 756716 and 757095  https://review.opendev.org/75714814:25
openstackgerritJan Kubovy proposed zuul/zuul master: Instantiate executor client, merger, nodepool and app within Scheduler  https://review.opendev.org/75714914:25
openstackgerritJan Kubovy proposed zuul/zuul master: Improve typings in context of 756304  https://review.opendev.org/75709714:25
*** vorotech has quit IRC14:32
*** jfoufas1 has quit IRC14:43
*** sgw has joined #zuul14:53
*** iurygregory has quit IRC14:57
*** iurygregory has joined #zuul14:57
openstackgerritJan Kubovy proposed zuul/zuul master: Driver event ingestion  https://review.opendev.org/71729915:06
openstackgerritMatthieu Huin proposed zuul/zuul master: [DNM] Test zuul-client builds subcommand  https://review.opendev.org/75814115:24
*** GomathiselviS has joined #zuul15:27
*** sgw has left #zuul15:39
*** sgw has joined #zuul15:43
*** hashar has joined #zuul15:44
*** jamesmcarthur has quit IRC15:44
*** jamesmcarthur has joined #zuul15:50
openstackgerritMatthieu Huin proposed zuul/zuul master: [DNM] Test zuul-client builds subcommand  https://review.opendev.org/75814115:50
*** jamesmcarthur has quit IRC15:50
*** jamesmcarthur has joined #zuul15:50
*** rpittau is now known as rpittau|afk15:52
*** jamesmcarthur has quit IRC15:57
*** jamesmcarthur has joined #zuul15:57
*** wuchunyang has joined #zuul16:06
*** wuchunyang has quit IRC16:10
*** jcapitao has quit IRC16:20
*** jpena is now known as jpena|off16:35
*** nils has quit IRC16:39
*** bhavikdbavishi has joined #zuul16:51
*** hashar is now known as hasharTurns4216:51
*** mach1na has quit IRC16:53
*** jamesmcarthur has quit IRC17:01
*** zenkuro has quit IRC17:01
*** hamalq has joined #zuul17:02
*** zenkuro has joined #zuul17:03
openstackgerritMatthieu Huin proposed zuul/zuul master: [DNM] Test zuul-client builds subcommand  https://review.opendev.org/75814117:05
*** jamesmcarthur has joined #zuul17:07
*** GomathiselviS has quit IRC18:02
*** zenkuro has quit IRC18:15
*** zenkuro has joined #zuul18:15
openstackgerritJan Kubovy proposed zuul/zuul master: Prepare Zookeeper for scale-out scheduler  https://review.opendev.org/71726918:21
openstackgerritJan Kubovy proposed zuul/zuul master: Mandatory Zookeeper connection for ZuulWeb in tests  https://review.opendev.org/72125418:21
openstackgerritJan Kubovy proposed zuul/zuul master: Merge Zookeeper connection methods  https://review.opendev.org/75436018:21
openstackgerritJan Kubovy proposed zuul/zuul master: Connect merger to Zookeeper  https://review.opendev.org/71622118:21
openstackgerritJan Kubovy proposed zuul/zuul master: Connect fingergw to Zookeeper  https://review.opendev.org/71687518:21
openstackgerritJan Kubovy proposed zuul/zuul master: Connect executor to Zookeeper  https://review.opendev.org/71626218:21
openstackgerritJan Kubovy proposed zuul/zuul master: Improve typings in context of 744416  https://review.opendev.org/75357818:21
openstackgerritJan Kubovy proposed zuul/zuul master: Make Zookeeper mandatory for Scheduler  https://review.opendev.org/75671618:21
openstackgerritJan Kubovy proposed zuul/zuul master: Make ConnectionRegistry mandatory for Scheduler  https://review.opendev.org/75709518:21
openstackgerritJan Kubovy proposed zuul/zuul master: Improve typings in context of 756716 and 757095  https://review.opendev.org/75714818:21
openstackgerritJan Kubovy proposed zuul/zuul master: Instantiate executor client, merger, nodepool and app within Scheduler  https://review.opendev.org/75714918:21
openstackgerritJan Kubovy proposed zuul/zuul master: Improve typings in context of 756304  https://review.opendev.org/75709718:21
openstackgerritJan Kubovy proposed zuul/zuul master: Driver event ingestion  https://review.opendev.org/71729918:21
openstackgerritJan Kubovy proposed zuul/zuul master: Switch to using zookeeper instead of gearman for jobs  https://review.opendev.org/74441618:21
*** jamesmcarthur has quit IRC18:31
*** jamesmcarthur has joined #zuul18:39
*** GomathiselviS has joined #zuul18:43
*** saneax has quit IRC18:52
*** zenkuro has quit IRC18:55
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Add nim roles and job  https://review.opendev.org/74786519:08
*** hasharTurns42 is now known as hashar19:22
*** hashar has quit IRC19:26
*** bhavikdbavishi has quit IRC19:50
*** wuchunyang has joined #zuul20:08
*** wuchunyang has quit IRC20:12
*** jamesmcarthur has quit IRC21:05
openstackgerritIan Wienand proposed zuul/zuul-jobs master: merge-output-to-logs: convert to untrusted executor safe code  https://review.opendev.org/75832521:08
*** zenkuro has joined #zuul21:14
*** jamesmcarthur has joined #zuul21:15
openstackgerritIan Wienand proposed zuul/zuul-jobs master: merge-output-to-logs: convert to untrusted executor safe code  https://review.opendev.org/75832521:21
*** jamesmcarthur has quit IRC21:22
*** jamesmcarthur has joined #zuul21:24
*** jamesmcarthur has quit IRC21:25
*** jamesmcarthur has joined #zuul21:25
*** jamesmcarthur has quit IRC21:48
openstackgerritIan Wienand proposed zuul/zuul-jobs master: merge-output-to-logs: convert to untrusted executor safe code  https://review.opendev.org/75832522:00
*** GomathiselviS has quit IRC22:05
*** rlandy|rover is now known as rlandy|rover|bbl22:09
*** armstrongs has joined #zuul22:10
openstackgerritIan Wienand proposed zuul/zuul-jobs master: merge-output-to-logs: convert to untrusted executor safe code  https://review.opendev.org/75832522:16
*** armstrongs has quit IRC22:20
*** hamalq has quit IRC22:32
*** hamalq has joined #zuul22:33
openstackgerritIan Wienand proposed zuul/zuul-jobs master: merge-output-to-logs: convert to untrusted executor safe code  https://review.opendev.org/75832522:53
*** holser has quit IRC23:18
*** holser has joined #zuul23:19
*** tosky has quit IRC23:23
*** holser has quit IRC23:43
*** jamesmcarthur has joined #zuul23:46

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