search

open-telemetry / opentelemetry-collector-contrib

Contrib repository for the OpenTelemetry Collector
https://opentelemetry.io
Apache License 2.0
2.93k stars 2.29k forks source link

Reconsider the need for specifying `otelcol_version` in builder manifest #31623

Open mx-psi opened 6 months ago

mx-psi commented 6 months ago

Component(s)

cmd/otelcontribcol, cmd/oteltestbedcol

Describe the issue you're reporting

During the 0.96.0 release I missed updating otelcol_version to 0.96.0, see https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/31551#discussion_r1511401702. @evan-bradley suggested we could remove otelcol_version entirely from the manifest and rely on the rest of core updates. Should we remove it?

github-actions[bot] commented 4 months ago

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.

github-actions[bot] commented 2 months ago

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.

github-actions[bot] commented 2 weeks ago

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.