Repository navigation
deploy: Unraid host networking + writable store volume for discovery; release-driven redeploy docs - #3
Merged
Conversation
…ease-driven redeploy The discovery feature (clock + _ember._tcp) needs multicast, which doesn't cross the default Docker bridge. Switch the Unraid template to Network: host, note the WebUI port is then informational + clock URL becomes optional (auto-discovered), and update the RUNBOOK: recreate the container with --network host (drop -p), and spell out that a redeploy is release-driven (no latest-from-main auto-build).
…Port config, expose EMBER_MDNS_ADVERTISE Self-review of the host-networking change: - Add a rw '/var/lib/ember' volume — the SQLite store (Pomodoro stats, Weather/ Reminders config, and the new Device-tab clock override) lives there and was NOT persisted by the template, so it reset on every update. - Drop the Type=Port WebUI config (meaningless under host networking; could inject a -p that conflicts) and hardcode the WebUI link to :8080. - Expose EMBER_MDNS_ADVERTISE as an advanced var; tidy the install steps.
tarakanof
changed the title
This was referenced Oct 4, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #1, hardening the Unraid deploy for the discovery feature + correcting the deploy docs. Reviewed (self-review pass included below).
deploy/unraid/ember.xmlNetwork: bridge→host— clock + server mDNS discovery (and_ember._tcp) need multicast, which doesn't cross the bridge./var/lib/embervolume (the SQLitedb_path). The template never persisted it, so Pomodoro stats, Weather/Reminders config, and the new Device-tab clock override reset on every container update. (Found during review — pre-existing gap, made impactful by the new feature.)Type="Port"WebUI config (meaningless under host networking; a stray-pconflicts with--network host) and hardcoded the WebUI link to:8080.EMBER_MDNS_ADVERTISEas an advanced var;awtrix.http_base_urlis now noted optional (auto-discovered); install steps note host networking + the free-:8080requirement.docs/RUNBOOK.mdvX.Y.ZRelease → workflow builds + pushes to Docker Hub → repoint the container; nolatest-from-mainauto-build).--network host(drops-p 8080:8080), with an Unraid note; dev section notes the-pform is fine except for discovery; refreshed the stalev0.1.xline.To ship the discovery feature (#1)
gh release create v0.3.0 --target main→ workflow publishesdtarakanov/ember:0.3.0+:latest→ on Unraid, the template now defaults to host networking + a persistent store; recreate the container (switching from bridge → host requires remove + re-add).XML validated (
xmllint). No code change.