Merge ~dannf/plainbox-provider-checkbox:lp1751167 into plainbox-provider-checkbox:master

[dann frazier]

[Jeff Lane  (bladernr)]

Good catch... I wonder why this never popped up before, because the same should be true on any HDD with multiple partitions:

ubuntu@xwing:~$ sudo parted -l
Model: LSI MR9260-8i (scsi)
Disk /dev/sda: 1000GB
Sector size (logical/physical): 512B/512B
Partition Table: gpt
Disk Flags:

Number Start End Size File system Name Flags
 1 1049kB 538MB 537MB fat32 boot, esp
 2 538MB 1000GB 999GB ext4

Maybe just lucky how the partitions are presented... either way, this works.

review: Approve

diff --git a/bin/disk_stress_ng b/bin/disk_stress_ng
index 0b5de83..df5694e 100755
--- a/bin/disk_stress_ng
+++ b/bin/disk_stress_ng
@@ -121,7 +121,7 @@ find_largest_partition() {
             exit 1
         fi
         local blkid_info=$(blkid -s TYPE $part_location | grep -E ext2\|ext3\|ext4\|xfs\|jfs\|btrfs\|LVM2_member)
-        if [ "$part_size" > "$largest_size" ] && [ -n "$blkid_info" ]  ; then
+        if [ "$part_size" -gt "$largest_size" ] && [ -n "$blkid_info" ]  ; then
             if [[ "$blkid_info" =~ .*LVM2_member.* ]] ; then
                 find_largest_lv
             else