Taragolis commented on code in PR #37558:
URL: https://github.com/apache/airflow/pull/37558#discussion_r1495763609


##########
tests/always/test_example_dags.py:
##########
@@ -36,6 +44,29 @@
         "The test is skipped because we are running in limited Pydantic 
environment", allow_module_level=True
     )
 
+# Some certain of examples/system tests might require additional dependencies,
+# which are not installed into specific CI check
+# Format of dictionary:
+# key: prefix of the file which need to be excluded,
+# values: dictionary with package distributions and optional specifier, e.g. 
>=2.3.4
+OPTIONAL_PROVIDERS_DEPENDENCIES: dict[str, dict[str, str | None]] = {
+    # Regression of https://github.com/apache/airflow/pull/37524
+    # It loads the module now eagerly instead of lazily
+    "tests/system/providers/common/io/example_file_transfer_local_to_s3.py": 
{"s3fs": None}

Review Comment:
   But yeah. This fix either not work well. It fix on one case, but not in the 
others
   



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