Remove project_imports method from usage_data
What does this MR do and why?
Related to #421501 (closed)
Remove a metric from usage_data.rb
as it's already defined using metric definition files.
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
Before | After |
---|---|
How to set up and validate locally
- Check out the master branch
- Open the rails console
- Generate new service ping payload:
sp = Gitlab::Usage::ServicePingReport.for(output: :all_metrics_values)
- Check out this MR's branch
- Update the rails console by executing
reload!
- Generate a new service ping payload:
sp2 = Gitlab::Usage::ServicePingReport.for(output: :all_metrics_values)
- Compare the new and old metrics by making sure this comparison returns
true
:sp['usage_activity_by_stage']['manage']['project_imports'] == sp2['usage_activity_by_stage']['manage']['project_imports']
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Edited by Michał Wielich