Skip to content

Conversation

@rrehbein
Copy link

No description provided.

@dejonghe
Copy link
Owner

Hey there,
did you mean to send this PR to me or to your fork of this repo?

@rrehbein
Copy link
Author

I wasn't sure if your project was meant to be a collection of examples, or a tool kit. If it wasn't your intention to have a collection of examples, then I should not be making a PR against your project.

@dejonghe
Copy link
Owner

Oh cool, well thanks!
The purpose of this was to have a deployable example to go with a talk I presented at our AWS Michigan Meetup. https://www.youtube.com/watch?v=SXf1HFuJP-4
If you want to add in the update for CFN, Build script, and update the README.md I'd gladly evaluate and merge it in. I'm slated to talk again next month about CloudFormation and lambda with Serverless Application Model.

@rrehbein
Copy link
Author

Added building to the scripts, and started an aurora.json.

With your structure, I'm not sure if I should include additional network configuration in the aurora.json, or if I should modify vpc.json for the network, or if there should be another stack for the subnets.

The RDS setup seems to end up with quite a few parameters. Technically not a problem, but may complicate a demo.

@rrehbein
Copy link
Author

Possibly another example would be something to populate MasterUserPassword. It could be a top parameter, passed through though.

@rrehbein rrehbein force-pushed the rds-get-roendpoint branch from 421b221 to de324a1 Compare May 26, 2017 20:27
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.

2 participants