Skip to content

acc: fix flaky selftest/timeout on Windows - #7255

Open
denik wants to merge 1 commit into
mainfrom
denik/acc-selftest-timeout-2s
Open

denik wants to merge 1 commit into
mainfrom
denik/acc-selftest-timeout-2s

Conversation

@denik

@denik denik commented Oct 29, 2026

Copy link
Copy Markdown
Contributor

Changes

Raise the timeout in acceptance/selftest/timeout from 500ms to 2s. The script still sleeps 4s, so it still checks that the script is killed.

Why

On a slow Windows runner bash did not reach trace sleep 4 within 500ms, so the expected >>> sleep 4 line was missing:

expected: "\n>>> sleep 4\n\nError: Test script killed due to a timeout (500ms)\n"
actual  : "\nError: Test script killed due to a timeout (500ms)\n"

Tests

Regenerated; passes locally.

This pull request and its description were written by Isaac.

Co-authored-by: Isaac <no-reply@databricks.com>
@github-actions

Copy link
Copy Markdown
Contributor

Commit: 471ac187c | Workflow run

Regression test report

Selected 1 changed tests (limit=20, 0 not selected)

1 tests: regression=0 unreleased=0 coverage=1 failing=0

test branch main (aed6702) latest (v1.23.0)
TestAccept/selftest/timeout ✅ ✅ ✅

@eng-dev-ecosystem-bot

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: 471ac18

Run: 40918332260

Env ✅​pass 🙈​skip Time
✅​ aws linux-2core-8gb 281 16 5:19
✅​ aws-windows-latest-4core-16gb 283 14 5:12
✅​ azure linux-2core-8gb 280 16 5:16
✅​ azure-windows-latest-4core-16gb 282 14 3:24
✅​ gcp linux-2core-8gb 281 16 5:24
✅​ gcp-windows-latest-4core-16gb 283 14 5:01
Top 6 slowest tests (at least 2 minutes):
duration env testname
5:03 aws-windows-latest-4core-16gb TestAccept
4:54 gcp-windows-latest-4core-16gb TestAccept
3:34 aws linux-2core-8gb TestAccept
3:33 gcp linux-2core-8gb TestAccept
3:30 azure linux-2core-8gb TestAccept
3:20 azure-windows-latest-4core-16gb TestAccept

@denik
denik marked this pull request as ready for review October 29, 2026 17:53
@denik
denik requested a review from a team as a code owner October 29, 2026 17:53
@denik
denik requested a review from rclarey October 29, 2026 17:53

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants