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

[BUG]: create-uber-principal command failed when updating the SQL warehouse config #2405

Closed
1 task done
qziyuan opened this issue Aug 9, 2024 · 1 comment · Fixed by #2409
Closed
1 task done
Assignees

Comments

@qziyuan
Copy link
Contributor

qziyuan commented Aug 9, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

  1. create-uber-principal will fail with ERROR [d.l.ucx.create-uber-principal] InvalidParameterValue: Endpoint security policy is required and must be one of NONE, DATA_ACCESS_CONTROL, PASSTHROUGH. when updating the SQL warehouse config to add the uber principal to data_access_config.
    This can be fixed by adding security_policy to the set_workspace_warehouse_config api call

  2. Furthermore, the same api call is still failing, after above is fixed, with this error: databricks.sdk.errors.platform.InvalidParameterValue: enable_serverless_compute is required..
    For a workspace with serverless enabled, the undocumented enable_serverless_compute field must be provided in the api request body, but python SDK does not support this field.
    UCX may have to call the REST API directly instead of using python SDK, if the SDK cannot support this field in the near future.

Expected Behavior

No response

Steps To Reproduce

No response

Cloud

Azure

Operating System

macOS

Version

latest via Databricks CLI

Relevant log output

No response

@qziyuan qziyuan changed the title [BUG]: create-uber-principal command failed when when the SQL warehouse config [BUG]: create-uber-principal command failed when updating the SQL warehouse config Aug 12, 2024
@qziyuan
Copy link
Contributor Author

qziyuan commented Aug 13, 2024

  1. PR Fix missing security_policy when updating SQL warehouse config #2409 will fix 1st Endpoint security policy is required and must be one of NONE, DATA_ACCESS_CONTROL, PASSTHROUGH. error.
  2. The 2nd enable_serverless_compute is required. error is pending upstream issue [ISSUE] Missing required enable_serverless_compute parameter databricks/databricks-sdk-py#305. PR Fix missing security_policy when updating SQL warehouse config #2409 will provide a workaround by printing the data access config in logs which can be then manually applied into the admin UI.

@qziyuan qziyuan self-assigned this Aug 13, 2024
@nfx nfx linked a pull request Aug 13, 2024 that will close this issue
nfx pushed a commit that referenced this issue Aug 14, 2024
1. Fix `InvalidParameterValue: Endpoint security policy is required and
must be one of NONE, DATA_ACCESS_CONTROL, PASSTHROUGH.` error in #2405
2. Print new SQL warehouse data access config in the log when
`databricks.sdk.errors.platform.InvalidParameterValue:
enable_serverless_compute is required.` error in #2405 occurs. With that
users can manually copy paste the config in UI to finish setting up the
uber principal,
@nfx nfx closed this as completed in #2409 Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

1 participant