-
Notifications
You must be signed in to change notification settings - Fork 0
Ore/terraform gh action #6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This pull request expands the observability module by introducing comprehensive tracing capabilities and numerous technical metrics to complement the existing cost monitoring, logging, and dashboards. The changes focus on providing end-to-end observability through distributed tracing and a comprehensive set of sample metrics for testing and development purposes.
- Adds distributed tracing infrastructure with OpenTelemetry Collector and Grafana Tempo configurations
- Introduces extensive sample Prometheus metrics covering technical, business, and security use cases
- Includes tracing documentation, parsers, and visualization components
- Updates CI workflow naming to remove special character
Reviewed Changes
Copilot reviewed 40 out of 47 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tracing/ | Complete tracing module with collector configs, exporters, parsers, dashboards, and documentation |
| metrics/grafana-stack/technical/technical_metrics/ | Comprehensive set of sample Prometheus metrics for various technical monitoring scenarios |
| metrics/grafana-stack/business/business_metrics/ | Sample business metrics for CLV, conversion rates, user journey analytics |
| logging/grafana-stack/ | Enhanced logging configurations and documentation |
| .github/workflows/dashboards.yml | Minor workflow name update |
Comments suppressed due to low confidence (1)
bit-blocks-starter/observability/metrics/grafana-stack/prometheus.yml:70
- The job name 'fake-/market-perf-local' contains an invalid forward slash character. Job names should use valid characters only. Consider renaming to 'fake-market-perf-local'.
- job_name: 'fake-/market-perf-local'
🧾 Terraform Plan for
|
🧾 Terraform Plan for
|
🧾 Terraform Plan for
|
🧾 Terraform Plan for
|
🧾 Terraform Plan for
|
🧾 Terraform Plan for
|
🧾 Terraform Plan for
|
🧾 Terraform Plan for
|
🧾 Terraform Plan for
|
🧾 Terraform Plan for
|
🧾 Terraform Plan for
|
This pull request introduces a comprehensive observability module for the BitBlock Starter Pack, focusing on cost monitoring, logging, and dashboards. It includes new configurations, dashboards, and documentation to enhance cloud-native application monitoring and cost transparency. Below are the most important changes grouped by theme:
Observability Documentation and Structure
README.mdin theobservabilityfolder to outline the module's purpose, folder structure, quick start guide, use cases, and best practices.Cost Monitoring Enhancements
cloud-cost-alerts.yamlwith Prometheus alert rules for cloud cost anomalies, such as monthly budget exceedance and daily spend spikes.cloud-cost-billing.jsonGrafana dashboard for visualizing cloud cost trends, provider breakdowns, and top services by cost.k8s-kube-cost-alerts.yamlfor alerts on namespace overspending and cost anomalies.kube-cost.jsonGrafana dashboard for Kubernetes cost analytics, including namespace costs, resource efficiency metrics, and savings opportunities.Logging Observability
loki-config.yamlandpromtail-config.yamlfor setting up Loki and Promtail for log aggregation and parsing. [1] [2]sample-logback.xml(Java) andsample-nginx.conf(NGINX). [1] [2]grafana-logs-dashboard.jsonGrafana dashboard for visualizing logs, including application, NGINX, and error logs.grok-patterns.mdandjson-parse-pattern.mdfor common formats like NGINX and JSON. [1] [2]Minor Changes
.github/workflows/dashboards.ymlto remove a special character.