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 new South America AZ(sa-east-1c) #154

Merged
merged 1 commit into from Feb 26, 2015

Conversation

ansoni
Copy link
Contributor

@ansoni ansoni commented Feb 26, 2015

add sa-east-1c and an error statement so we get more than an ugly NullPointer

… we get more than an ugly NullPointer exception
@cloudbees-pull-request-builder

ice-pull-requests #66 SUCCESS
This pull request looks good

@colinbjohnson
Copy link

I just implemented this and it resolved the issue described here https://groups.google.com/forum/#!topic/iceusers/B7NT4YkPEUI. Note that I was having the same issue.

vfilanovsky added a commit that referenced this pull request Feb 26, 2015
Add new South America AZ(sa-east-1c)
@vfilanovsky vfilanovsky merged commit 3505dd6 into Teevity:master Feb 26, 2015
@ckelner
Copy link

ckelner commented Feb 26, 2015

@vfilanovsky -- just want to point out here that this patch will still halt ice processor from bootstrapping by throwing an NPE. Is that desired?

Is there any internal attempt to use the AWS API to query for the zones rather than hard coding the zones? (see: http://docs.aws.amazon.com/AWSEC2/latest/APIReference/API_DescribeAvailabilityZones.html)

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.

None yet

5 participants