feat(bundle): cluster permissions from root bundle permissions #1859
Conversation
Mike-F-G
force-pushed
the
fix/cluster_permissions
branch
from
ab57b6c
to
faa7963
Compare
|
@Mike-F-G thanks for the PR! Just to clarify: you had issues when you defined top level bundle permissions but they were not working for clusters, is it correct? Because your PR adds support top level permissions but not really fixes the ones defined inside cluster config itself. Cluster permissions were fixed in this PR #1826 which was released last week in 0.231.0 version |
Mike-F-G
force-pushed
the
fix/cluster_permissions
branch
from
faa7963
to
6057cdc
Compare
Thank you for taking the time to consider this PR. Yes you are totally right, I was mistaken on the scope of the PR & the tests related to my change. My initial issue is indeed fixed with #1826. Therefore let met switch this PR to take into account the bundle level of permissions for the clusters. Aligning the initial modification with its usage. |
Mike-F-G
changed the title
Issue
Defining permissions at the root of the bundle with the recently added clusters as resources for all purpose clusters are not aligned with expected permissions.
Changes
Add permission conversion for the Databricks Clusters resources according to the cluster ACLs
Tests
Tests aligned with the expected definition