-
Notifications
You must be signed in to change notification settings - Fork 393
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
[ISSUE] databricks_permissions fails with Provider produced inconsistent result when provided upgraded to 0.3.0 #486
Comments
@nirav24 https://registry.terraform.io/providers/databrickslabs/databricks/latest/docs/resources/permissions
i'll add validation to prevent |
Thanks @nfx for quick reply. I am trying to apply policies which will restrict VM types can be used even for admins. Do you think, that policy will be surpassed as well? |
@nirav24 it's not possible to restrict anything for admins :) just remove people from admins group. |
I can raise a PR if you wish |
@nirav24 i would appreciate, review and merge it! :) |
I was running below code with Databricks provider
0.2.4
and it was working fine. I am currently looking to upgrade provider to latest version0.3.0
and terraform apply is failing with below error.Terraform Version
Run
terraform -v
to show the version. If you are not running the latest version of Terraform, please upgrade because your issue may have already been fixed.Affected Resource(s)
Please list the resources as a list, for example:
Environment variable names
To get relevant environment variable names please copypaste the output of the following command:
$ env | sort | grep -E 'DATABRICKS|AWS|AZURE|ARM|TEST' | awk -F= '{print $1}'
Terraform Configuration Files
Debug Output
Expected Behavior
I think, it should be able to assign policy to admins group. (I am not sure if Databricks doesn't allow to update admins group)
Actual Behavior
Steps to Reproduce
Please list the steps required to reproduce the issue, for example:
terraform apply
Important Factoids
Are there anything atypical about your accounts that we should know?
The text was updated successfully, but these errors were encountered: