Fix removal cancel

[ikozhukhov]

test failed due to fast operations on DilOS

original fail:

Test: /opt/zfs-tests/tests/functional/removal/removal_cancel.ksh (run as root) [00:05] [FAIL]

in log:
cat removal/removal_cancel/stderr:
cannot cancel removal: operation is not in progress

after update - increase test data - this test PASS on DilOS

ztest@zfs-tests2:~$ ./z2.sh
Test: /opt/zfs-tests/tests/functional/removal/removal_cancel.ksh (run as root) [00:30] [PASS]
Test: /opt/zfs-tests/tests/functional/removal/cleanup.ksh (run as root) [00:00] [PASS]

Results Summary
PASS       2

Running Time:   00:00:31
Percent passed: 100.0%
Log directory:  /var/tmp/test_results/20190830T192745

Motivation and Context

Description

How Has This Been Tested?

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Performance enhancement (non-breaking change which improves efficiency)
• Code cleanup (non-breaking change which makes code smaller or more readable)
• Breaking change (fix or feature that would cause existing functionality to change)
• Documentation (a change to man pages or other documentation)

Checklist:

• My code follows the ZFS on Linux code style requirements.
• I have updated the documentation accordingly.
• I have read the contributing document.
• I have added tests to cover my changes.
• All new and existing tests passed.
• All commit messages are properly formatted and contain Signed-off-by.

[behlendorf]

It looks like we've observed this at least once as well in the CI.

Test: /usr/share/zfs/zfs-tests/tests/functional/removal/removal_cancel (run as root) [00:17] [FAIL]
16:56:10.78 NOTE: begin default_setup_noexit
16:56:10.95 SUCCESS: zpool create -f testpool loop0
16:56:10.97 SUCCESS: zfs create testpool/testfs
16:56:11.04 SUCCESS: zfs set mountpoint=/mnt/testdir testpool/testfs
16:56:11.04 SUCCESS: default_setup_noexit loop0
16:56:11.05 SUCCESS: mkfile -n 1g //mnt/testdir/00
16:56:18.82 SUCCESS: randwritecomp //mnt/testdir/00 12500
16:56:19.19 SUCCESS: zpool add -f testpool loop2
16:56:26.84 SUCCESS: zpool remove testpool loop0
16:56:27.84 SUCCESS: sleep 1
16:56:27.85 SUCCESS: set_tunable32 zfs_removal_suspend_progress 1
16:56:27.85   pool: testpool
16:56:27.85  state: ONLINE
16:56:27.85   scan: none requested
16:56:27.85 remove: Removal of vdev 0 copied 801M in 0h0m, completed on Thu Aug 22 16:56:27 2019
16:56:27.85     54.3K memory used for removed device mappings
16:56:27.85 config:
16:56:27.85 
16:56:27.85 	NAME          STATE     READ WRITE CKSUM
16:56:27.85 	testpool      ONLINE       0     0     0
16:56:27.85 	  loop2       ONLINE       0     0     0
16:56:27.85 
16:56:27.85 errors: No known data errors
16:56:27.86 SUCCESS: zpool status testpool
16:56:27.86 cannot cancel removal: operation is not in progress
16:56:27.86 ERROR: zpool remove -s testpool exited 1

[codecov]

Codecov Report

Merging #9259 into master will decrease coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #9259      +/-   ##
==========================================
- Coverage   79.18%   78.97%   -0.22%     
==========================================
  Files         400      400              
  Lines      122035   122034       -1     
==========================================
- Hits        96634    96376     -258     
- Misses      25401    25658     +257

Flag	Coverage Δ
#kernel	79.63% <ø> (-0.14%)	⬇️
#user	66.52% <ø> (-0.24%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d39c71d...c39d901. Read the comment docs.