-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 v0.10.0 #981
Release v0.10.0 #981
Conversation
do we want to create a go.mod for the api package so people can import it by itself? |
@lizthegrey: sorry, probably should have pinged you in my last comment: #977 (comment)
Does this make sense to you? |
It does. Do we want to ship this current set of changes as v0.10, or do we want to add that in before v0.10 is cut? |
Good question. I was hoping (similar to you #977 (comment)) to accomplish all this in a single release. The remaining dependencies are in the instrumentation. I didn't want to make the current instrumentation into their own modules as they would likely be single release modules before they are renamed or moved to the contrib repo. And I can't move them to the contrib repo without having the STDOUT exporter published with a release. I could try instead to clean up the instrumentation of all SDK imports here. Which would allow for the SDK to be made into a module without any cyclic dependencies from the |
Yes, I think pulling out the SDK is the right thing to do, then we cut v0.10, then we refactor contrib, I think that's the right order of operations. But you're the full-time maintainer, I'm just an approver who pitches in sometimes. |
Agreed, I'll retry this after that work. |
0.10.0 - 2020-07-28
Added
NewExportPipeline
andInstallNewPipeline
constructor functions to match the common pattern.These function build a new exporter with default SDK options and register the exporter with the
global
package respectively. (Unify trace and metric exporter helpers #944)Changed
RegisterGlobal
Option
in the Jaeger exporter with anInstallNewPipeline
constructor function.This matches the other exporter constructor patterns and will register a new exporter after building it with default configuration. (Unify trace and metric exporter helpers #944)
go.opentelemetry.io/otel/exporters/trace/stdout
) and metric (go.opentelemetry.io/otel/exporters/metric/stdout
)stdout
exporters are now merged into a single exporter atgo.opentelemetry.io/otel/exporters/stdout
.This new exporter was made into its own Go module to follow the pattern of all exporters and decouple it from the
go.opentelemetry.io/otel
module. (Unify trace and metric stdout exporters #956)go.opentelemetry.io/otel/api/kv/value
package was merged into the parentgo.opentelemetry.io/otel/api/kv
package. (Remove sub-package value from kv #968)value.Bool
was replaced withkv.BoolValue
.value.Int64
was replaced withkv.Int64Value
.value.Uint64
was replaced withkv.Uint64Value
.value.Float64
was replaced withkv.Float64Value
.value.Int32
was replaced withkv.Int32Value
.value.Uint32
was replaced withkv.Uint32Value
.value.Float32
was replaced withkv.Float32Value
.value.String
was replaced withkv.StringValue
.value.Int
was replaced withkv.IntValue
.value.Uint
was replaced withkv.UintValue
.value.Array
was replaced withkv.ArrayValue
.Infer
toAny
in thego.opentelemetry.io/otel/api/kv
package. (Rename kv.Infer to kv.Any (#969) #972)Removed
IndexedAttribute
function from thego.opentelemetry.io/otel/api/label
package was removed in favor ofIndexedLabel
which it was synonymous with. (Remove IndexedAttribute from api/label #970)Fixed
global.Handle
for span export errors in the OTLP exporter. (use global handler for span export err in otlp #946)go.opentelemetry.io/otel/api
package. (Decouple API from SDK #977)