chore: removed curl call in release-deploy script - #228
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review infoConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughRemoved the HTTP POST curl request to reload Grafana dashboards provisioning from the release-deploy workflow. The curl call to Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
|
There was a problem hiding this comment.
Pull request overview
This PR removes an unnecessary manual Grafana dashboard provisioning reload call from the release-deploy workflow. The curl command that triggered a reload via the Grafana Admin API is no longer needed because Grafana is configured with automatic file-based provisioning that checks for changes every 10 seconds (configured in backend/grafana/provisioning/dashboards/dashboard.yml).
Changes:
- Removed redundant curl POST call to Grafana's
/api/admin/provisioning/dashboards/reloadendpoint from the deployment script
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.



Summary by cubic
Removed the Grafana dashboards reload curl from the release-deploy workflow. This simplifies the deploy step and avoids failures when the Grafana admin API is unavailable.
Written for commit f6b592a. Summary will update on new commits.
Summary by CodeRabbit