Commit 7bd9e8c2 authored by Pawel Kaminski's avatar Pawel Kaminski Committed by Jim Harris
Browse files

test/lvol: Assign big number to SIGTERM test case 


           to avoid renumbering test cases in the future.

SIGTERM should be called as last test case, because
it kills vhost instance. If it is called in the middle
another test case should start vhost again.
Want to avoid this.

Change-Id: Idd3e1492bd83e911d17e730c680187cd98f0c125
Signed-off-by: default avatarPawel Kaminski <pawelx.kaminski@intel.com>
Reviewed-on: https://review.gerrithub.io/399131


Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarKarol Latecki <karol.latecki@intel.com>
Reviewed-by: default avatarMaciej Szwed <maciej.szwed@intel.com>
Reviewed-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
parent 2d648558

autotest.sh

+1 −1
Original line number Diff line number Diff line
@@ -181,7 +181,7 @@ if [ $SPDK_TEST_LVOL -eq 1 ]; then 
	timing_enter lvol

	test_cases="1,50,51,52,53,100,101,102,250,251,252,253,255,"

	test_cases+="300,301,450,451,452,550,600,601,650,651,652,654,655,"

	test_cases+="700,701,750"

	test_cases+="700,701,10000"

	run_test ./test/lvol/lvol.sh --test-cases=$test_cases

	timing_exit lvol

fi

test/lvol/lvol.sh

+1 −1
Original line number Diff line number Diff line
@@ -58,7 +58,7 @@ function usage() { 
                                    655: 'thin_provisioning_filling_disks_less_than_lvs_size',

                                    700: 'tasting_positive',

                                    701: 'tasting_lvol_store_positive',

                                    750: 'SIGTERM'

                                    10000: 'SIGTERM'

                                    or

                                    all: This parameter runs all tests

                                    Ex: \"1,2,19,20\", default: all"

test/lvol/test_cases.py

+4 −4
Original line number Diff line number Diff line
@@ -61,7 +61,7 @@ def header(num): 
        655: 'thin_provisioning_filling_disks_less_than_lvs_size',

        700: 'tasting_positive',

        701: 'tasting_lvol_store_positive',

        750: 'SIGTERM',

        10000: 'SIGTERM',

    }

    print("========================================================")

    print("Test Case {num}: Start".format(num=num))
@@ -1067,8 +1067,8 @@ class TestCases(object): 
        footer(701)

        return fail_count



    def test_case750(self):

        header(750)

    def test_case10000(self):

        header(10000)

        pid_path = path.join(self.path, 'vhost.pid')



        base_name = self.c.construct_malloc_bdev(self.total_size,
@@ -1080,5 +1080,5 @@ class TestCases(object): 
                                                  self.cluster_size)



        fail_count += self._stop_vhost(pid_path)

        footer(750)

        footer(10000)

        return fail_count

test/lvol/test_plan.md

+21 −21
Original line number Diff line number Diff line
@@ -720,24 +720,9 @@ Expected result: 
- calls successful (lvol store should be tasted correctly), return code = 0

- no other operation fails



### SIGTERM



#### TEST CASE 750 - Name: SIGTERM

Call CTRL+C (SIGTERM) occurs after creating lvol store

Steps:

- create a malloc bdev

- construct_lvol_store on created malloc bdev

- check correct uuid values in response get_lvol_stores command

- Send SIGTERM signal to the application



Expected result:

- calls successful, return code = 0

- get_bdevs: no change

- no other operation fails



### snapshot and clone



#### TEST CASE 800 - Name: snapshot_readonly

#### TEST CASE 750 - Name: snapshot_readonly

- constrcut malloc bdev

- construct lvol store on malloc bdev

- construct lvol bdev
@@ -754,7 +739,7 @@ Expected result: 
- calls successful, return code = 0

- no other operation fails



#### TEST CASE 801 - Name: snapshot_compare_with_lvol_bdev

#### TEST CASE 751 - Name: snapshot_compare_with_lvol_bdev

- construct malloc bdev

- construct lvol store on malloc bdev

- construct thin provisioned lvol bdev with size less than 25% of lvs
@@ -776,7 +761,7 @@ Expected result: 
- removing snapshot should always end with success

- no other operation fails



#### TEST CASE 802 - Name: snapshot_during_io_traffic

#### TEST CASE 752 - Name: snapshot_during_io_traffic

- construct malloc bdev

- construct lvol store on malloc bdev

- construct thin provisioned lvol bdev
@@ -792,7 +777,7 @@ Expected result: 
- calls successful, return code = 0

- no other operation fails



#### TEST CASE 803 - Name: snapshot_of_snapshot

#### TEST CASE 753 - Name: snapshot_of_snapshot

- construct malloc bdev

- construct lvol store on malloc bdev

- construct thick provisioned lvol bdev
@@ -808,7 +793,7 @@ Expected result: 
- creating snapshot of snapshot should fail

- no other operation fails



#### TEST CASE 804 - Name: clone_bdev_only

#### TEST CASE 754 - Name: clone_bdev_only

- construct malloc bdev

- construct lvol store on malloc

- construct thick provisioned lvol bdev
@@ -831,7 +816,7 @@ Expected result: 
- cloning thick provisioned lvol bdev should fail

- no other operation fails



#### TEST CASE 806 - Name: clone_writing_to_clone

#### TEST CASE 755 - Name: clone_writing_to_clone

- construct with malloc bdev

- construct lvol store on malloc bdev

- construct thick provisioned lvol bdev
@@ -850,3 +835,18 @@ Expected result: 
Expected result:

- calls successful, return code = 0

- no other operation fails



### SIGTERM



#### TEST CASE 10000 - Name: SIGTERM

Call CTRL+C (SIGTERM) occurs after creating lvol store

Steps:

- create a malloc bdev

- construct_lvol_store on created malloc bdev

- check correct uuid values in response get_lvol_stores command

- Send SIGTERM signal to the application



Expected result:

- calls successful, return code = 0

- get_bdevs: no change

- no other operation fails