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

Bump github.com/prometheus/common from 0.47.0 to 0.52.3 #2955

Merged

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Apr 11, 2024

Bumps github.com/prometheus/common from 0.47.0 to 0.52.3.

Release notes

Sourced from github.com/prometheus/common's releases.

v0.52.3

What's Changed

Full Changelog: prometheus/common@v0.52.2...v0.52.3

v0.52.2

What's Changed

New Contributors

Full Changelog: prometheus/common@v0.51.1...v0.52.2

v0.51.1

What's Changed

Full Changelog: prometheus/common@v0.51.0...v0.51.1

v0.51.0

What's Changed

New Contributors

Full Changelog: prometheus/common@v0.50.0...v0.51.0

v0.50.0

What's Changed

... (truncated)

Commits
  • 0234594 Add support for go 1.20 (#617)
  • 4433931 Merge pull request #615 from prometheus/dependabot/go_modules/sigv4/github.co...
  • 728e910 Merge pull request #616 from srebhan/fix_protobuf_decode
  • 78585cf fix(protobuf): Correctly decode multi-messages streams
  • 5fef1df Bump github.com/aws/aws-sdk-go from 1.50.31 to 1.51.11 in /sigv4
  • 3dec13c Drop support for Go older than 1.18 (#612)
  • 95edf51 Update common Prometheus files (#613)
  • e821827 Retract v0.50.0 (#610)
  • 2807775 Update common Prometheus files (#609)
  • 617b51d Synchronize common files from prometheus/prometheus (#606)
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually 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 <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot requested review from a team as code owners April 11, 2024 02:21
@dependabot dependabot bot added dependencies Pull requests that update a dependency file go Pull requests that update Go code labels Apr 11, 2024
@dependabot dependabot bot requested review from ademidoff, BupycHuk, idoqo and JiriCtvrtka and removed request for a team April 11, 2024 02:21
@dependabot dependabot bot requested review from a team April 11, 2024 02:21
@github-actions github-actions bot enabled auto-merge (squash) April 11, 2024 02:21
Bumps [github.com/prometheus/common](https://github.com/prometheus/common) from 0.47.0 to 0.52.3.
- [Release notes](https://github.com/prometheus/common/releases)
- [Commits](prometheus/common@v0.47.0...v0.52.3)

---
updated-dependencies:
- dependency-name: github.com/prometheus/common
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot force-pushed the dependabot/go_modules/github.com/prometheus/common-0.52.3 branch from 2b0e3f2 to 7501be8 Compare April 15, 2024 13:30
go 1.21
go 1.22

toolchain go1.22.2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can drop the toolchain directive - less stuff to update on every new patch version release.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can't, you can see that CI failed because of not having that. By the last commit I had to re-add it back.

Copy link
Member

@ademidoff ademidoff Apr 17, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suppose this is because we need to have the same Go version inside the image/container (i.e. perconalab/pmm-server:dev-latest), which is not the case.

Copy link
Member

@ademidoff ademidoff Apr 17, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMO, if we bump it up here, we have to use the same version everywhere, including the build tools, like rpmbuild.

In the logs I'm seeing an attempt to download go (go: downloading go1.22 (linux/amd64), which should not be needed if we use the same version.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great part about it that our build system started using go1.22 after merging this PR and nothing is broken.

Copy link
Contributor Author

dependabot bot commented on behalf of github Apr 19, 2024

A newer version of github.com/prometheus/common exists, but since this PR has been edited by someone other than Dependabot I haven't updated it. You'll get a PR for the updated version as normal once this PR is merged.

@idoqo
Copy link
Contributor

idoqo commented Apr 21, 2024

@dependabot rebase

Copy link
Contributor Author

dependabot bot commented on behalf of github Apr 21, 2024

Looks like this PR has been edited by someone other than Dependabot. That means Dependabot can't rebase it - sorry!

If you're happy for Dependabot to recreate it from scratch, overwriting any edits, you can request @dependabot recreate.

Copy link

codecov bot commented Apr 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 42.27%. Comparing base (abeacee) to head (1bd7aae).
Report is 9 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2955   +/-   ##
=======================================
  Coverage   42.27%   42.27%           
=======================================
  Files         396      397    +1     
  Lines       50123    50122    -1     
=======================================
+ Hits        21188    21190    +2     
+ Misses      26935    26930    -5     
- Partials     2000     2002    +2     
Flag Coverage Δ
admin 11.59% <ø> (+0.03%) ⬆️
vmproxy 68.57% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions github-actions bot merged commit 24ec9c2 into main Apr 21, 2024
31 checks passed
@github-actions github-actions bot deleted the dependabot/go_modules/github.com/prometheus/common-0.52.3 branch April 21, 2024 21:27
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 go Pull requests that update Go code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants