Azure / bicep-registry-modules

Bicep registry modules
MIT License
465 stars 308 forks source link

[AVM Module Issue]: Add support for rest of the Workspace properties. #3087

Closed iliyansi closed 2 days ago

iliyansi commented 3 weeks ago

Check for previous/existing GitHub issues

Issue Type?

Feature Request

Module Name

avm/res/databricks/workspace

(Optional) Module Version

No response

Description

When I try to deploy data bricks workspace and configure default catalog I don't see such parameter while on resource api there is defaultCatalog property of type DefaultPropertiesCatalog object which sets initialName and InitialType of that catalog. There is also missing parameter for enhancedSecurityCompliance property and authorization.

(Optional) Correlation Id

No response

microsoft-github-policy-service[bot] commented 3 weeks ago

[!IMPORTANT] The "Needs: Triage :mag:" label must be removed once the triage process is complete!

[!TIP] For additional guidance on how to triage this issue/PR, see the BRM Issue Triage documentation.

avm-team-linter[bot] commented 3 weeks ago

@iliyansi, thanks for submitting this issue for the avm/res/databricks/workspace module!

[!IMPORTANT] A member of the @Azure/avm-res-databricks-workspace-module-owners-bicep or @Azure/avm-res-databricks-workspace-module-contributors-bicep team will review it soon!

clintgrove commented 1 week ago

Hi. I am busy working on a fix for this. It will take about another week

clintgrove commented 2 days ago

hi @iliyansi I have completed some work on this. PR is now open. One thing to note is that you cannot currently set the catalog name, the Resource Provider does not currently allow it. The thing you can set is whether its a HiveMetastore or a UnityCatalog. See how I have implemented it in the MAX version of the databricks deployment