-
Notifications
You must be signed in to change notification settings - Fork 395
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
Bump all the dependencies #1795
Conversation
@@ -6,89 +6,90 @@ require ( | |||
github.com/go-logr/logr v1.2.4 | |||
github.com/oklog/ulid/v2 v2.1.0 | |||
github.com/open-telemetry/opamp-go v0.7.0 | |||
github.com/open-telemetry/opentelemetry-operator v1.51.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should retract this version, it's what we have as latest in go.pkg.dev right now.
@@ -122,16 +121,17 @@ func NewMetricReporter( | |||
|
|||
// Create some metrics that will be reported according to OpenTelemetry semantic | |||
// conventions for process metrics (conventions are TBD for now). | |||
reporter.processCpuTime, err = reporter.meter.Float64ObservableGauge( | |||
reporter.processCpuTime, err = reporter.meter.Float64ObservableCounter( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is changing this metric type a breaking change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see below
) | ||
if err != nil { | ||
return nil, fmt.Errorf("can't create process time metric: %w", err) | ||
} | ||
reporter.processMemoryPhysical, err = reporter.meter.Float64ObservableCounter( | ||
|
||
reporter.processMemoryPhysical, err = reporter.meter.Float64ObservableGauge( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is changing this metric type a breaking change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it was previously incorrect, but also no one is using this image yet other than me pretty much because we don't have a way to deploy this yet.
* Bump all the deps * linter
* Bump all the deps * linter
Another dependabot doesn't know what the hell its doing PR. This bumps all our dependencies that dependabot is trying to bump.