Skip to content

Conversation

sunnylovestiramisu
Copy link
Contributor

What type of PR is this?
Update google.golang.org/grpc

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce an API-breaking change?:

Update google.golang.org/grpc


require (
google.golang.org/grpc v1.57.1
google.golang.org/protobuf v1.33.0
github.com/golang/protobuf v1.5.4
Copy link
Contributor

Choose a reason for hiding this comment

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

How did this deprecated link become a direct reference? It should be an indirect one used by google.golang.org/protobuf internally.

golang.org/x/net v0.38.0 // indirect
golang.org/x/sys v0.31.0 // indirect
golang.org/x/text v0.23.0 // indirect
google.golang.org/genproto/googleapis/rpc v0.0.0-20250324211829-b45e905df463 // indirect
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm getting this without generated files under lib/go/csi:

module github.com/container-storage-interface/spec

go 1.23.0

toolchain go1.24.4

require (
	google.golang.org/grpc v1.73.0
	google.golang.org/protobuf v1.36.6
)

require (
	golang.org/x/net v0.38.0 // indirect
	golang.org/x/sys v0.31.0 // indirect
	golang.org/x/text v0.23.0 // indirect
	google.golang.org/genproto/googleapis/rpc v0.0.0-20250324211829-b45e905df463 // indirect
)

@sunnylovestiramisu
Copy link
Contributor Author

/close

@sunnylovestiramisu
Copy link
Contributor Author

/close

Resolved by upgrading github.com/golang/protobuf v1.5.4

@sunnylovestiramisu
Copy link
Contributor Author

/close

1 similar comment
@sunnylovestiramisu
Copy link
Contributor Author

/close

@sunnylovestiramisu
Copy link
Contributor Author

Not needed anymore. Not sure why the close tag do not work anymore

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants