Skip to content
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

Add Jetstream dashboard and hidden integration #911

Merged
merged 2 commits into from
Feb 5, 2025
Merged

Conversation

yqlu
Copy link
Collaborator

@yqlu yqlu commented Feb 4, 2025

Developed at go/jetstream-dashboard-dev-notes. We'll validate this internally before setting the integration status from HIDDEN to GA.

@github-actions github-actions bot requested a review from EvanSimpson February 4, 2025 19:33
display_name: Jetstream Prometheus Overview
description: "This dashboard has charts displaying request rate, latency, queue sizes and other metrics for Jetstream inference."
related_integrations:
- id: jetstream
Copy link
Collaborator

Choose a reason for hiding this comment

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

FYI dashboard and integration will deploy separately

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ack. I don't think it's a big deal because:

  • the integration is being added as HIDDEN -- I'll wait for both to propagate fully, validate it internally and publish the corresponding docs before we mark it as GA
  • I believe our FE code fails gracefully if either deploys before the other

@@ -0,0 +1,1004 @@
{
"displayName": "JetStream Prometheus Overview",
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is the only place the s in JetStream is capitalized, should this + images be updated, or yaml files?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Good catch! Based on:

It looks like "JetStream" is right, so I've fixed everything else to match.

@EvanSimpson EvanSimpson merged commit b212292 into master Feb 5, 2025
5 checks passed
@EvanSimpson EvanSimpson deleted the jetstream branch February 5, 2025 14:35
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