Skip to content

Hack/grafana mcp #6696

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

Merged
merged 73 commits into from
Jun 13, 2025
Merged

Hack/grafana mcp #6696

merged 73 commits into from
Jun 13, 2025

Conversation

wdvr
Copy link
Contributor

@wdvr wdvr commented Jun 3, 2025

Adding the first version of /mcp_query, the Grafana dashboard creator. Linked to https://github.com/pytorch-labs/pytorch-gha-infra/pull/693 for the backend

Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

lintrunner found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@wdvr wdvr marked this pull request as ready for review June 12, 2025 23:17
Copy link
Contributor

@izaitsevfb izaitsevfb left a comment

Choose a reason for hiding this comment

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

stamping to unblock, but as per our discussion, we still need to:

  • extract components
  • fix any types
  • double-check (and probably simplify) the parsing
  • remove dead code

but since this is an isolated change, we can merge it now and improve later (#6758)

@wdvr
Copy link
Contributor Author

wdvr commented Jun 13, 2025

(#6758) is not merged into this PR, so landing this one.

@wdvr wdvr merged commit 4c04d68 into main Jun 13, 2025
6 checks passed
@wdvr wdvr deleted the hack/grafana_mcp branch June 13, 2025 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.