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]