geotrellis / geotrellis-site

Documentation and site for GeoTrellis.
Apache License 2.0
5 stars 10 forks source link

Remove default for SSL certificate ARN #83

Closed tnation14 closed 6 years ago

tnation14 commented 6 years ago

Overview

This PR removes the default value for ssl_certificate_arn (which is now outdated) from variables.tf, and updates scripts/infra to use a .tfvars file to make it easier to override other defaults in the future.

Changes

Fixes #82

Testing

hectcastro commented 6 years ago

The build is currently failing. Looks like the static Dockerfile is referencing a node package, but now it should be nodejs.

tnation14 commented 6 years ago

Fixed. I was also able to take care of the jai_core resolution failure by adding it to SBT librarayDependencies with an explicit URL.

hectcastro commented 6 years ago

If you are confident that fixed it, then consider closing https://github.com/geotrellis/geotrellis-site/issues/79 too.