New backup system delays startup as much as 2x #133904

bdraco · 2024-12-23T18:13:43Z

The problem

What version of Home Assistant Core has the issue?

2025.1.0b

What was the last working version of Home Assistant Core?

n/a

What type of installation are you running?

Home Assistant OS

Integration causing the issue

No response

Link to integration documentation on our website

No response

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

home-assistant · 2024-12-23T18:13:52Z

Hey there @home-assistant/core, mind taking a look at this issue as it has been labeled with an integration (backup) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of backup can trigger bot actions by commenting:

@home-assistant close Closes the issue.
@home-assistant rename Awesome new title Renames the issue.
@home-assistant reopen Reopen the issue.
@home-assistant unassign backup Removes the current integration label and assignees on the issue, add the integration domain after the command.
@home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
@home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

_{^{(message by CodeOwnersMention)}}

backup documentation
backup source
_{^{(message by IssueLinks)}}

bdraco · 2024-12-23T18:14:43Z

#133857
#133855
#133856

These did some work on this to help but isn't enough for all cases.

I'm guessing cloud/backup is still causing the delay.. but need to dig in

bdraco · 2024-12-23T18:25:33Z

recorder and cloud are having their integration platform imported when the import executor is busy which means everything gets in line behind it.

I also see it trying to load for config... need to dig into that.

cloud and recorder are strait-forward to fix

bdraco · 2024-12-23T18:25:54Z

2024-12-23 08:24:05.440 DEBUG (MainThread) [homeassistant.loader] Importing platforms for backup executor=[] loop=['config'] took 0.00s

bdraco · 2024-12-23T18:26:40Z

we also have it the other way around

2024-12-23 08:24:05.655 DEBUG (MainThread) [homeassistant.loader] Importing platforms for recorder executor=['backup'] loop=[] took 0.21s

bdraco · 2024-12-23T18:27:35Z

and integration platforms don't exclude themselves

…e import executor partially fixes #133904

bdraco · 2024-12-23T18:34:17Z

2024-12-23 08:33:39.305 DEBUG (MainThread) [homeassistant.loader] Importing platforms for backup executor=['backup'] loop=[] took 0.55s

…e import executor (#133907) * Ensure cloud and recorder backup platforms do not have to wait for the import executor partially fixes #133904 * backup.backup as well

Co-authored-by: Abílio Costa <abmantis@users.noreply.github.com> Co-authored-by: Joost Lekkerkerker <joostlek@outlook.com> Co-authored-by: VandeurenGlenn <8685280+VandeurenGlenn@users.noreply.github.com> Co-authored-by: Joostlek <joostlek@outlook.com> Co-authored-by: Allen Porter <allen.porter@gmail.com> Co-authored-by: J. Nick Koston <nick@koston.org> Co-authored-by: Teemu R. <tpr@iki.fi> Co-authored-by: Michael <35783820+mib1185@users.noreply.github.com> Co-authored-by: Franck Nijhof <git@frenck.dev> Co-authored-by: OzGav <gavnosp@hotmail.com> Co-authored-by: Franck Nijhof <frenck@frenck.nl> Co-authored-by: Simon <80467011+sorgfresser@users.noreply.github.com> Co-authored-by: Simon Sorg <simon.sorg@student.hpi.de> Co-authored-by: G Johansson <goran.johansson@shiftit.se> Co-authored-by: Sander Hoentjen <sander@hoentjen.eu> Co-authored-by: Paulus Schoutsen <paulus@home-assistant.io> Co-authored-by: Robert Resch <robert@resch.dev> Co-authored-by: Erik Montnemery <erik@montnemery.com> Co-authored-by: Richard Kroegel <42204099+rikroe@users.noreply.github.com> Co-authored-by: Josef Zweck <josef@zweck.dev> Co-authored-by: Álvaro Fernández Rojas <noltari@gmail.com> Co-authored-by: Maciej Bieniek <bieniu@users.noreply.github.com> Co-authored-by: Manu <4445816+tr4nt0r@users.noreply.github.com> Co-authored-by: Artur Pragacz <49985303+arturpragacz@users.noreply.github.com> Co-authored-by: Jan Bouwhuis <jbouwh@users.noreply.github.com> Co-authored-by: Arie Catsman <120491684+catsmanac@users.noreply.github.com> Co-authored-by: Norbert Rittel <norbert@rittel.de> Co-authored-by: Matthias Alphart <farmio@alphart.net> Co-authored-by: Tom <CoMPaTech@users.noreply.github.com> Co-authored-by: Steven B. <51370195+sdb9696@users.noreply.github.com> Co-authored-by: Glenn Vandeuren (aka Iondependent) <vandeurenglenn@gmail.com> Co-authored-by: Austin Mroczek <austin@mroczek.org> Co-authored-by: Mick Vleeshouwer <mick@imick.nl> Co-authored-by: PierreAronnax <pierre@trionax.com> Co-authored-by: Dave T <17680170+davet2001@users.noreply.github.com> Co-authored-by: jesperraemaekers <146726232+jesperraemaekers@users.noreply.github.com> Co-authored-by: Steven Looman <steven.looman@gmail.com> Co-authored-by: Barry vd. Heuvel <barry@fruitcake.nl> Co-authored-by: Raphael Hehl <7577984+RaHehl@users.noreply.github.com> Co-authored-by: Andre Lengwenus <alengwenus@gmail.com> Co-authored-by: dontinelli <73341522+dontinelli@users.noreply.github.com> Co-authored-by: Noah Husby <32528627+noahhusby@users.noreply.github.com> Co-authored-by: Lucas Gasenzer <lucasgasenzer@mac.com> Co-authored-by: jb101010-2 <168106462+jb101010-2@users.noreply.github.com> Co-authored-by: Christopher Fenner <9592452+CFenner@users.noreply.github.com> Co-authored-by: TheJulianJES <TheJulianJES@users.noreply.github.com> Co-authored-by: Martin Weinelt <mweinelt@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: jon6fingrs <53415122+jon6fingrs@users.noreply.github.com> Co-authored-by: mrtlhfr <10065880+mrtlhfr@users.noreply.github.com> Co-authored-by: Matrix <justin@yosmart.com> Co-authored-by: Duco Sebel <74970928+DCSBL@users.noreply.github.com> Co-authored-by: Marcel van der Veldt <m.vanderveldt@outlook.com> Co-authored-by: epenet <6771947+epenet@users.noreply.github.com> Co-authored-by: Omni Flux <omni.hyper.flux@gmail.com> Co-authored-by: starkillerOG <starkiller.og@gmail.com> Co-authored-by: Bram Kragten <mail@bramkragten.nl> Co-authored-by: Thomas55555 <59625598+Thomas55555@users.noreply.github.com> Co-authored-by: karwosts <32912880+karwosts@users.noreply.github.com> Co-authored-by: Jordi <Jordi1990@users.noreply.github.com> Co-authored-by: Martin Mrazik <mmrazik@users.noreply.github.com> Co-authored-by: Brett Adams <Bre77@users.noreply.github.com> Co-authored-by: G-Two <7310260+G-Two@users.noreply.github.com> Co-authored-by: Claudio Ruggeri - CR-Tech <41435902+crug80@users.noreply.github.com> Co-authored-by: Khole <29937485+KJonline@users.noreply.github.com> Co-authored-by: Philipp Danner <philipp@danner-web.de> Co-authored-by: Marc Mueller <30130371+cdce8p@users.noreply.github.com> Co-authored-by: Cyrill Raccaud <miaucl@users.noreply.github.com> Co-authored-by: Allen Porter <allen@thebends.org> Co-authored-by: Aaron Bach <bachya1208@gmail.com> Co-authored-by: Michael Hansen <mike@rhasspy.org> Co-authored-by: Paulus Schoutsen <balloob@gmail.com> Co-authored-by: Paul Daumlechner <paul.daumlechner@live.de> Co-authored-by: Adam Goode <agoode@google.com> Co-authored-by: Alberto Geniola <albertogeniola@users.noreply.github.com> Co-authored-by: tronikos <tronikos@users.noreply.github.com> Co-authored-by: Arne Keller <arne.keller@posteo.de> Co-authored-by: Andrew Jackson <andrew@codechimp.org> Co-authored-by: Brynley McDonald <brynley+github@zephire.nz> Co-authored-by: Simone Chemelli <simone.chemelli@gmail.com> Co-authored-by: Niels Mündler <niels.muendler@inf.ethz.ch> Co-authored-by: Craig Andrews <candrews@integralblue.com> Co-authored-by: Martin Hjelmare <marhje52@gmail.com> Co-authored-by: Matthew FitzGerald-Chamberlain <mattfitzgeraldchamberlain@proton.me> Co-authored-by: Adam Štrauch <cx@initd.cz> Co-authored-by: cdnninja <jaydenaphillips@gmail.com> Co-authored-by: Stefan Agner <stefan@agner.ch> Co-authored-by: Kenny Root <kenny@the-b.org> Co-authored-by: Krzysztof Dąbrowski <krzysdabro@live.com> Co-authored-by: Andrea Arcangeli <aagit@users.noreply.github.com> Co-authored-by: Robert Svensson <Kane610@users.noreply.github.com> Fix section translations check (#133683) Fix test coverage in workday (#133616) Fix spelling of "Gateway PIN" and remove two excessive spaces (#133716) Fix Peblar current limit user setting value (#133753) Fix binary_sensor typing in Overkiz (#133782) Fix errors in HitachiDHW in Overkiz (#133765) Fix typo in ElevenLabs (#133819) fixture from LCN tests (#133821) fix yesterday sensor extra_state invalid typing (#133425) Fix TypeError in maxcube climate action inference logic (#133853) Fix tplink camera entity unique id (#133880) Fix a history stats bug when window and tracked state change simultaneously (#133770) fixes #133904 Fix duplicate call to async_register_preload_platform (#133909) Fix missing % in string for generic camera (#133925) Fix Peblar import in data coordinator (#133926) Fix reload modbus component issue (#133820) Fix error when device goes offline (#133848) fix "Slow" response leads to "Could not find a charging station" #124129 (#133889) fix #124129 Fix swiss public transport line field none (#133964) fix #133116 Fix Nord Pool empty response (#134033) Fix KNX config flow translations and add data descriptions (#134078) Fix Wake on LAN Port input as Box instead of Slider (#134216) Fix duplicate sensor disk entities in Systemmonitor (#134139) Fix Onkyo volume rounding (#134157) Fix 400 This voice does not support speaking rate or pitch parameters at this time for Google Cloud Journey voices (#134255) Fix SQL sensor name (#134414) Fix a few small typos in peblar (#134481) Fix input_datetime.set_datetime not accepting 0 timestamp value (#134489) Fix backup dir not existing (#134506) Fix activating backup retention config on startup (#134523) fix generic component tests (#134569)

bdraco added this to the 2025.1.0 milestone Dec 23, 2024

bdraco added the integration: backup label Dec 23, 2024

bdraco added a commit that referenced this issue Dec 23, 2024

Ensure cloud and recorder backup platforms do not have to wait for th…

485cf91

…e import executor partially fixes #133904

This was referenced Dec 23, 2024

Ensure cloud and recorder backup platforms do not have to wait for the import executor #133907

Merged

Fix duplicate call to async_register_preload_platform #133909

Merged

bdraco mentioned this issue Dec 23, 2024

Ensure integration platform loader does not try to load same named platforms #133911

Closed

19 tasks

MartinHjelmare closed this as completed in #133907 Dec 23, 2024

MartinHjelmare closed this as completed in 3f441e7 Dec 23, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

New backup system delays startup as much as 2x #133904

New backup system delays startup as much as 2x #133904

bdraco commented Dec 23, 2024

home-assistant bot commented Dec 23, 2024

bdraco commented Dec 23, 2024

bdraco commented Dec 23, 2024

bdraco commented Dec 23, 2024

bdraco commented Dec 23, 2024

bdraco commented Dec 23, 2024

bdraco commented Dec 23, 2024

New backup system delays startup as much as 2x #133904

New backup system delays startup as much as 2x #133904

Comments

bdraco commented Dec 23, 2024

The problem

What version of Home Assistant Core has the issue?

What was the last working version of Home Assistant Core?

What type of installation are you running?

Integration causing the issue

Link to integration documentation on our website

Diagnostics information

Example YAML snippet

Anything in the logs that might be useful for us?

Additional information

home-assistant bot commented Dec 23, 2024

bdraco commented Dec 23, 2024

bdraco commented Dec 23, 2024

bdraco commented Dec 23, 2024

bdraco commented Dec 23, 2024

bdraco commented Dec 23, 2024

bdraco commented Dec 23, 2024