Simplify module vars and polish doc links

Author: onelrian

cert-manager/README.md

@@ -6,12 +6,10 @@ This directory contains the necessary configurations for **Cert-Manager**, which
 
 You can deploy Cert-Manager using one of the following methods:
 
-### 1. Terraform (Automated)
-This method uses the Terraform configuration located in the `terraform/` directory. It is the recommended approach for integration with the full observability stack.
+### 1. Automated Deployment
+This method uses the Terraform configuration located in the `terraform/` directory. It is the recommended approach for automation.
 
-- [**Terraform Deployment Guide**](../docs/cert-manager-terraform-deployment.md)
+For detailed instructions, see the [Terraform deployment guide](../docs/cert-manager-terraform-deployment.md).
 
 ### 2. Manual (Helm & Kubectl)
-If you prefer to deploy manually using CLI tools, you can follow the manual guide.
-
-- [**Manual Deployment Guide**](../docs/cert-manager-manual-deployment.md)
+If you prefer to deploy manually using CLI tools, you can follow the [manual deployment guide](../docs/cert-manager-manual-deployment.md).

cert-manager/terraform/main.tf

@@ -5,12 +5,12 @@ resource "helm_release" "cert_manager" {
   repository       = "https://charts.jetstack.io"
   chart            = "cert-manager"
   namespace        = var.namespace
-  create_namespace = var.create_namespace
+  create_namespace = true
   version          = var.cert_manager_version
 
   set {
     name  = "installCRDs"
-    value = var.install_crds
+    value = "true"
   }
 
   wait    = true

cert-manager/terraform/variables.tf

@@ -16,17 +16,7 @@ variable "namespace" {
   default     = "cert-manager"
 }
 
-variable "create_namespace" {
-  description = "Whether to create the namespace"
-  type        = bool
-  default     = true
-}
 
-variable "install_crds" {
-  description = "Whether to install CRDs"
-  type        = bool
-  default     = true
-}
 
 variable "letsencrypt_email" {
   description = "Email address for Let's Encrypt certificate notifications"

docs/cert-manager-terraform-deployment.md

@@ -68,3 +68,4 @@ cd observability
 | `cert_issuer_kind` | Kind of Issuer (`ClusterIssuer` or `Issuer`) | `ClusterIssuer` |
 | `issuer_namespace` | Namespace for Issuer (if Kind is Issuer). Defaults to install namespace. | `null` |
 | `ingress_class_name` | Ingress class for solving challenges | `nginx` |
+| `issuer_server` | ACME server URL | `https://acme-v02...` |

docs/kubernetes-observability.md

@@ -85,8 +85,8 @@ The stack automatically provisions **Cert-Manager** and **NGINX Ingress Controll
 > **Important**: Set these to `true` in your `terraform.tfvars` if you are doing a fresh install of the entire stack and need these components.
 
 If you prefer to deploy them standalone or manually, refer to their respective documentation:
-- [Cert-Manager Documentation](../cert-manager/README.md)
-- [Ingress Controller Documentation](../ingress-controller/README.md)
+- [Cert-Manager Documentation](../cert-manager/README.md) - *See [Terraform Deployment Guide](../docs/cert-manager-terraform-deployment.md#variables) for full configuration options.*
+- [Ingress Controller Documentation](../ingress-controller/README.md) - *See [Terraform Deployment Guide](../docs/ingress-controller-terraform-deployment.md#variables) for full configuration options.*
 
 ## Installation
 

ingress-controller/README.md

@@ -6,12 +6,10 @@ This directory contains the configurations for the **NGINX Ingress Controller**,
 
 You can deploy the Ingress Controller using one of the following methods:
 
-### 1. Terraform (Automated)
+### 1. Automated Deployment
 This method uses the Terraform configuration located in the `terraform/` directory.
 
-- [**Terraform Deployment Guide**](../docs/ingress-controller-terraform-deployment.md)
+For detailed instructions, see the [Terraform deployment guide](../docs/ingress-controller-terraform-deployment.md).
 
 ### 2. Manual (Helm)
-If you prefer to deploy manually using Helm, you can follow the manual guide.
-
-- [**Manual Deployment Guide**](../docs/ingress-controller-manual-deployment.md)
+If you prefer to deploy manually using Helm, you can follow the [manual deployment guide](../docs/ingress-controller-manual-deployment.md).

ingress-controller/terraform/main.tf

@@ -5,7 +5,7 @@ resource "helm_release" "nginx_ingress" {
   repository       = "https://kubernetes.github.io/ingress-nginx"
   chart            = "ingress-nginx"
   namespace        = var.namespace
-  create_namespace = var.create_namespace
+  create_namespace = true
   version          = var.nginx_ingress_version
 
   set {

ingress-controller/terraform/variables.tf

@@ -22,11 +22,6 @@ variable "namespace" {
   default     = "ingress-nginx"
 }
 
-variable "create_namespace" {
-  description = "Whether to create the namespace"
-  type        = bool
-  default     = true
-}
 
 variable "replica_count" {
   description = "Number of replicas for the controller"

lgtm-stack/README.md

@@ -14,7 +14,7 @@ The **LGTM** stack is a comprehensive open-source observability platform powered
 This repository provides two guides to help you deploy the stack:
 
 ### 1. Automated Deployment
-For a fully automated deployment using this stack, please follow the [Kubernetes Observability Guide](../docs/kubernetes-observability.md).
+For a fully automated deployment using this stack, please refer to the [observability guide](../docs/kubernetes-observability.md).
 
 ### 2. Manual Deployment
 If you prefer to configure and deploy components manually, or need to understand the individual steps, please refer to the [Manual LGTM Deployment Guide](../docs/manual-lgtm-deployment.md).

lgtm-stack/terraform/main.tf

@@ -160,10 +160,10 @@ module "cert_manager" {
   install_cert_manager = var.install_cert_manager
   cert_manager_version = var.cert_manager_version
   namespace            = "cert-manager"
-  install_crds         = true
-  letsencrypt_email    = var.letsencrypt_email
-  cert_issuer_name     = var.cert_issuer_name
-  cert_issuer_kind     = var.cert_issuer_kind
+
+  letsencrypt_email = var.letsencrypt_email
+  cert_issuer_name  = var.cert_issuer_name
+  cert_issuer_kind  = var.cert_issuer_kind
   # If Kind is Issuer, it must be in the observability namespace to be used by the ingress in that namespace.
   # If Kind is ClusterIssuer, this variable is ignored by the module logic.
   issuer_namespace   = var.namespace