jeans/core
jeans/core
1
0
Fork 0
mirror of https://github.com/home-assistant/core.git synced 2025-07-28 15:47:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

Simplify getting domains to resolve in bootstrap (#145829)

Browse Source
This commit is contained in:
Artur Pragacz 2025-07-22 10:10:59 +02:00 committed by GitHub
parent 2315bcbfe3
commit f5d68a4ea4
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 16 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
homeassistant/bootstrap.py
View File

@ -695,10 +695,10 @@ async def async_mount_local_lib_path(config_dir: str) -> str:
def _get_domains(hass: core.HomeAssistant, config: dict[str, Any]) -> set[str]: def _get_domains(hass: core.HomeAssistant, config: dict[str, Any]) -> set[str]:
"""Get domains of components to set up.""" """Get domains of components to set up."""
# Filter out the repeating and common config section [homeassistant] # The common config section [homeassistant] could be filtered here,
domains = { # but that is not necessary, since it corresponds to the core integration,
domain for key in config if (domain := cv.domain_key(key)) != core.DOMAIN # that is always unconditionally loaded.
} domains = {cv.domain_key(key) for key in config}
# Add config entry and default domains # Add config entry and default domains
if not hass.config.recovery_mode: if not hass.config.recovery_mode:
@ -726,34 +726,28 @@ async def _async_resolve_domains_and_preload(
together with all their dependencies. together with all their dependencies.
""" """
domains_to_setup = _get_domains(hass, config) domains_to_setup = _get_domains(hass, config)
platform_integrations = conf_util.extract_platform_integrations(
config, BASE_PLATFORMS # Also process all base platforms since we do not require the manifest
) # to list them as dependencies.
# Ensure base platforms that have platform integrations are added to `domains`, # We want to later avoid lock contention when multiple integrations try to load
# so they can be setup first instead of discovering them later when a config # their manifests at once.
# entry setup task notices that it's needed and there is already a long line
# to use the import executor.
# #
# Additionally process integrations that are defined under base platforms
# to speed things up.
# For example if we have # For example if we have
# sensor: # sensor:
# - platform: template # - platform: template
# #
# `template` has to be loaded to validate the config for sensor # `template` has to be loaded to validate the config for sensor.
# so we want to start loading `sensor` as soon as we know # The more platforms under `sensor:`, the longer
# it will be needed. The more platforms under `sensor:`, the longer
# it will take to finish setup for `sensor` because each of these # it will take to finish setup for `sensor` because each of these
# platforms has to be imported before we can validate the config. # platforms has to be imported before we can validate the config.
# #
# Thankfully we are migrating away from the platform pattern # Thankfully we are migrating away from the platform pattern
# so this will be less of a problem in the future. # so this will be less of a problem in the future.
domains_to_setup.update(platform_integrations) platform_integrations = conf_util.extract_platform_integrations(
config, BASE_PLATFORMS
# Additionally process base platforms since we do not require the manifest )
# to list them as dependencies.
# We want to later avoid lock contention when multiple integrations try to load
# their manifests at once.
# Also process integrations that are defined under base platforms
# to speed things up.
additional_domains_to_process = { additional_domains_to_process = {
*BASE_PLATFORMS, *BASE_PLATFORMS,
*chain.from_iterable(platform_integrations.values()), *chain.from_iterable(platform_integrations.values()),