Skip to content

Conversation

@XL-Zhao-23
Copy link
Contributor

[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

  • Modified log4j2-test.properties in runtime and e2e-test modules to support LOG_LEVEL environment variable
  • Updated .github/workflows/ci.yml to set LOG_LEVEL=INFO for Java tests, Java IT, and e2e tests
  • Default behavior remains unchanged (OFF) when environment variable is not set

API

No public API changes.

Documentation

  • doc-not-needed

@github-actions github-actions bot added priority/major Default priority of the PR or issue. fixVersion/0.2.0 The feature or bug should be implemented/fixed in the 0.2.0 version. doc-not-needed Your PR changes do not impact docs labels Feb 5, 2026
@XL-Zhao-23 XL-Zhao-23 marked this pull request as ready for review February 5, 2026 07:50
@XL-Zhao-23 XL-Zhao-23 marked this pull request as draft February 5, 2026 07:50
@XL-Zhao-23 XL-Zhao-23 marked this pull request as ready for review February 5, 2026 07:51
@XL-Zhao-23
Copy link
Contributor Author

This CI not passing could be due to network instability.

@wenjin272 wenjin272 added fixVersion/0.3.0 The feature or bug should be implemented/fixed in the 0.3.0 version. and removed fixVersion/0.2.0 The feature or bug should be implemented/fixed in the 0.2.0 version. labels Feb 6, 2026
@wenjin272
Copy link
Collaborator

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?

@XL-Zhao-23 XL-Zhao-23 force-pushed the tech-debt/#96-improve-observability-of-ci branch 2 times, most recently from 79d78e4 to 0d1c9e8 Compare February 10, 2026 07:09
@XL-Zhao-23 XL-Zhao-23 marked this pull request as draft February 10, 2026 07:10
@XL-Zhao-23 XL-Zhao-23 force-pushed the tech-debt/#96-improve-observability-of-ci branch from 2d97a21 to 1201797 Compare February 10, 2026 07:39
@XL-Zhao-23 XL-Zhao-23 marked this pull request as ready for review February 10, 2026 07:52
@XL-Zhao-23
Copy link
Contributor Author

Thanks for the suggestion!
Python tests now support configurable log levels: CI runs output INFO logs (via LOG_LEVEL: INFO in .github/workflows/ci.yml) while local runs remain quiet by default (CRITICAL), configurable with export LOG_LEVEL=INFO.
The tools/ut.sh script passes this to pytest via -o log_cli=true -o log_cli_level=${LOG_LEVEL:-CRITICAL}, keeping behavior consistent with Java tests and avoiding pyproject.toml schema issues.

@wenjin272
Copy link
Collaborator

LGTM. Could you take a look at your convenient @xintongsong ?

@xintongsong xintongsong merged commit f74d90a into apache:main Feb 10, 2026
20 checks passed
@xintongsong
Copy link
Contributor

Ported to release-0.2 in 270fa95.

@xintongsong xintongsong added the fixVersion/0.2.1 The feature or bug should be implemented/fixed in the 0.2.1 version. label Feb 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs fixVersion/0.2.1 The feature or bug should be implemented/fixed in the 0.2.1 version. fixVersion/0.3.0 The feature or bug should be implemented/fixed in the 0.3.0 version. priority/major Default priority of the PR or issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants