Till Westmann has posted comments on this change. Change subject: Add AWS automation scripts to asterix-server. ......................................................................
Patch Set 7: (3 comments) Just a few small comments so far. One bigger question I have if this should be a separate module. Maybe we can discuss this on the list? In any case this decision can be taken late on as well, so we don't need to block progress on this. https://asterix-gerrit.ics.uci.edu/#/c/1475/7/asterixdb/asterix-server/src/main/aws/ansible/instance_start.yml File asterixdb/asterix-server/src/main/aws/ansible/instance_start.yml: PS7, Line 26: 8u60 8u60 seems old - 8u121 is the current version. PS7, Line 29: jre Is a JRE enough? Just curious - it should be, but I've never tried that. https://asterix-gerrit.ics.uci.edu/#/c/1475/7/asterixdb/asterix-server/src/main/aws/conf/aws_settings.yml File asterixdb/asterix-server/src/main/aws/conf/aws_settings.yml: PS7, Line 24: us-west-1 Please don't use us-west-1 (N. California) as the default as it is quite expensive. us-west-2 (Oregon) seems more reasonable [1]. [1] https://www.concurrencylabs.com/blog/choose-your-aws-region-wisely/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1475 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If4061501e3561a649c3a2bb3068dc257f03c092d Gerrit-PatchSet: 7 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Yingyi Bu <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-HasComments: Yes
