FileStore: properly fill in XATTR_NO_SPILL_OUT tag
Right now, the only way the FileStore sets the XATTR_SPILL_OUT_NAME xattr to contain XATTR_NO_SPILL_OUT is when removing xattrs. We need to fill them in when creating objects, too. This is probably just a minor case adjustment in FileStore::_setattrs() to include one where we write set the xattr.
#2 Updated by Greg Farnum over 5 years ago
- Status changed from In Progress to Testing
Significantly revised the branch (it now removes the user_only stuff in the ObjectStore interface), and have scheduled a suite run against it.
It passed my trivial test (ran rados bench, checked file on disk, it now has the spillout xattr, where it did not with master).
#3 Updated by Greg Farnum about 5 years ago
- Status changed from Testing to In Progress
I ran this branch through testing and it wasn't passing; it appears to be some kind of issue with the spillout logic itself (that we didn't notice previously because it wasn't exercised), but maybe something else is going on. I'm discussing with Haomai on the mailing list.