🐛 fix([main.tf]): add null_resource to test and inform regarding serv…

…ice linked role for spot instances

README.md

@@ -26,8 +26,8 @@ The module deploys Karpenter needed AWS resources, namely in IAM. It copies the
 
 | Name | Version |
 |------|---------|
-| <a name="provider_aws"></a> [aws](#provider\_aws) | 6.0.0 |
-| <a name="provider_aws.eecr"></a> [aws.eecr](#provider\_aws.eecr) | 6.0.0 |
+| <a name="provider_aws"></a> [aws](#provider\_aws) | 5.100.0 |
+| <a name="provider_aws.eecr"></a> [aws.eecr](#provider\_aws.eecr) | 5.100.0 |
 | <a name="provider_helm"></a> [helm](#provider\_helm) | 3.0.2 |
 | <a name="provider_null"></a> [null](#provider\_null) | 3.2.4 |
 
@@ -42,7 +42,6 @@ The module deploys Karpenter needed AWS resources, namely in IAM. It copies the
 
 | Name | Type |
 |------|------|
-| [aws_iam_service_linked_role.spot](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/iam_service_linked_role) | resource |
 | [helm_release.karpenter](https://registry.terraform.io/providers/hashicorp/helm/latest/docs/resources/release) | resource |
 | [helm_release.nodepool_resources](https://registry.terraform.io/providers/hashicorp/helm/latest/docs/resources/release) | resource |
 | [null_resource.git_version](https://registry.terraform.io/providers/hashicorp/null/latest/docs/resources/resource) | resource |

main.tf

@@ -60,7 +60,7 @@ data "aws_ecr_image" "karpenter_image" {
 resource "helm_release" "karpenter" {
   depends_on = [
     data.aws_ecr_image.karpenter_image,
-    aws_iam_service_linked_role.spot
+    # aws_iam_service_linked_role.spot
   ]
   name             = "karpenter"
   chart            = "karpenter"
@@ -117,15 +117,15 @@ resource "helm_release" "nodepool_resources" {
 }
 
 # Create service-linked role for EC2 Spot (required for Karpenter spot instances)
-resource "aws_iam_service_linked_role" "spot" {
-  aws_service_name = "spot.amazonaws.com"
-  description      = "Service-linked role for EC2 Spot instances used by Karpenter"
-
-  # This will only create the role if it doesn't already exist
-  # If it exists, Terraform will import the existing role
-  lifecycle {
-    ignore_changes = [
-      description,
-    ]
-  }
-}
+# resource "aws_iam_service_linked_role" "spot" {
+#   aws_service_name = "spot.amazonaws.com"
+#   description      = "Service-linked role for EC2 Spot instances used by Karpenter"
+
+#   # This will only create the role if it doesn't already exist
+#   # If it exists, Terraform will import the existing role
+#   lifecycle {
+#     ignore_changes = [
+#       description,
+#     ]
+#   }
+# }