[LU-16739] sanity 400b fails due to recent lustre_disk.h changes Created: 14/Apr/23  Updated: 27/Dec/23  Resolved: 09/May/23

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

Type: Bug Priority: Minor
Reporter: James A Simmons Assignee: James A Simmons
Resolution: Fixed Votes: 0
Labels: None
Environment:

Any lustre client running maloo sanity test 400b


Severity: 3
Rank (Obsolete): 9223372036854775807

 Description   

The recent landing of ("LU-16524 nodemap: add rbac property to nodemap")  causes sanity 400b to fail. This test ensures the UAPI headers can be compiled by user land. The failure is:

lustre_disk.h:266:18: error: 'LUSTRE_NODEMAP_NAME_LENGTH' undeclared here (not in a function) 
 char   ncr_name[LUSTRE_NODEMAP_NAME_LENGTH + 1]; 
                 ^~~~~~~~~~~~~~~~~~~~~~~~~~ 
lustre_disk.h:267:20: error: 'ncr_flags' is narrower than values of its type [-Werror] 
 enum nm_flag_bits ncr_flags:8; 
                   ^~~~~~~~~ 
lustre_disk.h:267:20: error: field 'ncr_flags' has incomplete type 
lustre_disk.h:268:21: error: 'ncr_flags2' is narrower than values of its type [-Werror] 
 enum nm_flag2_bits ncr_flags2:8; 
                    ^~~~~~~~~~ 
lustre_disk.h:268:21: error: field 'ncr_flags2' has incomplete type 
lustre_disk.h:277:2: error: unknown type name 'lnet_nid_t' 
 lnet_nid_t nrr_start_nid; 
 ^~~~~~~~~~ 
lustre/lustre_disk.h:278:2: error: unknown type name 'lnet_nid_t' 
 lnet_nid_t nrr_end_nid; 
 ^~~~~~~~~~


 Comments   
Comment by James A Simmons [ 14/Apr/23 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/50641
Subject: LU-16739 uapi: make lustre_disk.h buildable in user land
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: b887e05ee6c8c58ce83338d60678a172a358a0ab

Comment by Gerrit Updater [ 09/May/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50641/
Subject: LU-16739 uapi: make lustre_disk.h buildable in user land
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 5a6725d19d4037026d7cab2442b0c639d1511e5d

Comment by Peter Jones [ 09/May/23 ]

Landed for 2.16

Comment by James A Simmons [ 22/Nov/23 ]

Test got broken again. Will submit a patch shortly.

Comment by Gerrit Updater [ 22/Nov/23 ]

"James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53208
Subject: LU-16739 uapi: restore LUSTRE_NODEMAP_NAME_LENGTH in lustre_disk.h
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 8f8a3c79f0bc1996637c0b3d2f0e0e4358802f88

Comment by Gerrit Updater [ 13/Dec/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53208/
Subject: LU-16739 uapi: restore LUSTRE_NODEMAP_NAME_LENGTH in lustre_disk.h
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: fc67e5d18a70ca8b5f01abc89eabdbfc78381f7c

Generated at Sat Feb 10 03:29:35 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.