Skip to content

Conversation

@javidahmed64592
Copy link
Owner

This pull request removes built-in support for Prometheus and Grafana monitoring from the Python Template Server project. The Docker configuration, documentation, and codebase have been updated to eliminate references to Prometheus and Grafana, simplifying the stack to focus on the FastAPI server. The most important changes are grouped below.

Docker and Deployment Configuration:

  • Removed Prometheus and Grafana services, volumes, and related configuration from docker-compose.yml. The stack now only includes the FastAPI server and essential volumes. [1] [2] [3]
  • Updated the Dockerfile to stop copying grafana/ and prometheus/ directories, and removed setup steps for monitoring configs. [1] [2] [3]
  • Cleaned up the CI workflow in .github/workflows/build.yml by removing checks and directory listings for Prometheus and Grafana. [1] [2]
  • Removed health checks for Prometheus and Grafana from the custom GitHub Action.

Documentation Updates:

  • Revised README.md, docs/API.md, and docs/DOCKER_DEPLOYMENT.md to remove all references to Prometheus metrics, Grafana dashboards, and related endpoints. Documentation now focuses solely on the FastAPI server and its features. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]

Internal Project Structure:

  • Updated project guides and instructions to remove setup and references for metrics middleware, Prometheus handlers, and Grafana dashboards. [1] [2] [3] [4] [5]

These changes streamline the project by removing external observability dependencies, making the template easier to deploy and maintain.

…ication, health, and rate limiting metrics; update project description and dependencies in pyproject.toml; delete Prometheus configuration files; clean up lock file by removing unused packages.
@javidahmed64592 javidahmed64592 merged commit c9356e4 into main Dec 30, 2025
10 checks passed
@javidahmed64592 javidahmed64592 deleted the remove-metrics-dashboards branch December 30, 2025 16:36
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