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

build(go.mod): project-wide bump google.golang.org/protobuf to v1.33.0 #3087

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Mar 13, 2024

Bumps google.golang.org/protobuf from 1.30.0 to 1.33.0.

Dependabot compatibility score

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 <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)
    You can disable automated security fix PRs for this repo from the Security Alerts page.

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

@dependabot dependabot bot added dependencies Pull requests that update a dependency file go Pull requests that update Go code labels Mar 13, 2024
@sandeepnRES
Copy link
Contributor

There are CI tests failing in some dependabot's PRs, I'll investigate and fix this, please don't merge these before that.
Merge those PRs where CI tests are passing.

@petermetz
Copy link
Contributor

There are CI tests failing in some dependabot's PRs, I'll investigate and fix this, please don't merge these before that. Merge those PRs where CI tests are passing.

@sandeepnRES OK, no worries! I'll close all the other ones and keep this one open with the contents of all of them combined (e.g. a mass upgrade of the package in question to v1.33.0.
Then we can test/verify the problems here and keep the conversation in a single place as well.

Copy link
Contributor

@petermetz petermetz left a comment

Choose a reason for hiding this comment

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

Holding this off until we investigate with @sandeepnRES what's going wrong exactly.
So not really a change request just blocking the merge for now as discussed in the comments.

@petermetz petermetz changed the title build(deps): bump google.golang.org/protobuf from 1.30.0 to 1.33.0 in /weaver/core/network/fabric-interop-cc/interfaces/asset-mgmt build(go.mod): project-wide bump google.golang.org/protobuf to v1.33.0 Mar 28, 2024
@petermetz
Copy link
Contributor

@petermetz
Copy link
Contributor

@sandeepnRES First thing I noticed is that the tests complain about the lock file not being updated (the sum file) so I'll do a bunch of go mod tidy calls in all the sub-directories and hopefully that'll make at least some of our issues go away.

1. Bumps google.golang.org/protobuf from 1.30.0 to 1.33.0 throughout the project.
2. Also ran `go mod tidy` in all 14 sub-directories where we have the go.mod
files in question located.

---
updated-dependencies:
- dependency-name: google.golang.org/protobuf

Co-authored-by: Peter Somogyvari <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Peter Somogyvari <[email protected]>
@petermetz petermetz force-pushed the dependabot/go_modules/weaver/core/network/fabric-interop-cc/interfaces/asset-mgmt/google.golang.org/protobuf-1.33.0 branch from 2280652 to 39ea37d Compare March 28, 2024 20:57
@petermetz
Copy link
Contributor

@sandepnRES The go mod tidy changes got it done. All the (required) tests are passing now.

@petermetz petermetz self-requested a review March 29, 2024 03:25
Copy link
Contributor

@jagpreetsinghsasan jagpreetsinghsasan left a comment

Choose a reason for hiding this comment

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

LGTM

@sandeepnRES
Copy link
Contributor

sandeepnRES commented May 21, 2024

Hi @petermetz Sorry I didn't realise about this PR, I fixed other PRs by dependabot which did the same upgrades. One is merged PR #3249, and other two are pending to be merged #3263 and PR #3275
So I'll close this PR.

Copy link
Contributor Author

dependabot bot commented on behalf of github May 21, 2024

OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting @dependabot ignore this major version or @dependabot ignore this minor version.

If you change your mind, just re-open this PR and I'll resolve any conflicts on it.

@dependabot dependabot bot deleted the dependabot/go_modules/weaver/core/network/fabric-interop-cc/interfaces/asset-mgmt/google.golang.org/protobuf-1.33.0 branch May 21, 2024 08:41
@petermetz
Copy link
Contributor

Hi @petermetz Sorry I didn't realise about this PR, I fixed other PRs by dependabot which did the same upgrades. One is merged PR #3249, and other two are pending to be merged #3263 and PR #3275 So I'll close this PR.

@sandeepnRES Please try to let me know earlier next time so I don't waste my time.

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