Closed CloudNiner closed 8 years ago
This PR does not currently fit the format of our other ansible-*
repositories. Additional work would need to be done for that. But I figured that wasn't terribly pressing atm since our primary use case is to use this repo as a provisioner for a cartodb dev environment, rather than using it in other ansible roles.
Might make sense to rename the repository to vagrant-cartodb
to address your concerns over it not aligning with the existing ansible-*
repositories.
Looks good to me as a first start. I'll let this run overnight to test.
I like the suggestion to change the repo name to vagrant-cartodb
, I'll do that once this PR is merged.
@hectcastro @ddohler Updated with a few more commits that should address almost all the comments and cleanup a few extra minor things. Feel free to take another look, I'm provisioning a fresh box atm.
Current changeset provisioned successfully, although I did get a pip error about mismatched hashes when downloading packages for the requirements.txt file. No issues on followup reprovision.
:+1:
Provisioned clearly here too. Good job wrangling all of these things together. :+1:
Thanks for the second round of feedback, will make a few adjustments, then merge and update the repository name.
Generally follows the install instructions from: http://cartodb.readthedocs.org/en/latest/install.html
Instructions for use are in README.md