Fix intermittent EINVAL in ZFSPool.sync #194
Conversation
Sometimes ZFSPool.sync doesn't pass the pool_name argument to lzc_sync correctly. lzc_sync sees an empty string. Replacing the self.name property access with the self.handle instance variable access seems to fix the problem, though I don't understand why. Fixes truenas#193
|
cc @alek-p |
| def sync(self, force=False): | ||
| cdef int ret | ||
| cdef const char *c_name = self.name | ||
| cdef const char *c_name = libzfs.zpool_get_name(self.handle) |
There was a problem hiding this comment.
I don't follow, it's doing the same thing in name property ?
Do you have a reproduction case which I could perhaps try locally ?
There was a problem hiding this comment.
Yes, it's doing the exact same thing as in the name property. But no, I don't have a very portable reproduction case.
There was a problem hiding this comment.
Here's a short script that frequently fails. Though if I change the pool name to the much smaller and less busy zroot, then it passes.
import libzfs
zfs = libzfs.ZFS()
ds = zfs.get_dataset("atl-rb22b-ss.b.24")
pool = ds.pool
pool.sync()There was a problem hiding this comment.
@asomers can you please create a ticket at https://ixsystems.atlassian.net and link this script there and we can investigate as to why this could be failing potentially. Do mention your scale version please as well. Thanks!
There was a problem hiding this comment.
What is a "scale version"?
There was a problem hiding this comment.
BTW I'm not a TrueNAS user. I'm using a custom Python application on FreeBSD stable/13.
There was a problem hiding this comment.
@asomers interesting. I am not able to reproduce using the same script ( i am executing the same code path in a loop ). I wonder if it's a cython/python version issue ?
Could you share some more details wrt cython/python version you have please ? Thanks!
There was a problem hiding this comment.
Python 3.9.13
Cython version 0.29.30
2 participants
Sometimes ZFSPool.sync doesn't pass the pool_name argument to lzc_sync
correctly. lzc_sync sees an empty string. Replacing the self.name
property access with the self.handle instance variable access seems to
fix the problem, though I don't understand why.
Fixes #193