go/opentelemetry-go
1
0
Fork 0
mirror of https://github.com/open-telemetry/opentelemetry-go.git synced 2025-10-08 23:21:56 +02:00
Code Issues Releases Activity

chore(deps): update dependency codespell to v2.3.0 (#5409)

Browse Source 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Robert Pająk <pellared@hotmail.com>
This commit is contained in:
renovate[bot]
2024-05-24 18:09:31 +02:00
committed by GitHub
parent bd24d549b6
commit 7c5e64cccc
10 changed files with 19 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
.codespellignore
View File

@@ -5,3 +5,5 @@ collison
consequentially consequentially
ans ans
nam nam
valu
thirdparty

2
.codespellrc
View File

@@ -5,6 +5,6 @@ check-filenames =
check-hidden = check-hidden =
ignore-words = .codespellignore ignore-words = .codespellignore
interactive = 1 interactive = 1
skip = .git,go.mod,go.sum,semconv,venv,.tools skip = .git,go.mod,go.sum,go.work,go.work.sum,semconv,venv,.tools
uri-ignore-words-list = * uri-ignore-words-list = *
write = write =

12
CHANGELOG.md
View File

@@ -250,7 +250,7 @@ See our [versioning policy](VERSIONING.md) for more information about these stab
## [1.20.0/0.43.0] 2023-11-10 ## [1.20.0/0.43.0] 2023-11-10
This release brings a breaking change for custom trace API implementations. Some interfaces (`TracerProvider`, `Tracer`, `Span`) now embed the `go.opentelemetry.io/otel/trace/embedded` types. Implementors need to update their implementations based on what they want the default behavior to be. See the "API Implementations" section of the [trace API] package documentation for more information about how to accomplish this. This release brings a breaking change for custom trace API implementations. Some interfaces (`TracerProvider`, `Tracer`, `Span`) now embed the `go.opentelemetry.io/otel/trace/embedded` types. Implementers need to update their implementations based on what they want the default behavior to be. See the "API Implementations" section of the [trace API] package documentation for more information about how to accomplish this.
### Added ### Added
@@ -282,15 +282,15 @@ This release brings a breaking change for custom trace API implementations. Some
- `go.opentelemetry.io/otel/bridge/opencensus.NewMetricProducer` returns a `*MetricProducer` struct instead of the metric.Producer interface. (#4583) - `go.opentelemetry.io/otel/bridge/opencensus.NewMetricProducer` returns a `*MetricProducer` struct instead of the metric.Producer interface. (#4583)
- The `TracerProvider` in `go.opentelemetry.io/otel/trace` now embeds the `go.opentelemetry.io/otel/trace/embedded.TracerProvider` type. - The `TracerProvider` in `go.opentelemetry.io/otel/trace` now embeds the `go.opentelemetry.io/otel/trace/embedded.TracerProvider` type.
This extends the `TracerProvider` interface and is is a breaking change for any existing implementation. This extends the `TracerProvider` interface and is is a breaking change for any existing implementation.
Implementors need to update their implementations based on what they want the default behavior of the interface to be. Implementers need to update their implementations based on what they want the default behavior of the interface to be.
See the "API Implementations" section of the `go.opentelemetry.io/otel/trace` package documentation for more information about how to accomplish this. (#4620) See the "API Implementations" section of the `go.opentelemetry.io/otel/trace` package documentation for more information about how to accomplish this. (#4620)
- The `Tracer` in `go.opentelemetry.io/otel/trace` now embeds the `go.opentelemetry.io/otel/trace/embedded.Tracer` type. - The `Tracer` in `go.opentelemetry.io/otel/trace` now embeds the `go.opentelemetry.io/otel/trace/embedded.Tracer` type.
This extends the `Tracer` interface and is is a breaking change for any existing implementation. This extends the `Tracer` interface and is is a breaking change for any existing implementation.
Implementors need to update their implementations based on what they want the default behavior of the interface to be. Implementers need to update their implementations based on what they want the default behavior of the interface to be.
See the "API Implementations" section of the `go.opentelemetry.io/otel/trace` package documentation for more information about how to accomplish this. (#4620) See the "API Implementations" section of the `go.opentelemetry.io/otel/trace` package documentation for more information about how to accomplish this. (#4620)
- The `Span` in `go.opentelemetry.io/otel/trace` now embeds the `go.opentelemetry.io/otel/trace/embedded.Span` type. - The `Span` in `go.opentelemetry.io/otel/trace` now embeds the `go.opentelemetry.io/otel/trace/embedded.Span` type.
This extends the `Span` interface and is is a breaking change for any existing implementation. This extends the `Span` interface and is is a breaking change for any existing implementation.
Implementors need to update their implementations based on what they want the default behavior of the interface to be. Implementers need to update their implementations based on what they want the default behavior of the interface to be.
See the "API Implementations" section of the `go.opentelemetry.io/otel/trace` package documentation for more information about how to accomplish this. (#4620) See the "API Implementations" section of the `go.opentelemetry.io/otel/trace` package documentation for more information about how to accomplish this. (#4620)
- `go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetricgrpc` does no longer depend on `go.opentelemetry.io/otel/exporters/otlp/otlpmetric`. (#4660) - `go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetricgrpc` does no longer depend on `go.opentelemetry.io/otel/exporters/otlp/otlpmetric`. (#4660)
- `go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp` does no longer depend on `go.opentelemetry.io/otel/exporters/otlp/otlpmetric`. (#4660) - `go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp` does no longer depend on `go.opentelemetry.io/otel/exporters/otlp/otlpmetric`. (#4660)
@@ -926,7 +926,7 @@ The next release will require at least [Go 1.19].
- Exported `Status` codes in the `go.opentelemetry.io/otel/exporters/zipkin` exporter are now exported as all upper case values. (#3340) - Exported `Status` codes in the `go.opentelemetry.io/otel/exporters/zipkin` exporter are now exported as all upper case values. (#3340)
- `Aggregation`s from `go.opentelemetry.io/otel/sdk/metric` with no data are not exported. (#3394, #3436) - `Aggregation`s from `go.opentelemetry.io/otel/sdk/metric` with no data are not exported. (#3394, #3436)
- Re-enabled Attribute Filters in the Metric SDK. (#3396) - Re-enabled Attribute Filters in the Metric SDK. (#3396)
- Asynchronous callbacks are only called if they are registered with at least one instrument that does not use drop aggragation. (#3408) - Asynchronous callbacks are only called if they are registered with at least one instrument that does not use drop aggregation. (#3408)
- Do not report empty partial-success responses in the `go.opentelemetry.io/otel/exporters/otlp` exporters. (#3438, #3432) - Do not report empty partial-success responses in the `go.opentelemetry.io/otel/exporters/otlp` exporters. (#3438, #3432)
- Handle partial success responses in `go.opentelemetry.io/otel/exporters/otlp/otlpmetric` exporters. (#3162, #3440) - Handle partial success responses in `go.opentelemetry.io/otel/exporters/otlp/otlpmetric` exporters. (#3162, #3440)
- Prevent duplicate Prometheus description, unit, and type. (#3469) - Prevent duplicate Prometheus description, unit, and type. (#3469)
@@ -1971,7 +1971,7 @@ with major version 0.
- `NewExporter` from `exporters/otlp` now takes a `ProtocolDriver` as a parameter. (#1369) - `NewExporter` from `exporters/otlp` now takes a `ProtocolDriver` as a parameter. (#1369)
- Many OTLP Exporter options became gRPC ProtocolDriver options. (#1369) - Many OTLP Exporter options became gRPC ProtocolDriver options. (#1369)
- Unify endpoint API that related to OTel exporter. (#1401) - Unify endpoint API that related to OTel exporter. (#1401)
- Optimize metric histogram aggregator to re-use its slice of buckets. (#1435) - Optimize metric histogram aggregator to reuse its slice of buckets. (#1435)
- Metric aggregator Count() and histogram Bucket.Counts are consistently `uint64`. (1430) - Metric aggregator Count() and histogram Bucket.Counts are consistently `uint64`. (1430)
- Histogram aggregator accepts functional options, uses default boundaries if none given. (#1434) - Histogram aggregator accepts functional options, uses default boundaries if none given. (#1434)
- `SamplingResult` now passed a `Tracestate` from the parent `SpanContext` (#1432) - `SamplingResult` now passed a `Tracestate` from the parent `SpanContext` (#1432)

2
log/DESIGN.md
View File

@@ -566,7 +566,7 @@ The `Logger` does not have methods like `Enabled`, `SetSeverity`, etc.
as the Bridge API needs to follow (be compliant with) as the Bridge API needs to follow (be compliant with)
the [specification](https://opentelemetry.io/docs/specs/otel/logs/bridge-api/) the [specification](https://opentelemetry.io/docs/specs/otel/logs/bridge-api/)
Moreover, the Bridge API is intendend to be used to implement bridges. Moreover, the Bridge API is intended to be used to implement bridges.
Applications should not use it directly. The applications should use logging packages Applications should not use it directly. The applications should use logging packages
such as [`slog`](https://pkg.go.dev/log/slog), such as [`slog`](https://pkg.go.dev/log/slog),
[`logrus`](https://pkg.go.dev/github.com/sirupsen/logrus), [`logrus`](https://pkg.go.dev/github.com/sirupsen/logrus),

2
requirements.txt
View File

@@ -1 +1 @@
codespell==2.2.6 codespell==2.3.0

6
sdk/log/DESIGN.md
View File

@@ -116,12 +116,12 @@ single `Expoter` interface.[^2]
However, introducing a `Processor` interface makes it easier However, introducing a `Processor` interface makes it easier
to create custom processor decorators[^3] to create custom processor decorators[^3]
and makes the design more aligned with the specifiation. and makes the design more aligned with the specification.
### Embedd log.Record ### Embed log.Record
Because [`Record`](#record) and [`log.Record`](https://pkg.go.dev/go.opentelemetry.io/otel/log#Record) Because [`Record`](#record) and [`log.Record`](https://pkg.go.dev/go.opentelemetry.io/otel/log#Record)
are very similar, there was a proposal to embedd `log.Record` in `Record` definition. are very similar, there was a proposal to embed `log.Record` in `Record` definition.
[`log.Record`](https://pkg.go.dev/go.opentelemetry.io/otel/log#Record) [`log.Record`](https://pkg.go.dev/go.opentelemetry.io/otel/log#Record)
supports only adding attributes. supports only adding attributes.

2
sdk/log/batch.go
View File

@@ -109,7 +109,7 @@ func NewBatchProcessor(exporter Exporter, opts ...BatchProcessorOption) *BatchPr
exporter = defaultNoopExporter exporter = defaultNoopExporter
} }
// Order is important here. Wrap the timeoutExporter with the chunkExporter // Order is important here. Wrap the timeoutExporter with the chunkExporter
// to ensure each export completes in timeout (instead of all chuncked // to ensure each export completes in timeout (instead of all chunked
// exports). // exports).
exporter = newTimeoutExporter(exporter, cfg.expTimeout.Value) exporter = newTimeoutExporter(exporter, cfg.expTimeout.Value)
// Use a chunkExporter to ensure ForceFlush and Shutdown calls are batched // Use a chunkExporter to ensure ForceFlush and Shutdown calls are batched

4
sdk/log/exporter.go
View File

@@ -73,7 +73,7 @@ type chunkExporter struct {
} }
// newChunkExporter wraps exporter. Calls to the Export will have their records // newChunkExporter wraps exporter. Calls to the Export will have their records
// payload chuncked so they do not exceed size. If size is less than or equal // payload chunked so they do not exceed size. If size is less than or equal
// to 0, exporter is returned directly. // to 0, exporter is returned directly.
func newChunkExporter(exporter Exporter, size int) Exporter { func newChunkExporter(exporter Exporter, size int) Exporter {
if size <= 0 { if size <= 0 {
@@ -82,7 +82,7 @@ func newChunkExporter(exporter Exporter, size int) Exporter {
return &chunkExporter{Exporter: exporter, size: size} return &chunkExporter{Exporter: exporter, size: size}
} }
// Export exports records in chuncks no larger than c.size. // Export exports records in chunks no larger than c.size.
func (c chunkExporter) Export(ctx context.Context, records []Record) error { func (c chunkExporter) Export(ctx context.Context, records []Record) error {
n := len(records) n := len(records)
for i, j := 0, min(c.size, n); i < n; i, j = i+c.size, min(j+c.size, n) { for i, j := 0, min(c.size, n); i < n; i, j = i+c.size, min(j+c.size, n) {

2
sdk/log/provider.go
View File

@@ -92,7 +92,7 @@ func NewLoggerProvider(opts ...LoggerProviderOption) *LoggerProvider {
// Logger returns a new [log.Logger] with the provided name and configuration. // Logger returns a new [log.Logger] with the provided name and configuration.
// //
// If p is shut down, a [noop.Logger] instace is returned. // If p is shut down, a [noop.Logger] instance is returned.
// //
// This method can be called concurrently. // This method can be called concurrently.
func (p *LoggerProvider) Logger(name string, opts ...log.LoggerOption) log.Logger { func (p *LoggerProvider) Logger(name string, opts ...log.LoggerOption) log.Logger {

2
sdk/log/record.go
View File

@@ -154,7 +154,7 @@ func (r *Record) WalkAttributes(f func(log.KeyValue) bool) {
func (r *Record) AddAttributes(attrs ...log.KeyValue) { func (r *Record) AddAttributes(attrs ...log.KeyValue) {
n := r.AttributesLen() n := r.AttributesLen()
if n == 0 { if n == 0 {
// Avoid the more complex duplicate map lookups bellow. // Avoid the more complex duplicate map lookups below.
attrs, r.dropped = dedup(attrs) attrs, r.dropped = dedup(attrs)
var drop int var drop int