Skip to content

Commit

Permalink
tests: Fix test_flush_over_640_gibibytes_with_fs
Browse files Browse the repository at this point in the history
Don't interrupt the test if prerequisites are not met

Signed-off-by: Michal Mielewczyk <michal.mielewczyk@huawei.com>
  • Loading branch information
mmichal10 committed Oct 3, 2024
1 parent 32fbb74 commit c5c62a5
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions test/functional/tests/lazy_writes/test_flush_huge_dirty_data.py
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ def test_flush_over_640_gibibytes_with_fs(cache_mode, fs):

with TestRun.step("Validate test file and read its crc32 sum."):
if test_file_main.size != file_size:
TestRun.fail("Created test file hasn't reached its target size.")
TestRun.LOGGER.error(f"Expected test file size {file_size}. Got {test_file_main.size}")
test_file_crc32sum_main = test_file_main.crc32sum(timeout=timedelta(hours=4))

with TestRun.step("Write data to exported object."):
Expand All @@ -97,12 +97,16 @@ def test_flush_over_640_gibibytes_with_fs(cache_mode, fs):

with TestRun.step(f"Check if dirty data exceeded {file_size * 0.98} GiB."):
minimum_4KiB_blocks = int((file_size * 0.98).get_value(Unit.Blocks4096))
if int(cache.get_statistics().usage_stats.dirty) < minimum_4KiB_blocks:
TestRun.fail("There is not enough dirty data in the cache!")
actual_dirty_blocks = int(cache.get_statistics().usage_stats.dirty)
if actual_dirty_blocks < minimum_4KiB_blocks:
TestRun.LOGGER.error(
f"Expected at least: {minimum_4KiB_blocks} dirty blocks."
f"Got {actual_dirty_blocks"
)

with TestRun.step("Unmount core and stop cache with flush."):
core.unmount()
# this operation could take few hours, depending on core disk
# this operation could take a few hours, depending on the core disk
output = TestRun.executor.run(stop_cmd(str(cache.cache_id)), timedelta(hours=12))
if output.exit_code != 0:
TestRun.fail(f"Stopping cache with flush failed!\n{output.stderr}")
Expand Down

0 comments on commit c5c62a5

Please sign in to comment.