Skip to content

feat: remove GKEHubv1alpha2#10415

Merged
zhumin8 merged 2 commits intomainfrom
removeGKEHubv1alpha2
Mar 4, 2024
Merged

feat: remove GKEHubv1alpha2#10415
zhumin8 merged 2 commits intomainfrom
removeGKEHubv1alpha2

Conversation

@alicejli
Copy link
Contributor

GKEHubv1alpha2 protos are removed. This PR removes those modules and updates the .OwlBot.yaml file to ensure they are not copied over from googleapis-gen in the future.

@alicejli alicejli requested a review from a team February 21, 2024 20:40
@snippet-bot
Copy link

snippet-bot bot commented Feb 21, 2024

Here is the summary of changes.

You are about to delete 32 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

@alicejli alicejli added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Feb 21, 2024
@alicejli
Copy link
Contributor Author

Adding do not merge until confirmation from the team that this is the correct thing to do in order to remove this package.

@@ -28,13 +28,9 @@ deep-copy-regex:
dest: "/owl-bot-staging/java-gkehub/v1beta/proto-google-cloud-gkehub-v1beta/src"
- source: "/google/cloud/gkehub/v1beta1/.*-java/proto-google-.*/src/main/java/com/google/cloud/gkehub"
dest: "/owl-bot-staging/java-gkehub/v1beta1/proto-google-cloud-gkehub-v1beta1/src/main/java/com/google/cloud/gkehub"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
dest: "/owl-bot-staging/java-gkehub/v1beta1/proto-google-cloud-gkehub-v1beta1/src/main/java/com/google/cloud/gkehub"
- source: "/google/cloud/gkehub/(v.*)/.*-java/proto-google-.*/src"
dest: "/owl-bot-staging/java-gkehub/$1/proto-google-cloud-alloydb-$1/src"

There's no v1alpha2 in googleapis-gen. Could you change these two lines without specifying the version?

Copy link
Contributor

@JoeWang1127 JoeWang1127 left a comment

Choose a reason for hiding this comment

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

Thanks.

@zhumin8 zhumin8 removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Mar 4, 2024
@zhumin8 zhumin8 merged commit 3e95fa4 into main Mar 4, 2024
@zhumin8 zhumin8 deleted the removeGKEHubv1alpha2 branch March 4, 2024 22:51
zhumin8 added a commit that referenced this pull request Mar 6, 2024
followup on #10415.
existing-version-check on release branch reported snapshot versions:
com.google.api.grpc:grpc-google-cloud-gkehub-v1alpha2:0.43.0-SNAPSHOT:
com.google.api.grpc:proto-google-cloud-gkehub-v1alpha2:0.43.0-SNAPSHOT

GKEHubv1alpha2 modules are removed, but owl-bot is not able to cleanup these pom files.
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.

3 participants