Vamsi-klu commented on PR #69828:
URL: https://github.com/apache/airflow/pull/69828#issuecomment-4966077609
This one is straightforward. The `if args.verbose: celery_log_level =
"DEBUG"` sits right after the `CELERY_LOGGING_LEVEL`/`LOGGING_LEVEL` resolution
in `worker()` and before `celery_log_level` gets dropped into the `--loglevel`
slot of `options`, so `-v` actually rides through to `celery_app.worker_main`
now. `args.verbose` is the shared `ARG_VERBOSE` store_true wired into the
worker command in `definition.py`, so it is always set and forcing DEBUG only
ever raises verbosity, never lowers a configured level. The added test asserts
the exact `--loglevel DEBUG` slot. Only stray thought is whether the generic
`-v` help text ("Make logging output more verbose") makes this discoverable for
the celery worker case, but that is non blocking.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
