[Events] Save panel

[kavithacm]

Signed-off-by: Kavitha Conjeevaram Mohan [email protected]

Description

Include save as metric workflow to save custom metric under event analytics

Issues Resolved

opensearch-project/dashboards-observability#139

Check List

• New functionality includes testing.
  • All tests pass, including unit test, integration test and doctest
• New functionality has been documented.
  • New functionality has javadoc added
  • New functionality has user manual doc added
• Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[codecov-commenter]

Codecov Report

Merging #1185 (7d2ee8f) into feature/metrics (116fa97) will not change coverage.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##             feature/metrics    opensearch-project/observability#1185   +/-   ##
==================================================
  Coverage              71.87%   71.87%           
  Complexity               291      291           
==================================================
  Files                     42       42           
  Lines                   2311     2311           
  Branches                 240      240           
==================================================
  Hits                    1661     1661           
  Misses                   509      509           
  Partials                 141      141           

Flag	Coverage Δ
opensearch-observability	71.87% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[sejli]

Milliseconds is no longer necessary; can you confirm @pjfitzgibbons?

[kavithacm]

oh okay, i'll remove it

[pjfitzgibbons]

I think we should exclude it. Milliseconds doesn't make sense for a time-span. Thanks and Kindest Regards, Peter Fitzgibbons (224) 307-9689

…

On Wed, Oct 26, 2022 at 1:23 PM Sean Li ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In dashboards-observability/common/constants/explorer.ts <#1185 (comment)> : > @@ -269,3 +269,14 @@ export enum ConfigChartOptionsEnum { export const CUSTOM_LABEL = 'customLabel'; export const BREAKDOWNS = 'breakdowns'; + +// Metrics constants +export const UNITS_OF_MEASURE = [ + 'milliseconds (ms)', Milliseconds is no longer necessary; can you confirm @pjfitzgibbons <https://github.com/pjfitzgibbons>? — Reply to this email directly, view it on GitHub <#1185 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAACYNXK3VFUDTJIMISWL7DWFGHNVANCNFSM6AAAAAARORAQ2Y> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[kavithacm]

I think we should exclude it. Milliseconds doesn't make sense for a time-span. Thanks and Kindest Regards, Peter Fitzgibbons (224) 307-9689
…
On Wed, Oct 26, 2022 at 1:23 PM Sean Li @.> wrote: @.* commented on this pull request. ------------------------------ In dashboards-observability/common/constants/explorer.ts <#1185 (comment)> : > @@ -269,3 +269,14 @@ export enum ConfigChartOptionsEnum { export const CUSTOM_LABEL = 'customLabel'; export const BREAKDOWNS = 'breakdowns'; + +// Metrics constants +export const UNITS_OF_MEASURE = [ + 'milliseconds (ms)', Milliseconds is no longer necessary; can you confirm @pjfitzgibbons https://github.com/pjfitzgibbons? — Reply to this email directly, view it on GitHub <#1185 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAACYNXK3VFUDTJIMISWL7DWFGHNVANCNFSM6AAAAAARORAQ2Y . You are receiving this because you were mentioned.Message ID: @.***>
oh okay, i'll remove it