Re: [ovirt-devel] python path conflicts issues while running python scripts from vdsm verbs
בתאריך 2 בדצמ׳ 2016 10:04 לפנה״צ, "Dan Kenigsberg" כתב: On Thu, Dec 01, 2016 at 11:32:07PM -0500, Ramesh Nachimuthu wrote: > Thank you all for helping me in this issue. Finally vdsm.gluster > modules are moved python site-packages. > Yay! There's a bit more work to be done: https://travis-ci.org/oVirt/vdsm/jobs/180630800 is failing since its docker images do not yet include python-magic and python-blivet. Nir, would you take https://gerrit.ovirt.org/#/c/67727/ and rebuild them? Sure, thanks. ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] python path conflicts issues while running python scripts from vdsm verbs
On Thu, Dec 01, 2016 at 11:32:07PM -0500, Ramesh Nachimuthu wrote: > Thank you all for helping me in this issue. Finally vdsm.gluster > modules are moved python site-packages. > Yay! There's a bit more work to be done: https://travis-ci.org/oVirt/vdsm/jobs/180630800 is failing since its docker images do not yet include python-magic and python-blivet. Nir, would you take https://gerrit.ovirt.org/#/c/67727/ and rebuild them? ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] python path conflicts issues while running python scripts from vdsm verbs
Thank you all for helping me in this issue. Finally vdsm.gluster modules are moved python site-packages. Regards, Ramesh - Original Message - > From: "Dan Kenigsberg" > To: "Ramesh Nachimuthu" > Cc: "Aravinda" , igoih...@redhat.com, "Nir Soffer" > , "devel" > , "Piotr Kliczewski" , "Yaniv Bronheim" > , "Sahina Bose" > > Sent: Thursday, December 1, 2016 5:16:34 PM > Subject: Re: python path conflicts issues while running python scripts from > vdsm verbs > > On Thu, Dec 01, 2016 at 03:38:36AM -0500, Ramesh Nachimuthu wrote: > > > > > > > > > > - Original Message - > > > From: "Dan Kenigsberg" > > > To: "Ramesh Nachimuthu" > > > Cc: "Aravinda" , igoih...@redhat.com, "Nir Soffer" > > > , "devel" > > > , "Piotr Kliczewski" , "Yaniv > > > Bronheim" , "Sahina Bose" > > > > > > 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" > > > > > To: "Aravinda" , igoih...@redhat.com > > > > > Cc: "Ramesh Nachimuthu" , "Nir Soffer" > > > > > , "devel" , "Piotr > > > > > Kliczewski" , "Yaniv Bronheim" > > > > > , "Sahina Bose" > > > > > 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
Re: [ovirt-devel] python path conflicts issues while running python scripts from vdsm verbs
On Thu, Dec 01, 2016 at 03:38:36AM -0500, Ramesh Nachimuthu wrote: > > > > > - Original Message - > > From: "Dan Kenigsberg" > > To: "Ramesh Nachimuthu" > > Cc: "Aravinda" , igoih...@redhat.com, "Nir Soffer" > > , "devel" > > , "Piotr Kliczewski" , "Yaniv > > Bronheim" , "Sahina Bose" > > > > 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" > > > > To: "Aravinda" , igoih...@redhat.com > > > > Cc: "Ramesh Nachimuthu" , "Nir Soffer" > > > > , "devel" , "Piotr > > > > Kliczewski" , "Yaniv Bronheim" > > > > , "Sahina Bose" > > > > 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
Re: [ovirt-devel] python path conflicts issues while running python scripts from vdsm verbs
- Original Message - > From: "Dan Kenigsberg" > To: "Ramesh Nachimuthu" > Cc: "Aravinda" , igoih...@redhat.com, "Nir Soffer" > , "devel" > , "Piotr Kliczewski" , "Yaniv Bronheim" > , "Sahina Bose" > > 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" > > > To: "Aravinda" , igoih...@redhat.com > > > Cc: "Ramesh Nachimuthu" , "Nir Soffer" > > > , "devel" , "Piotr > > > Kliczewski" , "Yaniv Bronheim" > > > , "Sahina Bose" > > > 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. Regards, Ramesh ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] python path conflicts issues while running python scripts from vdsm verbs
If vdsm setting PYTHONPATH as below export PYTHONPATH=/usr/share/vdsm:$PYTHONPATH change it as export PYTHONPATH=$PYTHONPATH:/usr/share/vdsm With this change, /usr/share/vdsm will not get first preference compared to /usr/lib*/python2.7/site-packages regards Aravinda On 11/30/2016 10:25 AM, Ramesh Nachimuthu wrote: - Original Message - From: "Nir Soffer" To: "Ramesh Nachimuthu" Cc: "devel" , "Dan Kenigsberg" , "Piotr Kliczewski" , "Yaniv Bronheim" , "Aravinda Vishwanathapura Krishna Murthy" , "Sahina Bose" Sent: Tuesday, November 29, 2016 7:20:57 PM Subject: Re: python path conflicts issues while running python scripts from vdsm verbs On Tue, Nov 29, 2016 at 3:26 PM, Ramesh Nachimuthu wrote: Hi, I am trying to run a python script '/sbin/gluster-eventsapi' in vdsm verb which internally imports some python modules from /usr/lib/python2.7/site-packages/gluster/cliutils/. But it fails with the import error. Following error is seen in the supervdsm log. MainProcess|Thread-1::DEBUG::2016-11-28 16:54:35,130::commands::93::root::(execCmd) FAILED: = 'Traceback (most recent call last):\n File "/sbin/gluster-eventsapi", line 25, in \nfrom gluster.cliutils.cliutils import (Cmd, execute, node_output_ok, node_output_notok,\nImportError: No module named cliutils.cliutils\n'; = 1 I think the import statement "from gluster.cliutils.cliutils import (Cmd, execute, node_output_ok, node_output_notok)" in the python script resolves to '/usr/share/vdsm/gluster' instead of /usr/lib/python2.7/site-packages/gluster/cliutils. I see the following in python sys.path while executing a python script from vdsm. ['/usr/libexec/glusterfs', '/usr/share/vdsm', '/usr/lib64/python27.zip', '/usr/lib64/python2.7', '/usr/lib64/python2.7/plat-linux2', '/usr/lib64/python2.7/lib-tk', '/usr/lib64/python2.7/lib-old', '/usr/lib64/python2.7/lib-dynload', '/usr/lib64/python2.7/site-packages', '/usr/lib64/python2.7/site-packages/gtk-2.0', '/usr/lib/python2.7/site-packages'] Looks like '/usr/share/vdsm' take precedence over '/usr/lib64/python2.7/site-packages'. Can someone suggests a way to fix this issue? Note: '/sbin/gluster-eventsapi' works perfectly while running directly from CLI. Related vdsm patch: https://gerrit.ovirt.org/#/c/67168/2/vdsm/gluster/events.py If an import fails because /usr/share/vdsm/foo hides a package in /usr/lib/python2.7/site-packages/vdsm it means that you have a wrong import, you need to import anything like this: from vdsm.foo import bar So names in /usr/share/vdsm/* cannot hide names in vdsm package. Here its a different case. Some external python script (which I am trying to run from vdsm verb) is trying to import some gluster package from /usr/lib/python2.7/site-packages/gluster. But it is conflicting with gluster modules under /usr/share/vdsm/gluster Python Script which I am trying to run: https://github.com/gluster/glusterfs/blob/master/events/src/peer_eventsapi.py Regards, Ramesh Nir ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] 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" > > To: "Aravinda" , igoih...@redhat.com > > Cc: "Ramesh Nachimuthu" , "Nir Soffer" > > , "devel" , "Piotr > > Kliczewski" , "Yaniv Bronheim" , > > "Sahina Bose" > > 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. ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] python path conflicts issues while running python scripts from vdsm verbs
- Original Message - > From: "Dan Kenigsberg" > To: "Aravinda" , igoih...@redhat.com > Cc: "Ramesh Nachimuthu" , "Nir Soffer" > , "devel" , "Piotr > Kliczewski" , "Yaniv Bronheim" , > "Sahina Bose" > 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. Regards, Ramesh > > On Wed, Nov 30, 2016 at 11:27:06AM +0530, Aravinda wrote: > > If vdsm setting PYTHONPATH as below > > > > export PYTHONPATH=/usr/share/vdsm:$PYTHONPATH > > > > change it as > > > > export PYTHONPATH=$PYTHONPATH:/usr/share/vdsm > > > > With this change, /usr/share/vdsm will not get first preference compared to > > /usr/lib*/python2.7/site-packages > > > > regards > > Aravinda > > > > On 11/30/2016 10:25 AM, Ramesh Nachimuthu wrote: > > > > > > > > > > > > - Original Message - > > > > From: "Nir Soffer" > > > > To: "Ramesh Nachimuthu" > > > > Cc: "devel" , "Dan Kenigsberg" , > > > > "Piotr Kliczewski" , "Yaniv > > > > Bronheim" , "Aravinda Vishwanathapura Krishna > > > > Murthy" , "Sahina Bose" > > > > > > > > Sent: Tuesday, November 29, 2016 7:20:57 PM > > > > Subject: Re: python path conflicts issues while running python scripts > > > > from vdsm verbs > > > > > > > > On Tue, Nov 29, 2016 at 3:26 PM, Ramesh Nachimuthu > > > > > > > > wrote: > > > > > Hi, > > > > > > > > > > I am trying to run a python script '/sbin/gluster-eventsapi' in > > > > > vdsm verb > > > > > which internally imports some python modules from > > > > > /usr/lib/python2.7/site-packages/gluster/cliutils/. But it fails > > > > > with the > > > > > import error. Following error is seen in the supervdsm log. > > > > > > > > > > > > > > > MainProcess|Thread-1::DEBUG::2016-11-28 > > > > > 16:54:35,130::commands::93::root::(execCmd) FAILED: = > > > > > 'Traceback > > > > > (most recent call last):\n File "/sbin/gluster-eventsapi", line 25, > > > > > in > > > > > \nfrom gluster.cliutils.cliutils import (Cmd, execute, > > > > > node_output_ok, node_output_notok,\nImportError: No module named > > > > > cliutils.cliutils\n'; = 1 > > > > > > > > > > > > > > > I think the import statement "from gluster.cliutils.cliutils import > > > > > (Cmd, > > > > > execute, node_output_ok, node_output_notok)" in the python script > > > > > resolves > > > > > to '/usr/share/vdsm/gluster' instead of > > > > > /usr/lib/python2.7/site-packages/gluster/cliutils. > > > > > > > > > > I see the following in python sys.path while executing a python > > > > > script from > > > > > vdsm. > > > > > > > > > > ['/usr/libexec/glusterfs', '/usr/share/vdsm', > > > > > '/usr/lib64/python27.zip', > > > > > '/usr/lib64/python2.7', '/usr/lib64/python2.7/plat-linux2', > > > > > '/usr/lib64/python2.7/lib-tk', '/usr/lib64/python2.7/lib-old', > > > > > '/usr/lib64/python2.7/lib-dynload', > > > > > '/usr/lib64/python2.7/site-packages', > > > > > '/usr/lib64/python2.7/site-packages/gtk-2.0', > > > > > '/usr/lib/python2.7/site-packages'] > > > > > > > > > > Looks like '/usr/share/vdsm' take precedence over > > > > > '/usr/lib64/python2.7/site-packages'. > > > > > > > > > > Can someone suggests a way to fix this issue? > > > > > > > > > > Note: '/sbin/gluster-eventsapi' works perfectly while running > > > > > directly from > > > > > CLI. > > > > > > > > > > Related vdsm patch: > > > > > https://gerrit.ovirt.org/#/c/67168/2/vdsm/gluster/events.py > > > > If an import fails because /usr/share/vdsm/foo hides a package in > > > > /usr/lib/python2.7/site-packages/vdsm it means that you have a wrong > > > > import, you need to import anything like this: > > > > > > > > from vdsm.foo import bar > > > > > > > > So names in /usr/share/vdsm/* cannot hide names in vdsm package. > > > > > > > Here its a different case. Some external python script (which I am trying > > > to run from vdsm verb) is trying to import some gluster package from > > > /usr/lib/python2.7/site-packages/gluster. But it is conflicting with > > > gluster modules under /usr/share/vdsm/gluster > > > > > > Python Script which I am trying to run: > > > https://github.com/gluster/glusterfs/blob/master/events/src/peer_eventsapi.py > > > > > > Regards, > > > Ramesh > ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] 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. On Wed, Nov 30, 2016 at 11:27:06AM +0530, Aravinda wrote: > If vdsm setting PYTHONPATH as below > > export PYTHONPATH=/usr/share/vdsm:$PYTHONPATH > > change it as > > export PYTHONPATH=$PYTHONPATH:/usr/share/vdsm > > With this change, /usr/share/vdsm will not get first preference compared to > /usr/lib*/python2.7/site-packages > > regards > Aravinda > > On 11/30/2016 10:25 AM, Ramesh Nachimuthu wrote: > > > > > > > > - Original Message - > > > From: "Nir Soffer" > > > To: "Ramesh Nachimuthu" > > > Cc: "devel" , "Dan Kenigsberg" , > > > "Piotr Kliczewski" , "Yaniv > > > Bronheim" , "Aravinda Vishwanathapura Krishna > > > Murthy" , "Sahina Bose" > > > > > > Sent: Tuesday, November 29, 2016 7:20:57 PM > > > Subject: Re: python path conflicts issues while running python scripts > > > from vdsm verbs > > > > > > On Tue, Nov 29, 2016 at 3:26 PM, Ramesh Nachimuthu > > > wrote: > > > > Hi, > > > > > > > > I am trying to run a python script '/sbin/gluster-eventsapi' in vdsm > > > > verb > > > > which internally imports some python modules from > > > > /usr/lib/python2.7/site-packages/gluster/cliutils/. But it fails with > > > > the > > > > import error. Following error is seen in the supervdsm log. > > > > > > > > > > > > MainProcess|Thread-1::DEBUG::2016-11-28 > > > > 16:54:35,130::commands::93::root::(execCmd) FAILED: = 'Traceback > > > > (most recent call last):\n File "/sbin/gluster-eventsapi", line 25, in > > > > \nfrom gluster.cliutils.cliutils import (Cmd, execute, > > > > node_output_ok, node_output_notok,\nImportError: No module named > > > > cliutils.cliutils\n'; = 1 > > > > > > > > > > > > I think the import statement "from gluster.cliutils.cliutils import > > > > (Cmd, > > > > execute, node_output_ok, node_output_notok)" in the python script > > > > resolves > > > > to '/usr/share/vdsm/gluster' instead of > > > > /usr/lib/python2.7/site-packages/gluster/cliutils. > > > > > > > > I see the following in python sys.path while executing a python script > > > > from > > > > vdsm. > > > > > > > > ['/usr/libexec/glusterfs', '/usr/share/vdsm', '/usr/lib64/python27.zip', > > > > '/usr/lib64/python2.7', '/usr/lib64/python2.7/plat-linux2', > > > > '/usr/lib64/python2.7/lib-tk', '/usr/lib64/python2.7/lib-old', > > > > '/usr/lib64/python2.7/lib-dynload', > > > > '/usr/lib64/python2.7/site-packages', > > > > '/usr/lib64/python2.7/site-packages/gtk-2.0', > > > > '/usr/lib/python2.7/site-packages'] > > > > > > > > Looks like '/usr/share/vdsm' take precedence over > > > > '/usr/lib64/python2.7/site-packages'. > > > > > > > > Can someone suggests a way to fix this issue? > > > > > > > > Note: '/sbin/gluster-eventsapi' works perfectly while running directly > > > > from > > > > CLI. > > > > > > > > Related vdsm patch: > > > > https://gerrit.ovirt.org/#/c/67168/2/vdsm/gluster/events.py > > > If an import fails because /usr/share/vdsm/foo hides a package in > > > /usr/lib/python2.7/site-packages/vdsm it means that you have a wrong > > > import, you need to import anything like this: > > > > > > from vdsm.foo import bar > > > > > > So names in /usr/share/vdsm/* cannot hide names in vdsm package. > > > > > Here its a different case. Some external python script (which I am trying > > to run from vdsm verb) is trying to import some gluster package from > > /usr/lib/python2.7/site-packages/gluster. But it is conflicting with > > gluster modules under /usr/share/vdsm/gluster > > > > Python Script which I am trying to run: > > https://github.com/gluster/glusterfs/blob/master/events/src/peer_eventsapi.py > > > > Regards, > > Ramesh ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] python path conflicts issues while running python scripts from vdsm verbs
- Original Message - > From: "Nir Soffer" > To: "Ramesh Nachimuthu" > Cc: "devel" , "Dan Kenigsberg" , "Piotr > Kliczewski" , "Yaniv > Bronheim" , "Aravinda Vishwanathapura Krishna Murthy" > , "Sahina Bose" > > Sent: Tuesday, November 29, 2016 7:20:57 PM > Subject: Re: python path conflicts issues while running python scripts from > vdsm verbs > > On Tue, Nov 29, 2016 at 3:26 PM, Ramesh Nachimuthu > wrote: > > Hi, > > > > I am trying to run a python script '/sbin/gluster-eventsapi' in vdsm verb > > which internally imports some python modules from > > /usr/lib/python2.7/site-packages/gluster/cliutils/. But it fails with the > > import error. Following error is seen in the supervdsm log. > > > > > > MainProcess|Thread-1::DEBUG::2016-11-28 > > 16:54:35,130::commands::93::root::(execCmd) FAILED: = 'Traceback > > (most recent call last):\n File "/sbin/gluster-eventsapi", line 25, in > > \nfrom gluster.cliutils.cliutils import (Cmd, execute, > > node_output_ok, node_output_notok,\nImportError: No module named > > cliutils.cliutils\n'; = 1 > > > > > > I think the import statement "from gluster.cliutils.cliutils import (Cmd, > > execute, node_output_ok, node_output_notok)" in the python script resolves > > to '/usr/share/vdsm/gluster' instead of > > /usr/lib/python2.7/site-packages/gluster/cliutils. > > > > I see the following in python sys.path while executing a python script from > > vdsm. > > > > ['/usr/libexec/glusterfs', '/usr/share/vdsm', '/usr/lib64/python27.zip', > > '/usr/lib64/python2.7', '/usr/lib64/python2.7/plat-linux2', > > '/usr/lib64/python2.7/lib-tk', '/usr/lib64/python2.7/lib-old', > > '/usr/lib64/python2.7/lib-dynload', '/usr/lib64/python2.7/site-packages', > > '/usr/lib64/python2.7/site-packages/gtk-2.0', > > '/usr/lib/python2.7/site-packages'] > > > > Looks like '/usr/share/vdsm' take precedence over > > '/usr/lib64/python2.7/site-packages'. > > > > Can someone suggests a way to fix this issue? > > > > Note: '/sbin/gluster-eventsapi' works perfectly while running directly from > > CLI. > > > > Related vdsm patch: > > https://gerrit.ovirt.org/#/c/67168/2/vdsm/gluster/events.py > > If an import fails because /usr/share/vdsm/foo hides a package in > /usr/lib/python2.7/site-packages/vdsm it means that you have a wrong > import, you need to import anything like this: > > from vdsm.foo import bar > > So names in /usr/share/vdsm/* cannot hide names in vdsm package. > Here its a different case. Some external python script (which I am trying to run from vdsm verb) is trying to import some gluster package from /usr/lib/python2.7/site-packages/gluster. But it is conflicting with gluster modules under /usr/share/vdsm/gluster Python Script which I am trying to run: https://github.com/gluster/glusterfs/blob/master/events/src/peer_eventsapi.py Regards, Ramesh > Nir > ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] python path conflicts issues while running python scripts from vdsm verbs
On Tue, Nov 29, 2016 at 3:26 PM, Ramesh Nachimuthu wrote: > Hi, > > I am trying to run a python script '/sbin/gluster-eventsapi' in vdsm verb > which internally imports some python modules from > /usr/lib/python2.7/site-packages/gluster/cliutils/. But it fails with the > import error. Following error is seen in the supervdsm log. > > > MainProcess|Thread-1::DEBUG::2016-11-28 > 16:54:35,130::commands::93::root::(execCmd) FAILED: = 'Traceback (most > recent call last):\n File "/sbin/gluster-eventsapi", line 25, in \n > from gluster.cliutils.cliutils import (Cmd, execute, node_output_ok, > node_output_notok,\nImportError: No module named cliutils.cliutils\n'; = > 1 > > > I think the import statement "from gluster.cliutils.cliutils import (Cmd, > execute, node_output_ok, node_output_notok)" in the python script resolves to > '/usr/share/vdsm/gluster' instead of > /usr/lib/python2.7/site-packages/gluster/cliutils. > > I see the following in python sys.path while executing a python script from > vdsm. > > ['/usr/libexec/glusterfs', '/usr/share/vdsm', '/usr/lib64/python27.zip', > '/usr/lib64/python2.7', '/usr/lib64/python2.7/plat-linux2', > '/usr/lib64/python2.7/lib-tk', '/usr/lib64/python2.7/lib-old', > '/usr/lib64/python2.7/lib-dynload', '/usr/lib64/python2.7/site-packages', > '/usr/lib64/python2.7/site-packages/gtk-2.0', > '/usr/lib/python2.7/site-packages'] > > Looks like '/usr/share/vdsm' take precedence over > '/usr/lib64/python2.7/site-packages'. > > Can someone suggests a way to fix this issue? > > Note: '/sbin/gluster-eventsapi' works perfectly while running directly from > CLI. > > Related vdsm patch: > https://gerrit.ovirt.org/#/c/67168/2/vdsm/gluster/events.py If an import fails because /usr/share/vdsm/foo hides a package in /usr/lib/python2.7/site-packages/vdsm it means that you have a wrong import, you need to import anything like this: from vdsm.foo import bar So names in /usr/share/vdsm/* cannot hide names in vdsm package. Nir ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
