From dee59a40147856b8d8215cf1b8acee927cb037e6 Mon Sep 17 00:00:00 2001
From: James Cross <james.cross@colibridigital.io>
Date: Fri, 24 Apr 2020 11:35:12 +0100
Subject: [PATCH] Paramaterize ingress namespace

---
 aws/eks/ingress.tf | 1 +
 aws/eks/vars.tf    | 6 +++++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/aws/eks/ingress.tf b/aws/eks/ingress.tf
index fe43e17..c63125b 100644
--- a/aws/eks/ingress.tf
+++ b/aws/eks/ingress.tf
@@ -2,6 +2,7 @@ resource "helm_release" "nginx-ingress" {
   name       = "nginx-ingress"
   repository = data.helm_repository.stable.metadata[0].name
   chart      = "nginx-ingress"
+  namespace  = var.dslab_namespace
 
   set {
     name  = "controller.name"
diff --git a/aws/eks/vars.tf b/aws/eks/vars.tf
index 5466108..a95516e 100644
--- a/aws/eks/vars.tf
+++ b/aws/eks/vars.tf
@@ -69,7 +69,6 @@ variable "eks_cluster_config_path" {
   type        = string
 }
 
-
 variable "map_users" {
   description = "Additional IAM users to add to the aws-auth configmap."
   type = list(object({
@@ -88,3 +87,8 @@ variable "aws_region" {
   description = "AWS region"
   type        = string
 }
+
+variable "ingress_deployment_namespace" {
+  description = "Namespace to deploy Nginx to"
+  type        = string
+}