Skip to content

chore(managed-identities): rm unused config API path#12856

Merged
zhumin8 merged 5 commits intomainfrom
rm-managed-identities-v1beta1
Apr 24, 2026
Merged

chore(managed-identities): rm unused config API path#12856
zhumin8 merged 5 commits intomainfrom
rm-managed-identities-v1beta1

Conversation

@zhumin8
Copy link
Copy Markdown
Contributor

@zhumin8 zhumin8 commented Apr 20, 2026

Removing unused API path from config for managed-identities, so that it matches the generated code status.

v1beta1 of java-managed-identities was added to the generation_config.yaml as part of initial migration to hermetic build: 14b7346, but client library code for v1beta1 is not present in java-managed-identities/google-cloud-managed-identities dir. This is due to deep-copy-regex being too strict and does not copy /v1beta1 from staging dir to final output.

Expect generation workflow to remove sample code related to v1beta1.

For #12736
For googleapis/librarian#5370

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request removes the v1beta1 proto path for the Managed Identities API from the generation configuration. I have no feedback to provide.

@zhumin8
Copy link
Copy Markdown
Contributor Author

zhumin8 commented Apr 20, 2026

hmm, we got no changed libraries

+ changed_libraries=
+ echo 'Changed libraries are: No changed library.'
Changed libraries are: No changed library.

@sonarqubecloud
Copy link
Copy Markdown

@zhumin8
Copy link
Copy Markdown
Contributor Author

zhumin8 commented Apr 21, 2026

Forced a code generation for this library by a temp change (3e4f22a)
Generated sample code under this dir are removed as expected: java-managed-identities/samples/snippets/generated/com/google/cloud/managedidentities/v1beta1

reverted above temp change.

@sonarqubecloud
Copy link
Copy Markdown

@zhumin8 zhumin8 marked this pull request as ready for review April 21, 2026 19:23
@zhumin8 zhumin8 requested a review from a team as a code owner April 21, 2026 19:23
@snippet-bot
Copy link
Copy Markdown

snippet-bot Bot commented Apr 21, 2026

Here is the summary of changes.

You are about to delete 53 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@zhumin8 zhumin8 requested a review from blakeli0 April 21, 2026 19:23
@zhumin8 zhumin8 enabled auto-merge (squash) April 21, 2026 19:23
@zhumin8
Copy link
Copy Markdown
Contributor Author

zhumin8 commented Apr 22, 2026

@blakeli0 PTAL.
The removed files are sample code within
java-managed-identities/samples/snippets/generated/com/google/cloud/managedidentities/v1beta1/ I think it's reasonable to remove since the relevant code for v1beta1 does not exists and samples are not published.

@zhumin8 zhumin8 requested review from diegomarquezp and lqiu96 April 23, 2026 20:56
@lqiu96
Copy link
Copy Markdown
Member

lqiu96 commented Apr 23, 2026

This is due to deep-copy-regex being too strict and does not copy /v1beta1 from staging dir to final output.

Please correct me if I'm misunderstanding this, but does this mean that the reason we don't have v1beta1 library is because of a config error on our end? If so, do we want to fix this so that managedidentities v1beta1 is generated properly?

@lqiu96
Copy link
Copy Markdown
Member

lqiu96 commented Apr 23, 2026

NVM, I see in the librarian ticket:

Agreed. If they were never generated, I think it's OK to not generate them before migration.

Ok, I think removing the samples if fine for now. Do we have a ticket to generate it after migration (if we still want to)?

@lqiu96 lqiu96 disabled auto-merge April 23, 2026 21:12
@blakeli0
Copy link
Copy Markdown
Contributor

Do we have a ticket to generate it after migration (if we still want to)?

Since this is v1beta1 and we never generated before, I think it's OK to not generate it for now. We can always add it back if someone needs it.

@zhumin8 zhumin8 enabled auto-merge (squash) April 24, 2026 14:26
@zhumin8
Copy link
Copy Markdown
Contributor Author

zhumin8 commented Apr 24, 2026

updated branch in attempt to trigger the required "conditional-required-check" test

@zhumin8 zhumin8 merged commit 436571e into main Apr 24, 2026
122 checks passed
@zhumin8 zhumin8 deleted the rm-managed-identities-v1beta1 branch April 24, 2026 15:05
suztomo pushed a commit to suztomo/google-cloud-java that referenced this pull request Apr 24, 2026
Removing unused API path from config for managed-identities, so that it
matches the generated code status.

v1beta1 of java-managed-identities was added to the
generation_config.yaml as part of initial migration to hermetic build:
googleapis@14b7346,
but client library code for v1beta1 is not present in
java-managed-identities/google-cloud-managed-identities dir. This is due
to deep-copy-regex being too strict and does not copy /v1beta1 from
staging dir to final output.

Expect generation workflow to remove sample code related to v1beta1.

For googleapis#12736
For googleapis/librarian#5370

---------

Co-authored-by: cloud-java-bot <cloud-java-bot@google.com>
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.

4 participants