From 1700271cee1a24297d13eb70c5a7e8493f35ba3c Mon Sep 17 00:00:00 2001
From: JMGaljaard <J.M.Galjaard-1@student.tudelft.nl>
Date: Fri, 9 Sep 2022 09:52:42 +0200
Subject: [PATCH] Update README + notebook

---
 README.md                        |  2 +-
 jupyter/terraform_notebook.ipynb | 20 ++++++++++++--------
 2 files changed, 13 insertions(+), 9 deletions(-)

diff --git a/README.md b/README.md
index 73c1d475..69db713e 100644
--- a/README.md
+++ b/README.md
@@ -233,7 +233,7 @@ container. With the use of BuildKit, consecutive builds allow to use cached requ
 up your builds when adding Python dependencies to your project.
 
 ``` bash
-DOCKER_BUILDKIT=1 docker build . --tag gcr.io/<project-id>/fltk
+DOCKER_BUILDKIT=1 docker build --platform linux/amd64 . --tag gcr.io/<project-id>/fltk
 docker push gcr.io/<project-id>/fltk
 ```
 
diff --git a/jupyter/terraform_notebook.ipynb b/jupyter/terraform_notebook.ipynb
index 2e5b672c..553de9eb 100644
--- a/jupyter/terraform_notebook.ipynb
+++ b/jupyter/terraform_notebook.ipynb
@@ -82,7 +82,9 @@
     "\n",
     "* [`terraform-gke/variables.tf`](terraform-gke/variables.tf)\n",
     "* [`terraform-dependencies/variables.tf`](terraform-dependencies/variables.tf)\n",
-    "\n"
+    "\n",
+    "\n",
+    "⚠️ Change the `PROJECT_ID` parameter to a unique project name, remember to update the paramter in the variables files!"
    ]
   },
   {
@@ -96,6 +98,7 @@
    "outputs": [],
    "source": [
     "ACCOUNT_ID=\"terraform-iam-service-account\"\n",
+    "# CHANGE ME!\n",
     "PROJECT_ID=\"test-bed-fltk\"\n",
     "PRIVILEGED_ACCOUNT_ID=\"${ACCOUNT_ID}@${PROJECT_ID}.iam.gserviceaccount.com\"\n",
     "CLUSTER_NAME=\"fltk-testbed-cluster\"\n",
@@ -309,7 +312,8 @@
    },
    "outputs": [],
    "source": [
-    "cd ../terraform/terraform-gke"
+    "cd ../terraform/terraform-gke\n",
+    "echo $PWD"
    ]
   },
   {
@@ -333,7 +337,7 @@
    },
    "outputs": [],
    "source": [
-    "terraform init -reconfigure"
+    "terraform init "
    ]
   },
   {
@@ -357,7 +361,7 @@
    },
    "outputs": [],
    "source": [
-    "terraform plan"
+    "terraform plan -var project_id=$PROJECT_ID"
    ]
   },
   {
@@ -387,7 +391,7 @@
    },
    "outputs": [],
    "source": [
-    "terraform apply -auto-approve"
+    "terraform apply -auto-approve -var project_id=$PROJECT_ID"
    ]
   },
   {
@@ -514,7 +518,7 @@
    },
    "outputs": [],
    "source": [
-    "terraform plan -out dependencies.out"
+    "terraform plan -var project_id=$PROJECT_ID"
    ]
   },
   {
@@ -536,7 +540,7 @@
    },
    "outputs": [],
    "source": [
-    "terraform apply -auto-approve"
+    "terraform apply -auto-approve -var project_id=$PROJECT_ID"
    ]
   },
   {
@@ -617,4 +621,4 @@
  },
  "nbformat": 4,
  "nbformat_minor": 1
-}
\ No newline at end of file
+}