[FEATURE] Status history panel series sorting #187
Conversation
abelyakin
force-pushed
the
status-history-sorting
branch
3 times, most recently
from
64b17b6 to
4705252
Compare
Signed-off-by: Alexander Belyakin <[email protected]>
abelyakin
force-pushed
the
status-history-sorting
branch
from
4705252 to
a1e5811
Compare
|
@abelyakin mind resolving the conflicts so we can merge this?, thnx. Again sorry for the delay on the response. |
abelyakin
requested review from
a team,
AntoineThebaud,
Nexucis and
andreasgerstmayr
as code owners
abelyakin
force-pushed
the
status-history-sorting
branch
from
bc6c3a6 to
b1e417f
Compare
Signed-off-by: Alexander Belyakin <[email protected]>
|
@jgbernalp sorry fro delay. updated PR. |
|
|
||
| kind: "StatusHistoryChart" | ||
| spec: { | ||
| sorting: "asc" |
There was a problem hiding this comment.
is the sorting option available in grafana in some way? or is not and we default to "asc"
There was a problem hiding this comment.
AFAIS there's no equivalent on Grafana side
There was a problem hiding this comment.
grafana doesn't have such option but it looks like it sorts series by default under the hood.
| value={value.legend} | ||
| onChange={handleLegendChange} | ||
| /> | ||
| <SortSelector value={value.sorting} onChange={handleSortChange} /> |
There was a problem hiding this comment.
It currently looks like the new control belongs to the Legend category, while it doesnt
Could you add a split by having a "Visual" header like we have in other cases e.g here:
?
There was a problem hiding this comment.
nice idea! moved to separate group
Signed-off-by: Alexander Belyakin <[email protected]>
3 participants
Description
Summary
Implemented series sorting functionality for the Status History panel to group series with the same name together, improving data visualization clarity.
Problem
When displaying multiple series with identical names (e.g., "kafka-1") from different Prometheus queries in the Status History panel, the series were displayed in random order, making it difficult to visually correlate related data points.
Solution
Added sorting configuration option to the Status History panel schema with "asc" | "desc" values
Implemented series sorting logic to group series by name alphabetically
Series with the same name from different queries are now displayed consecutively
Screenshots
Checklist
[<catalog_entry>] <commit message>naming convention using one of thefollowing
catalog_entryvalues:FEATURE,ENHANCEMENT,BUGFIX,BREAKINGCHANGE,DOC,IGNORE.UI Changes