From 8e7cad4ea5eaa01385f9b2ebdeb521287051f3ae Mon Sep 17 00:00:00 2001
From: dd di cesare <didi@posteo.net>
Date: Tue, 18 Jun 2024 15:55:58 +0200
Subject: [PATCH 1/3] [workflow] Release ID not needed

---
 .github/workflows/release-helm-chart.yaml | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/.github/workflows/release-helm-chart.yaml b/.github/workflows/release-helm-chart.yaml
index e6d627ea..eef2f6be 100644
--- a/.github/workflows/release-helm-chart.yaml
+++ b/.github/workflows/release-helm-chart.yaml
@@ -18,10 +18,6 @@ on:
         description: Limitador version
         default: latest
         type: string
-      releaseId:
-        description: Release ID
-        default: 0
-        type: string
 
 jobs:
   chart_release:

From dd752c7906d1c85700f64037f44c838ed754bfa4 Mon Sep 17 00:00:00 2001
From: dd di cesare <didi@posteo.net>
Date: Tue, 18 Jun 2024 15:59:50 +0200
Subject: [PATCH 2/3] [workflow] Removing chart build step

* No need to build if the tag already has the manifests
* Removing input variable for limitador version
* Added description
---
 .github/workflows/release-helm-chart.yaml | 15 ++++-----------
 1 file changed, 4 insertions(+), 11 deletions(-)

diff --git a/.github/workflows/release-helm-chart.yaml b/.github/workflows/release-helm-chart.yaml
index eef2f6be..84607830 100644
--- a/.github/workflows/release-helm-chart.yaml
+++ b/.github/workflows/release-helm-chart.yaml
@@ -1,4 +1,7 @@
-name: release-helm-chart
+# Description: This workflow is used to release the Helm chart to the GitHub repository. The chart manifests should be already
+# built with the target `helm-build` and the manifests changes already committed to the tag to be released.
+
+name: Release Helm Chart
 on:
 # TODO: The following commented lines should be used depending on the release strategy
 #  release:
@@ -14,10 +17,6 @@ on:
           description: Operator bundle version
           default: 0.0.0
           type: string
-      limitadorVersion:
-        description: Limitador version
-        default: latest
-        type: string
 
 jobs:
   chart_release:
@@ -37,12 +36,6 @@ jobs:
         git config user.name "$GITHUB_ACTOR"
         git config user.email "$GITHUB_ACTOR@users.noreply.github.com"
 
-    - name: Build the Helm Chart manifests
-      run: |
-        make helm-build \
-          VERSION=${{ inputs.operatorVersion }} \
-          LIMITADOR_VERSION=${{ inputs.limitadorVersion }}
-
     - name: Package Helm Chart
       run: |
         make helm-package

From 7fc5906e3865237000bdfe992aeebeecc9c8cf3c Mon Sep 17 00:00:00 2001
From: dd di cesare <didi@posteo.net>
Date: Tue, 18 Jun 2024 16:13:42 +0200
Subject: [PATCH 3/3] [workflow] Fix path to built chart package

---
 .github/workflows/release-helm-chart.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/release-helm-chart.yaml b/.github/workflows/release-helm-chart.yaml
index 84607830..9d59143f 100644
--- a/.github/workflows/release-helm-chart.yaml
+++ b/.github/workflows/release-helm-chart.yaml
@@ -45,7 +45,7 @@ jobs:
       id: upload-chart
       with:
         repo_token: ${{ secrets.GITHUB_TOKEN }}
-        file: charts/limitador-operator-${{ inputs.operatorVersion }}.tgz
+        file: limitador-operator-${{ inputs.operatorVersion }}.tgz
         asset_name: chart-limitador-operator-${{ inputs.operatorVersion }}.tgz
         tag: ${{ github.ref }}
         overwrite: true