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.

Sign up for GitHub

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

Jump to bottom

Handle DescribeInstances failures when creating new instance #414

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

Handle DescribeInstances failures when creating new instance #414

wants to merge 2 commits into from
+26 −5

Commits on Oct 31, 2019

  1. Wait up to 1 minute to describe new instance 

    Sometimes DescribeInstances takes a very long time to return data for
new instances. Wait up to 1 minute in this case before assuming that the
instance is dead.
    @dbnicholson
    dbnicholson committed Oct 31, 2019
    Configuration menu
    Copy the full SHA
    18143a9 View commit details
    Browse the repository at this point in the history

Commits on Nov 1, 2019

  1. Try to terminate new instance if no data could be retrieved 

    If DescribeInstances did not return any data, send a termination request
for it. If it shows up later, it would be included in capacity
calculations and Jenkins could refuse to instantiate new nodes if the
maximum capacity was reached.
    @dbnicholson
    dbnicholson committed Nov 1, 2019
    Configuration menu
    Copy the full SHA
    3d39e47 View commit details
    Browse the repository at this point in the history