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