WIP: Add an output for the role arn to the single-node-asg module.

mcgirr commented 4 years ago

This PR adds the asg_iam_role_arn output to the single-node-asg module
This PR also makes a fix for the single-node-asg module by removing the azs variable which is no longer a part of the parent asg module since https://github.com/fpco/terraform-aws-foundation/pull/247

[ ] Update the changelog
[x] Make sure that modules and files are documented. This can be done inside the module and files.
[x] Make sure that new modules directories contain a basic README.md file.
[x] Make sure that the module is added to tests/main.tf
[ ] Make sure that the linting passes on CI.
[ ] Make sure that there is an up to date example for your code:
- For new modules this would entail example code for how to use the module or some explanation in the module readme.
- For new examples please provide a README explaining how to run the example. It's also ideal to provide a basic makefile to use the example as well.
[ ] Make sure that there is a manual CI trigger that can test the deployment.

ketzacoatl commented 4 years ago

@mcgirr can you paste/screenshot/etc some output that confirms this works as expected?

mcgirr commented 4 years ago

@ketzacoatl Sure thing. And I will remove the WIP at that point as well.

mcgirr commented 4 years ago

This change is no longer needed.

fpco / terraform-aws-foundation