[denik]

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"

Dashboard: test acceptance.TestAccept/selftest/timeout; error 1044 (Incomplete output, 42 runs).

# 1044, task test (windows, direct)
Error:      	Not equal:
            	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.

[github-actions]

Commit: 2faf992f8 | 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 (3aa5c96)	latest (v1.23.0)
TestAccept/selftest/timeout	✅	✅	✅

[eng-dev-ecosystem-bot]

Integration test report

Commit: 2faf992

Run: 41436024514

	Env	✅​pass	🙈​skip	Time
✅​	aws linux-2core-8gb	281	16	5:30
✅​	aws-windows-latest-4core-16gb	283	14	3:50
✅​	azure linux-2core-8gb	280	16	5:22
✅​	azure-windows-latest-4core-16gb	282	14	3:49
✅​	gcp linux-2core-8gb	281	16	4:52
✅​	gcp-windows-latest-4core-16gb	283	14	3:49

Top 3 slowest tests (at least 2 minutes):

duration	env	testname
3:44	aws-windows-latest-4core-16gb	TestAccept
3:43	gcp-windows-latest-4core-16gb	TestAccept
3:39	azure-windows-latest-4core-16gb	TestAccept