feat: add flower-docker extension for Celery monitoring - #178
Conversation
…p#117 - Link docs/AUTHORING.es.md from README.md Documentation table - Link Spanish doc from CONTRIBUTING.md with canonical English note - Add CI L0-L3 brief mention to docs/AUTHORING.es.md intro Addresses acceptance criteria for issue Create-Python-App#117: - [x] Spanish doc exists and is linked - [x] English remains canonical - [x] Mentions CI L0-L3 briefly
…ate-Python-App#112) Defines the standard span kinds and attribute schema AI extensions must use when emitting LLM/tool/retrieval/guardrail spans via the primitives from fastapi-mlflow-tracing (Create-Python-App#81). - Documents 4 span kinds: llm_inference, tool_call, retrieval, guardrail_check - Specifies required attributes for each kind - Defines span shape rules (linear span tree for guardrail rejections) - Documents privacy policy (LLM_TRACE_PAYLOAD opt-in, off in CI) - Links all related issues (Create-Python-App#81, Create-Python-App#77, Create-Python-App#78, Create-Python-App#79, Create-Python-App#80, Create-Python-App#82, Create-Python-App#91) - Includes illustrative Python example using maybe_start_span
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…-118 feat: add architecture diagram for docs/ARCHITECTURE.md (issue Create-Python-App#118)
|
Flower monitoring is a welcome addition, and the extension layout itself looks right. But the branch mixes in unrelated changes: Spanish authoring-doc edits (overlapping #162), Please open a fresh feature branch off current upstream |
…patibleWith (#185) - extensions/fastapi-auth-jwt: add tests/test_auth.py (16 tests) covering password hashing (salted Argon2), JWT create/decode round-trip, extra claims, algorithm, secret rotation, expiry, Pydantic schemas, and router wire-up via TestClient ($auth endpoints). - extensions/fastapi-sqlalchemy: add tests/test_sqlalchemy.py (9 tests) covering DeclarativeBase registration, engine/session factory smoke, get_db generator contract, isolated in-memory round-trip, Alembic env wiring via static content check, DATABASE_URL override, and sqlite connect_args style guard. - docs/AUTHORING.md: document upcoming celery-docker vs flower-docker incompatibleWith prep for PR #178 (symmetric validation via scripts/ci/validate-registry.py / templates.schema.json). - extensions/all-postgres: fix .env.example.append style consistency (remove duplicate header and leading blank line). Co-authored-by: cpa-templates W3 task Co-authored-by: ulises-jeremias <ulises.jeremias@example.com>
|
Thanks @emberb170d for the Flower extension! A couple of items need work before this can be merged: 1. Missing
Current diff has no 2. Branch hygiene — head is fork's
3. Housekeeping:
Once |
Adds Flower dashboard for Celery workers (port 5555) as a Docker Compose overlay. Mutual incompatibility with celery-docker declared symmetrically (both ship Dockerfile/compose.yml for celery-worker and would overwrite the same paths). - extensions/flower-docker: compose.yml (+ healthchecks, flower service), compose.prod.yml, Dockerfile, .dockerignore (valid patterns), pyproject.toml (flower>=2.0.1), .env.example (FLOWER_BASIC_AUTH, FLOWER_PORT), docs/FLOWER_GUIDE.md, docs/README.md.append - templates.json: add flower-docker (observability, type celery-worker, labels Flower/Celery/Monitoring) with incompatibleWith celery-docker; add symmetric incompatibleWith to celery-docker - scripts/ci/validate-registry.py: allow flower-docker for celery-worker (canonical tool name, otherwise would require celery- prefix) Closes #128 Refs #178 (replaces flawed PR: fork main, stale .editorconfig, markdown .dockerignore) Co-authored-by: ulises-jeremias <ulises.jeremias@example.com>
|
Superseded by #188 which adds flower-docker with proper incompatibleWith symmetric to celery-docker, valid .dockerignore, and full CI green. Thanks @emberb170d for the initial contribution! Closing #128 as well via #188. |
Flower Monitoring Extension for Celery
Adds Flower monitoring dashboard for Celery workers.
Problem
users often want Flower for local/dev monitoring.
Solution
Extension compatible with celery worker type: docker-compose service or dependency + docs. Keeps secrets in env examples only. L2 against .
What's included
Files added
Usage
Then access the dashboard at
http://localhost:5555.Fixes #128