Remove PodLogs Redis counter and related code logic
Related #355591 (comment 892807856)
Summary
Remove PodLogs and Common class in order to avoid bad patterns to spread.
Details
Gitlab::UsageCounters::PodLogs used in https://gitlab.com/gitlab-org/gitlab/-/blob/master/app/controllers/projects/logs_controller.rb#L31
I see that the related service ping metric was removed in 14.0 https://gitlab.com/gitlab-org/gitlab/-/blob/master/config/metrics/counts_all/20210216175021_pod_logs_usages_total.yml
MR removing the metric !62962 (merged) and issue #321786 (comment 562109341)
I think we should remove the related tracking code logic too.
I would suggest:
- Remove the code remaining code logic for tracking
- As this is the only class implemented based on Common we can remove this type of counter for Service Ping.
- Product Intelligence can initiate the removal and we could ask for a review from ~"group::monitor" to be sure that we cover any concerns.
Edited by Alina Mihaila