Skip to content

Commit

Permalink
Making MPI operator optional for users
Browse files Browse the repository at this point in the history
  • Loading branch information
sanjeevrg89 committed Nov 6, 2023
1 parent 1457d05 commit 0a46f3a
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 3 deletions.
5 changes: 3 additions & 2 deletions ai-ml/trainium-inferentia/addons.tf
Original file line number Diff line number Diff line change
Expand Up @@ -512,7 +512,8 @@ data "kubectl_file_documents" "mpi_operator_yaml" {
}

resource "kubectl_manifest" "mpi_operator" {
for_each = data.kubectl_file_documents.mpi_operator_yaml.manifests
for_each = var.enable_mpi_operator ? data.kubectl_file_documents.mpi_operator_yaml.manifests : {}
yaml_body = each.value
depends_on = [module.eks.eks_cluster_id]
}
}

8 changes: 7 additions & 1 deletion ai-ml/trainium-inferentia/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -41,4 +41,10 @@ variable "enable_amazon_prometheus" {
variable "mpi_operator_version" {
description = "The version of the MPI Operator to install"
default = "v0.4.0"
}
}

variable "enable_mpi_operator" {
description = "Flag to enable the MPI Operator deployment"
type = bool
default = false
}

0 comments on commit 0a46f3a

Please sign in to comment.