Update pre-commit, cleanup old comments

yuvipanda · Aug 28, 2023 · 432f115 · 432f115
1 parent a735a08
commit 432f115
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 12 deletions.
diff --git a/.pre-commit-config-terraform.yaml b/.pre-commit-config-terraform.yaml
@@ -8,12 +8,14 @@
 # - terraform
 # - tflint
 
-# Currently only aws/curvenote is checked
-files: "^terraform/aws/curvenote/"
+# Currently only aws/ is checked
+files: "^terraform/aws/"
+exclude: "^terraform/aws/pangeo/"
+
 repos:
   # We can't run any CI tests on production Terraform code, so use as many static linters as possible
   - repo: https://github.com/antonbabenko/pre-commit-terraform
-    rev: v1.81.0
+    rev: v1.83.0
     hooks:
       - id: terraform_fmt
       - id: terraform_tflint

diff --git a/terraform/aws/binder-eks/provider.tf b/terraform/aws/binder-eks/provider.tf
@@ -0,0 +1,20 @@
+terraform {
+  required_providers {
+    aws = {
+      source  = "hashicorp/aws"
+      version = "~> 5.5"
+    }
+
+    kubernetes = {
+      source  = "hashicorp/kubernetes"
+      version = "~> 2.21"
+    }
+
+    tls = {
+      source  = "hashicorp/tls"
+      version = "~> 4.0"
+    }
+  }
+
+  required_version = ">= 1.4.6"
+}
diff --git a/terraform/aws/curvenote/main.tf b/terraform/aws/curvenote/main.tf
@@ -9,19 +9,11 @@ terraform {
       source  = "hashicorp/kubernetes"
       version = "~> 2.21"
     }
-
-    tls = {
-      source  = "hashicorp/tls"
-      version = "~> 4.0"
-    }
   }
 
   required_version = ">= 1.4.6"
 
-  # Bootstrapping
-  # 1. comment this out for a new deployment
-  # 2. run the deployment to create the S3 bucket
-  # 3. uncomment this and migrate the tfstate to S3
+  # Bootstrapping: Create the bucket and DynamoDB table using the ./bootstrap directory
   backend "s3" {
     bucket         = "binderhub-tfstate-7rjazazm1c7k"
     key            = "tfstate/dev/binderhub-dev"