add readme and fix whitespace in cloudformation template

Author: smiller171

README.md

@@ -0,0 +1,74 @@
+# terraform-aws-imagebuilder-pipeline [![](https://github.com/rhythmictech/terraform-aws-imagebuilder-pipeline/workflows/pre-commit-check/badge.svg)](https://github.com/rhythmictech/terraform-aws-imagebuilder-pipeline/actions) <a href="https://twitter.com/intent/follow?screen_name=RhythmicTech"><img src="https://img.shields.io/twitter/follow/RhythmicTech?style=social&logo=RhythmicTech" alt="follow on Twitter"></a>
+Terraform module for creating EC2 Image Builder Pipelines from Cloudformation
+
+## Example
+Here's what using the module will look like
+```hcl
+module "test_pipeline" {
+  source  = "rhythmictech/imagebuilder-recipe/aws"
+  version = "~> 0.2.0"
+
+  description = "Testing pipeline"
+  name        = "test-pipeline"
+  tags        = local.tags
+  recipe_arn  = module.test_recipe.recipe_arn
+  public      = false
+}
+```
+
+## About
+Allows the creation of EC2 Image Builder Pipelines with Cloudformation until native support is added to TF
+
+<!-- BEGINNING OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
+## Requirements
+
+| Name | Version |
+|------|---------|
+| terraform | >= 0.12.2 |
+| aws | ~> 2.44 |
+
+## Providers
+
+| Name | Version |
+|------|---------|
+| aws | ~> 2.44 |
+
+## Inputs
+
+| Name | Description | Type | Default | Required |
+|------|-------------|------|---------|:--------:|
+| name | name to use for component | `string` | n/a | yes |
+| recipe\_arn | ARN of the recipe to use. Must change with Recipe version | `string` | n/a | yes |
+| additional\_iam\_policy\_arns | List of ARN policies for addional builder permissions | `list(string)` | `[]` | no |
+| cloudformation\_timeout | How long to wait (in minutes) for CFN to apply before giving up | `number` | `10` | no |
+| description | description of component | `string` | `null` | no |
+| enabled | Whether pipeline is ENABLED or DISABLED | `bool` | `true` | no |
+| instance\_types | Instance types to create images from. It's unclear why this is a list. Possibly because different types can result in different images (like ARM instances) | `list(string)` | <pre>[<br>  "t3.medium"<br>]</pre> | no |
+| key\_pair | EC2 key pair to add to the default user on the builder | `string` | `null` | no |
+| license\_config\_arns | If you're using License Manager, your ARNs go here | `list(string)` | `null` | no |
+| log\_bucket | Bucket to store logs in. If this is ommited logs will not be stored | `string` | `null` | no |
+| log\_prefix | S3 prefix to store logs at. Recommended if sharing bucket with other pipelines | `string` | `null` | no |
+| public | Whether resulting AMI should be public | `bool` | `false` | no |
+| regions | n/a | `list` | <pre>[<br>  "us-east-1",<br>  "us-east-2",<br>  "us-west-1",<br>  "us-west-2",<br>  "ca-central-1"<br>]</pre> | no |
+| schedule | Schedule expression for when pipeline should run automatically https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-imagebuilder-imagepipeline-schedule.html | <pre>object({<br>    PipelineExecutionStartCondition = string<br>    ScheduleExpression              = string<br>  })</pre> | <pre>{<br>  "PipelineExecutionStartCondition": "EXPRESSION_MATCH_AND_DEPENDENCY_UPDATES_AVAILABLE",<br>  "ScheduleExpression": "cron(0 0 * * mon)"<br>}</pre> | no |
+| security\_group\_ids | Security group IDs for the Image Builder | `list(string)` | `null` | no |
+| shared\_account\_ids | AWS accounts to share AMIs with. If this is left null AMIs will be public | `list(string)` | `[]` | no |
+| sns\_topic\_arn | SNS topic to notify when new images are created | `string` | `null` | no |
+| subnet | Subnet ID to use for builder | `string` | `null` | no |
+| tags | map of tags to use for CFN stack and component | `map(string)` | `{}` | no |
+| terminate\_on\_failure | Change to false if you want to ssh into a builder for debugging after failure | `bool` | `true` | no |
+| test\_config | Whether to run tests during image creation and maximum time to allow tests to run | <pre>object({<br>    ImageTestsEnabled = bool<br>    TimeoutMinutes    = number<br>  })</pre> | <pre>{<br>  "ImageTestsEnabled": true,<br>  "TimeoutMinutes": 60<br>}</pre> | no |
+
+## Outputs
+
+| Name | Description |
+|------|-------------|
+| pipeline\_arn | n/a |
+
+<!-- END OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
+
+## The Giants underneath this module
+- pre-commit.com/
+- terraform.io/
+- github.com/tfutils/tfenv
+- github.com/segmentio/terraform-docs

cloudformation.yml.tpl

@@ -33,7 +33,7 @@ Resources:
         ${ indent(8, chomp(yamlencode(tags))) }
   infraConfig:
     Type: AWS::ImageBuilder::InfrastructureConfiguration
-    Properties: 
+    Properties:
       Name: ${name}-infrastructure-configuration
       %{~ if description != null ~}
       Description: ${description}