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

[FIX] Ability to set all Databricks cluster types using a class/enum #184

Open
2 of 4 tasks
zarembat opened this issue Dec 2, 2024 · 1 comment
Open
2 of 4 tasks
Assignees
Labels
bug Something isn't working enhancement New feature or request
Milestone

Comments

@zarembat
Copy link

zarembat commented Dec 2, 2024

Is your feature request related to a problem? Please describe.
Currently compute clusters are deployed as single-user clusters by default. This can be overwritten by setting the data_security_mode attribute for the Cluster object but the DataSecurityMode class which is used for that only has one property: SINGLE_USER. In order to configure a different cluster type one needs to know which value to hard code (eg. "USER_ISOLATION" for a Shared cluster) which is not convenient.

Cloud Information

  • AWS
  • Azure
  • GCP
  • Other

Describe the solution you'd like
Add other attributes to the DataSecurityMode class so that it can be used to configure all cluster types. For example:

@dataclass(frozen=True)
class DataSecurityMode:
    SINGLE_USER: str = "SINGLE_USER"
    SHARED: str = "USER_ISOLATION"
    NO_ISOLATION_SHARED: str = "XXX"

Describe alternatives you've considered
One can simply hard code the right value (like "USER_ISOLATION") directly when instantiating a Cluster.

@zarembat zarembat added the enhancement New feature or request label Dec 2, 2024
@riccamini
Copy link
Contributor

Patch release 1.3

@pariksheet pariksheet added the bug Something isn't working label Jan 14, 2025
@pariksheet pariksheet modified the milestones: v1.3.3, v1.3.1 Jan 14, 2025
@pariksheet pariksheet changed the title [FEATURE] Ability to set all Databricks cluster types using a class/enum [FIX] Ability to set all Databricks cluster types using a class/enum Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants