From 07e95c44db0b9759d4c9fa654e25e42350b8154a Mon Sep 17 00:00:00 2001
From: Cian Johnston <cian@coder.com>
Date: Thu, 20 Jul 2023 10:47:24 +0100
Subject: [PATCH 1/5] scaletest: stop kubernetes_secret from being constantly
 recreated

---
 scaletest/terraform/coder.tf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scaletest/terraform/coder.tf b/scaletest/terraform/coder.tf
index 13bd15205b6eb..8a63ae2484025 100644
--- a/scaletest/terraform/coder.tf
+++ b/scaletest/terraform/coder.tf
@@ -54,7 +54,7 @@ resource "random_password" "prometheus-postgres-password" {
 }
 
 resource "kubernetes_secret" "coder-db" {
-  type = "" # Opaque
+  type = "Opaque"
   metadata {
     name      = "coder-db-url"
     namespace = local.coder_namespace

From c6ad5e2c34054fecc6b424733ccd1adbac476be0 Mon Sep 17 00:00:00 2001
From: Cian Johnston <cian@coder.com>
Date: Thu, 20 Jul 2023 10:47:52 +0100
Subject: [PATCH 2/5] scaletest: ensure we do not get auto-upgraded

---
 scaletest/terraform/gcp_cluster.tf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scaletest/terraform/gcp_cluster.tf b/scaletest/terraform/gcp_cluster.tf
index a1b12678749ee..3021059598323 100644
--- a/scaletest/terraform/gcp_cluster.tf
+++ b/scaletest/terraform/gcp_cluster.tf
@@ -18,7 +18,7 @@ resource "google_container_cluster" "primary" {
 
   }
   release_channel {
-    channel = "STABLE"
+    channel = "UNSPECIFIED"
   }
   initial_node_count       = 1
   remove_default_node_pool = true

From ff706274813b30ccafd1770576c592ea52d99a9a Mon Sep 17 00:00:00 2001
From: Cian Johnston <cian@coder.com>
Date: Thu, 20 Jul 2023 10:48:20 +0100
Subject: [PATCH 3/5] scaletest: add external provisionerd deployment, the lazy
 way

---
 scaletest/terraform/coder.tf | 165 +++++++++++++++++++++++++++++++++++
 scaletest/terraform/vars.tf  |  31 +++++++
 2 files changed, 196 insertions(+)

diff --git a/scaletest/terraform/coder.tf b/scaletest/terraform/coder.tf
index 8a63ae2484025..b25823ec6a4b8 100644
--- a/scaletest/terraform/coder.tf
+++ b/scaletest/terraform/coder.tf
@@ -125,6 +125,9 @@ coder:
       value: "${var.coder_experiments}"
     - name: "CODER_DANGEROUS_DISABLE_RATE_LIMITS"
       value: "true"
+    # Disabling built-in provisioner daemons
+    - name: "CODER_PROVISIONER_DAEMONS"
+      value: "0"
   image:
     repo: ${var.coder_image_repo}
     tag: ${var.coder_image_tag}
@@ -242,6 +245,168 @@ resource "local_file" "kubernetes_template" {
   EOF
 }
 
+# TODO(cian): Remove this when we have support in the Helm chart.
+# Ref: https://github.com/coder/coder/issues/8243
+resource "local_file" "provisionerd_deployment" {
+    filename = "${path.module}/../.coderv2/provisionerd-deployment.yaml"
+    content = <<EOF
+apiVersion: apps/v1
+kind: Deployment
+metadata:
+  labels:
+    app.kubernetes.io/instance: ${var.name}
+    app.kubernetes.io/name: provisionerd
+  name: provisionerd
+  namespace: ${local.coder_namespace}
+spec:
+  replicas: ${var.provisionerd_replicas}
+  selector:
+    matchLabels:
+      app.kubernetes.io/instance: ${var.name}
+      app.kubernetes.io/name: provisionerd
+  strategy:
+    rollingUpdate:
+      maxSurge: 25%
+      maxUnavailable: 25%
+    type: RollingUpdate
+  template:
+    metadata:
+      creationTimestamp: null
+      labels:
+        app.kubernetes.io/instance: ${var.name}
+        app.kubernetes.io/name: provisionerd
+    spec:
+      affinity:
+        nodeAffinity:
+          requiredDuringSchedulingIgnoredDuringExecution:
+            nodeSelectorTerms:
+            - matchExpressions:
+              - key: cloud.google.com/gke-nodepool
+                operator: In
+                values:
+                - ${google_container_node_pool.coder.name}
+        podAntiAffinity:
+          preferredDuringSchedulingIgnoredDuringExecution:
+          - podAffinityTerm:
+              labelSelector:
+                matchExpressions:
+                - key: app.kubernetes.io/instance
+                  operator: In
+                  values:
+                  - ${var.name}
+              topologyKey: kubernetes.io/hostname
+            weight: 1
+      containers:
+      - args:
+        - server
+        command:
+        - /opt/coder
+        env:
+        - name: CODER_HTTP_ADDRESS
+          value: 0.0.0.0:8080
+        - name: CODER_PROMETHEUS_ADDRESS
+          value: 0.0.0.0:2112
+        - name: CODER_ACCESS_URL
+          value: ${local.coder_url}
+        - name: CODER_CACHE_DIRECTORY
+          value: /tmp/coder
+        - name: CODER_ENABLE_TELEMETRY
+          value: "false"
+        - name: CODER_LOGGING_HUMAN
+          value: /dev/null
+        - name: CODER_LOGGING_STACKDRIVER
+          value: /dev/stderr
+        - name: CODER_PG_CONNECTION_URL
+          valueFrom:
+            secretKeyRef:
+              key: url
+              name: coder-db-url
+        - name: CODER_PPROF_ENABLE
+          value: "true"
+        - name: CODER_PROMETHEUS_ENABLE
+          value: "true"
+        - name: CODER_PROMETHEUS_COLLECT_AGENT_STATS
+          value: "true"
+        - name: CODER_PROMETHEUS_COLLECT_DB_METRICS
+          value: "true"
+        - name: CODER_VERBOSE
+          value: "true"
+        - name: CODER_PROVISIONER_DAEMONS
+          value: "${var.provisionerd_concurrency}"
+        image: "${var.coder_image_repo}:${var.coder_image_tag}"
+        imagePullPolicy: IfNotPresent
+        lifecycle: {}
+        livenessProbe:
+          failureThreshold: 3
+          httpGet:
+            path: /api/v2/buildinfo
+            port: http
+            scheme: HTTP
+          periodSeconds: 10
+          successThreshold: 1
+          timeoutSeconds: 1
+        name: provisionerd
+        ports:
+        - containerPort: 8080
+          name: http
+          protocol: TCP
+        - containerPort: 2112
+          name: prometheus-http
+          protocol: TCP
+        readinessProbe:
+          failureThreshold: 3
+          httpGet:
+            path: /api/v2/buildinfo
+            port: http
+            scheme: HTTP
+          periodSeconds: 10
+          successThreshold: 1
+          timeoutSeconds: 1
+        resources:
+          limits:
+            cpu: "${var.provisionerd_cpu_limit}"
+            memory: "${var.provisionerd_mem_limit}"
+          requests:
+            cpu: "${var.provisionerd_cpu_request}"
+            memory: "${var.provisionerd_mem_request}"
+        securityContext:
+          allowPrivilegeEscalation: false
+          readOnlyRootFilesystem: true
+          runAsGroup: 1000
+          runAsNonRoot: true
+          runAsUser: 1000
+          seccompProfile:
+            type: RuntimeDefault
+        terminationMessagePath: /dev/termination-log
+        terminationMessagePolicy: File
+        volumeMounts:
+        - mountPath: /tmp
+          name: cache
+      dnsPolicy: ClusterFirst
+      restartPolicy: Always
+      serviceAccount: coder
+      serviceAccountName: coder
+      terminationGracePeriodSeconds: 60
+      volumes:
+      - emptyDir:
+          sizeLimit: 10Gi
+        name: cache
+    EOF
+}
+
+resource "null_resource" "provisionerd_deployment_apply" {
+  depends_on = [helm_release.coder-chart, local_file.provisionerd_deployment, null_resource.cluster_kubeconfig]
+  triggers = {
+    kubeconfig_path       = local.cluster_kubeconfig_path
+    manifest_path = local_file.provisionerd_deployment.filename
+  }
+  provisioner "local-exec" {
+    command = <<EOF
+      KUBECONFIG=${self.triggers.kubeconfig_path} kubectl apply -f ${self.triggers.manifest_path}
+    EOF
+  }
+}
+
 resource "local_file" "output_vars" {
   filename = "${path.module}/../.coderv2/url"
   content  = local.coder_url
diff --git a/scaletest/terraform/vars.tf b/scaletest/terraform/vars.tf
index d65711f1c07d3..aa404ee614684 100644
--- a/scaletest/terraform/vars.tf
+++ b/scaletest/terraform/vars.tf
@@ -130,6 +130,37 @@ variable "coder_mem_limit" {
   default     = "1024Mi"
 }
 
+// Allow independently scaling provisionerd resources
+variable "provisionerd_cpu_request" {
+  description = "CPU request to allocate to provisionerd."
+  default     = "500m"
+}
+
+variable "provisionerd_mem_request" {
+  description = "Memory request to allocate to provisionerd."
+  default     = "512Mi"
+}
+
+variable "provisionerd_cpu_limit" {
+  description = "CPU limit to allocate to provisionerd."
+  default     = "1000m"
+}
+
+variable "provisionerd_mem_limit" {
+  description = "Memory limit to allocate to provisionerd."
+  default     = "1024Mi"
+}
+
+variable "provisionerd_replicas" {
+  description = "Number of Provisionerd replicas."
+  default     = 1
+}
+
+variable "provisionerd_concurrency" {
+  description = "Number of concurrent provisioner jobs per provisionerd instance."
+  default     = 3
+}
+
 variable "coder_chart_version" {
   description = "Version of the Coder Helm chart to install. Defaults to latest."
   default     = null

From c2270e68622ef80769ad3ed219ff47abbbbeca17 Mon Sep 17 00:00:00 2001
From: Cian Johnston <cian@coder.com>
Date: Thu, 20 Jul 2023 11:09:54 +0100
Subject: [PATCH 4/5] Update scaletest/terraform/gcp_cluster.tf

---
 scaletest/terraform/gcp_cluster.tf | 1 +
 1 file changed, 1 insertion(+)

diff --git a/scaletest/terraform/gcp_cluster.tf b/scaletest/terraform/gcp_cluster.tf
index 3021059598323..d83ab99c8cb74 100644
--- a/scaletest/terraform/gcp_cluster.tf
+++ b/scaletest/terraform/gcp_cluster.tf
@@ -18,6 +18,7 @@ resource "google_container_cluster" "primary" {
 
   }
   release_channel {
+    # Setting release channel as STABLE can cause unexpected cluster upgrades.
     channel = "UNSPECIFIED"
   }
   initial_node_count       = 1

From dc8c3e231daedf178932e987d6dae0e2fb8f8d8f Mon Sep 17 00:00:00 2001
From: Cian Johnston <cian@coder.com>
Date: Thu, 20 Jul 2023 11:12:56 +0100
Subject: [PATCH 5/5] make fmt

---
 scaletest/terraform/coder.tf | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/scaletest/terraform/coder.tf b/scaletest/terraform/coder.tf
index b25823ec6a4b8..55a9e7a35721e 100644
--- a/scaletest/terraform/coder.tf
+++ b/scaletest/terraform/coder.tf
@@ -248,8 +248,8 @@ resource "local_file" "kubernetes_template" {
 # TODO(cian): Remove this when we have support in the Helm chart.
 # Ref: https://github.com/coder/coder/issues/8243
 resource "local_file" "provisionerd_deployment" {
-    filename = "${path.module}/../.coderv2/provisionerd-deployment.yaml"
-    content = <<EOF
+  filename = "${path.module}/../.coderv2/provisionerd-deployment.yaml"
+  content  = <<EOF
 apiVersion: apps/v1
 kind: Deployment
 metadata:
@@ -397,8 +397,8 @@ spec:
 resource "null_resource" "provisionerd_deployment_apply" {
   depends_on = [helm_release.coder-chart, local_file.provisionerd_deployment, null_resource.cluster_kubeconfig]
   triggers = {
-    kubeconfig_path       = local.cluster_kubeconfig_path
-    manifest_path = local_file.provisionerd_deployment.filename
+    kubeconfig_path = local.cluster_kubeconfig_path
+    manifest_path   = local_file.provisionerd_deployment.filename
   }
   provisioner "local-exec" {
     command = <<EOF