-
Notifications
You must be signed in to change notification settings - Fork 108
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
Add aws_lakeformation_* tables Closes #2397 #2417
Conversation
We need to add other tables for the Lakeformation service if required. |
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.
@ParthaI please take a look at the review comments. Thanks!!
Also should the tables be defined as aws_lake_formation_*
or aws_lakeformation_*
?
// Error: aws: operation error LakeFormation: ListPermissions, 1 validation error(s) found. | ||
// - missing required field, ListPermissionsInput.Resource.DataLocation.ResourceArn. | ||
// (SQLSTATE HV000 |
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.
Do we need these comments?
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.
It would be helpful to include the actual error in the comments, as it would provide a clear understanding of why the following logic was added and the specific issue it addresses.
@misraved, I have followed the naming convention used in Terraform. Here is the reference for your review: aws_lakeformation_resource. |
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.
PR Overview
This PR introduces three new Steampipe tables to query AWS Lake Formation permissions, resources, and LF-tags. The changes include documentation in markdown for each table, as well as new Go implementations that integrate with the AWS Lake Formation API.
Reviewed Changes
File | Description |
---|---|
docs/tables/aws_lakeformation_permission.md | Adds documentation for querying Lake Formation permissions |
docs/tables/aws_lakeformation_resource.md | Adds documentation for querying Lake Formation resources |
docs/tables/aws_lakeformation_tag.md | Adds documentation for querying Lake Formation LF-tags |
aws/table_aws_lakeformation_permission.go | Adds new table implementation for Lake Formation permissions |
aws/table_aws_lakeformation_tag.go | Adds new table implementation for Lake Formation LF-tags |
aws/table_aws_lakeformation_resource.go | Adds new table implementation for Lake Formation resources |
aws/plugin.go | Registers the new Lake Formation tables within the plugin |
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Hydrate: listLakeformationResources, | ||
Tags: map[string]string{"service": "lakeformation", "action": "ListResources"}, | ||
}, | ||
GetMatrixItemFunc: SupportedRegionMatrix(laekformationv1.EndpointsID), |
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.
There is a typo in the package alias 'laekformationv1'. It should be 'lakeformationv1' to ensure the correct endpoints are used.
GetMatrixItemFunc: SupportedRegionMatrix(laekformationv1.EndpointsID), | |
GetMatrixItemFunc: SupportedRegionMatrix(lakeformationv1.EndpointsID), |
Copilot is powered by AI, so mistakes are possible. Review output carefully before use.
Integration test logs
Logs
Example query results
Results