Subham-KRLX commented on PR #63739: URL: https://github.com/apache/airflow/pull/63739#issuecomment-4105626790
> Thanks @Subham-KRLX. > > Could you please rebase on the latest `main` again? There will also be a new CI test covering the E2E Elasticsearch remote logging flow. Please also add an integration test in `providers/elasticsearch/tests/integration/elasticsearch/log/test_es_remote_log_io.py` to verify reading exception/error details from Elasticsearch, and a corresponding test under `TestElasticsearchRemoteLogIO`. I have rebased the branch on the latest main, added the requested integration test in providers/elasticsearch/tests/integration/elasticsearch/log/test_es_remote_log_io.py , and included a corresponding unit test under TestElasticsearchRemoteLogIO in test_es_task_handler.py. Verified locally that the new tests pass and error_detail is correctly formatted. -- 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]
