cleanup and add iam role outputs (#1)

Author: cdaniluk

.github/workflows/check.yml

@@ -0,0 +1,20 @@
+name: check
+on: [push, pull_request]
+
+jobs:
+  build:
+    runs-on: macOS-latest
+    steps:
+    - uses: actions/checkout@v1
+
+    - name: Install prereq
+      run: |
+        brew install docker tfenv tflint
+        tfenv install
+
+    - name: tf fmt
+      run: |
+        terraform fmt
+    - name: tflint
+      run: |
+        tflint

.pre-commit-config.yaml

@@ -0,0 +1,13 @@
+---
+repos:
+- repo: git://github.com/antonbabenko/pre-commit-terraform
+  rev: v1.24.0
+  hooks:
+    - id: terraform_fmt
+    - id: terraform_docs
+- repo: https://github.com/pre-commit/pre-commit-hooks
+  rev: v2.4.0
+  hooks:
+    - id: end-of-file-fixer
+    - id: trailing-whitespace
+    - id: no-commit-to-branch

.terraform-version

@@ -0,0 +1 @@
+0.12.23

LICENSE

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2019 Rhythmic Technologies, Inc.
+Copyright (c) 2020 Rhythmic Technologies, Inc.
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

README.md

@@ -1,7 +1,7 @@
 # terraform-aws-eks-iam-cluster-autoscaler
 [![](https://github.com/rhythmictech/terraform-aws-eks-iam-cluster-autoscaler/workflows/check/badge.svg)](https://github.com/rhythmictech/terraform-aws-eks-iam-cluster-autoscaler/actions)
 
-Configure IAM role and related policies to use EKS Service Accounts with the external-dns service.
+Configure IAM role and related policies to use the EKS cluster autoscaler
 
 ## Usage
 ```
@@ -22,13 +22,14 @@ module "cluster_autoscaler" {
 |------|-------------|:----:|:-----:|:-----:|
 | cluster\_name |  | string | n/a | yes |
 | issuer\_url | OIDC issuer URL \(include prefix\) | string | n/a | yes |
-| kubernetes\_namespace | Namespace to operate in \(service accounts and pods must be in the same namespace\) | string | `"default"` | no |
-| service\_account | Name of service account to create \(computed based on cluster name if not specified\) | string | `""` | no |
+| kubernetes\_namespace | Namespace to operate in \(service accounts and pods must be in the same namespace\) | string | `"kube-system"` | no |
+| service\_account | Name of service account to create | string | `""` | no |
 
 ## Outputs
 
 | Name | Description |
 |------|-------------|
-| aws\_account\_id |  |
+| iam\_role\_cluster\_autoscaler\_arn |  |
+| iam\_role\_cluster\_autoscaler\_name |  |
 
-<!-- END OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
+<!-- END OF PRE-COMMIT-TERRAFORM DOCS HOOK -->

outputs.tf

@@ -1,3 +1,7 @@
-output "aws_account_id" {
-  value = data.aws_caller_identity.current.account_id
-}
+output "iam_role_cluster_autoscaler_arn" {
+  value = aws_iam_role.cluster_autoscaler.arn
+}
+
+output "iam_role_cluster_autoscaler_name" {
+  value = aws_iam_role.cluster_autoscaler.name
+}