Skip to content

chore: removed curl call in release-deploy script - #228

Merged
HardMax71 merged 1 commit into
mainfrom
fix/grafana-issues
Feb 24, 2026
Merged

chore: removed curl call in release-deploy script#228
HardMax71 merged 1 commit into
mainfrom
fix/grafana-issues

Conversation

@HardMax71

@HardMax71 HardMax71 commented Feb 24, 2026

Copy link
Copy Markdown
Owner

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

  • Chores
    • Simplified deployment pipeline by removing a provisioning reload step.

Copilot AI review requested due to automatic review settings February 24, 2026 11:27
@HardMax71
HardMax71 merged commit 66345ec into main Feb 24, 2026
7 of 8 checks passed
@HardMax71
HardMax71 deleted the fix/grafana-issues branch February 24, 2026 11:27
@coderabbitai

coderabbitai Bot commented Feb 24, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8a35101 and f6b592a.

📒 Files selected for processing (1)
  • .github/workflows/release-deploy.yml

📝 Walkthrough

Walkthrough

Removed the HTTP POST curl request to reload Grafana dashboards provisioning from the release-deploy workflow. The curl call to http://localhost:3000/api/admin/provisioning/dashboards/reload that executed after docker compose startup is deleted. Subsequent health check and image pruning steps remain intact.

Changes

Cohort / File(s) Summary
Workflow Configuration
.github/workflows/release-deploy.yml
Removed Grafana dashboard provisioning reload curl POST request (4 lines) that was invoked after docker compose up. Health check and image pruning steps unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • Fix/grafana rights2 #227: Directly related modification to the same Grafana provisioning reload curl in the release-deploy workflow.

Poem

🐰 The rabbit hops through workflows bright,
One curl request removed from sight,
No Grafana reload today,
Simpler deploys lead the way!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/grafana-issues

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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/reload endpoint from the deployment script

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

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