zuul/zuul-jobs
Code Issues Proposed changes

Drop Helm v2 support to fix v3 issue

Browse Source 
Helm v2 is long gone and there seems to be no interest in
supporting it any longer. [1]

Dropping support for v2 allows for an easy fix to the
command used so that it names the release properly
as it was supposed to instead of trying to set the
namespace name (the -n switch changed purpose [1]).

[1] https://lists.zuul-ci.org/archives/list/zuul-discuss@lists.zuul-ci.org/thread/WUWBM5F3PXXDLKK6JNSP4UR4VTWDNPZ4/

Story: 2010783
Task: 48201
Change-Id: I00676bcc7e8b4f3ec014f2292c6c0ee4c17acd96
This commit is contained in:
Radosław Piliszek 2023-06-20 17:06:36 +02:00
parent 29f27ebaaa
commit 0f662a23d0
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
roles/helm-template/tasks/main.yaml
View File

@ -4,14 +4,14 @@
chdir: "{{ zuul_work_dir }}" chdir: "{{ zuul_work_dir }}"
- name: Print templated charts - name: Print templated charts
command: "helm template -n zuul {{ helm_chart }}" command: "helm template zuul {{ helm_chart }}"
args: args:
chdir: "{{ zuul_work_dir }}" chdir: "{{ zuul_work_dir }}"
- name: Deploy templated charts - name: Deploy templated charts
shell: | shell: |
set -o pipefail set -o pipefail
helm template {% if helm_values_file is defined %}--values {{ helm_values_file }} {% endif %}-n {{ helm_release_name }} {{ helm_chart }} | kubectl apply -f- helm template {% if helm_values_file is defined %}--values {{ helm_values_file }} {% endif %}{{ helm_release_name }} {{ helm_chart }} | kubectl apply -f-
args: args:
executable: /bin/bash executable: /bin/bash
chdir: "{{ zuul_work_dir }}" chdir: "{{ zuul_work_dir }}"