Skip to content

Commit

Permalink
cleanup: remove unused input, output, and comments
Browse files Browse the repository at this point in the history
  • Loading branch information
mogul committed Mar 1, 2022
1 parent 1d162a1 commit f5d8caf
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 17 deletions.
4 changes: 0 additions & 4 deletions eks-service-definition.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,10 +49,6 @@ provision:
required: false
type: array
details: "A list of the desired AWS Compute types that the nodes will be launched with (e.g. [\"m5.large\"])"
- field_name: install_vpc_cni
required: false
type: boolean
details: "Control input to specify whether this module creates the vpc cni addon as part of the EKS deployment"
- field_name: aws_access_key_id
required: true
type: string
Expand Down
4 changes: 0 additions & 4 deletions terraform/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,6 @@ variable "write_kubeconfig" {
default = false
}


module "provision" {
source = "./modules/provision"
providers = {
Expand All @@ -81,7 +80,6 @@ module "provision" {
}
aws_access_key_id = var.aws_access_key_id
aws_secret_access_key = var.aws_secret_access_key
install_vpc_cni = true
instance_name = var.instance_name
labels = var.labels
mng_instance_types = var.mng_instance_types
Expand All @@ -94,8 +92,6 @@ module "provision" {
zone = var.zone
}

# There's more to be done on hoisting provider configuration out before we can
# uncomment this module!
module "bind" {
source = "./modules/bind"
providers = {
Expand Down
4 changes: 0 additions & 4 deletions terraform/modules/provision/outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,3 @@ output "token" {
value = data.kubernetes_secret.secret.data.token
description = "A cluster-admin token for use in constructing your own kubernetes configuration. NOTE: Do _not_ use this token when configuring the required_provider or you'll get a dependency cycle. Instead use exec with the same AWS credentials that were used for the required_providers aws provider."
}
output "admin_kubeconfig" {
value = data.template_file.admin_kubeconfig.rendered
description = "A kubeconfig for a cluster-admin service account."
}
5 changes: 0 additions & 5 deletions terraform/modules/provision/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -37,11 +37,6 @@ variable "mng_instance_types" {
default = ["m4.xlarge"]
}

variable "install_vpc_cni" {
type = bool
default = true
}

variable "labels" {
type = map(any)
default = {}
Expand Down

0 comments on commit f5d8caf

Please sign in to comment.