New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
glusterfs service: Ensure dirs needed by glusterfind
exist
#29071
Conversation
Don't merge yet, I have at least 2 more fixes to |
Fixes error File "/nix/store/lxpsl84km87xpk59nai6a33ihgpfs7qr-glusterfs-3.10.2/libexec/glusterfs/glusterfind/changelog.py", line 105, in populate_pgfid_and_inodegfid file_xattrs = xattr.list(p) AttributeError: 'module' object has no attribute 'list' when using `glusterfind pre`.
See: https://bugzilla.redhat.com/show_bug.cgi?id=1489610 Also add patch to correctly log stderr from remote nodes when glusterfind fails. This, too, should be removed when fixed upstream.
This fixes: Traceback (most recent call last): File "/nix/store/7f9arl3f9xyj8sm05mkanh2mlp217192-glusterfs-3.10.2/libexec/glusterfs/glusterfind/changelog.py", line 22, in <module> import libgfchangelog File "/nix/store/7f9arl3f9xyj8sm05mkanh2mlp217192-glusterfs-3.10.2/libexec/glusterfs/glusterfind/libgfchangelog.py", line 21, in <module> libgfc = CDLL("libgfchangelog.so", use_errno=True, mode=RTLD_GLOBAL) File "/nix/store/nlyr5ankhi7yvva8zndi718zj37js270-python-2.7.13-env/lib/python2.7/ctypes/__init__.py", line 362, in __init__ self._handle = _dlopen(self._name, mode) OSError: libgfchangelog.so: cannot open shared object file: No such file or directory Connection to 10.0.0.2 closed. when running `glusterfind pre`. Done by setting PYTHONPATH/LD_LIBRARY_PATH as for the other Python scripts.
I've pushed 3 more fixes. CC @bachp OK to merge again from my side. |
I applied the patch to my setup and everything is working. |
@nh2 you can open a PR with cherry-picked commits against release-17.09 if you like. I can also do the cherry-picks directly if you prefer, but pushing the green button is less work for me :) |
Motivation for this change
Without these dirs,
glusterfind
errors like this:Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)FYI @bachp who's another user of this service.
I would also like to have this change in 17.09, please let me know if I need to open a separate PR for that.