From 60122f60358d1c60cae5e546382ec6eb9b675c7d Mon Sep 17 00:00:00 2001 From: "Lennart Kats (databricks)" Date: Tue, 23 Apr 2024 21:36:25 +0200 Subject: [PATCH] Show a better error message for using wheel tasks with older DBR versions (#1373) ## Changes This is a minor improvement to the error about wheel tasks with older DBR versions, since we get questions about it every now and then. It also adds a pointer to the docs that were added since the original messages was committed. --------- Co-authored-by: Pieter Noordhuis --- bundle/python/warning.go | 2 +- bundle/python/warning_test.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/bundle/python/warning.go b/bundle/python/warning.go index 59c220a0..3da88b0d 100644 --- a/bundle/python/warning.go +++ b/bundle/python/warning.go @@ -25,7 +25,7 @@ func (m *wrapperWarning) Apply(ctx context.Context, b *bundle.Bundle) diag.Diagn } if hasIncompatibleWheelTasks(ctx, b) { - return diag.Errorf("python wheel tasks with local libraries require compute with DBR 13.1+. Please change your cluster configuration or set experimental 'python_wheel_wrapper' setting to 'true'") + return diag.Errorf("Python wheel tasks require compute with DBR 13.3+ to include local libraries. Please change your cluster configuration or use the experimental 'python_wheel_wrapper' setting. See https://docs.databricks.com/dev-tools/bundles/python-wheel.html for more information.") } return nil } diff --git a/bundle/python/warning_test.go b/bundle/python/warning_test.go index 990545ab..dd6397f7 100644 --- a/bundle/python/warning_test.go +++ b/bundle/python/warning_test.go @@ -102,7 +102,7 @@ func TestIncompatibleWheelTasksWithJobClusterKey(t *testing.T) { require.True(t, hasIncompatibleWheelTasks(context.Background(), b)) diags := bundle.Apply(context.Background(), b, WrapperWarning()) - require.ErrorContains(t, diags.Error(), "python wheel tasks with local libraries require compute with DBR 13.1+.") + require.ErrorContains(t, diags.Error(), "require compute with DBR 13.3") } func TestIncompatibleWheelTasksWithExistingClusterId(t *testing.T) {