Skip to content

refactor(scaletest/terraform): break up infra creation and k8s resource provisioning #9824

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Sep 22, 2023
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
rm unused vars
  • Loading branch information
johnstcn committed Sep 22, 2023
commit 50f84ec89e5cf0aa71727beb3043d4bd84160acd
90 changes: 0 additions & 90 deletions scaletest/terraform/k8s/vars.tf
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,6 @@ variable "state" {
default = "started"
}

# variable "project_id" {
# description = "The project in which to provision resources"
# }

variable "name" {
description = "Adds a prefix to resources."
}
Expand All @@ -31,92 +27,6 @@ variable "kubernetes_nodepool_misc" {
description = "Name of the nodepool on which to run everything else."
}

# variable "region" {
# description = "GCP region in which to provision resources."
# default = "us-east1"
# }

# variable "zone" {
# description = "GCP zone in which to provision resources."
# default = "us-east1-c"
# }

# variable "k8s_version" {
# description = "Kubernetes vversion to provision."
# default = "1.24"
# }

# variable "node_disk_size_gb" {
# description = "Size of the root disk for cluster nodes."
# default = 100
# }

# variable "node_image_type" {
# description = "Image type to use for cluster nodes."
# default = "cos_containerd"
# }

# // Preemptible nodes are way cheaper, but can be pulled out
# // from under you at any time. Caveat emptor.
# variable "node_preemptible" {
# description = "Use preemptible nodes."
# default = false
# }

// We create three nodepools:
// - One for the Coder control plane
// - One for workspaces
// - One for everything else (for example, load generation)

// These variables control the node pool dedicated to Coder.
# variable "nodepool_machine_type_coder" {
# description = "Machine type to use for Coder control plane nodepool."
# default = "t2d-standard-4"
# }

# variable "nodepool_size_coder" {
# description = "Number of cluster nodes for the Coder control plane nodepool."
# default = 1
# }

# // These variables control the node pool dedicated to workspaces.
# variable "nodepool_machine_type_workspaces" {
# description = "Machine type to use for the workspaces nodepool."
# default = "t2d-standard-4"
# }

# variable "nodepool_size_workspaces" {
# description = "Number of cluster nodes for the workspaces nodepool."
# default = 1
# }

# // These variables control the node pool for everything else.
# variable "nodepool_machine_type_misc" {
# description = "Machine type to use for the misc nodepool."
# default = "t2d-standard-4"
# }

# variable "nodepool_size_misc" {
# description = "Number of cluster nodes for the misc nodepool."
# default = 1
# }

# // These variables control the size of the database to be used by Coder.
# variable "cloudsql_version" {
# description = "CloudSQL version to provision"
# default = "POSTGRES_14"
# }

# variable "cloudsql_tier" {
# description = "CloudSQL database tier."
# default = "db-f1-micro"
# }

# variable "cloudsql_max_connections" {
# description = "CloudSQL database max_connections"
# default = 500
# }

// These variables control the Coder deployment.
variable "coder_replicas" {
description = "Number of Coder replicas to provision."
Expand Down