Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release v1.4.0/v0.29.0 #1639

Merged
merged 21 commits into from
Feb 14, 2022
Merged

Conversation

MrAlias
Copy link
Contributor

@MrAlias MrAlias commented Feb 11, 2022

  • Update otel dependencies
  • Drop exact support in datadog and dogstatsd exporters
  • Drop support for histograms in datadog exporter
  • Deprecate the datadog, cortext, and dogstatsd exporters
  • Update all otel/sdk/export/metric* package to use the non-deprecated replacement.
  • Bump versions
  • Update all version.go files (looks like this was missed last release).

Release notes

Added

Changed

  • All metric instruments from the go.opentelemetry.io/contrib/instrumentation/runtime package have been renamed from runtime.go.* to process.runtime.go.* so as to comply with OpenTelemetry semantic conventions. (add process prefix to runtime go metrics #1549)

Fixed

Deprecated

  • The go.opentelemetry.io/contrib/exporters/metric/datadog module is deprecated. (Release v1.4.0/v0.29.0 #1639)
  • The go.opentelemetry.io/contrib/exporters/metric/dogstatsd module is deprecated. (Release v1.4.0/v0.29.0 #1639)
  • The go.opentelemetry.io/contrib/exporters/metric/cortex module is deprecated.
    Use the go.opentelemetry.io/otel/exporters/otlp/otlpmetric exporter as a replacement to send data to a collector which can then export with its PRW exporter. (Release v1.4.0/v0.29.0 #1639)

Removed

@codecov
Copy link

codecov bot commented Feb 11, 2022

Codecov Report

Merging #1639 (7954c7b) into main (1005161) will increase coverage by 6.7%.
The diff coverage is 42.5%.

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #1639     +/-   ##
=======================================
+ Coverage   69.5%   76.2%   +6.7%     
=======================================
  Files        127     132      +5     
  Lines       5448    5456      +8     
=======================================
+ Hits        3787    4160    +373     
+ Misses      1519    1142    -377     
- Partials     142     154     +12     
Impacted Files Coverage Δ
detectors/aws/ec2/version.go 0.0% <0.0%> (ø)
detectors/aws/ecs/version.go 0.0% <0.0%> (ø)
detectors/aws/eks/version.go 0.0% <0.0%> (ø)
detectors/gcp/version.go 0.0% <0.0%> (ø)
exporters/metric/cortex/utils/config_utils.go 91.6% <ø> (ø)
exporters/metric/cortex/utils/version.go 0.0% <0.0%> (ø)
exporters/metric/cortex/version.go 0.0% <0.0%> (ø)
exporters/metric/datadog/datadog.go 57.1% <ø> (+9.7%) ⬆️
exporters/metric/datadog/version.go 0.0% <0.0%> (ø)
exporters/metric/dogstatsd/version.go 0.0% <0.0%> (ø)
... and 58 more

Do not fail lint for importing deprecated packages from inside those
deprecated packages.
CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Member

@pellared pellared left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please double-check if the following commit/PR should be included in CHANGELOG:

CHANGELOG.md Outdated Show resolved Hide resolved
@MrAlias
Copy link
Contributor Author

MrAlias commented Feb 14, 2022

Please double-check if the following commit/PR should be included in CHANGELOG:

I'll update the changelog to include this change. Thanks for the catch.

@MrAlias MrAlias merged commit c17e0db into open-telemetry:main Feb 14, 2022
@MrAlias MrAlias deleted the release-v1.4.0/v0.29.0 branch February 14, 2022 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants