chore: Fix pre-commit run errors (#387)

awslabs · Dec 13, 2023 · fe70fa0 · fe70fa0
2 parents 30ad93b + a288738
commit fe70fa0
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 14 deletions.
diff --git a/.codespellrc b/.codespellrc
@@ -3,4 +3,4 @@ skip = .git,*.pdf,*.svg,go.sum,package-lock.json,*.css,.codespellrc,*.sql,websit
 check-hidden = true
 # some embedded images and known typoed outputs
 ignore-regex = ^\s*"image/\S+": ".*|.*loopback adddress.*
-# ignore-words-list = 
+# ignore-words-list =
diff --git a/ai-ml/jupyterhub/helm/karpenter-resources/templates/node-class.yaml b/ai-ml/jupyterhub/helm/karpenter-resources/templates/node-class.yaml
@@ -5,16 +5,16 @@ metadata:
   name: {{ .Values.name }}
 spec:
   amiFamily: {{ .Values.amiFamily }}
-  subnetSelectorTerms:          
+  subnetSelectorTerms:
     - tags:
         karpenter.sh/discovery: {{ .Values.clusterName }}
         kubernetes.io/role/internal-elb: "1" # Make sure that it will be scheduled on private subs
-  securityGroupSelectorTerms:   
+  securityGroupSelectorTerms:
     - tags:
         Name: {{ .Values.clusterName }}-node
   role: {{ .Values.karpenterRole }}
   # Optional, propagates tags to underlying EC2 resources
-  tags:                  
+  tags:
     Name: {{ .Values.name }}
   metadataOptions:
     httpEndpoint: enabled
@@ -33,7 +33,7 @@ spec:
         encrypted: true
         deleteOnTermination: true
   detailedMonitoring: true
-  userData: |                   
+  userData: |
     MIME-Version: 1.0
     Content-Type: multipart/mixed; boundary="BOUNDARY"
 
@@ -52,4 +52,4 @@ spec:
     # Source extra environment variables in bootstrap script
     sed -i '/^set -o errexit/a\\nsource /etc/profile.d/bootstrap.sh' /etc/eks/bootstrap.sh
 
-    --BOUNDARY--
+    --BOUNDARY--
diff --git a/ai-ml/jupyterhub/helm/karpenter-resources/templates/node-pool.yaml b/ai-ml/jupyterhub/helm/karpenter-resources/templates/node-pool.yaml
@@ -43,4 +43,4 @@ spec:
   limits:
     cpu: "1000"
     memory: 1000Gi
-  weight: 10
+  weight: 10
diff --git a/ai-ml/jupyterhub/helm/karpenter-resources/values.yaml b/ai-ml/jupyterhub/helm/karpenter-resources/values.yaml
@@ -1,6 +1,6 @@
 name: default
 clusterName:
-karpenterRole: 
+karpenterRole:
 instanceSizes: ["xlarge", "2xlarge", "4xlarge", "8xlarge", "16xlarge", "24xlarge"]
 instanceFamilies: ["c5", "m5", "r5"]
 taints:

diff --git a/workshop/emr-eks/modules/eks/main.tf b/workshop/emr-eks/modules/eks/main.tf
@@ -124,10 +124,12 @@ module "karpenter" {
   source  = "terraform-aws-modules/eks/aws//modules/karpenter"
   version = "~> 19.15"
 
-  cluster_name                 = module.eks.cluster_name
-  irsa_oidc_provider_arn       = module.eks.oidc_provider_arn
-  create_irsa                  = false # EKS Blueprints add-on module creates IRSA
-  enable_spot_termination      = false # EKS Blueprints add-on module adds this feature
-  tags                         = local.tags
-  iam_role_additional_policies = ["arn:aws:iam::aws:policy/AmazonSSMManagedInstanceCore"]
+  cluster_name            = module.eks.cluster_name
+  irsa_oidc_provider_arn  = module.eks.oidc_provider_arn
+  create_irsa             = false # EKS Blueprints add-on module creates IRSA
+  enable_spot_termination = false # EKS Blueprints add-on module adds this feature
+  tags                    = local.tags
+  iam_role_additional_policies = {
+    AmazonSSMManagedInstanceCore = "arn:aws:iam::aws:policy/AmazonSSMManagedInstanceCore"
+  }
 }