[LU-5424] Test failuresanity test_154b: create lov data thru .lustre should fail Created: 28/Jul/14  Updated: 12/May/17  Resolved: 12/May/17

Status: Closed
Project: Lustre
Component/s: None
Affects Version/s: Lustre 2.7.0
Fix Version/s: Lustre 2.7.0

Type: Bug Priority: Critical
Reporter: Maloo Assignee: John Hammond
Resolution: Fixed Votes: 0
Labels: None

Issue Links:
Related
is related to LU-5762 Interop 2.6.0<->2.7 sanity test_154a:... Closed
is related to LU-6030 cleanup for ldiskfs patches Resolved
Severity: 3
Rank (Obsolete): 15081

 Description   

This issue was created by maloo for Nathaniel Clark <nathaniel.l.clark@intel.com>

This issue relates to the following test suite run: https://testing.hpdd.intel.com/test_sets/ef0a9a7c-15df-11e4-b29b-5254006e85c2
https://testing.hpdd.intel.com/test_sets/31ab086e-14a8-11e4-bb6a-5254006e85c2
https://testing.hpdd.intel.com/test_sets/f7d33720-148d-11e4-aae8-5254006e85c2

The sub-test test_154b failed with the following error:

create lov data thru .lustre should fail.

Info required for matching: sanity 154b



 Comments   
Comment by John Hammond [ 28/Jul/14 ]

Please see http://review.whamcloud.com/11248.

Comment by Andreas Dilger [ 28/Jul/14 ]

John, which patch was landed that made the change to mdt_create_data() that allowed open-by-FID to create file layouts, and why did that patch not cause test failures before it landed?

Comment by John Hammond [ 28/Jul/14 ]
commit 6d8c562350f9f0709a1280de5e0734deaac236c1
Author: Swapnil Pimpale <spimpale@ddn.com>
Date:   Mon Jan 6 20:12:27 2014 +0530

    LU-3181 mdt: mdt_cross_open should allow open by FID on MDT1
    
    Removed the check of fid_is_obf() in mdt_cross_open() to allow
    opening by FIDs on MDT1.
    
    Signed-off-by: Swapnil Pimpale <spimpale@ddn.com>
    Change-Id: I0a2e470137c99d75a7af01a03041de72c84d2771
    Reviewed-on: http://review.whamcloud.com/8738
    Tested-by: Jenkins
    Tested-by: Maloo <hpdd-maloo@intel.com>
    Reviewed-by: John L. Hammond <john.hammond@intel.com>
    Tested-by: John L. Hammond <john.hammond@intel.com>
    Reviewed-by: wangdi <di.wang@intel.com>
    Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
    Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>

Not sure why it passed. I'll take a look.

Comment by John Hammond [ 28/Jul/14 ]

It fails after the landing of

commit db5abf4b2b3fadc054a4e7d0d5a6b2fd9a99023c
Author: Oleg Drokin <oleg.drokin@intel.com>
Date:   Fri Jul 11 00:30:54 2014 -0400

    LU-4367 llite: Make revalidate return 0 for opens
    
    This is important so that we can properly engage opencache when needed.
    
    This also reshuffles some other logic in ll_revalidate_dentry to
    more properly handle intermediate component revalidate logic.
    
    Change-Id: Ib4f272155096d33fe3f45b781e21e5bd4f44ff58
    Signed-off-by: Oleg Drokin <oleg.drokin@intel.com>
    Reviewed-on: http://review.whamcloud.com/11062
    Tested-by: Jenkins
    Reviewed-by: Niu Yawei <yawei.niu@intel.com>
    Reviewed-by: Lai Siyao <lai.siyao@intel.com>
    Tested-by: Maloo <hpdd-maloo@intel.com>
Comment by John Hammond [ 01/Aug/14 ]

Patch landed to master.

Comment by Andreas Dilger [ 15/Dec/14 ]

This failed recently with https://testing.hpdd.intel.com/test_sets/32b56e16-82e4-11e4-a45f-5254006e85c2 even though it had this patch included. It isn't clear what the reason is yet, but I thought I should record it.

Comment by Jian Yu [ 25/Dec/14 ]

One more instance on master branch: https://testing.hpdd.intel.com/test_sets/1fdcd0fe-8c03-11e4-aeb3-5254006e85c2

Comment by Niu Yawei (Inactive) [ 21/Jan/15 ]

Another instance on master: https://testing.hpdd.intel.com/sub_tests/5c75b5aa-a128-11e4-9db6-5254006e85c2

Comment by Gerrit Updater [ 05/Feb/15 ]

Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: http://review.whamcloud.com/13657
Subject: LU-5424 tests: exclude sanity-sec test 4
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 7498ed606c2cf06638883221a9ca7872179f1d43

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