Skip to content

Tracing - Remove fixed boundaries from scatter chart

Daniele Rossetti requested to merge drosse/tracing-remove-scatter-chart-fixed-range into master

What does this MR do and why?

As discussed in https://gitlab.com/gitlab-org/opstrace/opstrace/-/issues/2519#note_1682661065, we decided to change the behaviour of the tracing scatter chart ( try it out here: gitlab-org/opstrace/sandbox/otel-demo/-/tracing ).

Current behaviour:

The chart time boundaries are fixed and matches the time range query filter. e.g. selecting "last 1h" filter set the chart time boundaries to the last hour.

This might cause the chart to have some gaps as the chart data is paginated and fetched while scrolling.

New behaviour:

The chart time boundaries are not fixed and they always match the data that is loaded. When fetching new data, the chart will adjust and append the new data, while changing the x time boundaries. This makes sure that there are no gaps in the chart.

Closes https://gitlab.com/gitlab-org/opstrace/opstrace/-/issues/2519

Screenshots or screen recordings

Before After
pre post

How to set up and validate locally

Testing this requires launching the full observability stack, which is not trivial and requires a VM. Will have to trust the screenshot on this. Or if feeling adventures, I can give access to my VM.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Daniele Rossetti

Merge request reports