feat: add InnoDB conversion, composite indexes, and optional Redis accounting (#5)

* feat: add InnoDB conversion, composite indexes, and optional Redis accounting

- Convert MyISAM tables (radcheck, radreply, radusergroup, radgroupcheck,
  radgroupreply) to InnoDB for transaction support and crash recovery
- Add composite indexes for freeradius-api query patterns
- Add optional Redis accounting: buffer Interim-Update packets in Redis
  for batch processing (ACCT_REDIS_ENABLED=true, default: off)
- Start/Stop packets always go to SQL for durability
- Add freeradius-redis package to Docker image
- Add Redis service to Docker Compose, Kubernetes, and Helm chart
- Falls back to SQL-only if Redis is unreachable at startup

* fix: update CI test for Redis service (3 healthy services, add Redis env vars)

Author: sumitroajiprabowo

.github/workflows/ci.yml

@@ -194,6 +194,11 @@ jobs:
           HEALTHCHECK_SECRET=testing123
           TZ=UTC
           RADIUS_ALLOW_PRIVATE_NETWORKS=true
+          ACCT_REDIS_ENABLED=false
+          REDIS_HOST=redis
+          REDIS_PORT=6379
+          REDIS_PASSWORD=
+          REDIS_DB=0
           EOF
           # Strip leading whitespace from .env
           sed -i 's/^[[:space:]]*//' .env
@@ -209,7 +214,7 @@ jobs:
             HEALTHY=$(docker compose ps --format json | grep -c '"healthy"' || true)
             TOTAL=$(docker compose ps --format json | wc -l | tr -d ' ')
             echo "  Healthy: $HEALTHY / $TOTAL ($i/90)"
-            if [ "$HEALTHY" -ge 2 ] 2>/dev/null; then
+            if [ "$HEALTHY" -ge 3 ] 2>/dev/null; then
               echo "All services are healthy!"
               break
             fi

.gitignore

@@ -10,4 +10,7 @@ backups/
 
 # Database dumps (prevent accidental commit of sensitive data)
 **/*.sql.gz
-**/*.sql
+**/*.sql
+
+# Allow SQL migration scripts
+!scripts/*.sql

CHANGELOG.md

@@ -7,8 +7,21 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Added
+
+- Post-schema migration system: automatically applies InnoDB conversion and composite indexes after FreeRADIUS default schema import
+- Optional Redis accounting: buffer Interim-Update packets in Redis for batch processing (`ACCT_REDIS_ENABLED=true`)
+- Redis service in Docker Compose (always available, accounting opt-in via env var)
+- Redis Deployment and Service for Kubernetes
+- Redis StatefulSet for Helm chart with persistence and external Redis support
+- New environment variables: `ACCT_REDIS_ENABLED`, `REDIS_HOST`, `REDIS_PORT`, `REDIS_PASSWORD`, `REDIS_DB`
+- `freeradius-redis` package in Docker image for `rlm_redis` module
+
 ### Changed
 
+- Tables `radcheck`, `radreply`, `radusergroup`, `radgroupcheck`, `radgroupreply` converted from MyISAM to InnoDB (enables transactions, row-level locking, crash recovery)
+- Added composite indexes: `radusergroup(username, groupname)`, `radgroupcheck(groupname, attribute)`, `radgroupreply(groupname, attribute)`, `radacct(username, acctstoptime)`, `radpostauth(authdate)`
+
 - Upgraded FreeRADIUS to 3.2.8 from Debian sid with apt pinning
 - Replaced all `:latest` image tags with semantic versioning (`3.2.8`)
 - Removed `:latest` tag from CI pipeline (only semantic versions pushed)

Dockerfile

@@ -32,6 +32,7 @@ RUN echo "deb http://deb.debian.org/debian sid main" > /etc/apt/sources.list.d/s
 RUN apt-get update && apt-get install -y --no-install-recommends \
     freeradius \
     freeradius-mysql \
+    freeradius-redis \
     freeradius-utils \
     mariadb-client \
     gosu \
@@ -55,6 +56,7 @@ RUN usermod -aG freerad root \
 
 # Copy entrypoint and set permissions
 COPY --link --chmod=755 scripts/entrypoint.sh /entrypoint.sh
+COPY --link scripts/post-schema.sql /entrypoint-post-schema.sql
 
 EXPOSE 1812/udp 1813/udp