]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Fix remaining memory issues with nfs backup unit tests
authorTom Barron <tpb@dyncloud.net>
Thu, 19 Mar 2015 23:10:54 +0000 (19:10 -0400)
committerTom Barron <tpb@dyncloud.net>
Thu, 4 Jun 2015 20:31:47 +0000 (20:31 +0000)
Here we fix another place in the nfs backup unit tests where
the configured file size was larger than the chunk size in
the corresponding Swift backup tests.  I missed this one in

   https://review.openstack.org/#/c/165394

and even though code with that fix ran fine in a bare 1GB VM,
there are still memory issues when run in CI with a VM of that
size.

Closes-Bug: 1452811
Change-Id: I20dcfb657699e1b415ae2e36ec63f8c6b0049285

cinder/tests/unit/backup/drivers/test_backup_nfs.py

index 7ac948506ee6f44f4c25c0ced0ebf702ddb59939..a48e049ba095d056e38aba080edde74aa323db8d 100644 (file)
@@ -610,7 +610,6 @@ class BackupNFSSwiftBasedTestCase(test.TestCase):
     def test_restore_uncompressed(self):
         self._create_backup_db_entry()
         self.flags(backup_compression_algorithm='none')
-        self.flags(backup_file_size=(1024 * 1024 * 1024))
         self.flags(backup_sha_block_size_bytes=32)
         service = nfs.NFSBackupDriver(self.ctxt)
         self.volume_file.seek(0)