fixing tags

Author: ssadasivuni

CHANGELOG.md

@@ -1,2 +1,5 @@
 # 0.1.0
 - Initial Release
+
+# 0.1.1
+- Fixing Tags

README.md

@@ -37,9 +37,6 @@ module "aws_iam_user" {
   name           = "test"
   force_destroy  = true
   pgp_key        = "keybase:test" # keybase user name
-  owned_by       = "example"
-  managed_by     = "terraform"
-  pipeline_id    = "123456" # this value is usually generated by your CI tool eg: Circle CI, Travis CI, Gitlab, etc.
 
   tags = {
     BusinessUnit = "tools"
@@ -65,18 +62,15 @@ module "aws_iam_user" {
 | create_iam_user_login_profile | Whether to create IAM user login profile. | `bool` | `true` | no |
 | create_user | Whether to create the IAM user. | `bool` | `true` | no |
 | force_destroy | When destroying this user, destroy even if it has non-Terraform-managed IAM access keys, login profile or MFA devices. Without force_destroy a user with non-Terraform-managed access keys and login profile will fail to be destroyed. | `bool` | `false` | no |
-| managed_by | The name of the tool that manages the resources. | `string` | `""` | yes |
 | name | Desired name for the IAM user. | `string` | `n/a` | yes |
-| owned_by | The name of the company who owns the resources. | `string` | `""` | yes |
 | password_length | The length of the generated password. | `number` | `20` | no |
 | password_reset_required | Whether the user should be forced to reset the generated password on first login. | `bool` | `true` | no |
 | path | Desired path for the IAM user. | `string` | `"/"` | no |
 | permissions_boundary | The ARN of the policy that is used to set the permissions boundary for the user. | `string` | `""` | no |
 | pgp_key | Either a base-64 encoded PGP public key, or a keybase username in the form `keybase:username`. Used to encrypt password and access key. | `string` | `""` | yes |
-| pipeline_id | The CI pipeline id number. | `number` | `n/a` | no |
 | ssh_key_encoding | Specifies the public key encoding format to use in the response. To retrieve the public key in ssh-rsa format, use SSH. To retrieve the public key in PEM format, use PEM. | `string` | `"SSH"` | no |
 | ssh_public_key | The SSH public key. The public key must be encoded in ssh-rsa format or PEM format. | `string` | `""` | no |
-| tags | Additional tags. | `map(string)` | `{}` | no |
+| tags | A map of tags for the resources. | `map(string)` | `{}` | no |
 | upload_iam_user_ssh_key | Whether to upload a public ssh key to the IAM user. | `bool` | `false` | no |
 
 ## Outputs

examples/main.tf

@@ -15,10 +15,8 @@ provider "aws" {
 module "aws_iam_user" {
   source = "../"
 
-  owned_by      = var.owned_by
-  managed_by    = var.managed_by
   name          = var.name
   force_destroy = var.force_destroy
   pgp_key       = var.pgp_key
-  pipeline_id   = var.pipeline_id
+  tags          = var.tags
 }

examples/terraform.tfvars

@@ -1,7 +1,8 @@
 region        = "us-east-1"
-owned_by      = "example"
-managed_by    = "terraform"
-pipeline_id   = 123456
 name          = "test"
 force_destroy = true
 pgp_key       = "keybase:test"
+tags          = {
+  BusinessUnit = "tools"
+  Application  = "cicd"
+}

examples/variables.tf

@@ -18,17 +18,7 @@ variable "pgp_key" {
   type        = string
 }
 
-variable "owned_by" {
-  description = "The name of the company who owns the resources."
-  type        = string
-}
-
-variable "managed_by" {
-  description = "The name of the tool that manages the resources."
-  type        = string
-}
-
-variable "pipeline_id" {
-  description = "The CI pipeline id number."
-  type        = number
+variable "tags" {
+  description = "A map of tags for the resources."
+  type        = map(string)
 }

main.tf

@@ -4,14 +4,7 @@ resource "aws_iam_user" "this" {
   path                 = var.path
   force_destroy        = var.force_destroy
   permissions_boundary = var.permissions_boundary
-  tags = merge(
-    {
-      OwnedBy     = var.owned_by
-      ManagedBy   = var.managed_by
-      Pipeline_Id = var.pipeline_id
-    },
-    var.tags,
-  )
+  tags                 = var.tags
 }
 
 resource "aws_iam_user_login_profile" "this" {

variables.tf

@@ -75,25 +75,8 @@ variable "permissions_boundary" {
   default     = ""
 }
 
-variable "owned_by" {
-  description = "The name of the company who owns the resources."
-  type        = string
-  default     = ""
-}
-
-variable "managed_by" {
-  description = "The name of the tool that manages the resources."
-  type        = string
-  default     = ""
-}
-
-variable "pipeline_id" {
-  description = "The CI pipeline id number."
-  type        = number
-}
-
 variable "tags" {
-  description = "Additional tags."
+  description = "A map of tags for the resources."
   type        = map(string)
   default     = {}
 }