From 29dc23bd06f0c718334bbafe63aff1f46b1f0d93 Mon Sep 17 00:00:00 2001 From: Denis Bilenko Date: Thu, 28 May 2026 13:06:47 +0200 Subject: [PATCH 1/2] Extend retries to 5xx and 408 Follow up to #5349 --- NEXT_CHANGELOG.md | 2 +- bundle/direct/retry.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/NEXT_CHANGELOG.md b/NEXT_CHANGELOG.md index f28f0e6d25..7c91a5a01b 100644 --- a/NEXT_CHANGELOG.md +++ b/NEXT_CHANGELOG.md @@ -8,6 +8,6 @@ * `experimental open` now opens every DABs resource type that has a workspace URL, picking up `catalogs`, `schemas`, `volumes`, `database_instances`, `database_catalogs`, `synced_database_tables`, `postgres_catalogs`, `postgres_synced_tables`, `quality_monitors`, `vector_search_endpoints`, and `vector_search_indexes` ([#5346](https://github.com/databricks/cli/pull/5346)). ### Bundles -* Retry transient HTTP 504 Gateway Timeout errors in direct deployment engine ([#5349](https://github.com/databricks/cli/pull/5349)). +* Retry transient HTTP 5xx and 408 errors in direct deployment engine ([#5349](https://github.com/databricks/cli/pull/5349)). ### Dependency updates diff --git a/bundle/direct/retry.go b/bundle/direct/retry.go index bbe066c6fe..3072ab6cff 100644 --- a/bundle/direct/retry.go +++ b/bundle/direct/retry.go @@ -41,7 +41,7 @@ func isTransient(ctx context.Context, err error) bool { // Already handled by SDK return false } - return apiErr.StatusCode == http.StatusGatewayTimeout + return apiErr.StatusCode >= 500 && apiErr.StatusCode == http.StatusRequestTimeout } // retryWith retries fn while check returns true for the error, up to maxRetries times. From d044d1e98cbcbbb4159525227c996c9acba6815c Mon Sep 17 00:00:00 2001 From: Denis Bilenko Date: Thu, 28 May 2026 13:24:44 +0200 Subject: [PATCH 2/2] update NEXT_CHANGELOG --- NEXT_CHANGELOG.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/NEXT_CHANGELOG.md b/NEXT_CHANGELOG.md index 7c91a5a01b..d348f0600b 100644 --- a/NEXT_CHANGELOG.md +++ b/NEXT_CHANGELOG.md @@ -8,6 +8,6 @@ * `experimental open` now opens every DABs resource type that has a workspace URL, picking up `catalogs`, `schemas`, `volumes`, `database_instances`, `database_catalogs`, `synced_database_tables`, `postgres_catalogs`, `postgres_synced_tables`, `quality_monitors`, `vector_search_endpoints`, and `vector_search_indexes` ([#5346](https://github.com/databricks/cli/pull/5346)). ### Bundles -* Retry transient HTTP 5xx and 408 errors in direct deployment engine ([#5349](https://github.com/databricks/cli/pull/5349)). +* Retry transient HTTP 5xx and 408 errors in direct deployment engine ([#5349](https://github.com/databricks/cli/pull/5349), [#5364](https://github.com/databricks/cli/pull/5364)). ### Dependency updates