Commit bfdb9de3 authored Apr 01, 2021 by Michal Berger Committed by Jim Harris Apr 14, 2021

test/vhost: Extend fio runtime during migration tests



Migration can take longer than 10s, hence we need to give fio more
time to spin in order to make sure it's still running on the target
vm after migration is complete.

On the other hand, reduce the timeout used to wait for fio after
the migration is done - there's no need to wait almost a minute,
half that time.

Signed-off-by: Michal Berger <michalx.berger@intel.com>
Change-Id: I1b53682d5c8bc72336a9eead28c395f97cd81fe7
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/7199


Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: Karol Latecki <karol.latecki@intel.com>

parent c2e0da84

test/vhost/migration/migration-tc1.job

+1 −1

Original line number	Diff line number	Diff line
		@@ -20,6 +20,6 @@ stonewall

		[randread]
		rw=randread
		runtime=10
		runtime=20
		time_based
		stonewall

test/vhost/migration/migration-tc1.sh

+1 −1

Original line number	Diff line number	Diff line
		@@ -93,7 +93,7 @@ function migration_tc1() {
		fi

		notice "Waiting for fio to finish"
		local timeout=40
		local timeout=20
		while is_fio_running $target_vm; do
		sleep 1
		echo -n "."

test/vhost/migration/migration-tc2.job

+1 −1

Original line number	Diff line number	Diff line
		@@ -15,6 +15,6 @@ verify_backlog=8

		[randwrite]
		rw=randwrite
		runtime=15
		runtime=20
		time_based
		stonewall

test/vhost/migration/migration-tc2.sh

+1 −1

Original line number	Diff line number	Diff line
		@@ -186,7 +186,7 @@ function migration_tc2() {
		fi

		notice "Waiting for fio to finish"
		local timeout=40
		local timeout=20
		while is_fio_running $target_vm; do
		sleep 1
		echo -n "."