Skip to content

fix(smoke): update e2e smoke defaults#248

Open
bihius wants to merge 1 commit into
mainfrom
fix/e2e-smoke
Open

fix(smoke): update e2e smoke defaults#248
bihius wants to merge 1 commit into
mainfrom
fix/e2e-smoke

Conversation

@bihius

@bihius bihius commented Jun 14, 2026

Copy link
Copy Markdown
Owner

Summary

  • isolate the smoke Compose project and HAProxy host port by default
  • make the smoke Host header configurable
  • check CRS submodule content before starting Compose
  • use /docs for the benign WAF-inspected request and wait for postgres health

Validation

  • bash -n benchmarks/smoke/e2e.sh
  • uv run --extra dev ruff check app/
  • uv run --extra dev mypy app/
  • uv run --extra dev pytest --cov=app
  • pnpm run type-check
  • pnpm run lint

Full Docker smoke was not run locally because this worktree lacks deploy/docker/.env, CRS submodule content, and Docker daemon access.

@bihius bihius marked this pull request as ready for review June 14, 2026 16:47
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