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

AWS ec2 instances to be charged by seconds from Oct 2nd #21

Open
ambirag opened this issue Sep 19, 2017 · 9 comments
Open

AWS ec2 instances to be charged by seconds from Oct 2nd #21

ambirag opened this issue Sep 19, 2017 · 9 comments

Comments

@ambirag
Copy link

ambirag commented Sep 19, 2017

This means the current logic of cost optimized termination has to be changed to immediately terminate nodes ( say 5mins idle time - or make it configurable)
Source:
https://aws.amazon.com/blogs/aws/new-per-second-billing-for-ec2-instances-and-ebs-volumes/

@mhardin
Copy link
Owner

mhardin commented Oct 2, 2017

@ambirag I did see this announcement. I'm not sure what the best way is going to be, as we're going to want to balance cost vs time to spin up new resources.

@ambirag
Copy link
Author

ambirag commented Oct 18, 2017

@mhardin also this is applicable only to AWS linux boxes (not other flavours)

@lenny1806
Copy link

@ambirag @mhardin
I was thinking about a 5 minute idle time, if no new tests are coming in during that time the instance will terminated. Also the possibility of setting this idle time via the property file would be nice.
What do you guys think?

@mhardin
Copy link
Owner

mhardin commented Oct 20, 2017

@lenny1806 a 5 minute / configurable amount would be fine. Only problem is this is likely going to require a large refactor to accomplish this logic.

@bloomm09
Copy link

@mhardin Has this issues been given any more thought? I'm currently reviewing the project to possibly tie it into my test framework. Thanks

@mhardin
Copy link
Owner

mhardin commented Jan 30, 2018

@bloomm09 unfortunately no update on my end

@ambirag
Copy link
Author

ambirag commented Mar 20, 2018

I have local changes which terminates nodes based on seconds, if you pass -D property name otherwise it will retain old behaviour

@ambirag
Copy link
Author

ambirag commented Mar 21, 2018

actually you dont need this feature, once your automation execution is completed, you can simply call curl -X "DELETE" http://HUBIP:4444/grid/admin/AutomationTestRunServlet and it will simply terminate all the nodes @bloomm09

@mhardin
Copy link
Owner

mhardin commented Mar 21, 2018

@ambirag calling the delete endpoint only works if you have at most one test run against a hub. If you have multiple test runs against a single hub, I don't think this will work well.

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

4 participants