You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
After an upload fails we try to delete the file(s) from the FS. This operation eventually checks whether the file is a directory (using the fs getinfomethod). In turn, this executes fs._query(QueryCode.XATTR, path) which is not a valid query for XATTR (it seems it's an unsupported operation).
Possible fix
From what I see, the client supports extended attributes listing which I believe would be what we need. I ran some tests in the CLI and seems to be OK:
fromxrootdpyfs.fsimportXRootDPyFSurl='...'fs=XRootDPyFS(url)
path='...'# Current code that failsfs._query(QueryCode.XATTR, path)
# Alternative codeextended_attr=fs._client.list_xattr(path)
In the example above, I realised that we have another issue when raising the fs exceptions (e.g. Unsupported). We must use raise Unsupported(msg=status.message). This is also being tracked by #122
If this is fixed, we could potentially have more visibility on why the upload failed on the first place (to be tested)
The text was updated successfully, but these errors were encountered:
Problem
After an upload fails we try to delete the file(s) from the FS. This operation eventually checks whether the file is a directory (using the fs
getinfo
method). In turn, this executesfs._query(QueryCode.XATTR, path)
which is not a valid query for XATTR (it seems it's an unsupported operation).Possible fix
From what I see, the client supports extended attributes listing which I believe would be what we need. I ran some tests in the CLI and seems to be OK:
In the example above, I realised that we have another issue when raising the fs exceptions (e.g.
Unsupported
). We must use raiseUnsupported(msg=status.message)
. This is also being tracked by #122If this is fixed, we could potentially have more visibility on why the upload failed on the first place (to be tested)
The text was updated successfully, but these errors were encountered: