Skip to content

feat: add Dockerfile and API smoke tests for Linux release#305

Merged
hanneshapke merged 3 commits intomainfrom
hanneshapke/dockerfile-api-test
Apr 6, 2026
Merged

feat: add Dockerfile and API smoke tests for Linux release#305
hanneshapke merged 3 commits intomainfrom
hanneshapke/dockerfile-api-test

Conversation

@hanneshapke
Copy link
Copy Markdown
Collaborator

Summary

  • Add a Dockerfile (Ubuntu 24.04, amd64) that downloads the latest Linux release tarball from GitHub Releases, verifies the SHA256 checksum, and installs it to /opt/kiji-proxy
  • Add docker-test-api.sh smoke test script that starts the proxy and validates /health, /version, /stats, DELETE /mappings, /api/model/info, POST /api/pii/check (with PII detection verification), and /api/pii/confidence endpoints
  • Fix ONNX runtime library resolution on Linux by adding .so search paths alongside the existing macOS .dylib paths in onnx_model_detector.go, with a platform-aware fallback using runtime.GOOS

Test plan

  • docker build -t kiji-proxy . succeeds
  • docker run --rm kiji-proxy /opt/kiji-proxy/docker-test-api.sh passes 10/10 API tests
  • docker run -p 8080:8080 kiji-proxy starts the server normally
  • Verify macOS builds are unaffected (.dylib paths unchanged)

🤖 Generated with Claude Code

hanneshapke and others added 3 commits April 3, 2026 15:42
Add a Dockerfile that downloads the latest Linux release from GitHub,
installs it on Ubuntu 24.04, and includes an API test script that
validates health, version, stats, model info, PII detection, and
confidence endpoints.

Also fix ONNX runtime library resolution on Linux by adding .so search
paths alongside the existing macOS .dylib paths.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 3, 2026

Warning

This PR touches 3+ distinct areas of the codebase.

Consider splitting into smaller, focused PRs — each covering a single semantic type.
This makes reviews easier and keeps the git history clean.

Categories found:

docs:

  • src/scripts/docker/README.md

code:

  • src/backend/pii/detectors/onnx_model_detector.go

chore:

  • src/scripts/docker/Dockerfile
  • src/scripts/docker/docker-test-api.sh

@hanneshapke hanneshapke merged commit 750d866 into main Apr 6, 2026
5 of 6 checks passed
@hanneshapke hanneshapke deleted the hanneshapke/dockerfile-api-test branch April 6, 2026 03:46
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