Skip to content
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

Issue with the Python path in the recent builds #8048

Closed
lekshmideepu opened this issue Jul 6, 2017 · 10 comments
Closed

Issue with the Python path in the recent builds #8048

lekshmideepu opened this issue Jul 6, 2017 · 10 comments

Comments

@lekshmideepu
Copy link

We have some issue with the wrong python path. Since yesterday evening, our builds are using the python from the /opt which is 2.7.13. We tried to debug the build. It seems like you are executing the /etc/profile.d/pyenv.sh. For our builds, we were using the system python. (2.7.6)
Problematic build is here.
Due to the wrong python path, it cannot find the mpi4py module.

@meatballhat
Copy link

@lekshmideepu Have you been able to work around this with an explicitly defined PYTHONAPATH?

jrfonseca added a commit to jrfonseca/xdot.py that referenced this issue Jul 6, 2017
It seems system packages can't be found when using
"3.4_with_system_site_packages" python.

Possibly related to travis-ci/travis-ci#8048
@lekshmideepu
Copy link
Author

@meatballhat yes, By explicitly defining the path works.

@jougs
Copy link

jougs commented Jul 7, 2017

@meatballhat: @lekshmideepu and I tried to export PATH=/usr/bin/:$PATH right before installing Python modules via apt-get and that seems to circumvent the problem. To our knowledge, PYTHONPATH is not set explicitly, but just implicitly by the sys.path in the python used.

We believe that the root cause for the problem is that etc/profile.d/pyenv.sh puts the bin directories of the Python installations in /opt/python/ before the system path /usr/bin instead of after. etc/profile.d/pyenv.sh apparently was not executed in the old images (cf. old, new).

@jougs
Copy link

jougs commented Jul 12, 2017

@meatballhat: do you expect this to be resolved quickly? If not, we probably should apply our workaround to our setup and restart the failing jobs. Thanks!

@lekshmideepu
Copy link
Author

@meatballhat : We are waiting for your response. Could you please let us know about the status of the issue.

@lekshmideepu
Copy link
Author

@jougs @meatballhat I have contacted the support via email and received an update about the issue.

@lekshmideepu
Copy link
Author

Travis team is working on this issue and they confirmed that it is a bug. One suggested workaround is to use "language: minimal" in the .travis.yml. This will route the builds to the Connie stack, which is the stack in which Python builds ran before the update. After the update, Python builds are now running in the Sugilite stack and it appears to be overwriting the PYTHONPATH.

@Lyoness
Copy link
Contributor

Lyoness commented Jul 20, 2017

Thanks for sharing the workaround @lekshmideepu and for submitting this issue.

We've been able to push up changes to all image stacks using python. Currently they are available in group: edge, which you can specify in your .travis.yml to test if the path issues get fixed. Let us know if this works for you. We're working on promoting edge to group: stable soon.

@lekshmideepu
Copy link
Author

lekshmideepu commented Jul 21, 2017

@Lyoness Thanks for the update. I tried with the group option edge, unfortunately PythonLibs for version3 are not found as shown here.
On the other hand, if I use the deprecated option, it works fine.

mintar added a commit to uos/sick_tim that referenced this issue Aug 14, 2017
Due to a change in Travis, the default Debian python path is now
ignored. This commit works around that issue by setting the PYTHONPATH
explicitly.
mintar added a commit to uos/basler_tof that referenced this issue Aug 14, 2017
Due to a change in Travis, the default Debian python path is now
ignored. This commit works around that issue by setting the PYTHONPATH
explicitly.
mintar added a commit to uos/diffdrive_gazebo_plugin that referenced this issue Aug 14, 2017
Due to a change in Travis, the default Debian python path is now
ignored. This commit works around that issue by setting the PYTHONPATH
explicitly.
mintar added a commit to uos/katana_driver that referenced this issue Aug 14, 2017
Due to a change in Travis, the default Debian python path is now
ignored. This commit works around that issue by setting the PYTHONPATH
explicitly.
mintar added a commit to ros-drivers/phidgets_drivers that referenced this issue Aug 14, 2017
Due to a change in Travis, the default Debian python path is now
ignored. This commit works around that issue by setting the PYTHONPATH
explicitly.
mintar added a commit to SICKAG/sick_ldmrs_laser that referenced this issue Aug 14, 2017
Due to a change in Travis, the default Debian python path is now
ignored. This commit works around that issue by setting the PYTHONPATH
explicitly.
mintar added a commit to uos/volksbot_driver that referenced this issue Aug 14, 2017
Due to a change in Travis, the default Debian python path is now
ignored. This commit works around that issue by setting the PYTHONPATH
explicitly.
@stale
Copy link

stale bot commented Apr 13, 2018

Thanks for contributing to this issue. As it has been 90 days since the last activity, we are automatically closing the issue. This is often because the request was already solved in some way and it just wasn't updated or it's no longer applicable. If that's not the case, please do feel free to either reopen this issue or open a new one. We'll gladly take a look again! You can read more here: https://blog.travis-ci.com/2018-03-09-closing-old-issues

@stale stale bot added the stale label Apr 13, 2018
@stale stale bot closed this as completed Apr 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants