dstandish commented on PR #39177: URL: https://github.com/apache/airflow/pull/39177#issuecomment-2270016628
@kahlstrm can you clarify what you mean here? > This PR implements https://github.com/apache/airflow/pull/32561 in a different way. This caused a regression for our use case, where non-running task try logs weren't shown in UI for running tasks. This is due to us storing the logs on the worker with a Persistent Volume. Specifically this part: > This is due to us storing the logs on the worker with a Persistent Volume What does storing logs on the worker with a PV have to do with anything? If you're storing logs on a PV, shouldn't the webserver have access to it, so it can read the logs directly from the PV? This PR definitely has introduced a bug, because now users cannot see served logs from triggerer while deferred. But I'm just not sure exactly what functionality here we need to preserve and implement in a different way. -- 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]
