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

Add customize diff for databricks_grant and databricks_grants for case insensitivity & spaces in grants #3657

Merged
merged 12 commits into from
Jun 7, 2024

Conversation

nkvuong
Copy link
Contributor

@nkvuong nkvuong commented Jun 6, 2024

Changes

Tests

  • make test run locally
  • covered with integration tests in internal/acceptance
  • relevant acceptance tests are passing
  • using Go SDK

@nkvuong nkvuong requested review from a team as code owners June 6, 2024 22:19
@nkvuong nkvuong requested review from hectorcast-db and removed request for a team June 6, 2024 22:19
@codecov-commenter
Copy link

codecov-commenter commented Jun 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.60%. Comparing base (bb5ffcb) to head (0759a41).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3657      +/-   ##
==========================================
+ Coverage   81.59%   81.60%   +0.01%     
==========================================
  Files         196      196              
  Lines       19719    19735      +16     
==========================================
+ Hits        16089    16105      +16     
  Misses       2671     2671              
  Partials      959      959              
Files Coverage Δ
catalog/permissions/permissions.go 96.77% <100.00%> (+0.10%) ⬆️
catalog/resource_grant.go 82.69% <100.00%> (+0.33%) ⬆️
catalog/resource_grants.go 86.66% <100.00%> (+1.05%) ⬆️

@nkvuong nkvuong changed the title Relaxed principal checks to be case insensitive for databricks_grant and databricks_grants Relaxed checks for databricks_grant and databricks_grants to allow successful apply Jun 7, 2024
@nkvuong nkvuong changed the title Relaxed checks for databricks_grant and databricks_grants to allow successful apply Add customize diff for databricks_grant and databricks_grants for case insensitivity & spaces in grants Jun 7, 2024
@nkvuong
Copy link
Contributor Author

nkvuong commented Jun 7, 2024

integration tests passed 🎉

@nkvuong nkvuong enabled auto-merge June 7, 2024 12:43
@nkvuong nkvuong added this pull request to the merge queue Jun 7, 2024
Merged via the queue into main with commit 22732d1 Jun 7, 2024
5 checks passed
@nkvuong nkvuong deleted the fix/grant_case_sensitive branch June 7, 2024 12:47
hectorcast-db added a commit that referenced this pull request Jun 25, 2024
### New Features and Improvements
* Add customize diff for `databricks_grant` and `databricks_grants` for case insensitivity & spaces in grants ([#3657](#3657)).
* Fix detection of local file changes in `databricks_file` ([#3662](#3662)).
* Apply all cluster validations to jobs cluster references ([#3651](#3651)).
* Fixed: Issue with `databricks_cluster` resource using `exporter` does not include cluster libraries ([#3674](#3674)).
* Relaxed cluster check for `databricks_sql_permissions` ([#3683](#3683)).
* Update jobs library fields from set to list ([#3669](#3669)).
* Added `CAN_MONITOR` permission to SQL warehouses in `databricks_permissions` ([#3681](#3681)).

### Documentation Changes
 * Fix errors in Unity Catalog data sources documentation ([#3656](#3656)).
 * Rename default_namespace_setting documentation to correct name ([#3682](#3682)).
 * Fix private DNS zone resource name in ADB PL Simplified guide ([#3664](#3664)).
 * Added links to Terraform modules and removed mention of E2 arch ([#3667](#3667)).

### Internal Changes
 * Update CodeQL actions to v3 ([#3648](#3648)).
 * Ensure jobs tests check for error ([#3666](#3666)).

### Dependency updates
 * Update to use Go 1.22 ([#3647](#3647)).
 * Bump github.com/databricks/databricks-sdk-go from 0.41.0 to 0.42.0 ([#3638](#3638)).
 * Bump github.com/databricks/databricks-sdk-go from 0.42.0 to 0.43.0 ([#3697](#3697)).
@hectorcast-db hectorcast-db mentioned this pull request Jun 25, 2024
github-merge-queue bot pushed a commit that referenced this pull request Jun 25, 2024
### New Features and Improvements
* Add customize diff for `databricks_grant` and `databricks_grants` for case insensitivity & spaces in grants ([#3657](#3657)).
* Fix detection of local file changes in `databricks_file` ([#3662](#3662)).
* Apply all cluster validations to jobs cluster references ([#3651](#3651)).
* Fixed: Issue with `databricks_cluster` resource using `exporter` does not include cluster libraries ([#3674](#3674)).
* Relaxed cluster check for `databricks_sql_permissions` ([#3683](#3683)).
* Update jobs library fields from set to list ([#3669](#3669)).
* Added `CAN_MONITOR` permission to SQL warehouses in `databricks_permissions` ([#3681](#3681)).

### Documentation Changes
 * Fix errors in Unity Catalog data sources documentation ([#3656](#3656)).
 * Rename default_namespace_setting documentation to correct name ([#3682](#3682)).
 * Fix private DNS zone resource name in ADB PL Simplified guide ([#3664](#3664)).
 * Added links to Terraform modules and removed mention of E2 arch ([#3667](#3667)).

### Internal Changes
 * Update CodeQL actions to v3 ([#3648](#3648)).
 * Ensure jobs tests check for error ([#3666](#3666)).

### Dependency updates
 * Update to use Go 1.22 ([#3647](#3647)).
 * Bump github.com/databricks/databricks-sdk-go from 0.41.0 to 0.42.0 ([#3638](#3638)).
 * Bump github.com/databricks/databricks-sdk-go from 0.42.0 to 0.43.0 ([#3697](#3697)).
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.

[ISSUE] Case sensitive Issue with databricks_grants on external_location resource
4 participants