[GitHub] [airflow] potiuk commented on pull request #24811: Add %z for %(asctime)s to fix timezone for logs on UI

[GitBox <gi...@apache.org>]

potiuk commented on PR #24811:
URL: https://github.com/apache/airflow/pull/24811#issuecomment-1181392738

   Should we possibly add a warning if someone does not use TimezoneAware formatter? I think it is no harm to add it (with clear instruction on what to do). I think writing it in the logs of tasks, would make it much more discoverable - you could simply print a warning "Beware, this log might contain wrong timezone and you should use this and that to fix it". 
   
   There are two kinds of users - those who read the "migration guides and changelog" and  those who don't and the latter kind of user will generally open an issue if they see logs with wrong timestamp. Producing a warning in the very logs that the user might complain about is as good as we can do to make the user self-serviced, especially if we are very explicit that "timestamps here might have wrong timezone" and add "this is how you fix it".


-- 
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: commits-unsubscribe@airflow.apache.org

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