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

Add necessary dependencies for paramiko #276

Merged
merged 4 commits into from
May 13, 2016
Merged

Conversation

toliver
Copy link
Contributor

@toliver toliver commented May 11, 2016

Fixes #275

Not sure why this has started happening now, as we've had a dependency on paramiko for a while.

DO NOT MERGE YET. Wait for Jenkins results.

@toliver toliver added the queue label May 11, 2016
@toliver
Copy link
Contributor Author

toliver commented May 11, 2016

Actually it failed with a new error on a plain Ubuntu 14.04.4 docker container. Error is the same reported here

ERROR! Unexpected Exception: The 'pycrypto>=2.6' distribution was not found and is required by ansible
the full traceback was:

Traceback (most recent call last):
  File "/tmp/ansible/bin/ansible-playbook", line 73, in <module>
    mycli = getattr(__import__("ansible.cli.%s" % sub, fromlist=[myclass]), myclass)
  File "/tmp/ansible/lib/ansible/cli/playbook.py", line 30, in <module>
    from ansible.executor.playbook_executor import PlaybookExecutor
  File "/tmp/ansible/lib/ansible/executor/playbook_executor.py", line 27, in <module>
    from ansible.executor.task_queue_manager import TaskQueueManager
  File "/tmp/ansible/lib/ansible/executor/task_queue_manager.py", line 28, in <module>
    from ansible.executor.play_iterator import PlayIterator
  File "/tmp/ansible/lib/ansible/executor/play_iterator.py", line 29, in <module>
    from ansible.playbook.block import Block
  File "/tmp/ansible/lib/ansible/playbook/__init__.py", line 25, in <module>
    from ansible.playbook.play import Play
  File "/tmp/ansible/lib/ansible/playbook/play.py", line 27, in <module>
    from ansible.playbook.base import Base
  File "/tmp/ansible/lib/ansible/playbook/base.py", line 35, in <module>
    from ansible.parsing.dataloader import DataLoader
  File "/tmp/ansible/lib/ansible/parsing/dataloader.py", line 33, in <module>
    from ansible.parsing.vault import VaultLib
  File "/tmp/ansible/lib/ansible/parsing/vault/__init__.py", line 67, in <module>
    from cryptography.hazmat.primitives.hashes import SHA256 as c_SHA256
  File "/usr/local/lib/python2.7/dist-packages/cryptography/hazmat/primitives/hashes.py", line 15, in <module>
    from cryptography.hazmat.backends.interfaces import HashBackend
  File "/usr/local/lib/python2.7/dist-packages/cryptography/hazmat/backends/__init__.py", line 7, in <module>
    import pkg_resources
  File "/usr/local/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 2927, in <module>
    @_call_aside
  File "/usr/local/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 2913, in _call_aside
    f(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 2940, in _initialize_master_working_set
    working_set = WorkingSet._build_master()
  File "/usr/local/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 635, in _build_master
    ws.require(__requires__)
  File "/usr/local/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 943, in require
    needed = self.resolve(parse_requirements(requirements))
  File "/usr/local/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 829, in resolve
    raise DistributionNotFound(req, requirers)
DistributionNotFound: The 'pycrypto>=2.6' distribution was not found and is required by ansible

@toliver
Copy link
Contributor Author

toliver commented May 12, 2016

Working correctly on Ubuntu 14.04.4 docker container.

@@ -22,6 +22,10 @@ rm -rf sr-build-tools &
rm -rf ansible &
wait

# This is only necessary for non-desktop images
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

setup_production_machine is for desktop machine

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

True, but maybe having this would allow to test the script on a docker container?

@toliver
Copy link
Contributor Author

toliver commented May 12, 2016

The problem with premature exit is due to the way we run the script with the one liner (see #277).

For the moment I will do a quick fix to merge this branch.

@AndriyPt AndriyPt self-assigned this May 13, 2016
@AndriyPt AndriyPt merged commit 57a5743 into master May 13, 2016
@AndriyPt AndriyPt deleted the B#275_fix_one_liner branch May 13, 2016 10:09
@AndriyPt AndriyPt removed the queue label May 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants