Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(iam/admin): regenerate client #11570

Merged
merged 2 commits into from
Feb 7, 2025
Merged

Conversation

codyoss
Copy link
Member

@codyoss codyoss commented Feb 7, 2025

This client has some handwritten code in it that causes failures when generating it normally. I generated the client and hand patched around handwritten code as to not create a breaking change. Changes made:

  • Keep old variants of RPCs that did not generate an iterator helper to avoid breaking changes: ListRoles, QueryGrantableRoles, and QueryTestablePermissions
  • Add variants of above RPCs that do return an iterator: ListRolesIter, QueryGrantableRolesIter, and QueryTestablePermissionsIter
  • Preserve API for GetIamPolicy and SetIamPolicy.

Fixes: #8219

This client has some handwritten code in it that causes failures
when generating it normally. I generated the client and hand patched
around handwritten code as to not create a breaking change.
@codyoss codyoss added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Feb 7, 2025
@product-auto-label product-auto-label bot added the api: iam Issues related to the Identity and Access Management API. label Feb 7, 2025
@codyoss codyoss removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Feb 7, 2025
@codyoss codyoss marked this pull request as ready for review February 7, 2025 17:32
@codyoss codyoss requested a review from a team as a code owner February 7, 2025 17:32
Copy link
Member

@quartzmo quartzmo left a comment

Choose a reason for hiding this comment

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

I generated the client and hand patched around handwritten code

Would it be possible to write a detailed step-set for regularly reproducing what you did? This seems to be a semi-permanent situation...

@codyoss
Copy link
Member Author

codyoss commented Feb 7, 2025

@quartzmo I will write a small doc internally. I think what Noah suggested in the linked issue may be the correct long term solution here -- we make a v2 and fix this. What I did was not hard per-say, but it is not super clean or maintainable to do regularly.

Or when we implement the selective generation feature, maybe that could help here.

@codyoss codyoss merged commit eab87d7 into googleapis:main Feb 7, 2025
9 checks passed
@codyoss codyoss deleted the iam-admin-update branch February 7, 2025 20:06
@release-please release-please bot mentioned this pull request Feb 7, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
api: iam Issues related to the Identity and Access Management API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

iam: mistmatch in exported functions in IAMClient and proto client
2 participants