4229 date filter

[punitdarira]

Telemetry API now has metrics and date filter.
Made changes in telemetry and metrics api to reuse code.

related to #4229

[CLAassistant]

All committers have signed the CLA.

[danielkelemen]

Hi @punitdarira, thank you for your PR!

I'd need a bit more time to review this thoroughly. It already looks pretty good at first sight.
I can already tell you, that we would need a few more tests in the TelemetryRestServiceTest for instance, to test the newly added parameters and the results. But I'll provide more feedback soon.

-Daniel

[punitdarira]

Hi @punitdarira, thank you for your PR!

I'd need a bit more time to review this thoroughly. It already looks pretty good at first sight. I can already tell you, that we would need a few more tests in the TelemetryRestServiceTest for instance, to test the newly added parameters and the results. But I'll provide more feedback soon.

-Daniel

Hi @danielkelemen,
I agree regarding the tests. I'll update this PR with the tests.

[tasso94]

Hi @punitdarira,

This PR got lost in the hustle and bustle. Sorry for replying only now.

Are you still interested in merging this? If yes, could you please resolve the merge conflicts before we review it?

Best,
Tassilo

[punitdarira]

@tasso94
Sure I'll resolve the merge conflicts

[punitdarira]

Hi @tasso94,
Since sending telemetry data is deprecated (#4486)
Can you please confirm whether this ticket is still relevant?