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

chore(deps): update accountingservice #1244

Merged
merged 2 commits into from
Nov 27, 2023
Merged

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Nov 11, 2023

Mend Renovate logo banner

This PR contains the following updates:

Package Type Update Change
github.com/IBM/sarama require patch v1.42.0 -> v1.42.1
go.opentelemetry.io/otel require minor v1.19.0 -> v1.21.0
go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc require minor v1.19.0 -> v1.21.0
go.opentelemetry.io/otel/sdk require minor v1.19.0 -> v1.21.0
go.opentelemetry.io/otel/trace require minor v1.19.0 -> v1.21.0
golang stage patch 1.21.3-alpine -> 1.21.4-alpine

Release Notes

IBM/sarama (github.com/IBM/sarama)

v1.42.1: Version 1.42.1 (2023-11-07)

Compare Source

What's Changed

🐛 Fixes

Full Changelog: IBM/sarama@v1.42.0...v1.42.1

open-telemetry/opentelemetry-go (go.opentelemetry.io/otel)

v1.21.0: /v0.44.0

Compare Source

Removed
  • Remove the deprecated go.opentelemetry.io/otel/bridge/opencensus.NewTracer. (#​4706)
  • Remove the deprecated go.opentelemetry.io/otel/exporters/otlp/otlpmetric module. (#​4707)
  • Remove the deprecated go.opentelemetry.io/otel/example/view module. (#​4708)
  • Remove the deprecated go.opentelemetry.io/otel/example/fib module. (#​4723)
Fixed
  • Do not parse non-protobuf responses in go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp. (#​4719)
  • Do not parse non-protobuf responses in go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp. (#​4719)

v1.20.0: /v0.43.0

Compare Source

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 about how to accomplish this.

Added
  • Add go.opentelemetry.io/otel/bridge/opencensus.InstallTraceBridge, which installs the OpenCensus trace bridge, and replaces opencensus.NewTracer. (#​4567)
  • Add scope version to trace and metric bridges in go.opentelemetry.io/otel/bridge/opencensus. (#​4584)
  • Add the go.opentelemetry.io/otel/trace/embedded package to be embedded in the exported trace API interfaces. (#​4620)
  • Add the go.opentelemetry.io/otel/trace/noop package as a default no-op implementation of the trace API. (#​4620)
  • Add context propagation in go.opentelemetry.io/otel/example/dice. (#​4644)
  • Add view configuration to go.opentelemetry.io/otel/example/prometheus. (#​4649)
  • Add go.opentelemetry.io/otel/metric.WithExplicitBucketBoundaries, which allows defining default explicit bucket boundaries when creating histogram instruments. (#​4603)
  • Add Version function in go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetricgrpc. (#​4660)
  • Add Version function in go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp. (#​4660)
  • Add Summary, SummaryDataPoint, and QuantileValue to go.opentelemetry.io/sdk/metric/metricdata. (#​4622)
  • go.opentelemetry.io/otel/bridge/opencensus.NewMetricProducer now supports exemplars from OpenCensus. (#​4585)
  • Add support for WithExplicitBucketBoundaries in go.opentelemetry.io/otel/sdk/metric. (#​4605)
  • Add support for Summary metrics in go.opentelemetry.io/otel/bridge/opencensus. (#​4668)
Deprecated
  • Deprecate go.opentelemetry.io/otel/bridge/opencensus.NewTracer in favor of opencensus.InstallTraceBridge. (#​4567)
  • Deprecate go.opentelemetry.io/otel/example/fib package is in favor of go.opentelemetry.io/otel/example/dice. (#​4618)
  • Deprecate go.opentelemetry.io/otel/trace.NewNoopTracerProvider.
    Use the added NewTracerProvider function in go.opentelemetry.io/otel/trace/noop instead. (#​4620)
  • Deprecate go.opentelemetry.io/otel/example/view package in favor of go.opentelemetry.io/otel/example/prometheus. (#​4649)
  • Deprecate go.opentelemetry.io/otel/exporters/otlp/otlpmetric. (#​4693)
Changed
  • 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.
    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.
    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.
    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.
    See the "API Implementations" section of the go.opentelemetry.io/otel/trace package documentation for more informationabout how to accomplish this. (#​4620)
  • 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.
    Implementors 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)
  • 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)
  • Retry for 502 Bad Gateway and 504 Gateway Timeout HTTP statuses in go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp. (#​4670)
  • Retry for 502 Bad Gateway and 504 Gateway Timeout HTTP statuses in go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp. (#​4670)
  • Retry for RESOURCE_EXHAUSTED only if RetryInfo is returned in go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetricgrpc. (#​4669)
  • Retry for RESOURCE_EXHAUSTED only if RetryInfo is returned in go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc. (#​4669)
  • Retry temporary HTTP request failures in go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp. (#​4679)
  • Retry temporary HTTP request failures in go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp. (#​4679)
Fixed
  • Fix improper parsing of characters such us +, / by Parse in go.opentelemetry.io/otel/baggage as they were rendered as a whitespace. (#​4667)
  • Fix improper parsing of characters such us +, / passed via OTEL_RESOURCE_ATTRIBUTES in go.opentelemetry.io/otel/sdk/resource as they were rendered as a whitespace. (#​4699)
  • Fix improper parsing of characters such us +, / passed via OTEL_EXPORTER_OTLP_HEADERS and OTEL_EXPORTER_OTLP_METRICS_HEADERS in go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetricgrpc as they were rendered as a whitespace. (#​4699)
  • Fix improper parsing of characters such us +, / passed via OTEL_EXPORTER_OTLP_HEADERS and OTEL_EXPORTER_OTLP_METRICS_HEADERS in go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp as they were rendered as a whitespace. (#​4699)
  • Fix improper parsing of characters such us +, / passed via OTEL_EXPORTER_OTLP_HEADERS and OTEL_EXPORTER_OTLP_TRACES_HEADERS in go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlptracegrpc as they were rendered as a whitespace. (#​4699)
  • Fix improper parsing of characters such us +, / passed via OTEL_EXPORTER_OTLP_HEADERS and OTEL_EXPORTER_OTLP_TRACES_HEADERS in go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlptracehttp as they were rendered as a whitespace. (#​4699)
  • In go.opentelemetry.op/otel/exporters/prometheus, the exporter no longer Collects metrics after Shutdown is invoked. (#​4648)
  • Fix documentation for WithCompressor in go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc. (#​4695)
  • Fix documentation for WithCompressor in go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetricgrpc. (#​4695)

Configuration

📅 Schedule: Branch creation - "every weekend" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

@renovate renovate bot requested a review from a team November 11, 2023 00:38
@renovate renovate bot added the dependencies Pull requests that update a dependency file label Nov 11, 2023
Copy link

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Nov 24, 2023
@julianocosta89
Copy link
Member

@secustor how can I rebase this PR without messing up renovate?
I've tried checking the PR checkbox, but nothing happened.

@secustor
Copy link
Member

secustor commented Nov 26, 2023

Looks correct.

The job has run which you can see in the repo logs which you can find behind the View repository logs here link.
https://developer.mend.io/github/open-telemetry/opentelemetry-demo

Tough I have no access to detailed logs as I'm missing permissions on this repo.

@julianocosta89
Copy link
Member

Looks correct.

But in order to merge it, we need the branch updated.
I remember that we had some issues with it in the past, because we manually updated the branch.
That's why I'm asking 😅

@secustor
Copy link
Member

If the branch is modified, then Renovate will not update in anymore until the PR is merged or closed. As it assumes that devs are fixing upgrade issues.

Using the check box these changes can be discarded.

If PR rules are defined that prevent merge commit merges, than Renovate should rebase the PR itself ( if the rule above is not triggered the PR itself )
See https://docs.renovatebot.com/configuration-options/#rebasewhen

From what I understand what happened on the other PRs is that upgrades have been fixed by cherry picking from the Renovate branches an then merging via another PR.
Then the main branch has been merged into the renovate/ branch, which led to that Renovate considered the PR as modified.

This is a theory tough, I have to still reproduce that.

Copy link
Contributor Author

renovate bot commented Nov 27, 2023

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

Warning: custom changes will be lost.

@austinlparker austinlparker merged commit 4f5c2cf into main Nov 27, 2023
46 checks passed
@austinlparker austinlparker deleted the renovate/accountingservice branch November 27, 2023 18:04
jmichalak9 pushed a commit to jmichalak9/opentelemetry-demo that referenced this pull request Mar 22, 2024
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Austin Parker <austin@ap2.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants