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

fix: Use ParentResourceFieldResolver instead of ParentIDResolver #2125

Merged
merged 2 commits into from
Sep 29, 2022

Conversation

erezrokah
Copy link
Contributor

Summary

Fixes Azure plugin to get the parent ID from the id column of the parent instead of using the CQ ID of the parent

@cq-bot cq-bot added the azure label Sep 28, 2022
@erezrokah erezrokah added the automerge Automatically merge once required checks pass label Sep 28, 2022
@kodiakhq kodiakhq bot merged commit 5039788 into cloudquery:main Sep 29, 2022
kodiakhq bot pushed a commit that referenced this pull request Oct 2, 2022
🤖 I have created a release *beep* *boop*
---


## [1.0.1-pre.0](plugins-source-azure-v1.0.0-pre.0...plugins-source-azure-v1.0.1-pre.0) (2022-10-02)


### Bug Fixes

* **deps:** Update golang.org/x/exp digest to 540bb73 ([#2169](#2169)) ([d183fea](d183fea))
* **deps:** Update module github.com/Azure/azure-sdk-for-go/sdk/azcore to v1.1.3 ([#2173](#2173)) ([d707d60](d707d60))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v0.10.2 ([#2048](#2048)) ([e407991](e407991))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v0.11.0 ([#2135](#2135)) ([1729467](1729467))
* Use ParentResourceFieldResolver instead of ParentIDResolver ([#2125](#2125)) ([5039788](5039788))
* Use TypeString for parent ids ([#2136](#2136)) ([a62f1a0](a62f1a0))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
@erezrokah erezrokah deleted the fix/azure_parent_id branch December 15, 2023 10:13
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.

None yet

3 participants