[Bugs] [Bug 1630804] libgfapi-python: test_listdir_with_stat and test_scandir failure on release 5 branch

bugzilla at redhat.com bugzilla at redhat.com
Fri Oct 12 07:41:47 UTC 2018


https://bugzilla.redhat.com/show_bug.cgi?id=1630804



--- Comment #14 from Soumya Koduri <skoduri at redhat.com> ---
Okay after a bit of thorough reading, I see that we do not skip dir entries
(with entry->inode NULL) in the response but just do not perform lookup/link
their inodes so that any further operation on that dirent, shall force lookup.

So I see two approaches to fix this particular issue -
* From the commit-msg,
"translators like readdir-ahead selectively retain entry information of iatt
(gfid and type) when rest of the iatt is invalidated (for write invalidating
ia_size, (m)(c)times etc)."

Since readdir-ahead needs to invalidate attr for only files but not
directories, it can set entry->inode to NULL and gfapi shall perform lookup to
link inode and fetch stat before sending the attr to application

* or if this case needs to be handled for directories as well -->

we need extra check in the above routine " glfd_entry_refresh()" to validate
stat and perform lookup if NULL. 

Thoughts?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are on the CC list for the bug.


More information about the Bugs mailing list