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

feat(ui): simplify flux labels #5997

Merged
merged 5 commits into from
Aug 23, 2022
Merged

feat(ui): simplify flux labels #5997

merged 5 commits into from
Aug 23, 2022

Conversation

sranka
Copy link
Contributor

@sranka sranka commented Aug 17, 2022

This PR simplifies labels of flux-based line charts.

Briefly describe your proposed changes:
A label value before this PR

_value[_start=2022-08-17T06:54:02.67759815Z][_stop=2022-08-17T07:54:02.657Z][_field=numSeries][_measurement=database][database=_internal][hostname=66fec3b3194f]

is herein simplified to

numSeries measurement=database database=_internal hostname=66fec3b3194f

with a help of the following rules

  • _field value is used in place of _value string, it just shows the field that was measured
  • _start, _stop and _field groups keys are removed from the label
  • measurement is used in place of _measurement
    • queries that do have no _value and _field columns are also handled properly

What was the problem?
image

What was the solution?
image

@sranka sranka marked this pull request as ready for review August 23, 2022 04:45
@sranka sranka requested a review from ivankudibal August 23, 2022 04:45
@sranka sranka force-pushed the feat/simplify_flux_labels branch from d591fc2 to 451b1fe Compare August 23, 2022 12:45
@sranka sranka merged commit c88dbeb into master Aug 23, 2022
@sranka sranka deleted the feat/simplify_flux_labels branch August 23, 2022 13:12
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