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 ec2 module #1571

Merged
merged 1 commit into from
Nov 9, 2012
Merged

add ec2 module #1571

merged 1 commit into from
Nov 9, 2012

Conversation

skvidal
Copy link
Contributor

@skvidal skvidal commented Nov 9, 2012

No description provided.

mpdehaan added a commit that referenced this pull request Nov 9, 2012
@mpdehaan mpdehaan merged commit 8069068 into ansible:devel Nov 9, 2012
@mpdehaan
Copy link
Contributor

mpdehaan commented Nov 9, 2012

Thanks! This is good to have.

I am curious since you mentioned it, what does the sys.argv.append do? It seems like that would append Ansible's sys.argv and it's already into Euca API territory at that point.

merged in!

@skvidal
Copy link
Contributor Author

skvidal commented Nov 9, 2012

I'm using the cli interface for the euca2ools - everything else you can pass in as an option to the class - except for the image id. You have to pass that in as an argument. Otherwise it will print out the --help text. :-/

@skvidal
Copy link
Contributor Author

skvidal commented Nov 9, 2012

I've been told that the next big release of euca2ools will have a new cli interface that lets things be more sane - but not yet.

Also since this module is being run somewhere else and since modules get their arguments included internally to them - it should be appending to sys.argv wherever it has been run - which should be going from [commandname] [commandname, imageid]

mpdehaan added a commit that referenced this pull request Nov 20, 2012
@ansible ansible locked and limited conversation to collaborators Apr 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants