From f1d54bb0ef963b818376e4d8190c8ff0519f84a4 Mon Sep 17 00:00:00 2001
From: Pablo Vigo <pvigo@collabora.com>
Date: Mon, 13 Jan 2025 11:08:51 +0100
Subject: [PATCH 1/2] inventory: force download chart
When running a check, an error occurs in the `Deploy` task because
the previous download step is skipped due to the simulation mode.
`Error: failed to download "jetstack/cert-manager"`
By adding the option `check_mode: no`, the chart download is forced,
allowing it to be evaluated afterward.
Added in cert-manager, metrics-server & vector ansible roles.
Signed-off-by: Pablo Vigo <pvigo@collabora.com>
---
digital-ocean-k8s/roles/cert-manager/tasks/main.yaml | 1 +
digital-ocean-k8s/roles/metrics-server/tasks/main.yaml | 1 +
digital-ocean-k8s/roles/vector/tasks/main.yaml | 1 +
3 files changed, 3 insertions(+)
diff --git a/digital-ocean-k8s/roles/cert-manager/tasks/main.yaml b/digital-ocean-k8s/roles/cert-manager/tasks/main.yaml
index dd1c45d..81ca48f 100644
--- a/digital-ocean-k8s/roles/cert-manager/tasks/main.yaml
+++ b/digital-ocean-k8s/roles/cert-manager/tasks/main.yaml
@@ -7,6 +7,7 @@
state: present
- name: "Add cert-manager chart repository"
+ check_mode: no
kubernetes.core.helm.helm_repository:
name: "jetstack"
repo_url: "https://charts.jetstack.io"
diff --git a/digital-ocean-k8s/roles/metrics-server/tasks/main.yaml b/digital-ocean-k8s/roles/metrics-server/tasks/main.yaml
index 3bc4ccf..36d3a36 100644
--- a/digital-ocean-k8s/roles/metrics-server/tasks/main.yaml
+++ b/digital-ocean-k8s/roles/metrics-server/tasks/main.yaml
@@ -7,6 +7,7 @@
state: present
- name: "Add cert-manager chart repository"
+ check_mode: no
kubernetes.core.helm.helm_repository:
name: "metrics-server"
repo_url: "https://kubernetes-sigs.github.io/metrics-server/"
diff --git a/digital-ocean-k8s/roles/vector/tasks/main.yaml b/digital-ocean-k8s/roles/vector/tasks/main.yaml
index a5f82b0..46497b3 100644
--- a/digital-ocean-k8s/roles/vector/tasks/main.yaml
+++ b/digital-ocean-k8s/roles/vector/tasks/main.yaml
@@ -7,6 +7,7 @@
state: present
- name: "Add vector chart repository"
+ check_mode: no
kubernetes.core.helm.helm_repository:
name: "vector"
repo_url: "https://helm.vector.dev"
--
GitLab
From 5e4bcc8e469add584afd59683a4a51561a89986d Mon Sep 17 00:00:00 2001
From: Pablo Vigo <pvigo@collabora.com>
Date: Mon, 13 Jan 2025 11:29:43 +0100
Subject: [PATCH 2/2] Fix title task error
A task title was copy from another role and the name of the task was
incorrect.
Signed-off-by: Pablo Vigo <pvigo@collabora.com>
---
digital-ocean-k8s/roles/metrics-server/tasks/main.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/digital-ocean-k8s/roles/metrics-server/tasks/main.yaml b/digital-ocean-k8s/roles/metrics-server/tasks/main.yaml
index 36d3a36..2782a8c 100644
--- a/digital-ocean-k8s/roles/metrics-server/tasks/main.yaml
+++ b/digital-ocean-k8s/roles/metrics-server/tasks/main.yaml
@@ -6,7 +6,7 @@
kind: Namespace
state: present
-- name: "Add cert-manager chart repository"
+- name: "Add metrics-server chart repository"
check_mode: no
kubernetes.core.helm.helm_repository:
name: "metrics-server"
--
GitLab