potiuk commented on code in PR #31920:
URL: https://github.com/apache/airflow/pull/31920#discussion_r1231051073


##########
docker_tests/test_prod_image.py:
##########
@@ -127,7 +127,7 @@ def test_pip_dependencies_conflict(self):
         "cncf.kubernetes": ["kubernetes", "cryptography"],
         "dask": ["cloudpickle", "distributed"],
         "docker": ["docker"],
-        "elasticsearch": ["elasticsearch", "es.elastic", "elasticsearch_dsl"],
+        "elasticsearch": ["elasticsearch"],

Review Comment:
   I guess.. No need because we aren't going to cherry-pick that one I guess. 
It would have been more complex if we actually decided to cherry-pick because 
we woud have to remember to cherry-pick both together (that's what you loose 
when you split such related PRs that they become non-cherry-pickable as single 
PR.



-- 
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]

Reply via email to