-
Notifications
You must be signed in to change notification settings - Fork 40
feat: Added VPC Endpoint Services and Configurations #1029
feat: Added VPC Endpoint Services and Configurations #1029
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks really great! A few minor comments
@mlozoya2 This pr looks good, can you expand a little bit on the issues you faced with the AWS SDK? |
/test sha=66f9682cc9d8bdb04966fe10fe02e405488d6791 |
Same as cloudquery/cq-provider-azure#331 (comment). Update:Ended up ignoring specific columns in tests instead of creating a specific configuration for those |
/test sha=07eb9141e3621441a64041fd170e39deec1abed9 |
@bbernays Sure thing. In my first attempt, I had both of these new tables under
This produced the following error for almost all of the fetches...
We were able to pull some results into the tables, but far fewer than expected. For example, we only got 1 entry into the |
@mlozoya2 - Thank you for that very thorough explanation, I fully agree that your initial proposed heirarchy is definitely the ideal solution. Do you happen to have the code from that initial attempt? I want to make sure that there wasn't an issue on the CQ SDK side of things that blocked you from finding all of the results you expected. |
@bbernays This is what I had for
|
@mlozoya2 - That looks good and it would work on small accounts, but I think the issue you ran into was probably throttling at the SDK level for large accounts. As every single VPC Endpoint Service would require 3+ API calls to resolve. Great job working around that issue! |
🤖 I have created a release *beep* *boop* --- ## [0.12.12](v0.12.11...v0.12.12) (2022-06-15) ### Features * Add VPC Endpoint Services and Configurations ([#1029](#1029)) ([668ea91](668ea91)) ### Bug Fixes * **deps:** Update module github.com/cloudquery/cq-provider-sdk to v0.11.2 ([#1062](#1062)) ([5b2bc76](5b2bc76)) * **deps:** Update module github.com/cloudquery/cq-provider-sdk to v0.11.3 ([#1063](#1063)) ([b81b84c](b81b84c)) * Resolvers Returning Early ([#1059](#1059)) ([449aefc](449aefc)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
🤖 I have created a release *beep* *boop* --- ## [0.12.12](cloudquery/cq-provider-aws@v0.12.11...v0.12.12) (2022-06-15) ### Features * Add VPC Endpoint Services and Configurations ([#1029](cloudquery/cq-provider-aws#1029)) ([668ea91](cloudquery/cq-provider-aws@668ea91)) ### Bug Fixes * **deps:** Update module github.com/cloudquery/cq-provider-sdk to v0.11.2 ([#1062](cloudquery/cq-provider-aws#1062)) ([5b2bc76](cloudquery/cq-provider-aws@5b2bc76)) * **deps:** Update module github.com/cloudquery/cq-provider-sdk to v0.11.3 ([#1063](cloudquery/cq-provider-aws#1063)) ([b81b84c](cloudquery/cq-provider-aws@b81b84c)) * Resolvers Returning Early ([#1059](cloudquery/cq-provider-aws#1059)) ([449aefc](cloudquery/cq-provider-aws@449aefc)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
🎉 Thank you for making CloudQuery awesome by submitting a PR 🎉
Summary
This PR adds
Ec2VpcEndpointServices
andEc2VpcEndpointServiceConfigurations
resources.Ideally, these new resources would be added as relation tables under the
Ec2VpcEndpoint
resource, but there are issues with the AWS-sdk when fetching them using theServiceName
/ServiceID
parameters. The fetch throws errors related to invalid formats forServiceName
andServiceID
. The workaround was to omit the parameter and grab all of the resources.Use the following steps to ensure your PR is ready to be reviewed
go fmt
to format your code 🖊golangci-lint run
🚨 (install golangci-lint here)go run ./docs/docs.go
and committing the changes 📃