diff options
author | Vladyslav Drok <vdrok@mirantis.com> | 2020-06-08 18:39:36 +0200 |
---|---|---|
committer | Vladyslav Drok <vdrok@mirantis.com> | 2020-06-08 18:42:27 +0200 |
commit | b9938230f992935e8332b6e288937be890724cd2 (patch) | |
tree | d41da308c9881720346b2fbf3e3ebed88579ba18 | |
parent | 00ccff5a0dec0a0a84ef656d839c188b149574b6 (diff) | |
download | oslo-utils-b9938230f992935e8332b6e288937be890724cd2.tar.gz |
Release greenthread when computing checksum4.2.1
Having time.sleep(0) after reading each block of data for computing
checksum will allow other greenthreads to run.
Closes-Bug: 1882569
Change-Id: I6e547d206de9e3e333e29ccad52bf6b519a86ff9
-rw-r--r-- | oslo_utils/fileutils.py | 3 | ||||
-rw-r--r-- | oslo_utils/tests/test_fileutils.py | 19 |
2 files changed, 22 insertions, 0 deletions
diff --git a/oslo_utils/fileutils.py b/oslo_utils/fileutils.py index e2b0917..909ed4b 100644 --- a/oslo_utils/fileutils.py +++ b/oslo_utils/fileutils.py @@ -25,6 +25,7 @@ import hashlib import os import stat import tempfile +import time from oslo_utils import excutils @@ -123,6 +124,8 @@ def compute_file_checksum(path, read_chunksize=65536, algorithm='sha256'): with open(path, 'rb') as f: for chunk in iter(lambda: f.read(read_chunksize), b''): checksum.update(chunk) + # Release greenthread, if greenthreads are not used it is a noop. + time.sleep(0) return checksum.hexdigest() diff --git a/oslo_utils/tests/test_fileutils.py b/oslo_utils/tests/test_fileutils.py index d34a38b..7e089c8 100644 --- a/oslo_utils/tests/test_fileutils.py +++ b/oslo_utils/tests/test_fileutils.py @@ -19,6 +19,8 @@ import os import shutil import stat import tempfile +import time +from unittest import mock import uuid from oslotest import base as test_base @@ -215,6 +217,23 @@ class TestComputeFileChecksum(test_base.BaseTestCase): self.assertEqual(expected_checksum.hexdigest(), actual_checksum) + def test_compute_checksum_sleep_0_called(self): + path = fileutils.write_to_tempfile(self.content) + self.assertTrue(os.path.exists(path)) + self.check_file_content(self.content, path) + + expected_checksum = hashlib.sha256() + expected_checksum.update(self.content) + + with mock.patch.object(time, "sleep") as sleep_mock: + actual_checksum = fileutils.compute_file_checksum( + path, read_chunksize=4) + + sleep_mock.assert_has_calls([mock.call(0)] * 3) + # Just to make sure that there were exactly 3 calls + self.assertEqual(3, sleep_mock.call_count) + self.assertEqual(expected_checksum.hexdigest(), actual_checksum) + def test_compute_checksum_named_algorithm(self): path = fileutils.write_to_tempfile(self.content) self.assertTrue(os.path.exists(path)) |