-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 Summary row to Gitpod overview dashboard #11506
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@aledbf Judging from the screenshot you shared workspace types are counted separately (maybe by cluster, or nodepool)?
LGTM besides ✔️
/hold
By cluster |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall it looks good!
It just needs a small change because we cannot make changes to the vendor
folder. This folder is always created by automation with jsonnet-bundler and whatever changes we make here will be overridden when going to production.
Also, I've noticed that people are more comfortable with raw JSON. Do you wanna keep using jsonnet for this dashboard?
# Custom version of https://github.com/grafana/grafonnet-lib/pull/274 | ||
cp components/bar_gauge_panel.libsonnet vendor/github.com/grafana/grafonnet-lib/grafonnet/bar_gauge_panel.libsonnet |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This works when developing locally, but won't work when importing those changes to our observability repository
We have another vendor file over there and it we can't move directories around when rendering dashboards with ArgoCD 😕
operations/observability/mixins/cross-teams/dashboards/gitpod-overview.libsonnet
Outdated
Show resolved
Hide resolved
I just want to add a Summary. If someone wants to own such change across the dashboards, go 👍 |
Description
By default the new row is collapsed
Release Notes
Werft options: