Skip to content

Commit

Permalink
Remove noisy azure_workspace_resource_id field not provided message…
Browse files Browse the repository at this point in the history
… for non-production Databricks deployments

JWT mismatched claims errors are only happening for dev and staging deployments of Databricks, adding the unnecessary noise to production users. `azure_workspace_resource_id` is not required for authentication to succeed.
  • Loading branch information
nfx committed Oct 17, 2023
1 parent 8ee6ff3 commit 11452d0
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion databricks/sdk/core.py
Original file line number Diff line number Diff line change
Expand Up @@ -280,7 +280,8 @@ def for_resource(cfg: 'Config', resource: str) -> 'AzureCliTokenSource':
return token
except OSError:
logger.warning("Failed to get token for subscription. Using resource only token.")
else:
elif cfg.effective_azure_login_app_id != ARM_DATABRICKS_RESOURCE_ID:
# the proper fix has to be done on Databricks Platform side
logger.warning(
"azure_workspace_resource_id field not provided. " +
"It is recommended to specify this field in the Databricks configuration to avoid authentication errors."
Expand Down

0 comments on commit 11452d0

Please sign in to comment.