[PR] [CELEBORN-2397][CLI] Expose logger level get/set endpoints in celeborn-cli [celeborn]

["yew1eb (via GitHub)" <gi...@apache.org>]

yew1eb opened a new pull request, #3774:
URL: https://github.com/apache/celeborn/pull/3774

   ### What changes were proposed in this pull request?
   
   Expose the already-generated `LoggerApi` (`GET /api/v1/loggers` and `POST /api/v1/loggers`) in `celeborn-cli`, for both the `master` and `worker` subcommands:
   
   - `--show-loggers` (with optional `--logger-name` filter) to query logger levels
   - `--set-loglevel --logger-name <name> --logger-level <level>` to set a logger's level at runtime
   
   Usage examples:
   
   ```bash
   # Show all configured loggers of a worker
   celeborn-cli worker --hostport host:port --show-loggers
   
   # Show the level of a single logger of the master
   celeborn-cli master --hostport host:port --show-loggers --logger-name org.apache.celeborn
   
   # Set a logger level at runtime
   celeborn-cli worker --hostport host:port --set-loglevel --logger-name org.apache.celeborn --logger-level DEBUG
   ```
   
   ### Why are the changes needed?
   
   The logger level endpoints are already served by Master and Worker, and the `LoggerApi` client is already generated in `celeborn-openapi-client`, but they are not exposed in `celeborn-cli`. Exposing them allows adjusting log levels for troubleshooting without restarting Master/Worker or editing `log4j2.xml`.
   
   ### Does this PR resolve a correctness bug?
   
   - [ ] Yes
   
   ### Does this PR introduce _any_ user-facing change?
   
   - [x] Yes. New `celeborn-cli` actions `--show-loggers` and `--set-loglevel` (with `--logger-name`/`--logger-level` options) for both `master` and `worker` subcommands.
   
   ### How was this patch tested?
   
   New E2E test cases in `TestCelebornCliCommands` (mini cluster): `master/worker --show-loggers` and `master/worker --set-loglevel`, the latter verified by querying the level back. All 45 tests in the suite pass.


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@celeborn.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [CELEBORN-2397][CLI] Expose logger level get/set endpoints in celeborn-cli [celeborn]

["zaynt4606 (via GitHub)" <gi...@apache.org>]

zaynt4606 merged PR #3774:
URL: https://github.com/apache/celeborn/pull/3774


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@celeborn.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org