jedcunningham opened a new pull request #16357: URL: https://github.com/apache/airflow/pull/16357
During the 2.0 upgrade, the external log link when using elasticsearch remote logs was broken. This fixes it, including it only being shown if `[elasticsearch] frontend` is set. This should be a safe change as nothing is currently using `ExternalLoggingMixin`. In 1.10.x, the external links were being toggled by the following, which is why we can't simply add `ExternalLoggingMixin` into the `ElasticsearchTaskHandler` and need the extra `supports_external_link` property: https://github.com/apache/airflow/blob/4aec433e48dcc66c9c7b74947c499260ab6be9e9/airflow/www_rbac/views.py#L1569 https://github.com/apache/airflow/blob/4aec433e48dcc66c9c7b74947c499260ab6be9e9/airflow/www_rbac/views.py#L1585 -- 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. For queries about this service, please contact Infrastructure at: [email protected]
