-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
fix(deps): update module github.com/prometheus/prometheus to v0.52.0 #33059
fix(deps): update module github.com/prometheus/prometheus to v0.52.0 #33059
Conversation
|
github.com/distribution/reference v0.5.0 // indirect | ||
github.com/docker/docker v25.0.5+incompatible // indirect | ||
github.com/docker/docker v26.0.1+incompatible // indirect |
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.
This breaks build
github.com/distribution/reference v0.5.0 // indirect | ||
github.com/docker/docker v25.0.5+incompatible // indirect | ||
github.com/docker/docker v26.0.1+incompatible // indirect |
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.
I think this breaks the build? Since failed build lint check show the otelcontribcol is not built successfully.
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.
The failed build is due to github.com/google/cadvisor
# github.com/google/cadvisor/container/docker/utils
../../../../../pkg/mod/github.com/google/[email protected]/container/docker/utils/docker.go:48:42: undefined: dockertypes.Info
../../../../../pkg/mod/github.com/google/[email protected]/container/docker/utils/docker.go:57:44: undefined: dockertypes.Info
../../../../../pkg/mod/github.com/google/[email protected]/container/docker/utils/docker.go:77:43: undefined: dockertypes.Info
../../../../../pkg/mod/github.com/google/[email protected]/container/docker/utils/docker.go:86:48: undefined: dockertypes.ImageSummary
I try the latest version of cadvisor 0.49.1 again, it stills failed in same reason.
Should this be solved by cadvisor first, then continue this pr? @songy23
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.
Yes. Problem is cadvisor depends on github.com/docker/docker v24.0.9+incompatible
https://github.com/google/cadvisor/blob/master/go.mod#L14 and that version does not work with the one here. Until cadvisor adopts v26.0.1+incompatible
this PR cannot move forward.
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.
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 there a reason they stick on the old version or can we bump it in their project?
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
Closed as inactive. Feel free to reopen if this PR is still being worked on. |
This PR contains the following updates:
v0.51.2-0.20240405174432-b4a973753c6e
->v0.52.0
Warning
Some dependencies could not be looked up. Check the Dependency Dashboard for more information.
Release Notes
prometheus/prometheus (github.com/prometheus/prometheus)
v0.52.0
Compare Source
v0.52.0-rc.1
Compare Source
v0.52.0-rc.0
Compare Source
Configuration
📅 Schedule: Branch creation - "on tuesday" (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.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Mend Renovate. View repository job log here.