Deprecate service::telemetry::metrics::address

[bogdandrutu]

Existing configs like:

service:
  telemetry:
    metrics:
      level: "basic"
      address: "localhost:8888"

Should change to:

service:
  telemetry:
    metrics:
      level: "basic"
      readers:
        - pull:
            exporter:
              prometheus:
                host: "localhost"
                port: 8888

[codecov]

Codecov Report

Attention: Patch coverage is 86.48649% with 5 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@d6f568d). Learn more about missing BASE report.
Report is 14 commits behind head on main.

Files with missing lines	Patch %	Lines
service/telemetry/config.go	85.18%	2 Missing and 2 partials ⚠️
service/telemetry/metrics.go	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #11205   +/-   ##
=======================================
  Coverage        ?   91.56%           
=======================================
  Files           ?      424           
  Lines           ?    20195           
  Branches        ?        0           
=======================================
  Hits            ?    18492           
  Misses          ?     1319           
  Partials        ?      384           

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

[dmitryax]

This is a much more verbose solution. Users would be wondering why are we doing it. Do we have an issue where we track the transition with the reasoning behind it?

[bogdandrutu]

This is a much more verbose solution. Users would be wondering why are we doing it. Do we have an issue where we track the transition with the reasoning behind it?

#7641
#7532

[codeboten]

This is a much more verbose solution. Users would be wondering why are we doing it. Do we have an issue where we track the transition with the reasoning behind it?

@dmitryax this was done in part to support the same configuration as is proposed by the opentelemetry-configuration schema, but also to support additional exporters, and more functionality like views in the future