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

Ability to create dynamic passthrough clusters based on members of the group #714

Merged
merged 4 commits into from
Jul 8, 2021

Conversation

nfx
Copy link
Contributor

@nfx nfx commented Jul 6, 2021

Fix #702 Add id attribute to user data resource

  • error out, when permissions are attempted to be added for the current user, as it's not possible to decrease administrative privileges
  • added a guide for dynamic single-user passthrough clusters

* error out, when permissions are attempted to be added for the current user, as it's not possible to decrease administrative privileges
* added a guide for dynamic single-user passthrough clusters
@nfx nfx added this to the v0.3.6 milestone Jul 6, 2021
@nfx nfx requested review from pietern and alexott July 6, 2021 12:30
@codecov
Copy link

codecov bot commented Jul 6, 2021

Codecov Report

Merging #714 (d04c36d) into master (0328535) will increase coverage by 0.04%.
The diff coverage is 93.93%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #714      +/-   ##
==========================================
+ Coverage   83.44%   83.49%   +0.04%     
==========================================
  Files          87       87              
  Lines        7991     8015      +24     
==========================================
+ Hits         6668     6692      +24     
  Misses        842      842              
  Partials      481      481              
Impacted Files Coverage Δ
access/resource_permissions.go 71.02% <60.00%> (-1.23%) ⬇️
identity/data_user.go 95.91% <100.00%> (+9.25%) ⬆️
identity/users.go 93.54% <0.00%> (+3.22%) ⬆️

Copy link
Contributor

@alexott alexott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm


# Dynamic Passthrough Clusters

This example addresses a pretty common use-case: data science team, which is managed as a group through SCIM provisioning, needs a collection of individual passthrough [databricks_cluster](../resources/cluster.md), which they should be able to restart. It could simply be achieved by [databricks_group](../data-sources/group.md) and [databricks_user](../data-sources/user.md) data resources to get the list of user names, that belong to a group. Terraform's `for_each` meta-attribute helps to do this easily.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

names, that -> names that

@nfx nfx merged commit f17e41f into master Jul 8, 2021
@nfx nfx deleted the issue/702 branch July 8, 2021 17:10
@nfx nfx mentioned this pull request Jul 14, 2021
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.

Filter user data source by user id, not just by user name
2 participants