*** tflink has quit IRC | 02:08 | |
*** tflink has joined #zuul | 02:37 | |
*** dvn has left #zuul | 03:44 | |
*** openstackgerrit has quit IRC | 05:18 | |
*** pcaruana has joined #zuul | 05:59 | |
*** openstackgerrit has joined #zuul | 06:00 | |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: executor: enable add_host for trusted play and persist inventory https://review.openstack.org/590092 | 06:00 |
---|---|---|
*** pcaruana has quit IRC | 06:05 | |
*** pcaruana has joined #zuul | 06:19 | |
*** ssbarnea has joined #zuul | 06:54 | |
openstackgerrit | Markus Hosch proposed openstack-infra/nodepool master: Add list of metrics provided to statsd https://review.openstack.org/590233 | 07:20 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-website master: Link to Zuul Sphinx extension https://review.openstack.org/591282 | 07:29 |
*** pcaruana has quit IRC | 07:43 | |
*** jpena|off is now known as jpena | 07:55 | |
*** pcaruana has joined #zuul | 07:57 | |
*** electrofelix has joined #zuul | 09:33 | |
*** panda is now known as panda|ruck | 10:15 | |
*** eumel8 has joined #zuul | 10:19 | |
openstackgerrit | Markus Hosch proposed openstack-infra/zuul master: Fix reject clauses in the absence of approvals https://review.openstack.org/589762 | 10:34 |
*** threestrands has quit IRC | 10:36 | |
openstackgerrit | Markus Hosch proposed openstack-infra/zuul master: Fix reject clauses in the absence of approvals https://review.openstack.org/589762 | 10:37 |
*** gouthamr has quit IRC | 10:44 | |
*** dmellado has quit IRC | 10:46 | |
*** sshnaidm is now known as sshnaidm|off | 11:04 | |
*** jpena is now known as jpena|lunch | 11:14 | |
*** panda|ruck is now known as panda|ruck|lunch | 11:23 | |
*** panda|ruck|lunch is now known as panda|ruck | 12:02 | |
*** jpena|lunch is now known as jpena | 12:13 | |
*** darkwisebear has joined #zuul | 12:25 | |
openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: Do not abort node launch if failed node cannot be deleted https://review.openstack.org/589854 | 12:45 |
*** dmellado has joined #zuul | 12:50 | |
openstackgerrit | Markus Hosch proposed openstack-infra/nodepool master: Add list of metrics provided to statsd https://review.openstack.org/590233 | 13:08 |
*** nhicher has joined #zuul | 13:12 | |
openstackgerrit | Markus Hosch proposed openstack-infra/nodepool master: Add list of metrics provided to statsd https://review.openstack.org/590233 | 13:14 |
*** pwhalen has joined #zuul | 13:16 | |
*** elyezer has quit IRC | 13:28 | |
*** elyezer has joined #zuul | 13:30 | |
*** darkwisebear has quit IRC | 13:45 | |
*** chmouel has joined #zuul | 14:46 | |
*** chmouel has quit IRC | 14:54 | |
*** pcaruana has quit IRC | 15:37 | |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: executor: enable add_host for trusted play and update inventory https://review.openstack.org/590092 | 15:47 |
*** gouthamr has joined #zuul | 15:54 | |
* SpamapS is annoyed he has to stop playing with Zuul-on-k8s to do other stuff now | 15:59 | |
openstackgerrit | Merged openstack-infra/zuul master: Fix web content copying in multi dashboard job https://review.openstack.org/591089 | 16:17 |
mordred | SpamapS: other stuff is dumb | 16:21 |
openstackgerrit | Paul Belanger proposed openstack-infra/zuul-jobs master: DNM - base-test https://review.openstack.org/591453 | 16:22 |
openstackgerrit | Paul Belanger proposed openstack-infra/zuul-jobs master: DNM - base-test https://review.openstack.org/591453 | 16:24 |
*** jpena is now known as jpena|off | 16:34 | |
openstackgerrit | Paul Belanger proposed openstack-infra/zuul-jobs master: Promote test-emit-job-header change https://review.openstack.org/591457 | 16:39 |
pabelanger | ^ should be safe to land, I've tested with both openstack-infra and rdoproject zuul. | 16:40 |
*** pcaruana has joined #zuul | 16:40 | |
openstackgerrit | Frank Kloeker proposed openstack-infra/zuul master: Remove non-working html code in div container https://review.openstack.org/591459 | 16:53 |
openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Map file comment line numbers https://review.openstack.org/590442 | 16:53 |
*** jimi|ansible has quit IRC | 17:00 | |
*** jimi|ansible has joined #zuul | 17:01 | |
*** openstackgerrit has quit IRC | 17:19 | |
corvus | mordred: you said you needed to rework https://review.openstack.org/589634 -- what's wrong with it? | 17:31 |
*** eandersson has quit IRC | 17:32 | |
*** openstackgerrit has joined #zuul | 17:32 | |
openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: WIP Extract pep8 messages for inline comments https://review.openstack.org/589634 | 17:32 |
openstackgerrit | James E. Blair proposed openstack-infra/zuul master: DNM: test pep8 failures https://review.openstack.org/591469 | 17:34 |
corvus | mordred: ^ | 17:34 |
*** ianychoi_ has joined #zuul | 17:38 | |
*** ianychoi has quit IRC | 17:41 | |
openstackgerrit | Merged openstack-infra/zuul-jobs master: Stop logging inventory in validate-host https://review.openstack.org/563790 | 17:42 |
mordred | corvus: the issue is that it's capturing stdout of a task that can fail - and the stdout is only interesting if the task fails | 17:53 |
mordred | corvus: 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 analysis | 17:54 |
mordred | corvus: 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 executor | 17:55 |
corvus | mordred: oh i see. fun. | 17:56 |
corvus | mordred: that makes sense. or maybe "flake8 --tee --output-file=flake8.txt" then have the post playbook run on flake8.txt | 17:57 |
corvus | though... that entails changing tox.ini doesn't it? | 17:58 |
corvus | so, i guess, nevermind. :) | 17:58 |
corvus | mordred: 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 IRC | 17:59 | |
mordred | corvus: yah - that's sort of what I was thinking - we could potentially even have such a processor hooked in to a base job | 18:00 |
mordred | *waves arms* | 18:00 |
corvus | mordred: 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 |
corvus | so 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 IRC | 18:19 | |
*** jpena|off is now known as jpena | 19:04 | |
*** jpena is now known as jpena|off | 19:06 | |
*** fdegir_ has joined #zuul | 19:17 | |
*** gundalow_ has joined #zuul | 19:18 | |
*** bstinson_ has joined #zuul | 19:23 | |
*** mgagne_ has joined #zuul | 19:24 | |
*** austinsun[m] has quit IRC | 19:25 | |
*** gundalow has quit IRC | 19:25 | |
*** fdegir has quit IRC | 19:25 | |
*** robcresswell has quit IRC | 19:25 | |
*** mgagne has quit IRC | 19:25 | |
*** bstinson has quit IRC | 19:25 | |
*** gundalow_ is now known as gundalow | 19:25 | |
*** fdegir_ is now known as fdegir | 19:25 | |
*** hwoarang has joined #zuul | 21:18 | |
openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Add icons to index files https://review.openstack.org/591535 | 23:36 |
openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Add icons to index files https://review.openstack.org/591535 | 23:40 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!