Skip to content

Migrate ecosystem redis_hll metrics to instrumentation class

What does this MR do and why?

With #338463 (closed) foundation for migration of redis hll metrics into isturmentation classes framework has been implemented. Additional first batch of metrics was migrated. This issue is a follow up to migrate the pending ones and remove completely automated addition of redis hll metrics into ServicePing payload via usage_data.rb file

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

How to set up and validate locally

pry(main)> Gitlab::UsageDataCounters::HLLRedisCounter.unique_events_data['ecosystem']
=> nil

pry(main)> ServicePing::BuildPayload.new.execute[:redis_hll_counters]['ecosystem']
=> {array of metrics}

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #368996 (closed)

Edited by Niko Belokolodov

Merge request reports