-
Notifications
You must be signed in to change notification settings - Fork 400
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 databricks_compliance_security_profile_workspace_setting resource #3564
Conversation
Hi @tanmay-db @alexott @nkvuong @hectorcast-db the test in this PR fails with the following error. Any idea what could be wrong? I already spent hours trying a bunch of stuff but I suspect I am not defining the
|
the issue is with
basically it tries to convert https://github.com/databricks/databricks-sdk-go/blob/27d08a67df5b0d35544c663f9d47577f85ffc6e4/service/settings/model.go#L199 to a Terraform resource and crashed. @edwardfeng-db you last worked on this function, any thoughts? |
Yeah it seems kind of strange to me that this is a |
@edwardfeng-db It is a ComplianceStandard because it is an
We do this for all enums, so I am not sure why would cause an issue now. |
Thanks @hectorcast-db, yeah this seems like a legit bug on the terraform side, I'll try to fix this |
@edwardfeng-db any ETA on the fix? This is blocking our public api launch which the customers have been asking for. |
@harshshah-db Hopefully today, still debugging |
Here's the fix - #3581 Verified it works with the failing test |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3564 +/- ##
==========================================
+ Coverage 82.57% 82.58% +0.01%
==========================================
Files 188 189 +1
Lines 19191 19207 +16
==========================================
+ Hits 15847 15863 +16
Misses 2414 2414
Partials 930 930
|
databricks_compliance_security_profile_workspace_setting
resource for ESC
databricks_compliance_security_profile_workspace_setting
resource for ESC### New Features and Improvements * Fix bug in collectionToMaps ([#3581](#3581)). * Added `databricks_enhanced_security_monitoring_workspace_setting ` resource for ESC (Enhanced Compliance and Security) settings ([#3563](#3563)). * Added `databricks_automatic_cluster_update_workspace_setting` resource ([#3444](#3444)). * Add databricks_compliance_security_profile_workspace_setting resource ([#3564](#3564)). * Make customizable error logs more readable ([#3583](#3583)). * Fix control run state failures for `databricks_job` resource ([#3585](#3585)). * Added support for popular column types for `resource_sql_table` ([#3528](#3528)). * Modify state upgrader to remove `ebs_volume_iops` set to zero ([#3601](#3601)). * Add route optimized option to model serving terraform ([#3572](#3572)). * Rename lakehouse monitor to quality monitor ([#3584](#3584)). ### Documentation Changes * Fix documentation for `databricks_storage_credential` and `databricks_external_location` data sources ([#3588](#3588)). ### Exporter * Add support for `databricks_mws_permission_assignment` resource ([#3562](#3562)). * Don't list directories in the incremental mode ([#3569](#3569)). ### Internal Changes * Add TestMwsAccServicePrincipalResourceOnAws to flaky tests ([#3580](#3580)). ### Dependency updates * Bump github.com/databricks/databricks-sdk-go from 0.40.1 to 0.41.0 ([#3604](#3604)). * Bump github.com/hashicorp/terraform-plugin-sdk/v2 from 2.33.0 to 2.34.0 ([#3594](#3594)).
* Release v1.45.0 ### New Features and Improvements * Fix bug in collectionToMaps ([#3581](#3581)). * Added `databricks_enhanced_security_monitoring_workspace_setting ` resource for ESC (Enhanced Compliance and Security) settings ([#3563](#3563)). * Added `databricks_automatic_cluster_update_workspace_setting` resource ([#3444](#3444)). * Add databricks_compliance_security_profile_workspace_setting resource ([#3564](#3564)). * Make customizable error logs more readable ([#3583](#3583)). * Fix control run state failures for `databricks_job` resource ([#3585](#3585)). * Added support for popular column types for `resource_sql_table` ([#3528](#3528)). * Modify state upgrader to remove `ebs_volume_iops` set to zero ([#3601](#3601)). * Add route optimized option to model serving terraform ([#3572](#3572)). * Rename lakehouse monitor to quality monitor ([#3584](#3584)). ### Documentation Changes * Fix documentation for `databricks_storage_credential` and `databricks_external_location` data sources ([#3588](#3588)). ### Exporter * Add support for `databricks_mws_permission_assignment` resource ([#3562](#3562)). * Don't list directories in the incremental mode ([#3569](#3569)). ### Internal Changes * Add TestMwsAccServicePrincipalResourceOnAws to flaky tests ([#3580](#3580)). ### Dependency updates * Bump github.com/databricks/databricks-sdk-go from 0.40.1 to 0.41.0 ([#3604](#3604)). * Bump github.com/hashicorp/terraform-plugin-sdk/v2 from 2.33.0 to 2.34.0 ([#3594](#3594)). * fixes
Changes
Tests
make test
run locallydocs/
folderinternal/acceptance