[ovirt-devel] python path conflicts issues while running python scripts from vdsm verbs

[Dan Kenigsberg]

On Thu, Dec 01, 2016 at 03:38:36AM -0500, Ramesh Nachimuthu wrote:
> 
> 
> 
> 
> ----- Original Message -----
> > From: "Dan Kenigsberg" <danken at redhat.com>
> > To: "Ramesh Nachimuthu" <rnachimu at redhat.com>
> > Cc: "Aravinda" <avishwan at redhat.com>, igoihman at redhat.com, "Nir Soffer" <nsoffer at redhat.com>, "devel"
> > <devel at ovirt.org>, "Piotr Kliczewski" <pkliczew at redhat.com>, "Yaniv Bronheim" <ybronhei at redhat.com>, "Sahina Bose"
> > <sabose at 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" <danken at redhat.com>
> > > > To: "Aravinda" <avishwan at redhat.com>, igoihman at redhat.com
> > > > Cc: "Ramesh Nachimuthu" <rnachimu at redhat.com>, "Nir Soffer"
> > > > <nsoffer at redhat.com>, "devel" <devel at ovirt.org>, "Piotr
> > > > Kliczewski" <pkliczew at redhat.com>, "Yaniv Bronheim"
> > > > <ybronhei at redhat.com>, "Sahina Bose" <sabose at 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.