Skip to content

feat(gcp-resources): Add vision#6786

Merged
kodiakhq[bot] merged 2 commits intocloudquery:mainfrom
shimonp21:feat_gcp_vision
Jan 13, 2023
Merged

feat(gcp-resources): Add vision#6786
kodiakhq[bot] merged 2 commits intocloudquery:mainfrom
shimonp21:feat_gcp_vision

Conversation

@shimonp21
Copy link
Copy Markdown
Contributor

Caveats:

  • Generated only most-imporant/top-level tables (skipped productsets).
  • no programatic way to get list of supported locations. I copy-pasted them from the APIs error message:
2:56PM ERR table resolver finished with error error="rpc error: code = InvalidArgument desc = Location not supported: 'europe-west3'. Must be one of 'us-west1', 'us-east1', 'asia-east1', 'europe-west1'" client=cq-playground module=gcp-src table=gcp_vision_product_sets

@github-actions
Copy link
Copy Markdown

This PR has the following changes to source plugin(s) tables:

  • Table gcp_vision_product_reference_images was added
  • Table gcp_vision_products was added

Copy link
Copy Markdown
Member

@erezrokah erezrokah left a comment

Choose a reason for hiding this comment

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

Looks great, we can remove one code comment I think. I like the usage of a multiplexer here and will port

once this is merged

@shimonp21 shimonp21 added the automerge Automatically merge once required checks pass label Jan 13, 2023
@kodiakhq kodiakhq bot merged commit 3dd6116 into cloudquery:main Jan 13, 2023
kodiakhq bot pushed a commit that referenced this pull request Jan 15, 2023

#### Summary

Follow up to #6786. We should add the location to the client ID as it has to be unique

<!--
erezrokah pushed a commit that referenced this pull request Jan 17, 2023
🤖 I have created a release *beep* *boop*
---


##
[7.2.0](plugins-source-gcp-v7.1.1...plugins-source-gcp-v7.2.0)
(2023-01-17)


### Features

* Add GCP vmmigration
([#6760](#6760))
([28d9825](28d9825))
* ADD GCP VPC Access
([#6755](#6755))
([59c32dc](59c32dc))
* Add GCP Web Security Scanner
([#6674](#6674))
([9b08d0e](9b08d0e))
* **gcp-resources:** Add Cloud Scheduler
([#6770](#6770))
([b0941af](b0941af))
* **gcp-resources:** Add Cloud Support
([#6771](#6771))
([7aefc2b](7aefc2b))
* **gcp-resources:** Add livestream
([#6788](#6788))
([6da9658](6da9658))
* **gcp-resources:** Add missing KMS resources
([#6769](#6769))
([9ecc4d7](9ecc4d7))
* **gcp-resources:** Add translate
([#6832](#6832))
([d827029](d827029))
* **gcp-resources:** Add video-transcoder
([#6806](#6806))
([d4fd41d](d4fd41d))
* **gcp-resources:** Add vision
([#6786](#6786))
([3dd6116](3dd6116))


### Bug Fixes

* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.24.2
([#6695](#6695))
([694ab9f](694ab9f))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.25.0
([#6745](#6745))
([9c41854](9c41854))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.25.1
([#6805](#6805))
([9da0ce2](9da0ce2))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.26.0
([#6839](#6839))
([6ccda8d](6ccda8d))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.27.0
([#6856](#6856))
([545799b](545799b))
* **gcp:** Add `Location` to client `ID` when present
([#6807](#6807))
([8b21b80](8b21b80))
* **gcp:** Prevent gcp_resourcemanager_folders rows with nil org_id
([#6577](#6577))
([266c191](266c191))

---
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

automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants