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

feat: Create databricks catalog ext loc modules #614

Merged

Conversation

alldoami
Copy link
Contributor

@alldoami alldoami commented Aug 14, 2024

Summary

This creates a bucket for databricks catalogs to use. It sets up the iam permissions that allows databricks access to read/write to the bucket. In order to associate a databricks catalog to a S3 bucket, it needs a storage credential which is connected to an external location (which is the S3 bucket).

The databricks-catalog module creates a new catalog associated to the bucket with the desired permissions.

Test Plan

Tested modules in shared-infra. Will be removing from shared-infra after merging and releasing.

@alldoami alldoami requested a review from a team as a code owner August 14, 2024 21:31
@alldoami alldoami changed the title feat: Create databricks catalog ext loc feat: Create databricks catalog ext loc modules Aug 14, 2024
@alldoami alldoami requested a review from jayengee August 14, 2024 21:31
@alldoami alldoami requested review from jayengee and removed request for alexlokshin-czi August 15, 2024 20:09
}
databricks = {
source = "databricks/databricks"
version = "1.49.1"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

required for the isolation_mode attribute for databricks_external_location

Copy link
Contributor

@jayengee jayengee left a comment

Choose a reason for hiding this comment

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

Main concern here is sharing the same storage root between multiple catalogs, and around support for multiple external locations per databricks workspace

Other than that, this looks good - let's try it out!

databricks-catalog-external-location/catalogs.tf Outdated Show resolved Hide resolved
databricks-catalog-external-location/main.tf Show resolved Hide resolved
databricks-catalog-external-location/main.tf Show resolved Hide resolved
databricks-catalog-external-location/main.tf Show resolved Hide resolved
@alldoami alldoami requested a review from jayengee August 16, 2024 03:22
@alldoami alldoami merged commit 8b47a1c into main Aug 16, 2024
105 checks passed
@alldoami alldoami deleted the CDI-3232-create-module-for-catalog-bucket-and-dbx-access branch August 16, 2024 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants