-
Notifications
You must be signed in to change notification settings - Fork 11.8k
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
Explore: Improve timeseries limit disclaimer #75587
Conversation
@@ -80,6 +80,7 @@ export function LogsVolumePanel(props: Props) { | |||
anchorToZero | |||
yAxisMaximum={allLogsVolumeMaximum} | |||
eventBus={props.eventBus} | |||
showAllTimeSeries |
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.
Should we also make this a state variable that the user can toggle at some point?
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.
whole context in https://raintank-corp.slack.com/archives/C02TXB92G6T/p1695815888660829
the TLDR is that this is just to unblock som other things, the logs volume panel can only have like 8 series i think (max 1 per each log level) so it's not really affected, we'll refactor this to make it better
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.
Logs changes look good 👍
Co-authored-by: Kristina <kristina.durivage@grafana.com>
Backend code coverage report for PR #75587 |
Frontend code coverage report for PR #75587
|
* Explore: improve timeseries limit disclaimer * use object notation for style definition * increase gap * Update public/app/features/explore/Graph/GraphContainer.tsx Co-authored-by: Kristina <kristina.durivage@grafana.com> --------- Co-authored-by: Kristina <kristina.durivage@grafana.com>
* Explore: improve timeseries limit disclaimer * use object notation for style definition * increase gap * Update public/app/features/explore/Graph/GraphContainer.tsx Co-authored-by: Kristina <kristina.durivage@grafana.com> --------- Co-authored-by: Kristina <kristina.durivage@grafana.com>
* Explore: improve timeseries limit disclaimer * use object notation for style definition * increase gap * Update public/app/features/explore/Graph/GraphContainer.tsx Co-authored-by: Kristina <kristina.durivage@grafana.com> --------- Co-authored-by: Kristina <kristina.durivage@grafana.com>
What is this feature?
Revamps the "Showing only 20 series" disclaimer in Explore:
Special notes for your reviewer:
The code is uglier then my hair right now, and the Graph is coupled with the container, but this should unblock #75499. Will create a follow up issue to do a tiny bit of refactoring as discussed with @gelicia to introduce a
limit
prop the the ExploreGraph (instead of the current solution) so it'll be a bit cleanerPlease check that: