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: Introduce automatic LRO polling, integrate latest compute API definitions #583

Merged
merged 10 commits into from
Dec 7, 2021

Conversation

Neenu1995
Copy link
Contributor

@Neenu1995 Neenu1995 commented Nov 29, 2021

feat: Introduce automatic LRO polling; all LRO methods now return OperationFuture<> instead of Operation and poling is done automatically; calling cancel() on the future will cancel polling but not the operation itself on the server side.
chore: migrate to owlbot
fix: rename IPProtocol to IPProtocolEnum

@Neenu1995 Neenu1995 requested review from a team as code owners November 29, 2021 22:42
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Nov 29, 2021
@product-auto-label product-auto-label bot added the api: compute Issues related to the googleapis/java-compute API. label Nov 29, 2021
@vam-google
Copy link
Contributor

@georgiyekkert PTAL. Please update Integration Test according to the changes.

The expected changes:

  1. Strings must be used instead of enums
  2. LRO methos are now in place (what used to return Operation now returns OpratoinFuture<Operation, Operation>)

@Neenu1995 Neenu1995 added automerge Merge the pull request once unit tests and other checks pass. and removed automerge Merge the pull request once unit tests and other checks pass. labels Dec 2, 2021
@Neenu1995 Neenu1995 changed the title chore: migrate to owlbot feat: integrate latest compute API definitions Dec 6, 2021
@vam-google vam-google changed the title feat: integrate latest compute API definitions feat: Introduce automatic LRO polling, integrate latest compute API definitions Dec 6, 2021
@Neenu1995
Copy link
Contributor Author

Hi @georgiyekkert,
The integration tests are failing. Would you mind taking a look?

@georgiyekkert
Copy link
Contributor

Looking

@vam-google vam-google added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 7, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 7, 2021
@georgiyekkert
Copy link
Contributor

I was able to reproduce the issue locally, an image insert fails with:

errors:

  • code: RESOURCE_OPERATION_RATE_EXCEEDED
    message: Operation rate exceeded for resource 'projects/debian-cloud/global/images/debian-10-buster-v20210721'.
    Too frequent operations from the source resource.

I'm going to try to fix or skip this test. The error doesn't look related to library changes.

@georgiyekkert
Copy link
Contributor

confirmed: it's not a library issue, the API has a limitation - you can create only 6 images per hour per project. I'll update test in a separate PR.

Copy link
Contributor

@vam-google vam-google left a comment

Choose a reason for hiding this comment

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

LGTM

@Neenu1995 Neenu1995 added the automerge Merge the pull request once unit tests and other checks pass. label Dec 7, 2021
@gcf-merge-on-green gcf-merge-on-green bot merged commit 90f7b41 into main Dec 7, 2021
@gcf-merge-on-green gcf-merge-on-green bot deleted the owlbot-migration branch December 7, 2021 17:46
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Dec 7, 2021
gcf-merge-on-green bot pushed a commit that referenced this pull request Dec 7, 2021
🤖 I have created a release \*beep\* \*boop\*
---
## [1.6.0](https://www.github.com/googleapis/java-compute/compare/v1.5.0...v1.6.0) (2021-12-07)


### Features

* Introduce automatic LRO polling, integrate latest compute API definitions ([#583](https://www.github.com/googleapis/java-compute/issues/583)) ([90f7b41](https://www.github.com/googleapis/java-compute/commit/90f7b41dfc91a45981786a2eeed2f21bf82febeb))
* Introduce automatic LRO polling; all LRO methods now return `OperationFuture<>` instead of `Operation` and poling is done automatically; calling `cancel()` on the future will cancel polling but not the operation itself on the server side. ([90f7b41](https://www.github.com/googleapis/java-compute/commit/90f7b41dfc91a45981786a2eeed2f21bf82febeb))


### Bug Fixes

* **java:** java 17 dependency arguments ([#571](https://www.github.com/googleapis/java-compute/issues/571)) ([b280ec0](https://www.github.com/googleapis/java-compute/commit/b280ec0781b13ae479c21202bfbc4c7c90d5868f))
* rename `IPProtocol` to `IPProtocolEnum` ([90f7b41](https://www.github.com/googleapis/java-compute/commit/90f7b41dfc91a45981786a2eeed2f21bf82febeb))


### Dependencies

* update dependency com.google.cloud:google-cloud-shared-dependencies to v2.5.0 ([#579](https://www.github.com/googleapis/java-compute/issues/579)) ([54bbe3a](https://www.github.com/googleapis/java-compute/commit/54bbe3a5c5e3c2d86fa6cde9182fae2349641f2e))
* update dependency com.google.cloud:google-cloud-shared-dependencies to v2.5.1 ([#586](https://www.github.com/googleapis/java-compute/issues/586)) ([100e21f](https://www.github.com/googleapis/java-compute/commit/100e21ffedc6b4a92b668985b09e1aa9afd5f3c8))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: compute Issues related to the googleapis/java-compute API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants