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

feat(gkeconnect/gateway): start generating apiv1beta1 #4235

Merged
merged 4 commits into from
Jun 9, 2021
Merged

feat(gkeconnect/gateway): start generating apiv1beta1 #4235

merged 4 commits into from
Jun 9, 2021

Conversation

codyoss
Copy link
Member

@codyoss codyoss commented Jun 8, 2021

No description provided.

@codyoss codyoss requested a review from a team as a code owner June 8, 2021 20:35
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jun 8, 2021
@codyoss codyoss requested a review from noahdietz June 8, 2021 20:35
@codyoss codyoss added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jun 8, 2021
@codyoss
Copy link
Member Author

codyoss commented Jun 8, 2021

@noahdietz For some reason the import for gatewaypb is not being generated into the imports. Any ideas why? Looks like the RPC don't deal with types from that package but just httpbody. Maybe that is why?

@noahdietz
Copy link
Contributor

Oop yes this was fixed in v0.20.2 of the generator. Turns out the proto has no request/response messages of its own (they are are google.api.HttpBody) and the generator apparently was only adding the API's pb import when it resolved request/response messages. Updating the generator in the pipeline should fix it. Sorry about that.

@codyoss
Copy link
Member Author

codyoss commented Jun 8, 2021

No worries, thanks for the info!

@codyoss codyoss added automerge Merge the pull request once unit tests and other checks pass. and removed do not merge Indicates a pull request not ready for merge, due to either quality or timing. labels Jun 9, 2021
@codyoss codyoss merged commit 1c3e968 into googleapis:master Jun 9, 2021
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jun 9, 2021
@codyoss codyoss deleted the gen-gat-v1b1 branch June 9, 2021 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants