On Thu, Dec 01, 2016 at 03:38:36AM -0500, Ramesh Nachimuthu wrote: > > > > > ----- Original Message ----- > > From: "Dan Kenigsberg" <dan...@redhat.com> > > To: "Ramesh Nachimuthu" <rnach...@redhat.com> > > Cc: "Aravinda" <avish...@redhat.com>, igoih...@redhat.com, "Nir Soffer" > > <nsof...@redhat.com>, "devel" > > <devel@ovirt.org>, "Piotr Kliczewski" <pklic...@redhat.com>, "Yaniv > > Bronheim" <ybron...@redhat.com>, "Sahina Bose" > > <sab...@redhat.com> > > Sent: Wednesday, November 30, 2016 4:54:59 PM > > Subject: Re: python path conflicts issues while running python scripts from > > vdsm verbs > > > > On Wed, Nov 30, 2016 at 03:49:02AM -0500, Ramesh Nachimuthu wrote: > > > > > > > > > > > > > > > ----- Original Message ----- > > > > From: "Dan Kenigsberg" <dan...@redhat.com> > > > > To: "Aravinda" <avish...@redhat.com>, igoih...@redhat.com > > > > Cc: "Ramesh Nachimuthu" <rnach...@redhat.com>, "Nir Soffer" > > > > <nsof...@redhat.com>, "devel" <devel@ovirt.org>, "Piotr > > > > Kliczewski" <pklic...@redhat.com>, "Yaniv Bronheim" > > > > <ybron...@redhat.com>, "Sahina Bose" <sab...@redhat.com> > > > > Sent: Wednesday, November 30, 2016 2:02:43 PM > > > > Subject: Re: python path conflicts issues while running python scripts > > > > from vdsm verbs > > > > > > > > > > > > Regardless of the solution you plan to take, let it serve as a reminder > > > > about moving gluster bits out of /usr/share/vdsm/ and into > > > > site-packages. I am willing to pay the price of an ugly > > > > exception-handling in xmlrpc, if this is the cost. > > > > > > Yes. I think this is the right time to revive the patch > > > https://gerrit.ovirt.org/#/c/57530/ and > > > https://gerrit.ovirt.org/#/c/57524/ to move all the gluster code to > > > site-packages. Let me re-post these packages for review. > > > > Please do. > > > > IIRC https://gerrit.ovirt.org/#/c/57530/9 is broken in xmlrpc. > > > > Since xmlrpc is going to be used only by the command line in 4.1, I'd be > > pleased to see the problem solved via a simplistic hack. > > > > > I have updated the patch https://gerrit.ovirt.org/#/c/57530/ for moving > vdsm/gluster into site-packages. I took the patch > https://gerrit.ovirt.org/#/c/57524/ as an workaround to solve the ClientIF > issue. > > Dan: Please let me know if you have any simple fix for clientIF issue.
Thanks. I think that the approach of your https://gerrit.ovirt.org/#/c/67660/ is much better. _______________________________________________ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel