Teevity / ice

AWS Usage Tool
2.86k stars 435 forks source link

src/java/ice.properties has an s3- before prepend to FQDN #277

Open Virtui opened 7 years ago

Virtui commented 7 years ago

Causes host resolution fail when determining bucket location. Currently causes UnknownHostException: bucketname.s3-us-east-1.amazonaws.com instead of bucketname.s3.us-east-1.amazonaws.com

ltagliamonte commented 6 years ago

@Virtui leave empty ice.billing_s3bucketregion= works for me, because of this https://github.com/Teevity/ice/blob/master/src/java/com/netflix/ice/common/AwsUtils.java#L98

Juberstine commented 6 years ago

I'd imagine a conditional could be added that if set to us-east-1, it works correctly. Anyways, thanks for the info @ltagliamonte. I actually figured it out on my own and then found the issue here later. Took about 2 hours to debug.

benwaffle commented 6 years ago

s3-us-east-1.amazonaws.com is not a valid endpoint

https://docs.aws.amazon.com/general/latest/gr/rande.html?shortFooter=true#s3_region

nfonrose commented 6 years ago

Hi Jeff,

Thanks for the suggestion. You’re right that a condition could be added to make this problem go away.

Are you familiar with Groovy/Java and how to make Pull Requests? If you’re not, it’s not an issue. Just let us know and we’ll do it.

Nicolas

Le mer. 17 janv. 2018 à 22:05, Jeff Uberstine notifications@github.com a écrit :

I'd imagine a conditional could be added that if set to us-east-1, it works correctly. Anyways, thanks for the info @ltagliamonte https://github.com/ltagliamonte. I actually figured it out on my own and then found the issue here later. Took about 2 hours to debug.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Teevity/ice/issues/277#issuecomment-358445004, or mute the thread https://github.com/notifications/unsubscribe-auth/AACUpYlONTA44TxcEx8VL_tQlrWTJdkRks5tLmCwgaJpZM4Qi0kK .

--

Nicolas Fonrose | Teevity | Founder +33.6.61.35.43.31 https://teevity.com - Cloud Costs Analytics built on NetflixOSS twitter - @nfonrose / @teevity