fix(hass): copy the Honeywell package into /config/packages, and check all three places #216

Merged

kblack0610 merged 1 commit from fix/honeywell-package-copy into master

2026-09-01 19:48:13 +00:00

kblack0610 commented

2026-09-01 19:47:54 +00:00

Owner

#215 fixed one of two independent omissions, so the bug survived it - live HA still reported zero honeywell entities after it rolled.

Getting one package file to Home Assistant takes three agreeing edits:

#	File	Miss it and...
1	`config/packages/<name>.yaml`	nothing exists
2	`kustomization.yaml` configMapGenerator entry	not in the ConfigMap (fixed in #215)
3	`deployment.yaml` init-container `cp` line	in the ConfigMap, never lands in `/config/packages` (this PR)

b41a897 was missing 2 and 3. Each alone is sufficient to break it, which is why fixing only 2 changed nothing observable.

check-ha-packages.py now asserts all three agree instead of two. Negative control on each failure mode separately:

both present:        ok: all 17 ...                         exit 0
cp line removed:     FAIL ... deployment.yaml needs:  cp ... exit 1
kustomization line:  FAIL ... kustomization.yaml needs: ...  exit 1
restored:            ok: all 17 ...                         exit 0

Will confirm the entities actually appear in live HA after this rolls, rather than calling it done at merge.

#215 fixed one of **two independent** omissions, so the bug survived it - live HA still reported zero honeywell entities after it rolled. Getting one package file to Home Assistant takes three agreeing edits: | # | File | Miss it and... | |---|---|---| | 1 | `config/packages/<name>.yaml` | nothing exists | | 2 | `kustomization.yaml` configMapGenerator entry | not in the ConfigMap (fixed in #215) | | 3 | `deployment.yaml` init-container `cp` line | in the ConfigMap, never lands in `/config/packages` (this PR) | b41a897 was missing 2 **and** 3. Each alone is sufficient to break it, which is why fixing only 2 changed nothing observable. `check-ha-packages.py` now asserts all three agree instead of two. Negative control on each failure mode separately: ``` both present: ok: all 17 ... exit 0 cp line removed: FAIL ... deployment.yaml needs: cp ... exit 1 kustomization line: FAIL ... kustomization.yaml needs: ... exit 1 restored: ok: all 17 ... exit 0 ``` Will confirm the entities actually appear in live HA after this rolls, rather than calling it done at merge.

kblack0610 added 1 commit

2026-09-01 19:47:54 +00:00

fix(hass): copy the Honeywell package into /config/packages, and check all three places

checks / fleet-roster (pull_request) Successful in 6s

Details

checks / kustomize-builds (pull_request) Successful in 6s

Details

a22928dafa

#215 fixed one of two independent omissions. Getting a package to HA needs three
agreeing edits - the file, a configMapGenerator entry, and an explicit cp line in
the init container - and b41a897 was missing both wiring edits, so registering it
in the ConfigMap alone still left /config/packages without it. Live HA still
reported zero honeywell entities after #215 rolled.

check-ha-packages.py now asserts all three agree rather than two. Both failure
modes verified independently before trusting it: dropping either line alone
reports 1 of 17 incomplete, exit 1.