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

Salt-cloud crypto dependency fails. #59

Closed
andrew-vant opened this issue Oct 31, 2014 · 6 comments
Closed

Salt-cloud crypto dependency fails. #59

andrew-vant opened this issue Oct 31, 2014 · 6 comments

Comments

@andrew-vant
Copy link
Contributor

I rebuilt my salt master today and the salt.cloud state refuses to run, saying that the attempt to pip.install package 'crypto' failed. I tried it manually and it also failed. I looked at the package index located here: https://pypi.python.org/pypi?%3Aaction=index and see no package named 'crypto.'

I am confused, because this worked a month ago. Did the package get renamed in pip recently, or something? Shouldn't the distribution-provided crypto package be getting used anyway, if possible?

@nmadhok
Copy link
Member

nmadhok commented Oct 31, 2014

@andrew-vant Can you post more details such as salt --versions-report, what distro and OS are you on, what does your state look like, log output or any other relevant information?

@andrew-vant
Copy link
Contributor Author

Ubuntu 14.04. salt --versions-report gives me this:

           Salt: 2014.1.13
         Python: 2.7.6 (default, Mar 22 2014, 22:59:56)
         Jinja2: 2.7.2
       M2Crypto: 0.21.1
 msgpack-python: 0.3.0
   msgpack-pure: Not Installed
       pycrypto: 2.6.1
         PyYAML: 3.10
          PyZMQ: 14.0.1
            ZMQ: 4.0.4

The state I'm trying to run is salt.cloud, which reported succeded:12, failed: 2. The output from the two that failed is:


      ID: salt-cloud
Function: pkg.installed
  Result: False
 Comment: One or more requisite failed
 Changes:   

      ID: crypto
Function: pip.installed
  Result: False
 Comment: Failed to install package crypto. Error: Downloading/unpacking crypto
            Could not find any downloads that satisfy the requirement crypto
          Cleaning up...
          No distributions at all found for crypto
          Storing debug log for failure in /root/.pip/pip.log
 Changes:   

I gathered from this that the salt-cloud package install has a require: for the crypto pip install, which is what's actually failing. I checked the source and that was the case (see line 23-29 in salt/cloud.sls). To determine if the problem was the salt state or something with the package itself, I tried to install it from the command line:

root@ajv-salt:/srv# pip install crypto
Downloading/unpacking crypto
  Could not find any downloads that satisfy the requirement crypto
Cleaning up...
No distributions at all found for crypto
Storing debug log for failure in /root/.pip/pip.log

Which looks to me like the package doesn't exist. I checked the PyPI index mentioned in the previous comment and don't see it, which seems to confirm that. At that point I came to report it here: cloud.sls has a requisite for a package that doesn't appear to exist.

You can repro by running salt-bootstrap on a clean Ubuntu 14.04 install, followed by applying the salt.cloud state.

@andrew-vant
Copy link
Contributor Author

A bit more confirmation: I pulled up another salt master I set up a month or so ago, using the same formula. It has a pip package called crypto installed. Trying to upgrade that package fails with the same message given above.

So it definitely was there before, and definitely isn't now. Someone renamed or removed it from PyPI.

nmadhok added a commit that referenced this issue Nov 6, 2014
@nmadhok nmadhok added the bug label Nov 6, 2014
@nmadhok nmadhok assigned nmadhok and techhat and unassigned nmadhok and techhat Nov 6, 2014
@nmadhok
Copy link
Member

nmadhok commented Nov 13, 2014

@playfab-configmanagement Can you try again with the updated formula?

@highlyunavailable
Copy link
Contributor

Sorry, posted with the wrong account.

Yes, I made a fork that has crypto removed after posting this - consider this resolved for me.

@gravyboat
Copy link
Contributor

Closing.

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