Add CLI support for time_frame arrays in metrics
All threads resolved!
All threads resolved!
Compare changes
There are no changes yet
No changes between version 6 and version 6
Related to #478008 (closed)
After !167201 (merged) has been deployed & tested on production, we now know that metrics with time_frame
array are working.
Now, we add official support for them, by making the CLI use them as the default & adding them to our documentation.
This also updates all of the tests that were previously creating a separate metric definition for each time_frame.
Please include cross links to any resources that are relevant to this MR This will give reviewers and future readers helpful context to give an efficient review of the changes introduced.
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
Before | After |
---|---|
To test this, run the internal events CLI:
ruby scripts/internal_events/cli.rb
And try to create new metric definition with time_frame array. Then, verify that the new metrics are working, using the Manual testing in GDK
commands listed by the CLI when using the view usage
command of the CLI.
No changes between version 6 and version 6