Make all-in-one metric names match metrics from standalone components

[yurishkuro]

Which problem is this PR solving?

• Resolves [Bug]: Jaeger prometheus metrics are missing "collector" prefix? #5007

Description of the changes

• provide proper namespaces for different sub-components (agent, collector, query) running in all-in-one, to match the metrics names produces by these components when running standalone

How was this change tested?

• go run ./cmd/all-in-one
• curl http://localhost:14269/metrics | grep -v '#'
• compare before/after textcompare-diff.pdf

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b5f73ae) 95.64% compared to head (5c41fce) 95.62%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5008      +/-   ##
==========================================
- Coverage   95.64%   95.62%   -0.02%     
==========================================
  Files         319      319              
  Lines       18786    18786              
==========================================
- Hits        17968    17965       -3     
- Misses        657      659       +2     
- Partials      161      162       +1     

Flag	Coverage Δ
cassandra-3.x	25.63% <ø> (ø)
cassandra-4.x	25.63% <ø> (ø)
elasticsearch-5.x	19.89% <ø> (-0.02%)	⬇️
elasticsearch-6.x	19.90% <ø> (ø)
elasticsearch-7.x	20.04% <ø> (ø)
elasticsearch-8.x	20.13% <ø> (ø)
grpc-badger	19.52% <ø> (ø)
kafka	14.12% <ø> (ø)
opensearch-1.x	20.04% <ø> (+0.01%)	⬆️
opensearch-2.x	20.02% <ø> (-0.02%)	⬇️
unittests	93.37% <ø> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[albertteoh]

LGTM. I wonder if it's easy to write a unit test to validate the namespaces attached to a metrics factory to prevent future regressions?

[yurishkuro]

main isn't easily testable without serious refactoring. This will be better in jaeger-v2