opentelemetry-collector icon indicating copy to clipboard operation
opentelemetry-collector copied to clipboard

[cmd/mdatagen] Add an option to generate setters from callbacks

Open dmitryax opened this issue 2 years ago • 4 comments
trafficstars

Component(s)

No response

Is your feature request related to a problem? Please describe.

To avoid doing unnecessary work fetching data from external sources for disabled metrics or resource attributes. Originally posted by @mx-psi in https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/24360#discussion_r1266593277

Describe the solution you'd like

The control of how to generate the setter can be set in metadata.yaml similar to https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/receiver/apachereceiver/metadata.yaml#L66

Describe alternatives you've considered

No response

Additional context

No response

dmitryax avatar Jul 18 '23 15:07 dmitryax

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

  • cmd/mdatagen: @dmitryax

See Adding Labels via Comments if you do not have permissions to add labels yourself.

github-actions[bot] avatar Sep 18 '23 03:09 github-actions[bot]

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

  • cmd/mdatagen: @dmitryax

See Adding Labels via Comments if you do not have permissions to add labels yourself.

github-actions[bot] avatar Dec 20 '23 03:12 github-actions[bot]

This issue has been closed as inactive because it has been stale for 120 days with no activity.

github-actions[bot] avatar Feb 18 '24 05:02 github-actions[bot]

transferred issue as mdatagen lives in the core repo now

codeboten avatar Aug 16 '24 15:08 codeboten