Add service methods for immutable Compute resources and tests by mziccard · Pull Request #696 · googleapis/google-cloud-java · GitHub

This PR adds spi and service methods for manipulating the following resources:

DiskType
MachineType
License
Zone
Region

In c18730c integration tests are also added.

I think the coverage drop is the usual one we see when we add the spi layer that is only tested through ITs (which do not concur to compute the coverage).

Nice work! Done with first iteration. Any reason we don't have codacy analysis here?

@aozarov I addressed the comments in ff8feb5

Add support for filters: filter method to options + filter classes
Removed unnecessary fields + added missing ones
Removed redundant "throws" from method signatures
Add static method in DefaultComputeRpc to handle the NOT_FOUND case

I also had to add dedicated option classes for aggregated lists as fields option is not completely supported (see #711).

Looks good. Few minor comments. Feel free to merge after they are resolved.

- Add missing resource fields and remove non-needed ones - Add selector classes and selector options - Add dedicated option classes for aggregated options - Remove unnecessary "throws" from method signature - Add static function to handle 404 exceptions

Add service methods for immutable Compute resources and tests

…onfig to v1.2.5 (googleapis#696) * build(deps): update dependency com.google.cloud:google-cloud-shared-config to v1.2.5 * 🦉 Updates from OwlBot See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

Source-Link: googleapis/synthtool@7a220e2 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:6d4e3a15c62cfdcb823d60e16da7521e7c6fc00eba07c8ff12e4de9924a57d28

🤖 I have created a release *beep* *boop* --- ### Updating meta-information for bleeding-edge SNAPSHOT release. --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

Making CLIRR not required. The version bumps are now controlled by the Release Please and OwlBot. The CL authors create appropriate change description to control major version bumps.

🤖 I have created a release *beep* *boop* --- ## [1.11.0](googleapis/java-compute@v1.10.0...v1.11.0) (2022-06-30) ### Features * Update Compute Engine API to revision 20220526 ([googleapis#722](googleapis/java-compute#722)) ([b409524](googleapis/java-compute@b409524)) ### Bug Fixes * **compute:** remove proto3_optional from parent_id ([googleapis#712](googleapis/java-compute#712)) ([b409524](googleapis/java-compute@b409524)) * **compute:** replace missing REQUIRED for parent_id ([googleapis#711](googleapis/java-compute#711)) ([b409524](googleapis/java-compute@b409524)) * **compute:** revert proto3_optional, required removal on parent_id ([googleapis#714](googleapis/java-compute#714)) ([b409524](googleapis/java-compute@b409524)) ### Dependencies * update dependency com.google.cloud:google-cloud-shared-dependencies to v2.13.0 ([googleapis#697](googleapis/java-compute#697)) ([f708f6b](googleapis/java-compute@f708f6b)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

Source-Link: googleapis/synthtool@18d4e9b Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:f8374176dc59291f05dd3fec927a9da2cda687a9ef4de32e77f699a2be12ab45

…plugin to v3 (#696) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [org.apache.maven.plugins:maven-deploy-plugin](https://maven.apache.org/plugins/) | `2.8.2` -> `3.0.0` | [![age](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-deploy-plugin/3.0.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-deploy-plugin/3.0.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-deploy-plugin/3.0.0/compatibility-slim/2.8.2)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-deploy-plugin/3.0.0/confidence-slim/2.8.2)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, click this checkbox. --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-game-servers).

…-plugin to v3.4.1 (#696) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [org.apache.maven.plugins:maven-javadoc-plugin](https://maven.apache.org/plugins/) | `3.4.0` -> `3.4.1` | [![age](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-javadoc-plugin/3.4.1/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-javadoc-plugin/3.4.1/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-javadoc-plugin/3.4.1/compatibility-slim/3.4.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-javadoc-plugin/3.4.1/confidence-slim/3.4.0)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Renovate will not automatically rebase this PR, because other commits have been found. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, click this checkbox. ⚠ **Warning**: custom changes will be lost. --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-webrisk).

…1.1 (#696) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [com.google.cloud:libraries-bom](https://cloud.google.com/java/docs/bom) ([source](https://togithub.com/googleapis/java-cloud-bom)) | `26.1.0` -> `26.1.1` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.1/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.1/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.1/compatibility-slim/26.1.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.1/confidence-slim/26.1.0)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, click this checkbox. --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-billing).

…logback to v0.123.4-alpha (#696) [![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [com.google.cloud:google-cloud-logging-logback](https://togithub.com/googleapis/java-logging-logback) | `0.123.3-alpha` -> `0.123.4-alpha` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-logging-logback/0.123.4-alpha/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-logging-logback/0.123.4-alpha/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-logging-logback/0.123.4-alpha/compatibility-slim/0.123.3-alpha)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-logging-logback/0.123.4-alpha/confidence-slim/0.123.3-alpha)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Renovate will not automatically rebase this PR, because other commits have been found. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, click this checkbox. --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-logging-logback).

🤖 I have created a release *beep* *boop* --- ### Updating meta-information for bleeding-edge SNAPSHOT release. --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

….90 (#696)

…eapis#696)

….90 (#696)

…jdbc to v2.5.6 (googleapis#696)

* changes without context autosynth cannot find the source of changes triggered by earlier changes in this repository, or by version upgrades to tools such as linters. * chore(java): remove readme autosynth job config README generation is now handled by owlbot Source-Author: Jeff Ching <chingor@google.com> Source-Date: Wed Sep 8 12:11:28 2021 -0700 Source-Repo: googleapis/synthtool Source-Sha: df5093b950d4aafd49a4c7758d74c44485263ada Source-Link: googleapis/synthtool@df5093b * chore: remove readme synth.py config Source-Author: Jeff Ching <chingor@google.com> Source-Date: Wed Sep 8 13:23:41 2021 -0700 Source-Repo: googleapis/synthtool Source-Sha: 2d31a9243781b282202b4f76dc7bbc8b45803196 Source-Link: googleapis/synthtool@2d31a92 * chore: add gcf-owl-bot to list of trusted contributors Source-Author: Jeff Ching <chingor@google.com> Source-Date: Wed Sep 8 13:42:12 2021 -0700 Source-Repo: googleapis/synthtool Source-Sha: a6b97202771f89a4b76873d43ea9a07d7fc95f91 Source-Link: googleapis/synthtool@a6b9720 * chore(java): update shared-config and shared-dependencies version in pom template Source-Author: Neenu Shaji <Neenu1995@users.noreply.github.com> Source-Date: Wed Sep 8 17:28:48 2021 -0400 Source-Repo: googleapis/synthtool Source-Sha: 396d9b84a1e93880f5bf88b59ecd38a0a6dffc5e Source-Link: googleapis/synthtool@396d9b8

This PR was generated using Autosynth. 🌈 <details><summary>Log from Synthtool</summary> ``` 2021-06-07 19:02:35,177 synthtool [DEBUG] > Executing /root/.cache/synthtool/java-pubsub/.github/readme/synth.py. On branch autosynth-readme nothing to commit, working tree clean 2021-06-07 19:02:36,573 synthtool [DEBUG] > Wrote metadata to .github/readme/synth.metadata/synth.metadata. ``` </details> Full log will be available here: https://source.cloud.google.com/results/invocations/adbb05f8-f040-494d-881e-808a85e5c63e/targets - [ ] To automatically regenerate this PR, check this box. (May take up to 24 hours.)

… to v1.21.3 (googleapis#696)

mziccard added the api: compute Issues related to the Compute Engine API. label Mar 1, 2016

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 1, 2016

mziccard added 2 commits March 1, 2016 19:24

Add service methods for immutable Compute resources

8073b20

Add RemoteComputeHelper and integration tests

3af1497

mziccard force-pushed the compute branch from c18730c to 3af1497 Compare March 1, 2016 18:30

aozarov reviewed Mar 1, 2016
View reviewed changes

mziccard force-pushed the compute branch from 28ed6e8 to ff8feb5 Compare March 4, 2016 15:50

aozarov reviewed Mar 8, 2016
View reviewed changes

mziccard force-pushed the compute branch from ff8feb5 to 8ee83d8 Compare March 8, 2016 12:00

Refactor compute service

29f35f7

- Add missing resource fields and remove non-needed ones - Add selector classes and selector options - Add dedicated option classes for aggregated options - Remove unnecessary "throws" from method signature - Add static function to handle 404 exceptions

mziccard force-pushed the compute branch from 8ee83d8 to 29f35f7 Compare March 8, 2016 12:02

mziccard added a commit that referenced this pull request Mar 8, 2016

Merge pull request #696 from mziccard/compute

4e16b54

Add service methods for immutable Compute resources and tests

mziccard merged commit 4e16b54 into googleapis:compute-alpha Mar 8, 2016

meltsufin pushed a commit that referenced this pull request Dec 22, 2025

chore: change branch master to main in sync-repo-settings.yaml (#696 )

bf80799

chingor13 pushed a commit that referenced this pull request Feb 24, 2026

chore(deps): update dependency com.google.appengine:appengine to v1.9…

4cde5b2

….90 (#696)

suztomo pushed a commit to suztomo/google-cloud-java that referenced this pull request Mar 11, 2026

chore: change branch master to main in sync-repo-settings.yaml (googl…

12ec32a

…eapis#696)

chingor13 pushed a commit that referenced this pull request Mar 12, 2026

chore(deps): update dependency com.google.appengine:appengine to v1.9…

af8a706

….90 (#696)

suztomo pushed a commit to suztomo/google-cloud-java that referenced this pull request Mar 23, 2026

chore(deps): update dependency com.google.cloud:google-cloud-spanner-…

8236d38

…jdbc to v2.5.6 (googleapis#696)

benjaminp pushed a commit to benjaminp/google-cloud-java that referenced this pull request Jul 13, 2026

chore(deps): update dependency com.google.cloud:google-cloud-bigtable…

07b0415

… to v1.21.3 (googleapis#696)

+                    ImmutableList.Builder<DiskType> builder = ImmutableList.builder();
+                    Map<String, DiskTypesScopedList> scopedList = aggregatedList.getItems();
+                    if (scopedList != null) {
+                      for (String key : scopedList.keySet()) {

Add service methods for immutable Compute resources and tests by mziccard · Pull Request #696 · googleapis/google-cloud-java · GitHub

Uh oh!

Add service methods for immutable Compute resources and tests - #696

Add service methods for immutable Compute resources and tests#696
mziccard merged 3 commits into
googleapis:compute-alphafrom
mziccard:compute

mziccard commented Mar 1, 2016

Uh oh!

mziccard commented Mar 1, 2016

Uh oh!

This comment was marked as spam.

Uh oh!

aozarov commented Mar 4, 2016

Uh oh!

mziccard commented Mar 4, 2016

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

aozarov commented Mar 8, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

+                  private final ComparisonOperator operator;
+                  private final Object value;
+                  enum ComparisonOperator {

Add service methods for immutable Compute resources and tests by mziccard · Pull Request #696 · googleapis/google-cloud-java · GitHub

Uh oh!

Add service methods for immutable Compute resources and tests - #696

Add service methods for immutable Compute resources and tests#696mziccard merged 3 commits intogoogleapis:compute-alphagoogleapis/google-cloud-java:compute-alphafrom mziccard:computemziccard/gcloud-java:computeCopy head branch name to clipboard

Conversation

mziccard commented Mar 1, 2016

Uh oh!

mziccard commented Mar 1, 2016

Uh oh!

This comment was marked as spam.

Uh oh!

aozarov commented Mar 4, 2016

Uh oh!

mziccard commented Mar 4, 2016

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

aozarov commented Mar 8, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Add service methods for immutable Compute resources and tests#696
mziccard merged 3 commits into
googleapis:compute-alphafrom
mziccard:compute