honeycombio / opentelemetry-collector-configs

Honeycomb Metrics-flavored OpenTelemetry Collector configuration
Apache License 2.0
7 stars 4 forks source link

Create and package an otel-collector build w/ timestamp processor #5

Closed hazeledmands closed 3 years ago

hazeledmands commented 3 years ago

Much of this code was copied & modified from the opentelemetry-collector base repository. Now, running make release will:

Here is an example of the result: https://github.com/honeycombio/opentelemetry-collector-configs/releases/tag/1.1.0-prerelease-3

The compiled version of the collector comes with:

In order to accomplish this I had to some things that felt gross, in the name of dealing with conflicting versions:

Also! I noticed that OpenTelemetry collector at version 32+ is no longer compatible with our metricstransform configuration, which is a bummer. (At that version our config will silently delete metrics instead of renaming them.) So, I added a test that will prevent us from deploying a version that runs into this problem.

hazeledmands commented 3 years ago

Will we support both versions in the end - .30 and .32

I would prefer to just officially support the latest version, which at this point is 0.33.0, but there are a few things blocking that still!