----- Original Message ----- From: "Humble Chirammal" <hchir...@redhat.com> To: "Dan Kenigsberg" <dan...@redhat.com>, "Darshan Narayana Murthy" <dnara...@redhat.com> Cc: vdsm-de...@fedorahosted.org, "Vijay Bellur" <vbel...@redhat.com> Sent: Wednesday, April 9, 2014 11:42:09 AM Subject: Re: [vdsm] Help Needed
On 04/08/2014 02:22 PM, Dan Kenigsberg wrote: > On Tue, Apr 08, 2014 at 02:02:37AM -0400, Darshan Narayana Murthy wrote: >> >> ----- Original Message ----- >> From: "Darshan Narayana Murthy" <dnara...@redhat.com> >> To: vdsm-de...@fedorahosted.org >> Sent: Tuesday, April 8, 2014 11:21:27 AM >> Subject: Re: [vdsm] Help Needed >> >> >> >> ----- Original Message ----- >> From: "Dan Kenigsberg" <dan...@redhat.com> >> To: "Darshan Narayana Murthy" <dnara...@redhat.com> >> Sent: Monday, April 7, 2014 4:53:30 PM >> Subject: Re: Help Needed >> >> On Mon, Apr 07, 2014 at 06:52:36AM -0400, Darshan Narayana Murthy wrote: >>> Hi Dan, >>> >>> I sent a patch for vdsm to get the gluster volume capacity >>> statistics using libgf api ( patch : http://gerrit.ovirt.org/#/c/26343 ), >>> This patch requires glusterfs-devel package for build. >>> >>> It looks like jenkins does not have this package and so the build >>> for this patch is failing. Does jenkins automatically pull the required >>> package or is there anything to be done to get this package in jenkins ?. >>> >>> Can you please help me to resolve this. >> Generally speaking, if you want a new package installed, you should ask >> that on in...@ovirt.org. >> >> However, I am not at all happy with adding C code into Vdsm. What is it? >> Python binding for glfs_statvfs ? Could this be implemented elsewhere >> (such as an independent python-glfs package)? >> >> Dan. >> >> >> Hi, >> >> We are making use of libgf-api for getting the statistics related >> to a glusterfs volume, as it is more efficient than we mounting a volume >> and getting the statistics. >> >> libgfapi is a c api. Initially we tried using ctypes to wrap the >> required functions in libgfapi. But because of a limitation in glusterfs >> when these functions were invoked through supervdsm it would break. > I hope Toni can help out here, he found outr that when using threads, > you must declare the function prototype explicitly, but I'm not at all > sure this is your issue. >> >> Issue with the above approach was: >> https://lists.fedorahosted.org/pipermail/vdsm-devel/2013-August/002537.html >> >> So we thought of having an extension module that makes use of libgfapi >> and provides the statistics, which can be used in vdsm. >> >> what would be the better approach to resolve this ? Please provide us >> your suggestions. > In my opionion, the python module that you suggest makes sense - but it > should be part of libgf - it is their's python binding (or part > thereof). I have added glfs_statvfs in libgfapi python binding and its working , hopefully we can avoid all these issues via that. Darshan, if we are getting any issue ( in vdsm ) even after using new method, we will dig further.. -- Cheers, Humble Chirammal yes, I will try and use libgfapi python for this. Thanks, Darshan N Sr. Software Engineer Red Hat India Pvt. Ltd _______________________________________________ vdsm-devel mailing list vdsm-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
