mobuchowski commented on code in PR #32692:
URL: https://github.com/apache/airflow/pull/32692#discussion_r1272367426
##########
airflow/providers_manager.py:
##########
@@ -1024,6 +1039,20 @@ def _discover_config(self) -> None:
if provider.data.get("config"):
self._provider_configs[provider_package] =
provider.data.get("config")
+ def _discover_plugins(self) -> None:
+ """Retrieve all plugins defined in the providers."""
+ for provider_package, provider in self._provider_dict.items():
+ if provider.data.get("plugins"):
+ for plugin_dict in provider.data["plugins"]:
+ if _correctness_check(provider_package,
plugin_dict["plugin-class"], provider):
+ self._plugins_set.add(
+ PluginInfo(
+ name=plugin_dict["name"],
+ plugin_class=plugin_dict["plugin-class"],
+ provider_name=provider_package,
+ )
+ )
Review Comment:
Done. Also added log warning without details - `_correctness_check` logs the
details itself.
##########
airflow/plugins_manager.py:
##########
@@ -267,6 +274,27 @@ def load_plugins_from_plugin_directory():
import_errors[file_path] = str(e)
+def load_providers_plugins():
+ from airflow.providers_manager import ProvidersManager
+
+ global import_errors
Review Comment:
Removed.
--
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: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
