o-nikolas commented on code in PR #69768:
URL: https://github.com/apache/airflow/pull/69768#discussion_r3678976791


##########
airflow-core/src/airflow/cli/commands/team_command.py:
##########
@@ -163,6 +201,9 @@ def team_list(args, *, session=NEW_SESSION):
 @provide_session
 def team_sync(args, *, session=NEW_SESSION):
     """Sync missing teams from the dag bundle config."""
+    if not conf.getboolean("core", "multi_team"):
+        return

Review Comment:
   +1 to this one



##########
airflow-core/newsfragments/69768.feature.rst:
##########
@@ -0,0 +1 @@
+Add automatic creation of team default pools in multi-team deployments and 
assign tasks without an explicitly configured pool to their team's default pool 
during DAG parsing. Existing multi-team deployments should run ``airflow teams 
sync`` after upgrading to provision default pools for existing teams.

Review Comment:
   ```suggestion
   Add automatic creation of team default pools in multi-team deployments and 
assign tasks without an explicitly configured pool to their team's default pool 
during Dag parsing. Existing multi-team deployments should run ``airflow teams 
sync`` after upgrading to provision default pools for existing teams.
   ```



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