GPFS unit tests fail on Mac

Bug #1203075 reported by Avishay Traeger on 2013-07-19
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Cinder
Undecided
Bill Owen

Bug Description

Similar to bug 1188972, the GPFS tests use 'dd' and 'truncate', which fail on Max OSX.

Haomai Wang (haomai) on 2013-07-19
Changed in cinder:
status: New → Confirmed
assignee: nobody → Haomai Wang (haomai)
Haomai Wang (haomai) wrote :

The whole test is awful and ugly to refactor. I recommend the driver vendor to refactor it.

Changed in cinder:
assignee: Haomai Wang (haomai) → nobody
assignee: nobody → Haomai Wang (haomai)
assignee: Haomai Wang (haomai) → nobody
John Griffith (john-griffith) wrote :

Might have to do a skip if Mac OS-X in the short term.

Long term the test shouldn't be doing truncates on the users/testers filesystem.

Bill Owen (billowen) on 2013-07-23
Changed in cinder:
assignee: nobody → Bill Owen (billowen)

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

Changed in cinder:
status: Confirmed → In Progress

Reviewed: https://review.openstack.org/38496
Committed: http://github.com/openstack/cinder/commit/72080f9cf538d9d029f953dbe18ac30fbd9f365b
Submitter: Jenkins
Branch: master

commit 72080f9cf538d9d029f953dbe18ac30fbd9f365b
Author: Bill Owen <email address hidden>
Date: Wed Jul 24 09:08:16 2013 -0700

    GPFS stub calls to truncate and dd in unit tests

    gpfs unit tests had calls to truncate, and allowed
    driver call to dd and df. Modify tests to stub these
    out.

    Also remove references to image_path. This was a temp file
    created using truncate that is no longer used.

    Change-Id: I437b56505ba025b256b051dfaf43d8124005f854
    Fixes: bug #1203075

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx) on 2013-09-05
Changed in cinder:
milestone: none → havana-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2013-10-17
Changed in cinder:
milestone: havana-3 → 2013.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers