fix remaining nfs backup unit test memory issue

Bug #1452811 reported by Tom Barron
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Tom Barron

Bug Description

We need to 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.

Tom Barron (tpb)
Changed in cinder:
assignee: nobody → Tom Barron (tpb)
tags: added: backup-service nfs unit-tests
Eric Harney (eharney)
tags: added: tests
removed: unit-tests
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/188577

Changed in cinder:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/188577
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=1c1d16e77fe8e57dbf9995dcf042bb6aa2b77309
Submitter: Jenkins
Branch: master

commit 1c1d16e77fe8e57dbf9995dcf042bb6aa2b77309
Author: Tom Barron <email address hidden>
Date: Thu Mar 19 19:10:54 2015 -0400

    Fix remaining memory issues with nfs backup unit tests

    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

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → liberty-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: liberty-1 → 7.0.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.