From b61239665803d0ff5f1108277616b5d33986442a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Steven!=20Ragnar=C3=B6k?= <steven@nuclearsandwich.com>
Date: Wed, 6 May 2020 15:46:24 -0400
Subject: [PATCH 1/2] Add Groovy-based Python warning parser.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The parser itself is actually part of the primary Jenkisn configuration
and requires separate documentation or being contributed as an upstream
warning parser.

Groovy-based parsers are not allowed to run on console output so this
scans the stderr.log files from colcon test output.

Signed-off-by: Steven! Ragnarök <steven@nuclearsandwich.com>
---
 job_templates/snippet/publisher_warnings_ng.xml.em | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/job_templates/snippet/publisher_warnings_ng.xml.em b/job_templates/snippet/publisher_warnings_ng.xml.em
index d6c4bc395..6a389d278 100644
--- a/job_templates/snippet/publisher_warnings_ng.xml.em
+++ b/job_templates/snippet/publisher_warnings_ng.xml.em
@@ -42,6 +42,14 @@
 @[else]@
 @{assert False, 'Unknown os_name: ' + os_name}@
 @[end if]@
+    <io.jenkins.plugins.analysis.warnings.groovy.GroovyScript>
+      <id />
+      <name />
+      <pattern>ws/log/latest_test/**/stderr.log</pattern>
+      <reportEncoding />
+      <skipSymbolicLinks>false</skipSymbolicLinks>
+      <parserId>python-warnings</parserId>
+    </io.jenkins.plugins.analysis.warnings.groovy.GroovyScript>
   </analysisTools>
   <sourceCodeEncoding></sourceCodeEncoding>
   <sourceDirectory></sourceDirectory>

From 89fa854e6ca783dd76165b97bb3d641086bda9e7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Steven!=20Ragnar=C3=B6k?= <steven@nuclearsandwich.com>
Date: Wed, 6 May 2020 16:07:57 -0400
Subject: [PATCH 2/2] Parse colcon log decorations and pytest output.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

We're getting these warnings secondhand via a pytest warnings summary
via colcon logs and there were additional items that needed to be parsed
out in order for the warning data to be accurate.

Signed-off-by: Steven! Ragnarök <steven@nuclearsandwich.com>
---
 job_templates/snippet/publisher_warnings_ng.xml.em | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/job_templates/snippet/publisher_warnings_ng.xml.em b/job_templates/snippet/publisher_warnings_ng.xml.em
index 6a389d278..8d0d7d76d 100644
--- a/job_templates/snippet/publisher_warnings_ng.xml.em
+++ b/job_templates/snippet/publisher_warnings_ng.xml.em
@@ -48,7 +48,7 @@
       <pattern>ws/log/latest_test/**/stderr.log</pattern>
       <reportEncoding />
       <skipSymbolicLinks>false</skipSymbolicLinks>
-      <parserId>python-warnings</parserId>
+      <parserId>colcon-log-pytest-warnings</parserId>
     </io.jenkins.plugins.analysis.warnings.groovy.GroovyScript>
   </analysisTools>
   <sourceCodeEncoding></sourceCodeEncoding>