Skip to content

fix(updater): fail health check fast, signal busy during boot provisioning, keep overlay through UI restart - #340

Open
gmmcosta15 wants to merge 9 commits into
devfrom
fix/updater-health-failfast-restart-flash
Open

gmmcosta15 wants to merge 9 commits into
devfrom
fix/updater-health-failfast-restart-flash

Conversation

@gmmcosta15

@gmmcosta15 gmmcosta15 commented Sep 29, 2026 •

Copy link
Copy Markdown
Collaborator

Description

  • Feature
  • Bug fix
  • Code refactor
  • Documentation

Fixes the double Spoolman start, the loading-screen flash on update, and the boot-time Spoolman install.

  • Fail-fast Spoolman health check, no double start, overlay held until fresh status.
  • Skip the background apt upgrade when a daemon restart is pending, so the restart can't SIGKILL dpkg.
  • install-updater.sh enables BlocksScreen.service, so the UI starts at boot.
  • Boot provisioning: busy is set before the D-Bus export, so the overlay shows before MainWindow, and provisioning retries every 3 s (up to 10 times) while boot reconcile holds the lock.
  • The overlay reads "Missing component, installing ..." then "Installing : ".
  • replay_busy() re-emits a busy=True that fired before MainWindow wired its slots.

Tests

  • Unit tests added for each change.
  • On an RF50, Release-2.0 -> this branch:
    • Spoolman installs and health returns 200.
    • The UI recovers.
    • Boot with Spoolman removed shows the overlay first, installs it, and comes up healthy.
    • The apt skip was seen in the journal.

@gmmcosta15 gmmcosta15 added the enhancement New feature or request. label Sep 29, 2026
@gmmcosta15 gmmcosta15 self-assigned this Sep 29, 2026
@gmmcosta15
gmmcosta15 requested a review from a team September 29, 2026 14:35
@gmmcosta15
gmmcosta15 marked this pull request as draft September 29, 2026 15:51
@gmmcosta15
gmmcosta15 marked this pull request as ready for review September 29, 2026 16:48

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant