newslynx / opportunities

A garden of NewsLynx futures
1 stars 0 forks source link

Make aws settings optional in automation / core #176

Closed mhkeller closed 8 years ago

mhkeller commented 8 years ago

Currently automation requires two AWS configuration settings. The first is in secrets.yaml which control provisioning I believe when deploying to aws. This throws an error if deploying locally into virtualbox but should be optional.

The install will also throw an error if the s3 settings for backups are not set in config.yaml. These settings could be made optional and if not provided, the backup cron is never set.

mhkeller commented 8 years ago

The ssh key params of keypair_name and ssh_private_key_path are definitely required for aws but when provisioning with virtual box are these two settings optional as well @abelsonlive: https://github.com/newslynx/automation/blob/master/secrets.yaml.sample#L3-L4

I added this to the readme since it didn't previously discuss secrets.yaml. If i've got it wrong about the ssh stuff being optional with virtualbox deploy, tweak away! https://github.com/newslynx/automation/commit/a2e3515a5e201b7b92c295956ec99221603ddb2c