Comment 2 for bug 1212502

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/42045
Committed: http://github.com/openstack/cinder/commit/6dc2193813d1e8ee951288c5386296b0a2a5e7b9
Submitter: Jenkins
Branch: master

commit 6dc2193813d1e8ee951288c5386296b0a2a5e7b9
Author: John Griffith <email address hidden>
Date: Wed Aug 14 20:06:05 2013 -0600

    Replace os.unlink with delete_if_exists

    Shouldn't care when doing unlink on our temp files
    if they exist or not. In fact this causes problems
    when you do things like with tempfile/dir and happen
    to try and unlink after it's already been removed.

    This replaces these calls with the safer
    common.fileutils.delete_if_exists which will
    ignore the os exception of the object DNE.

    Fixes bug: 1212502

    Change-Id: Ica86c95f736411da486335aec5512e59247bfbc0