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

Future python 2.7 module missing with linchpin 1.7.6.2 #1269

Closed
ssegalredhat opened this issue Aug 13, 2019 · 5 comments
Closed

Future python 2.7 module missing with linchpin 1.7.6.2 #1269

ssegalredhat opened this issue Aug 13, 2019 · 5 comments

Comments

@ssegalredhat
Copy link

Describe the bug

Linchpin 1.7.6.2 fails on RHEL 7.6 with Python 2.7:
Traceback (most recent call last):
File "/usr/bin/linchpin", line 7, in
from linchpin.shell import runcli
File "/usr/lib/python2.7/site-packages/linchpin/init.py", line 19, in
from linchpin.hooks.state import State
File "/usr/lib/python2.7/site-packages/linchpin/hooks/init.py", line 52, in
import git
File "/usr/lib/python2.7/site-packages/git/init.py", line 46, in
from git.repo import Repo # @nomove @IgnorePep8
File "/usr/lib/python2.7/site-packages/git/repo/init.py", line 4, in
from .base import *
File "/usr/lib/python2.7/site-packages/git/repo/base.py", line 7, in
from builtins import str
ImportError: No module named builtins

To Reproduce
Steps to reproduce the behavior:

  1. on RHEL 7.6, install linchpin 1.7.6.2 - pip install linchpin==1.7.6.2
  2. linchpin -w linchpin-openstack -p PinFile.fedora --creds-path linchpin-openstack/credentials -v up
    Expected behavior
    Will not fail on missing module
@samvarankashyap
Copy link
Contributor

@ssegalredhat
It should work once you install future package

pip install future

@ssegalredhat
Copy link
Author

ssegalredhat commented Aug 13, 2019

@samvarankashyap yes, but it should be installed as dependency when I install linchpin, no?

@samvarankashyap
Copy link
Contributor

@ssegalredhat
I agree with you.
However, Due to the lack of testing on rhel images we were not aware of that.
Will try to fix this in next release.

@samvarankashyap
Copy link
Contributor

@ssegalredhat #1270 should fix this issue

@samvarankashyap
Copy link
Contributor

fixed by #1270

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

2 participants