kahlstrm commented on PR #39177: URL: https://github.com/apache/airflow/pull/39177#issuecomment-2270123467
> Thanks @kahlstrm, kind of you to follow up. What i'm going with now in #41272 is basically reverting everything -- the fix before yours (that introduced the bug you found), and your two fixes for the bugs introduced by that fix. I am just not sure it's worth the complexity just to avoid an edge case `403` error message that isn't of much consequence. If someone has time to reintroduce a better approach to suppressing the 403 in that case (e.g. perhaps just suppress the 403) then they can. But for now, I just want to fix the inability to access logs while in deferred state. Sounds good to me 👍 I agree with you on this, that adding this amount of logical complexity just to avoid a single request error is not worth it, but didn't myself want to revert the wanted behavior of #32561 immediately. As it now has turned out, having this amount of bugs/unwanted behavior come out of such change is not worth it IMO. -- 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]
