Skip to content

ADFA-5229 fix(rootfs): matomo-only tier actually enables matomo - #465

Merged
luisguzman-adfa merged 1 commit into
mainfrom
fix/ADFA-5229-matomo-tier-enable
Aug 22, 2026
Merged

ADFA-5229 fix(rootfs): matomo-only tier actually enables matomo#465
luisguzman-adfa merged 1 commit into
mainfrom
fix/ADFA-5229-matomo-tier-enable

Conversation

@luisguzman-adfa

Copy link
Copy Markdown
Collaborator

The build-rootfs "matomo" tier passes AUTO_INSTALL_SIZE=4, but the installer never honored 4: it fell through to the Medium template and still force-enabled code, and nothing set matomo — so matomo stayed False and the role was SKIPPED (the build-matomo-*.log had zero TASK [matomo : ...]). Now size 4 uses the Small base, sets MATOMO_ONLY, turns the content roles (kiwix/maps/osm_vector_maps) OFF on all arches, does NOT force code, and turns matomo_install/matomo_enabled ON so the role runs (MariaDB + PHP come on demand as its deps). Unblocks capturing matomo.txt for the determinate install bar. bash -n clean; build-rootfs already maps 4->matomo.

The build-rootfs "matomo" tier passes AUTO_INSTALL_SIZE=4, but the installer never
honored 4: it fell through to the Medium template and still force-enabled `code`,
and nothing set matomo — so matomo stayed False and the role was SKIPPED (the
build-matomo-*.log had zero `TASK [matomo : ...]`). Now size 4 uses the Small base,
sets MATOMO_ONLY, turns the content roles (kiwix/maps/osm_vector_maps) OFF on all
arches, does NOT force code, and turns matomo_install/matomo_enabled ON so the role
runs (MariaDB + PHP come on demand as its deps). Unblocks capturing matomo.txt for
the determinate install bar. bash -n clean; build-rootfs already maps 4->matomo.
@luisguzman-adfa
luisguzman-adfa merged commit ddf866b into main Aug 22, 2026
3 checks passed
@luisguzman-adfa
luisguzman-adfa deleted the fix/ADFA-5229-matomo-tier-enable branch August 22, 2026 02:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant