Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Normalizes Maven build logs #2270

Merged
merged 1 commit into from
May 6, 2021
Merged

Conversation

orpiske
Copy link
Contributor

@orpiske orpiske commented May 5, 2021

Normalizes the Maven build log format so that the build messages go
through the operator log. This makes the logs more easily parseable
by log collection tools due to the format of the previous Maven build
log being different.

This solves GH issue #2268

Release Note

Normalized Maven build log so that it is more easily parseable by log collection tools

@orpiske
Copy link
Contributor Author

orpiske commented May 5, 2021

This is how it looks like now:

Uploading normalized-maven-log.png…

@orpiske
Copy link
Contributor Author

orpiske commented May 5, 2021

This is how it looks like now:

{"level":"info","ts":1620228691.0135958,"logger":"camel-k.maven.build","msg":"Downloaded from central: https://repo.maven.apache.org/maven2/io/quarkus/quarkus-bom-quarkus-platform-properties/1.13.0.Final/quarkus
-bom-quarkus-platform-properties-1.13.0.Final.properties (92 B at 730 B/s)"} 

@orpiske orpiske force-pushed the mvn-build-correlation branch 2 times, most recently from bc62501 to ba10454 Compare May 5, 2021 15:46
Normalizes the Maven build log format so that the build messages go
through the operator log. This makes the logs more easily parseable
by log collection tools due to the format of the previous Maven build
log being different.

This solves GH issue apache#2268
Copy link
Member

@astefanutti astefanutti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome!

@astefanutti
Copy link
Member

There is the Spectrum logs left, so that we'll have the operator logs fully JSONified, just saying 😉.

@astefanutti
Copy link
Member

CI is failing because of apache/camel-kamelets#238.

@orpiske
Copy link
Contributor Author

orpiske commented May 5, 2021

There is the Spectrum logs left, so that we'll have the operator logs fully JSONified, just saying 😉.

We'll get there :D /me adds that to the TODO list

@oscerd
Copy link
Contributor

oscerd commented May 6, 2021

CI is failing because of apache/camel-kamelets#238.

It should be fixed now.

@orpiske orpiske merged commit 06b464c into apache:main May 6, 2021
@nicolaferraro nicolaferraro mentioned this pull request Jul 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants