-
Notifications
You must be signed in to change notification settings - Fork 158
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
explicitly set data security mode to SINGLE_USER #179
explicitly set data security mode to SINGLE_USER #179
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @shichengzhou-db ! Would you mind also fixing the tests that are failing?
@@ -3,6 +3,7 @@ new_cluster: &new_cluster | |||
num_workers: 3 | |||
spark_version: 15.3.x-cpu-ml-scala2.12 | |||
node_type_id: {{template `cloud_specific_node_type_id` .}} | |||
data_security_mode: "SINGLE_USER" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we confirm these workflows still run on AWS, Azure, and GCP?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
confirmed this works on dogfood(aws), will test the other in a bit
3badd2a
to
9b6faa2
Compare
9b6faa2
to
037b706
Compare
@arpitjasa-db (Will test the other workflows on all clouds as well tmmrw and document in the PR description) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @shichengzhou-db !
Title.
Manual tested all workflows on all clouds (aws/azure/gcp), no failures related to data_security_mode. Although the workflows seems to be broken for various issues that need to be looked into separately.