[salt-cloud] rackconnect and managedcloud fix #10184

Merged
merged 2 commits into from Feb 4, 2014

Projects

None yet

3 participants

@gtmanfred
Member

salt-cloud tries to check .get those values from the nodelist entry. I just added
the default option for each one.

@techhat please take a look.

This also requires libcloud 0.14.0 for the access_ip to be set in the extra dictionary

Fixes #10158

Thanks,
Daniel Wallace

gtmanfred added some commits Feb 4, 2014
@gtmanfred gtmanfred [Salt-cloud] Add function for checking libcloud versions.
Everything else should need 0.13.2, but rackconnect will need 0.14.0.
Add a function to make custom requirements for libcloud as you would
need them.
fefa620
@gtmanfred gtmanfred [Salt-cloud] use list_nodes_full for extra dict
Rackconnect and ManagedCloud require the extra dict in the node data.  I
have also added the default options for those because the values are
not set immediately.
81ce248
@techhat
Member
techhat commented Feb 4, 2014

Thanks @gtmanfred, waiting for the build to finish.

@salt-jenkins
Contributor

Test FAILed.
Refer to this link for build results: http://jenkins.saltstack.com/job/salt-pr-build/1133/

@techhat techhat merged commit 77bece6 into saltstack:develop Feb 4, 2014

1 check failed

default Merged build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment