Skip to content

Commit 1319dda

Browse files
ai: apply changes for #919 (1 review thread)
Addresses: - #3828445075 at src/databricks/sql/backend/kernel/auth_bridge.py:285 Signed-off-by: peco-engineer-bot[bot] <peco-engineer-bot[bot]@users.noreply.github.com>
1 parent e0858b5 commit 1319dda

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

src/databricks/sql/backend/kernel/auth_bridge.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -282,7 +282,7 @@ def kernel_auth_kwargs(
282282
if opts.get(name) is not None
283283
]
284284
if ignored_signals:
285-
logger.debug(
285+
logger.warning(
286286
"auth_type='azure-sp-m2m' selected; ignoring conflicting "
287287
"credential signal(s) %s (Azure SP credentials in the azure_* "
288288
"namespace take precedence on the kernel path).",

tests/unit/test_kernel_auth_bridge.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -748,7 +748,7 @@ def test_azure_sp_m2m_ignores_conflicting_oauth_signal(self, conflicting_signal)
748748
# signal as a hard "Ambiguous auth" error, but an explicit azure-sp-m2m
749749
# selector carries its creds in the azure_* namespace, so a stray
750750
# oauth_*/credentials_provider value is NOT a routing collision — it is
751-
# silently ignored (logger.debug breadcrumb only) and the Azure SP flow
751+
# silently ignored (logger.warning breadcrumb only) and the Azure SP flow
752752
# still wins. This guards against a refactor accidentally promoting the
753753
# ignored signal to an error (see the routing note in auth_bridge.py).
754754
opts = dict(self._CREDS, **conflicting_signal)

0 commit comments

Comments
 (0)