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

Testing Issue: breaking changes related to grpc generator #1554

Open
shollyman opened this issue Aug 12, 2024 · 1 comment
Open

Testing Issue: breaking changes related to grpc generator #1554

shollyman opened this issue Aug 12, 2024 · 1 comment
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@shollyman
Copy link
Contributor

As part of showcase testing in this generator, CI tests effectively compare a recent release of the showcase generated code to one that's output by the generator using apidiff.

The grpc plugin (https://pkg.go.dev/google.golang.org/grpc/cmd/protoc-gen-go-grpc) has a new option (use_generic_streams_experimental) that recently defaulted to true. Because our showcase testing grabs the plugin latest version, we were picking up that new change.

To resolve our immediate issue, we need to add the experimental flag to our build process to disable the newly enabled behavior, and verify what further actions are needed.

@shollyman shollyman added priority: p2 Moderately-important priority. Fix may not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. labels Aug 12, 2024
@quartzmo
Copy link
Member

Great diagnosis, thank you!

quartzmo added a commit to quartzmo/gapic-generator-go that referenced this issue Aug 29, 2024
* upgrade bazel_gazelle to 0.38.0
* upgrade com_google_protobuf to 25.4
* add rules_proto 6.0.2
* remove -use_generic_streams_experimental
  * fix error: protoc-gen-go-grpc: no such flag -use_generic_streams_experimental

refs: googleapis#1554
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Development

No branches or pull requests

2 participants