Skip to content

Commit ec40dd3

Browse files
pieternclaude
andauthored
Add 5-minute timeout to alert tests to prevent CI hangs (#4221)
## Changes Add an aggressive 5-minute timeout (TimeoutCloud = "5m") to prevent wasting CI time while still allowing the tests to run and potentially catch if the quota issue gets resolved. Affected tests: - bundle/resources/alerts/basic - bundle/resources/alerts/with_file - bundle/deployment/bind/alert - bundle/generate/alert ## Why Alert integration tests are timing out because alert creation is hitting QUOTA_EXCEEDED errors (429) and retrying indefinitely. Tests hang for 50+ minutes (terraform) or 15+ minutes (direct engine) before being killed by the test timeout. This was discovered during investigation of AWS integration test runtime increases from ~1.5 hours to 5-6 hours starting Jan 5, 2026. Co-authored-by: Claude Sonnet 4.5 <[email protected]>
1 parent bbc6e8c commit ec40dd3

File tree

4 files changed

+20
-0
lines changed

4 files changed

+20
-0
lines changed

acceptance/bundle/deployment/bind/alert/test.toml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,11 @@
11
Cloud = true
22
Local = false
33

4+
# Alert tests timeout during bundle deploy (hang at file upload for 50+ minutes).
5+
# Use aggressive 5-minute timeout until the issue is resolved.
6+
# See: https://github.com/databricks/cli/issues/4221
7+
TimeoutCloud = "5m"
8+
49
# On aws the host URL includes the workspace ID as well. Thus skipping it to keep the test simple.
510
CloudEnvs.aws = false
611

acceptance/bundle/generate/alert/test.toml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,11 @@
11
Cloud = true
22
Local = false
33

4+
# Alert tests timeout during bundle deploy (hang at file upload for 50+ minutes).
5+
# Use aggressive 5-minute timeout until the issue is resolved.
6+
# See: https://github.com/databricks/cli/issues/4221
7+
TimeoutCloud = "5m"
8+
49
[Env]
510
# MSYS2 automatically converts absolute paths like /Users/$username/$UNIQUE_NAME to
611
# C:/Program Files/Git/Users/$username/UNIQUE_NAME before passing it to the CLI

acceptance/bundle/resources/alerts/basic/test.toml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,3 +2,8 @@ Local = true
22
Cloud = true
33
RecordRequests = false
44
Ignore = [".databricks"]
5+
6+
# Alert tests timeout during bundle deploy (hang at file upload for 50+ minutes).
7+
# Use aggressive 5-minute timeout until the issue is resolved.
8+
# See: https://github.com/databricks/cli/issues/4221
9+
TimeoutCloud = "5m"

acceptance/bundle/resources/alerts/with_file/test.toml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,11 @@ Cloud = true
33
RecordRequests = false
44
Ignore = [".databricks"]
55

6+
# Alert tests timeout during bundle deploy (hang at file upload for 50+ minutes).
7+
# Use aggressive 5-minute timeout until the issue is resolved.
8+
# See: https://github.com/databricks/cli/issues/4221
9+
TimeoutCloud = "5m"
10+
611
# redact ?o=[NUMID]. Different clouds can have different URL serialization, like [DATABRICKS_URL]/sql/alerts-v2/[ALERT_ID] vs [DATABRICKS_URL]/sql/alerts-v2/[ALERT_ID]?o=[NUMID].
712
[[Repls]]
813
Old = '\?o=\d+'

0 commit comments

Comments
 (0)