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 fails to honor 'password' in cloud options before raising an exception #24871

Closed
dkiser opened this issue Jun 22, 2015 · 0 comments
Closed

Comments

@dkiser
Copy link
Contributor

dkiser commented Jun 22, 2015

salt-cloud bootstrap method https://github.com/saltstack/salt/blob/develop/salt/utils/cloud.py#L318 does not honor 'password' or 'win_password' options in the cloud config but explicitly requires them to be set in the context of the vm_. We should fix this so the cloud options are honored if the vm_ doesn't explicitly have a password set, but one is set higher in the deep copy hierarchy.

salt versions:

           Salt: 2015.5.2
         Python: 2.7.5 (default, Jun 17 2014, 18:11:42)
         Jinja2: 2.7.3
       M2Crypto: 0.21.1
 msgpack-python: 0.4.6
   msgpack-pure: Not Installed
       pycrypto: 2.6.1
        libnacl: Not Installed
         PyYAML: 3.11
          ioflo: Not Installed
          PyZMQ: 14.3.1
           RAET: Not Installed
            ZMQ: 3.2.5
           Mako: Not Installed

Will probably throw i a PR on 2015.5 branch for fix as I feel its a bug and use latest develop that includes the win_password to kill 2 birds with one stone.

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

No branches or pull requests

2 participants