[LU-3096] Sanity test_1b doesn't check rmdir result Created: 03/Apr/13  Updated: 16/Oct/13  Resolved: 14/Jun/13

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: None
Fix Version/s: Lustre 2.5.0

Type: Bug Priority: Minor
Reporter: Vladimir Saveliev Assignee: Keith Mannthey (Inactive)
Resolution: Fixed Votes: 0
Labels: patch

Severity: 3
Project: Test Infrastructure
Rank (Obsolete): 7519

 Description   

test_1b() {
rmdir $DIR/d1/d2
rmdir $DIR/d1
$CHECKSTAT -a $DIR/d1 || error
}

Test falsely pass if started separately - no there is dirs to remove but test doesn't check rmdir result. Need to create directory before removing or (preferable) unite this test with 1a.

 Comments   
Comment by Vladimir Saveliev [ 03/Apr/13 ]

This is created to request upstream landing for http://jira-nss.xy01.xyratex.com:8080/browse/MRP-814

Comment by Vladimir Saveliev [ 03/Apr/13 ]

http://review.whamcloud.com/5929

Comment by Keith Mannthey (Inactive) [ 03/Apr/13 ]

http://jira-nss.xy01.xyratex.com:8080/browse/MRP-814 I don't seem to be able to load anything when I click on the jira ticket.

Comment by Vladimir Saveliev [ 04/Apr/13 ]

Yes, that is my fault, I added it just for reference and later realized that it is not public.

Description of this ticket is a replica of description of MRP-814.
Patch for your review (http://review.whamcloud.com/5929) is port of patch landed on Xyratex's branch already.
The only unique comment in that bug is Nathan's comment:
<quota>
Nathan Rutman added a comment - 07/Jan/13 10:53 PM
Please combine any dependent tests into a single test.
(e.g. if 1b depends on running 1a first, combine them both into "test 1"). Add error exit messages indicating what failed.
</quota>

Comment by Vladimir Saveliev [ 22/May/13 ]

the patch http://review.whamcloud.com/5929 is updated, please take a look

Comment by Keith Mannthey (Inactive) [ 14/Jun/13 ]

Patched Merged.

Generated at Sat Feb 10 01:30:56 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.