Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add Makefile #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add Makefile #5

wants to merge 1 commit into from

Conversation

mroch
Copy link
Contributor

@mroch mroch commented Jul 31, 2016

I scripted the installation instructions from the README (super detailed, thanks!) using the aws CLI.

just run aws configure first to set up your credentials, then make should set everything up. there are a couple variables at the top of Makefile in case you want to rename policies, roles, etc. still need to manually patch conf in dyndns53.py though.

it creates a bunch of state files in a build/ directory which it uses to keep track of ARNs and IDs it needs, and to make it idempotent so you can run make multiple times without getting errors. if you update dyndns53.py and rerun make, it'll just upload the changes.

tested on Mac.

@sTywin
Copy link
Owner

sTywin commented Sep 4, 2016

Thanks, this looks great :)

I haven't forgotten about it, I've just been super tied up, so haven't gotten around to testing it. I want to make sure I can use it reliably end-to-end before I pull it in. I'll want to update the README as well.

@asheiduk
Copy link

asheiduk commented Mar 5, 2020

Thank you for the Makefile, it is makes the setup much easier.
I've noticed two rough things:

  • If the build directory does not exist, it is not created but errors occur.
  • I have changed lambda_region at the top of the Makefile to something other than the default-region used by the aws CLI. The result were strange and hard to correct errors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants