-
Notifications
You must be signed in to change notification settings - Fork 92
[infra] Add configurable log level for CI test runs #518
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
[infra] Add configurable log level for CI test runs #518
Conversation
|
This CI not passing could be due to network instability. |
|
Hi, @XL-Zhao-23, thanks for your contribution. Since currently we don't have too many test cases, I think printing the log to screen is acceptable. Can we also print the log for python test either? |
79d78e4 to
0d1c9e8
Compare
2d97a21 to
1201797
Compare
|
Thanks for the suggestion! |
|
LGTM. Could you take a look at your convenient @xintongsong ? |
|
Ported to release-0.2 in 270fa95. |
[infra] Add configurable log level for CI test runs
Linked issue: #96
Purpose of change
Improve observability of CI test runs by making log levels configurable through environment variables. This allows enabling INFO level logging in CI workflows for debugging purposes while keeping local test runs quiet by default.
Tests
log4j2-test.propertiesin runtime and e2e-test modules to supportLOG_LEVELenvironment variable.github/workflows/ci.ymlto setLOG_LEVEL=INFOfor Java tests, Java IT, and e2e testsAPI
No public API changes.
Documentation
doc-not-needed