search

confluentinc / common-docker

Confluent Commons with support for building and testing Docker images.
Apache License 2.0
4 stars 69 forks source link

CPBR-1662 Set the root logger level to ERROR for tests (#7) #516

Closed wadhwa1 closed 3 months ago

wadhwa1 commented 3 months ago

Change Description

common-docker maven logs are huge, occupying over 95+MB size.. semaphoreCI trims the logs and put it in workflow artifact, need to download logs every time to check issue/ troubleshooting which is not good experience.. it was observed that most of these are actually unit test logs, so updated the root logger level to ERROR (for unit tests)

will also raise another PR (following airlock workflow) to suppress artifacts download logs as well with -ntp flag

[INFO] Downloading from third-party-artifacts: https://confluent-524708692776.d.codeartifact.us-west-2.amazonaws.com/maven/confluent-maven/io/confluent/assembly-plugin-boilerplate/7.2.12-6/assembly-plugin-boilerplate-7.2.12-6.pom00:03
[INFO] Downloading from confluent-codeartifact-internal: https://confluent-519856050701.d.codeartifact.us-west-2.amazonaws.com/maven/maven/io/confluent/assembly-plugin-boilerplate/7.2.12-6/assembly-plugin-boilerplate-7.2.12-6.pom

Testing

PR check workflow https://semaphore.ci.confluent.io/jobs/28fb8443-fcde-4e65-848b-6b60ac4369c2 shows logs are directly visible in semaphoreCI console.

rahejaprince commented 3 months ago

Should this be raised on 7.0.x?

wadhwa1 commented 3 months ago

Should this be raised on 7.0.x? I see excessive log output issue starting 7.2.x https://semaphore.ci.confluent.io/jobs/80ad8d22-c61a-47dd-a769-d821a124f917 , don't think there are any tests run for 7.0.x and 7.1.x and logs are visible https://semaphore.ci.confluent.io/jobs/85d6f2b0-a556-45d4-9fd7-388f7d42b93e (7.0.x) and https://semaphore.ci.confluent.io/jobs/384653ea-d033-497a-96ff-eb04a0ff4100 (7.1.x)