Skip to content

Fix CI pytest failure, add .dockerignore, complete render.yaml - #4

Merged
CoreyLeath-code merged 2 commits into
mainfrom
copilot/add-deployment-ready-structure
Apr 11, 2026
Merged

Fix CI pytest failure, add .dockerignore, complete render.yaml#4
CoreyLeath-code merged 2 commits into
mainfrom
copilot/add-deployment-ready-structure

Conversation

Copilot AI commented Apr 11, 2026

Copy link
Copy Markdown
Contributor

CI was broken due to pytest: command not found (exit code 127) — pip install places the binary in ~/.local/bin which isn't guaranteed on $PATH in GitHub runners. Render deployment config was a stub. No .dockerignore existed, meaning Docker builds included .git, secrets templates, Terraform state, and other noise.

Changes

CI (ci.yml)

  • pytest tests/ -vpython -m pytest tests/ -v — bypasses PATH resolution entirely

.dockerignore (new)

Excludes from all image builds:

  • .git/, .github/, *.md, LICENSE
  • .env, .env.* (secrets — !.env.example kept)
  • Python/Go/C++ build artifacts and caches
  • terraform/, helm/, k8s/, monitoring/ (infra-only, mounted as volumes in compose)
  • OS/IDE files

render.yaml

Expanded from a 5-line stub to define all 4 Docker services with correct dockerfilePath, dockerContext, healthCheckPath, and env vars. Sensitive values (DATABASE_URL, OLLAMA_HOST) marked sync: false for secure injection via the Render dashboard.

Service Image source Health path
sentinel-dashboard streamlit-dashboard/Dockerfile /_stcore/health
sentinel-ingestion ingestion-service/Dockerfile /health
sentinel-drift-engine drift-engine/Dockerfile /health
sentinel-llm-guard llm-guard/Dockerfile /health

Copilot AI changed the title [WIP] Add deployment-ready structure for the application Fix CI pytest failure, add .dockerignore, complete render.yaml Apr 11, 2026
Copilot AI requested a review from CoreyLeath-code April 11, 2026 17:56
@CoreyLeath-code
CoreyLeath-code marked this pull request as ready for review April 11, 2026 17:56
@CoreyLeath-code
CoreyLeath-code merged commit 9402e4e into main Apr 11, 2026
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.

2 participants