astefanutti commented on a change in pull request #2311:
URL: https://github.com/apache/camel-k/pull/2311#discussion_r634991871
##########
File path: pkg/util/maven/maven.go
##########
@@ -178,17 +178,24 @@ func Run(ctx Context) error {
Log.Debug("About to start parsing the Maven output")
+ structuredLogs := HasStructuredLog()
+
for scanner.Scan() {
Review comment:
Would trying parsing the first line and falling back to unstructured be
a solution? It seems it'd be a more robust heuristic than relying on
`MAVEN_OPTS` and `logback.configurationFile`, that do not guarantee structured
logging is actually enabled.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
