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 Micro instance for EC2 by replacing the underlying SDK. #12

Closed
wants to merge 4 commits into from
Closed

Add Micro instance for EC2 by replacing the underlying SDK. #12

wants to merge 4 commits into from

Conversation

frankbille
Copy link
Contributor

In order to fix JENKINS-10331 (adding micro instance type), it was
needed to replace the old Amazon AWS library (Typica) with Amazons own
SDK for Java. Typica hasn't been updated in over a year and the project
seems to have stagnated. Now that Amazon has picked up the ball we might
as well use that.

Not all tests pass right now.

In order to fix JENKINS-10331 (adding micro instance type), it was
needed to replace the old Amazon AWS library (Typica) with Amazons own
SDK for Java. Typica hasn't been updated in over a year and the project
seems to have stagnated. Now that Amazon has picked up the ball we might
as well use that.
will override what has been set in parent, causing errors in the unit
tests.
@frankbille
Copy link
Contributor Author

The unit tests run again.

@francisu
Copy link
Member

Hi Frank, thanks for this work. I have been trying to get this working and having trouble. I'm also interested in being the maintainer on this plugin, so that I can get your work committed and do some of my own (built on top of what you have done). Here is a link to my tale of woe: https://groups.google.com/group/jenkinsci-dev/browse_thread/thread/157848c64d07590a?hl=en If you have any ideas of what I'm doing wrong, I would appreciate the help. And if you are interested in being the maintainer for this, I'm happy to support you doing it (I think all you have to do is ask on the list). I don't want to step on any toes about this.

@francisu
Copy link
Member

(Also to be clear, two of the unit tests seem to be failing in the current version of the ec2-plugin, both from the main repository and from your patch, so I think that either I'm doing something wrong or something changed in the core that broke them)

@frankbille
Copy link
Contributor Author

I have replied on your thread. I am interested in this plugin, and I have a plan of refactor it so it would be able to support Windows AMI's as well. I don't think I am ready to maintain it though, due to lack of time.

@francisu
Copy link
Member

I would prefer to use #14 instead of this pull request if that's OK with everyone.

@frankbille
Copy link
Contributor Author

Sounds fine, lets take it there.

@frankbille frankbille closed this Jan 26, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants