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: remove GKEHubv1alpha2 #10415

Merged
merged 2 commits into from
Mar 4, 2024
Merged

feat: remove GKEHubv1alpha2 #10415

merged 2 commits into from
Mar 4, 2024

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 as a code owner February 21, 2024 20:40
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
32 checks passed
@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