Skip to content

Commit

Permalink
fix!: remove the ArgoCD namespace variable
Browse files Browse the repository at this point in the history
Since we are hardcoding the namespace variable on all modules, the variable to set the ArgoCD namespace will no longer be needed as well.
  • Loading branch information
lentidas committed Jan 19, 2024
1 parent ee1eeed commit 5dd52e9
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 11 deletions.
7 changes: 2 additions & 5 deletions main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,7 @@ resource "argocd_project" "this" {

metadata {
name = "argocd"
namespace = var.argocd_namespace
annotations = {
"devops-stack.io/argocd_namespace" = var.argocd_namespace
}
namespace = "argocd"
}

spec {
Expand Down Expand Up @@ -57,7 +54,7 @@ data "utils_deep_merge_yaml" "values" {
resource "argocd_application" "this" {
metadata {
name = "argocd"
namespace = var.argocd_namespace
namespace = "argocd"
labels = merge({
"application" = "argocd"
"cluster" = "in-cluster"
Expand Down
6 changes: 0 additions & 6 deletions variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -12,12 +12,6 @@ variable "base_domain" {
type = string
}

variable "argocd_namespace" {
description = "Namespace used by Argo CD where the Application and AppProject resources should be created. Normally, it should take the outputof the namespace from the bootstrap module."
type = string
default = "argocd"
}

variable "argocd_project" {
description = "Name of the Argo CD AppProject where the Application should be created. If not set, the Application will be created in a new AppProject only for this Application."
type = string
Expand Down

0 comments on commit 5dd52e9

Please sign in to comment.