googleapis / googleapis/gapic-generator-go

Fix issue with google.golang.org/protobuf v1.36.X+

Open
#1,608 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
305
Forks
60
Avg merge
1d 8h
Merged PRs (30d)
13

Description

When we upgrade our bazel codegen to this new version we get a strange compile error from our gapic targets. Running `bazelisk build --sandbox_debug //google/bigtable/admin/v2:admin_go_gapic` generates lots of error like: `cannot use name (type string) as type int32 in field value`. This happens for all LROs. For now we are manually patching in an old version but this should be removed ASAP once we have identified the defect.

Contributor guide

Open the contributing guide

Research direction

Start by reproducing the failure with `bazelisk build --sandbox_debug //google/bigtable/admin/v2:admin_go_gapic` using google.golang.org/protobuf v1.36.X, then compare the generated LRO targets with the older version currently used as a workaround. Trace the generated fields involved in the `string` versus `int32` errors. Done means the targets build without manual patching on the new protobuf version.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
build-system, tooling
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.