vincbeck commented on code in PR #69502:
URL: https://github.com/apache/airflow/pull/69502#discussion_r3536859640


##########
airflow-core/src/airflow/plugins_manager.py:
##########
@@ -422,3 +422,40 @@ def get_priority_weight_strategy_plugins() -> dict[str, 
type[PriorityWeightStrat
 def get_import_errors() -> dict[str, str]:
     """Get import errors encountered during plugin loading."""
     return _get_plugins()[1]
+
+
+def validate_plugin_teams() -> None:

Review Comment:
   I am only wondering if this is necessary to be honest. The person managing 
the environment might not be the same as the one managing the plugins? If a 
team gets deleted (I know, rare event), then all the plugins mentioning this 
team must be updated as well. In the API server we had this check and some 
users were complaining about it, thus we converted to warning instead of 
exception because it is too aggressive



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