linux-system-roles / storage

Ansible role for linux storage management
https://linux-system-roles.github.io/storage/
MIT License
100 stars 59 forks source link

test: disk size is ok if less than 4 percent of expected #420

Closed richm closed 8 months ago

richm commented 8 months ago

QE reported a test failure like this:

TASK [Show actual size] ********************************************************
ok: [localhost] => {
    "storage_test_actual_size": {
        "bytes": 1649267441664,
        "changed": false,
        "failed": false,
        "lvm": "1t",
        "parted": "1TiB",
        "size": "1 TiB"
    }
}

TASK [Show expected size] ******************************************************
ok: [localhost] => {
    "storage_test_expected_size": "1600321314816"
}

TASK [Assert expected size is actual size] *************************************
fatal: [localhost]: FAILED! => {
    "assertion": "(storage_test_expected_size | int - storage_test_actual_size.bytes) | abs / storage_test_expected_size | int < 0.02",
    "changed": false,
    "evaluated_to": false,
    "msg": "Volume test1 has unexpected size (expected: 1600321314816 / actual: 1649267441664)"
}

PLAY RECAP *********************************************************************

In this case,

>>> abs(1600321314816-1649267441664)/1600321314816
0.03058518710889486

I think this is correct, so raising the percentage to check to 4.

Signed-off-by: Rich Megginson rmeggins@redhat.com

richm commented 8 months ago

@vojtechtrefny @japokorn @yizhanglinux this is the second time we have had to increase the percentage here. I think it is still correct, and hoping that there is a better way to determine if the actual size is the expected size.