Commit a8a53c7c authored Jul 15, 2019 by Vitaliy Mysak Committed by Darek Stojaczyk Jul 18, 2019

test/spdkcli: dont use -v flag for app/match



app/match/match with -v flag prints content of the files
  even if match is successful. Such output is usually big
  and not useful, so it is better to print content only
  for failed cases.
This patch removes -v flag for match program
  to achieve described desired behaviour.

Change-Id: I3e2f24e35f5fa3aeead2e3945a44053bc9ff3240
Signed-off-by: Vitaliy Mysak <vitaliy.mysak@intel.com>
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/462023


Reviewed-by: Karol Latecki <karol.latecki@intel.com>
Reviewed-by: Darek Stojaczyk <dariusz.stojaczyk@intel.com>
Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

parent 4739f623

test/spdkcli/common.sh

+1 −1

Original line number	Diff line number	Diff line
		@@ -46,6 +46,6 @@ function run_vhost_tgt() {

		function check_match() {
		$rootdir/scripts/spdkcli.py ll $SPDKCLI_BRANCH > $testdir/match_files/${MATCH_FILE}
		$rootdir/test/app/match/match -v $testdir/match_files/${MATCH_FILE}.match
		$rootdir/test/app/match/match $testdir/match_files/${MATCH_FILE}.match
		rm -f $testdir/match_files/${MATCH_FILE}
		}

test/spdkcli/vhost.sh

+1 −1

Original line number	Diff line number	Diff line
		@@ -77,7 +77,7 @@ timing_exit spdkcli_save_config

		timing_enter spdkcli_check_match_details
		$rootdir/scripts/spdkcli.py bdevs/split_disk/Nvme0n1p0 show_details \| jq -r -S '.' > $testdir/match_files/spdkcli_details_vhost.test
		$rootdir/test/app/match/match -v $testdir/match_files/spdkcli_details_vhost.test.match
		$rootdir/test/app/match/match $testdir/match_files/spdkcli_details_vhost.test.match
		rm -f $testdir/match_files/spdkcli_details_vhost.test
		timing_exit spdkcli_check_match_details