ashb commented on code in PR #63365:
URL: https://github.com/apache/airflow/pull/63365#discussion_r2926088039
##########
airflow-core/src/airflow/utils/db.py:
##########
@@ -947,8 +947,9 @@ def check_and_run_migrations():
if sys.stdout.isatty() and verb:
print()
question = f"Please confirm database {verb} (or wait 4 seconds to skip
it). Are you sure? [y/N]"
+ print_fn = log.info if conf.getboolean("logging", "json_logs",
fallback=False) else print
try:
- answer = helpers.prompt_with_timeout(question, timeout=4,
default=False)
+ answer = helpers.prompt_with_timeout(question, timeout=4,
default=False, print=print_fn)
Review Comment:
That is a very good question. Probably cos `check_and_run_migrations` isn't
exercised in tests (we put "assume yes" on or something)
--
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]
