Monday, 2018-08-13

*** tflink has quit IRC02:08
*** tflink has joined #zuul02:37
*** dvn has left #zuul03:44
*** openstackgerrit has quit IRC05:18
*** pcaruana has joined #zuul05:59
*** openstackgerrit has joined #zuul06:00
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: executor: enable add_host for trusted play and persist inventory  https://review.openstack.org/59009206:00
*** pcaruana has quit IRC06:05
*** pcaruana has joined #zuul06:19
*** ssbarnea has joined #zuul06:54
openstackgerritMarkus Hosch proposed openstack-infra/nodepool master: Add list of metrics provided to statsd  https://review.openstack.org/59023307:20
openstackgerritAndreas Jaeger proposed openstack-infra/zuul-website master: Link to Zuul Sphinx extension  https://review.openstack.org/59128207:29
*** pcaruana has quit IRC07:43
*** jpena|off is now known as jpena07:55
*** pcaruana has joined #zuul07:57
*** electrofelix has joined #zuul09:33
*** panda is now known as panda|ruck10:15
*** eumel8 has joined #zuul10:19
openstackgerritMarkus Hosch proposed openstack-infra/zuul master: Fix reject clauses in the absence of approvals  https://review.openstack.org/58976210:34
*** threestrands has quit IRC10:36
openstackgerritMarkus Hosch proposed openstack-infra/zuul master: Fix reject clauses in the absence of approvals  https://review.openstack.org/58976210:37
*** gouthamr has quit IRC10:44
*** dmellado has quit IRC10:46
*** sshnaidm is now known as sshnaidm|off11:04
*** jpena is now known as jpena|lunch11:14
*** panda|ruck is now known as panda|ruck|lunch11:23
*** panda|ruck|lunch is now known as panda|ruck12:02
*** jpena|lunch is now known as jpena12:13
*** darkwisebear has joined #zuul12:25
openstackgerritMatthieu Huin proposed openstack-infra/nodepool master: Do not abort node launch if failed node cannot be deleted  https://review.openstack.org/58985412:45
*** dmellado has joined #zuul12:50
openstackgerritMarkus Hosch proposed openstack-infra/nodepool master: Add list of metrics provided to statsd  https://review.openstack.org/59023313:08
*** nhicher has joined #zuul13:12
openstackgerritMarkus Hosch proposed openstack-infra/nodepool master: Add list of metrics provided to statsd  https://review.openstack.org/59023313:14
*** pwhalen has joined #zuul13:16
*** elyezer has quit IRC13:28
*** elyezer has joined #zuul13:30
*** darkwisebear has quit IRC13:45
*** chmouel has joined #zuul14:46
*** chmouel has quit IRC14:54
*** pcaruana has quit IRC15:37
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: executor: enable add_host for trusted play and update inventory  https://review.openstack.org/59009215:47
*** gouthamr has joined #zuul15:54
* SpamapS is annoyed he has to stop playing with Zuul-on-k8s to do other stuff now15:59
openstackgerritMerged openstack-infra/zuul master: Fix web content copying in multi dashboard job  https://review.openstack.org/59108916:17
mordredSpamapS: other stuff is dumb16:21
openstackgerritPaul Belanger proposed openstack-infra/zuul-jobs master: DNM - base-test  https://review.openstack.org/59145316:22
openstackgerritPaul Belanger proposed openstack-infra/zuul-jobs master: DNM - base-test  https://review.openstack.org/59145316:24
*** jpena is now known as jpena|off16:34
openstackgerritPaul Belanger proposed openstack-infra/zuul-jobs master: Promote test-emit-job-header change  https://review.openstack.org/59145716:39
pabelanger^ should be safe to land, I've tested with both openstack-infra and rdoproject zuul.16:40
*** pcaruana has joined #zuul16:40
openstackgerritFrank Kloeker proposed openstack-infra/zuul master: Remove non-working html code in div container  https://review.openstack.org/59145916:53
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Map file comment line numbers  https://review.openstack.org/59044216:53
*** jimi|ansible has quit IRC17:00
*** jimi|ansible has joined #zuul17:01
*** openstackgerrit has quit IRC17:19
corvusmordred: you said you needed to rework https://review.openstack.org/589634  -- what's wrong with it?17:31
*** eandersson has quit IRC17:32
*** openstackgerrit has joined #zuul17:32
openstackgerritJames E. Blair proposed openstack-infra/zuul-jobs master: WIP Extract pep8 messages for inline comments  https://review.openstack.org/58963417:32
openstackgerritJames E. Blair proposed openstack-infra/zuul master: DNM: test pep8 failures  https://review.openstack.org/59146917:34
corvusmordred: ^17:34
*** ianychoi_ has joined #zuul17:38
*** ianychoi has quit IRC17:41
openstackgerritMerged openstack-infra/zuul-jobs master: Stop logging inventory in validate-host  https://review.openstack.org/56379017:42
mordredcorvus: the issue is that it's capturing stdout of a task that can fail - and the stdout is only interesting if the task fails17:53
mordredcorvus: so trying to capture it in the playbook directly means you'd have to also do a failed_when: false - and then probably a followup fail: when: registered_output.rc != 0 after the log analysis17:54
mordredcorvus: my thinking was to rework it to be a thing that could be run in a post playbook - that would look at the console.log on the executor17:55
corvusmordred: oh i see. fun.17:56
corvusmordred: that makes sense.  or maybe "flake8 --tee --output-file=flake8.txt" then have the post playbook run on flake8.txt17:57
corvusthough... that entails changing tox.ini doesn't it?17:58
corvusso, i guess, nevermind.  :)17:58
corvusmordred: the more generic thing might actually work for a bunch of types of jobs too.  if we make it iterate over a list of regexes, that could be pretty flexible.17:59
*** pcaruana has quit IRC17:59
mordredcorvus: yah - that's sort of what I was thinking - we could potentially even have such a processor hooked in to a base job18:00
mordred*waves arms*18:00
corvusmordred: i'd probably suggest checking any potential filename matches against those files actually being in the work dir, that should filter out any false matches that happen to have the right pattern of text,colon:number, etc.18:00
corvusso then you've got something that is basically "if a line looks like a file,line number,character,message tuple, and the file is a real file on disk, treat it as an inline comment"18:01
mordred++18:02
*** electrofelix has quit IRC18:19
*** jpena|off is now known as jpena19:04
*** jpena is now known as jpena|off19:06
*** fdegir_ has joined #zuul19:17
*** gundalow_ has joined #zuul19:18
*** bstinson_ has joined #zuul19:23
*** mgagne_ has joined #zuul19:24
*** austinsun[m] has quit IRC19:25
*** gundalow has quit IRC19:25
*** fdegir has quit IRC19:25
*** robcresswell has quit IRC19:25
*** mgagne has quit IRC19:25
*** bstinson has quit IRC19:25
*** gundalow_ is now known as gundalow19:25
*** fdegir_ is now known as fdegir19:25
*** hwoarang has joined #zuul21:18
openstackgerritJames E. Blair proposed openstack-infra/zuul-jobs master: Add icons to index files  https://review.openstack.org/59153523:36
openstackgerritJames E. Blair proposed openstack-infra/zuul-jobs master: Add icons to index files  https://review.openstack.org/59153523:40

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!