potiuk commented on code in PR #70755:
URL: https://github.com/apache/airflow/pull/70755#discussion_r3711975689
##########
shared/configuration/src/airflow_shared/configuration/parser.py:
##########
@@ -77,6 +77,34 @@ def _collect_kwarg_env_vars(prefix: str) -> dict[str, str]:
ConfigSectionSourcesType = dict[str, str | tuple[str, str]]
ConfigSourcesType = dict[str, ConfigSectionSourcesType]
ENV_VAR_PREFIX = "AIRFLOW__"
+# Separates the team name from the base section name in a team scoped config
file section.
+TEAM_SECTION_SEPARATOR = "="
+
+
+def team_section_name(team_name: str, section: str) -> str:
+ """
+ Build the config file section name that holds the team scoped overrides of
``section``.
+
+ :param team_name: name of the team the overrides belong to
+ :param section: base section name that is being overridden
+ :return: the team scoped section name, e.g. ``team_a=celery``
+ """
+ return f"{team_name}{TEAM_SECTION_SEPARATOR}{section}"
+
+
+def base_section_name(section: str) -> str:
Review Comment:
They are the reusable, shared configuration-related small util methods, so
.. it checks all the boxes IMHO
--
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]
