Skip to content

fix: Update apisix-grafana-dashboard.json #12100

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

alibabakhanlu12
Copy link

@alibabakhanlu12 alibabakhanlu12 commented Mar 30, 2025

Grafana has officially deprecated and removed support for Angular-based panels from Grafana 8.x onward. All new and existing panels now require React as the base framework.

Relevant Grafana documentation:
https://grafana.com/developers/plugin-tools/migration-guides/angular-react/
https://grafana.com/docs/grafana/latest/developers/angular_deprecation/

Description

This PR updates the existing Apache APISIX Grafana dashboard JSON to replace deprecated Angular-based panels with their React-based equivalents:

Singlestat → Stat
Graph → Time Series

Replaced "type": "singlestat" with "type": "stat".

Replaced "type": "graph" with "type": "timeseries".

Which issue(s) this PR fixes:

The current Apache APISIX Grafana dashboard uses panels that no longer function in Grafana v8.x and later due to Grafana’s removal of AngularJS support. Without migrating these panels, users encounter errors:

"This panel requires Angular support in Grafana."

This update resolves the issue by ensuring compatibility with current and future Grafana releases.

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

grafana decrepted all panels that made using angular panels and added react as base framework for new panels. 
due to that reason i edited this json file and updated new things for supporting react as base library
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. enhancement New feature or request labels Mar 30, 2025
@alibabakhanlu12 alibabakhanlu12 changed the title Update apisix-grafana-dashboard.json fix: Update apisix-grafana-dashboard.json Mar 30, 2025
@Baoyuantop Baoyuantop moved this to 🏗 In progress in ⚡️ Apache APISIX Roadmap Apr 1, 2025
@Baoyuantop Baoyuantop moved this from 🏗 In progress to 👀 In review in ⚡️ Apache APISIX Roadmap Apr 1, 2025
@Baoyuantop Baoyuantop self-requested a review April 1, 2025 09:17
@juzhiyuan
Copy link
Member

Hi @Baoyuantop, any chance to review the changes?

@Baoyuantop
Copy link
Contributor

Hi @Baoyuantop, any chance to review the changes?

Yes, I'm on it.

Copy link
Contributor

@Baoyuantop Baoyuantop left a comment

Choose a reason for hiding this comment

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

Please format json with 2 spaces for a clearer view diff.

@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Apr 4, 2025
@alibabakhanlu12
Copy link
Author

alibabakhanlu12 commented Apr 4, 2025

I changed the file’s indentation to 2 spaces, and now the changes are visible
@Baoyuantop

@juzhiyuan
Copy link
Member

@Baoyuantop good to merge?

@Baoyuantop Baoyuantop requested review from juzhiyuan and bzp2010 April 11, 2025 00:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request size:M This PR changes 30-99 lines, ignoring generated files.
Projects
Status: 👀 In review
Development

Successfully merging this pull request may close these issues.

3 participants