Skip to content

Conversation

@asnare
Copy link
Contributor

@asnare asnare commented Nov 18, 2025

This PR updates the logging set up during handoff from the Databricks CLI so that TRACE-level logging doesn't trigger a crash. Because Python doesn't support TRACE as a logging level we map this to DEBUG.

In addition, rather than crash when an unknown log-level is encountered we now log a warning and proceed with INFO instead.

Resolves #317.

This has been tested manually with lakebridge.

@asnare asnare self-assigned this Nov 18, 2025
@asnare asnare added this to UCX Nov 18, 2025
@asnare asnare added the bug Something isn't working label Nov 18, 2025
@codecov
Copy link

codecov bot commented Nov 18, 2025

Codecov Report

❌ Patch coverage is 60.00000% with 6 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (main@ac509c1). Learn more about missing BASE report.

Files with missing lines Patch % Lines
src/databricks/labs/blueprint/cli.py 60.00% 4 Missing and 2 partials ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #318   +/-   ##
=======================================
  Coverage        ?   80.05%           
=======================================
  Files           ?       17           
  Lines           ?     2396           
  Branches        ?      516           
=======================================
  Hits            ?     1918           
  Misses          ?      354           
  Partials        ?      124           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@asnare asnare moved this to Ready for Review in UCX Nov 18, 2025
@asnare asnare marked this pull request as ready for review November 18, 2025 16:14
@asnare asnare requested a review from nfx as a code owner November 18, 2025 16:14
@github-actions
Copy link

✅ 40/40 passed, 2 skipped, 1m44s total

Running from acceptance #369

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Ready for Review

Development

Successfully merging this pull request may close these issues.

Databricks CLI integration fails with --log-level=trace

2 participants