Skip to content

Commit

Permalink
fix(cert-mounter): Added helm_release_name (#379)
Browse files Browse the repository at this point in the history
* added helm_release_name

* fix helm_release_name max charts

* fix prometheus docs

* pre-commit fixs
  • Loading branch information
diegolagospagopa authored Nov 25, 2024
1 parent 3fc1daf commit 61c6484
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 1 deletion.
1 change: 1 addition & 0 deletions cert_mounter/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@ No modules.
|------|-------------|------|---------|:--------:|
| <a name="input_cert_mounter_chart_version"></a> [cert\_mounter\_chart\_version](#input\_cert\_mounter\_chart\_version) | (Optional) Cert mounter chart version | `string` | `"2.0.1"` | no |
| <a name="input_certificate_name"></a> [certificate\_name](#input\_certificate\_name) | (Required) Name of the certificate stored in the keyvault, that will be installed as a secret in aks | `string` | n/a | yes |
| <a name="input_helm_release_name"></a> [helm\_release\_name](#input\_helm\_release\_name) | (Optional) Cert Mounter release name. Changed this field force the helm destroy and re-install | `string` | `"cert-mounter-blueprint"` | no |
| <a name="input_kv_name"></a> [kv\_name](#input\_kv\_name) | (Required) Key vault name where to retrieve the certificate | `string` | n/a | yes |
| <a name="input_namespace"></a> [namespace](#input\_namespace) | (Required) Namespace where the cert secret will be created | `string` | n/a | yes |
| <a name="input_pod_cpu"></a> [pod\_cpu](#input\_pod\_cpu) | Pod request and limit for CPU (in `m`) | `number` | `10` | no |
Expand Down
2 changes: 1 addition & 1 deletion cert_mounter/main.tf
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
resource "helm_release" "cert_mounter" {
name = "cert-mounter-blueprint"
name = replace(substr(var.helm_release_name, 0, 53), ".", "-")
repository = "https://pagopa.github.io/aks-helm-cert-mounter-blueprint"
chart = "cert-mounter-blueprint"
version = local.chart_version
Expand Down
6 changes: 6 additions & 0 deletions cert_mounter/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,12 @@ locals {
chart_version = var.workload_identity_enabled ? var.cert_mounter_chart_version : "1.0.4"
}

variable "helm_release_name" {
type = string
description = "(Optional) Cert Mounter release name. Changed this field force the helm destroy and re-install"
default = "cert-mounter-blueprint"
}

variable "namespace" {
type = string
description = "(Required) Namespace where the cert secret will be created"
Expand Down
1 change: 1 addition & 0 deletions kubernetes_prometheus_install/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ module "aks_prometheus_install" {
prometheus_namespace = "monitoring"
storage_class_name = "default-zrs" #example of ZRS storage class created by kubernetes_storage_class
prometheus_crds_enabled = true
}
```

Expand Down

0 comments on commit 61c6484

Please sign in to comment.