search

heroku / libcnb.rs

A framework for writing Cloud Native Buildpacks in Rust
BSD 3-Clause "New" or "Revised" License
35 stars 6 forks source link

Bump the opentelemetry group with 3 updates #799

Closed dependabot[bot] closed 4 months ago

dependabot[bot] commented 7 months ago

Updates the requirements on opentelemetry, opentelemetry_sdk and opentelemetry-stdout to permit the latest version. Updates opentelemetry to 0.22.0

Release notes

Sourced from opentelemetry's releases.

v0.22.0

API

Added

Changed

Removed

SDK

Deprecated

  • XrayIdGenerator in the opentelemetry-sdk has been deprecated and moved to version 0.10.0 of the opentelemetry-aws crate.

Added

  • #1410 Add experimental synchronous gauge

  • #1471 Configure batch log record processor via OTEL_BLRP_* environment variables and via OtlpLogPipeline::with_batch_config

  • #1503 Make the documentation for In-Memory exporters visible.

  • #1526 Performance Improvement : Creating Spans and LogRecords are now faster, by avoiding expensive cloning of Resource for every Span/LogRecord.

Changed

  • Breaking #1313 #1350 Changes how Span links/events are stored to achieve performance gains. See below for details:

    Behavior Change: When enforcing max_links_per_span, max_events_per_span from SpanLimits, links/events are kept in the first-come order. The previous "eviction" based approach is no longer performed.

    Breaking Change Affecting Exporter authors:

    SpanData now stores links as SpanLinks instead of EvictedQueue where SpanLinks is a struct with a Vec of links and dropped_count.

    SpanData now stores events as SpanEvents instead of EvictedQueue where SpanEvents is a struct with a Vec of events and dropped_count.

  • Breaking Remove TextMapCompositePropagator #1373. Use TextMapCompositePropagator in opentelemetry API.

... (truncated)

Commits


Updates opentelemetry_sdk to 0.22.1

Release notes

Sourced from opentelemetry_sdk's releases.

0.22.1 of OpenTelemetry sdk

Fixed open-telemetry/opentelemetry-rust#1576 Fix Span kind is always set to "internal".

Commits


Updates opentelemetry-stdout to 0.3.0

Release notes

Sourced from opentelemetry-stdout's releases.

v0.3.0

Added

  • New Base64 propagator
  • New SpanBuilder api
  • Zipkin Exporter crate

Changed

  • Switch to SpanId and TraceId from u64 and u128
  • Remove &mut self requirements for Span API

Fixed

  • circular Tracer debug impl
Changelog

Sourced from opentelemetry-stdout's changelog.

v0.3.0

Added

  • New Base64 propagator
  • New SpanBuilder api
  • Zipkin Exporter crate

Changed

  • Switch to SpanId and TraceId from u64 and u128
  • Remove &mut self requirements for Span API

Fixed

  • circular Tracer debug impl

v0.2.0

Added

  • Make trace and metrics features optional
  • ExportResult as specified in the specification
  • Add Futures compatibility API
  • Added serde serialise support to SpanData
  • Separate OpenTelemetry Jaeger crate

Changed

  • Rename HttpTraceContextPropagator to TraceContextPropagator
  • Rename HttpB3Propagator to B3Propagator
  • Switch to Apache 2 license
  • Resolve agent addresses to allow non-static IP
  • Remove tracer name prefix from span name

Removed

  • Remove add_link from spans

v0.1.5

Added

  • trace-context propagator

Changed

  • Prometheus API cleanup

v0.1.4

Added

  • Parent option for default sampler

Fixed

  • SDK tracer default span id

v0.1.3

... (truncated)

Commits


You can trigger a rebase of this PR by commenting @dependabot rebase.

Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore major version` will close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself) - `@dependabot ignore minor version` will close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself) - `@dependabot ignore ` will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself) - `@dependabot unignore ` will remove all of the ignore conditions of the specified dependency - `@dependabot unignore ` will remove the ignore condition of the specified dependency and ignore conditions

Note Automatic rebases have been disabled on this pull request as it has been open for over 30 days.

edmorley commented 7 months ago

@joshwlewis The tests are failing with:

---- tracing::tests::test_tracing stdout ----
tracing_contents: {"resourceSpans":[{"resource":{"attributes":[{"key":"service.version","value":{"stringValue":"0.0.99"}},{"key":"service.name","value":{"stringValue":"company.com/foo"}}]},"scopeSpans":[{"scope":{"name":"libcnb","version":"0.19.0"},"spans":[{"traceId":"7deae25accd2519109a45a7d05bb5cb2","spanId":"f599048ca08001f7","parentSpanId":"0000000000000000","name":"company_com_foo-bar","kind":1,"startTimeUnixNano":1709116666323150356,"startTime":"2024-02-28 10:37:46.323","endTimeUnixNano":1709116666323166256,"endTime":"2024-02-28 10:37:46.323","attributes":[{"key":"buildpack_id","value":{"stringValue":"company.com/foo"}},{"key":"buildpack_name","value":{"stringValue":"Foo buildpack for company.com"}},{"key":"buildpack_version","value":{"stringValue":"0.0.99"}},{"key":"buildpack_homepage","value":{"stringValue":""}}],"droppedAttributesCount":0,"events":[{"name":"baz-event","attributes":[],"droppedAttributesCount":0,"timeUnixNano":1709116666323158401,"time":"2024-02-28 10:37:46.323"},{"name":"exception","attributes":[{"key":"exception.message","value":{"stringValue":"it's broken"}}],"droppedAttributesCount":0,"timeUnixNano":1709116666323165324,"time":"2024-02-28 10:37:46.323"}],"droppedEventsCount":0,"flags":1,"droppedLinksCount":0,"status":{"message":"Custom { kind: Other, error: \"it's broken\" }","code":2}}]}]}]}

thread 'tracing::tests::test_tracing' panicked at libcnb/src/tracing.rs:193:9:
assertion failed: tracing_contents.contains("\"code\":1")

Could you take a look?

edmorley commented 7 months ago

I've also removed the "skip changelog" label to remind us to confirm whether this is a breaking change or not for consumers of the /tmp/libcnb-telemetry file or not. ie: Will cnb-otel-collector need to be updated in lockstep?

dependabot[bot] commented 4 months ago

Superseded by #828.