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

Dev #3

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Dev #3

wants to merge 15 commits into from

Conversation

bodepd
Copy link
Owner

@bodepd bodepd commented Mar 13, 2013

No description provided.

Dan Bode added 14 commits December 13, 2012 14:39
vms may be in the stopping or destroyed state should
not be treated as present.
so that subsequent objects can inspect it.
to 600 so that ssh will use them by default
just adds some extra checking to prevent nil
exceptions when there are no networks defined.
Adds extra error checking to prevent failures
when there are not key pairs defined.
b/c some of the properties being set are not set
in the resource, but parsed from the creation
response.
This resource supports installing puppet
enterprise, and classifying agents.
to demonstrate creation of supported objects.
cleans up some commented out code, fixes formatting
so that you can refer to previously created key pairs.

also, ensure that any specified key pairs are 
auto-required.
@bodepd
Copy link
Owner Author

bodepd commented Mar 15, 2013

@ke4qqq do you mind reviewing this and giving me the +1 to merge? Don't worry about the puppet_node stuff, I am probably going to rip it out and make it a separate project.

@ke4qqq
Copy link
Contributor

ke4qqq commented Mar 16, 2013

Yeah - I'll look at it next week.

@bodepd
Copy link
Owner Author

bodepd commented Mar 18, 2013

@ke4qqq if you want to set up a time, it could go faster if we just paired on this for a couple of hours.

@larstobi
Copy link

@ke4qqq @bodepd Are there any updates on this?

@bodepd
Copy link
Owner Author

bodepd commented Nov 21, 2013

I never quite looped back around to it and don't have a cloudstack environment to test against.

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.

3 participants