Re: [PR] openlineage: replace dt.now with airflow.utils.timezone.utcnow [airflow]

2024-07-22 Thread via GitHub
mobuchowski merged PR #40887: URL: https://github.com/apache/airflow/pull/40887 -- 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-unsubscr...@airf

Re: [PR] openlineage: replace dt.now with airflow.utils.timezone.utcnow [airflow]

2024-07-19 Thread via GitHub
kacpermuda commented on PR #40887: URL: https://github.com/apache/airflow/pull/40887#issuecomment-2239020080 > Yes got it. only the task FAIL was missing timezone info. DAG FAIL already has the timezone. Thanks for confirming :) Ah, yes, now i saw that you mean the DAG 😄 Yes, DAG FAIL

Re: [PR] openlineage: replace dt.now with airflow.utils.timezone.utcnow [airflow]

2024-07-19 Thread via GitHub
rahul-madaan commented on PR #40887: URL: https://github.com/apache/airflow/pull/40887#issuecomment-2239015988 Yes got it. only the task FAIL was missing timezone info. DAG FAIL already has the timezone. Thanks for confirming :) -- This is an automated message from the Apache Git Serv

Re: [PR] openlineage: replace dt.now with airflow.utils.timezone.utcnow [airflow]

2024-07-19 Thread via GitHub
kacpermuda commented on PR #40887: URL: https://github.com/apache/airflow/pull/40887#issuecomment-2238999877 > Just want to make sure, does this change also update the time format in DAG FAIL event? Not sure i understand what you mean by time format, but now we should receive a prope

Re: [PR] openlineage: replace dt.now with airflow.utils.timezone.utcnow [airflow]

2024-07-19 Thread via GitHub
rahul-madaan commented on PR #40887: URL: https://github.com/apache/airflow/pull/40887#issuecomment-2238986247 Just want to make sure, does this change also updates the time format in DAG FAIL event? -- This is an automated message from the Apache Git Service. To respond to the message, p

[PR] openlineage: replace dt.now with airflow.utils.timezone.utcnow [airflow]

2024-07-19 Thread via GitHub
kacpermuda opened a new pull request, #40887: URL: https://github.com/apache/airflow/pull/40887 Fix missing timezone information in task FAIL event. Related: https://github.com/OpenLineage/OpenLineage/issues/2858 --- **^ Add meaningful description above**