chore: refresh docker-compose.yaml template to align with prod (#71)#90
Merged
Conversation
The in-repo template predated Redis and the tickets endpoint, declared deprecated `version: '3'`, and bundled a database + nginx that the live api stack doesn't carry. Refresh aligned with the live `7cav-api` stack: - drop deprecated `version: '3'` - add `redis` service (caching has been required since 2.1.0; the template never had it) - drop the `database` service — the DB is sourced from the `xenforo` stack via the `xenforo_internal` external network, not bundled in the api compose - drop the local-TLS-test `nginx` service — TLS is terminated at a different stack; `make certs` is independent and unchanged for devs who want local TLS - networks: switch to the live `edge` + `xenforo_internal` external pair - env vars: add full set — `REDIS_HOST` / `REDIS_PORT` / `REDIS_PASSWORD`, plus commented `FORUM_BASE_URL` and `REFERENCE_CACHE_REFRESH_INTERVAL` for the tickets endpoint - live patterns: rename api container to `7cav-api`, add the watchtower-enable label, `restart: always` - `DB_HOST` defaults to `xenforo-db` (the cross-stack hostname) - header comment documents the prerequisites Makes Dependabot #85 (mariadb 11.4 → 12.2.2) moot — the image reference is gone. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
SyniRon
force-pushed
the
chore/mariadb-12-eval
branch
from
b14b1ae to
0cb42f5
Compare
SyniRon
changed the title
This was referenced May 22, 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.
Summary
Closes #71. Single-commit refresh of the in-repo
docker-compose.yamltemplate aligned with the live7cav-apistack (which the user shared sans secrets).version: '3'.databaseservice — the DB is sourced from thexenforostack via thexenforo_internalexternal network, not bundled in the api compose. This makes Dependabot Bump mariadb from 11.4 to 12.2.2 #85 (mariadb 11.4 → 12.2.2) moot — the image reference is gone.nginxservice — TLS terminates at a different stack.make certsremains independent and unchanged.redisservice (caching has been required since 2.1.0; the template never had it).edge+xenforo_internalexternal pair.REDIS_HOST/REDIS_PORT/REDIS_PASSWORD, plus commented-outFORUM_BASE_URLandREFERENCE_CACHE_REFRESH_INTERVALfor the tickets endpoint.7cav-api, add the watchtower-enable label,restart: always.DB_HOSTdefaults toxenforo-db(the cross-stack hostname).docker network create edge/xenforo_internal, xenforo stack running).Test plan
docker compose config --quietvalidates the refreshed YAMLManual smoke against
mariadb:12.2.2(the original intent of this branch) confirmed XenForo schema imports clean and every API path returns identical data — see prior history of this PR. With the DB service removed from the template, the bump itself is no longer carried here.Closes #71. Makes #85 moot — recommend closing #85 with a comment pointing here.
🤖 Generated with Claude Code