11#! /bin/bash
2+ set -eu -o pipefail
23
3- if [ " ${# } " == " 0" ; then
4+ helm_files=(${HELM_FILES[@]} )
5+ if [ " ${# helm_files[@]} " == " 0" ; then
46 echo " No Helm files specified, nothing to do"
57 exit
68fi
7- helm_files=( " ${@ } "
8- echo " Helm files to render: ${helm_files[*]} "
9+ echo " ${# helm_files[@]} Helm file(s) to render: ${helm_files[*]} "
910
1011helm_template () {
11- if [ -z " ${2} " ; then
12+ set -eu -o pipefail
13+ if [ -z " ${1} " ; then
1214 echo " Error: Need file name to template" >&2
13- return 1
15+ return 2
1416 fi
1517
18+ # 'head' or 'base' ref - used for logging output
19+ ref=" ${1%%/* } "
20+
1621 # Set test = <something> to run against Helm teplates under test/
17- if [ -z " ${test } " ; then
18- helm_file=" ${2 } "
22+ if [ -z " ${TEST :- } " ; then
23+ helm_file=" ${1 } "
1924 else
20- helm_file=" test/${2 } "
25+ helm_file=" test/${1 } "
2126 fi
2227
2328 if [ ! -f " ${helm_file} " ; then
24- echo " Error : File \" ${helm_file} \" not found, skipping diff "
25- echo " Error: File \" ${helm_file} \" not found, skipping diff " >&2
29+ # echo "Warn : File \"${helm_file}\" not found, skipping"
30+ echo " File \" ${helm_file} \" not found, skipping" >&2
2631 return 1
2732 fi
2833
34+ # Determine repo type - HelmRepository or OCIRepository
35+ # https://fluxcd.io/flux/components/source/helmrepositories/
36+ # https://fluxcd.io/flux/components/source/ocirepositories/
37+ # https://fluxcd.io/flux/components/source/gitrepositories/
38+ if [[ " HelmRepository" == " $( yq ' . | select(.kind == "HelmRelease").spec.chart.spec.sourceRef.kind' " ${helm_file} " ) " ; then
39+ repo_type=helm
40+ repo_name=$( yq ' . | select(.kind == "HelmRelease").spec.chart.spec.sourceRef.name' " ${helm_file} " )
41+ chart=$( yq ' . | select(.kind == "HelmRelease").spec.chart.spec.chart' " ${helm_file} " )
42+ url=$( yq ' . | select(.kind == "HelmRepository").spec.url' " ${helm_file} " )
43+ version=$( yq ' . | select(.kind == "HelmRelease").spec.chart.spec.version' " ${helm_file} " )
44+ if [[ " ${url} " = " oci://" * ]]; then
45+ url=" ${url} /${chart} " # Syntax for chart repos is different from OCI repos (as HelmRepo kind)
46+ fi
47+ elif [[ " OCIRepository" == " $( yq ' . | select(.kind == "HelmRelease").spec.chartRef.kind' " ${helm_file} " ) " ; then
48+ repo_type=oci
49+ repo_name=$( yq ' . | select(.kind == "HelmRelease").spec.chartRef.name' " ${helm_file} " )
50+ chart=" ${repo_name} "
51+ url=$( yq ' . | select(.kind == "OCIRepository").spec.url' " ${helm_file} " )
52+ version=$( yq ' . | select(.kind == "OCIRepository").spec.ref.tag' " ${helm_file} " )
53+ else
54+ echo " Unable to determine repo type, skipping"
55+ echo " Unable to determine repo type, skipping" >&2
56+ return 2
57+ fi
58+
2959 # Extracting chart properties
3060 name=$( yq ' . | select(.kind == "HelmRelease").metadata.name' " ${helm_file} " )
3161 namespace=$( yq ' . | select(.kind == "HelmRelease").metadata.namespace' " ${helm_file} " )
32- version=$( yq ' . | select(.kind == "HelmRelease") | .spec.chart.spec.version' " ${helm_file} " )
33- url=$( yq ' . | select(.kind == "HelmRepository") | .spec.url' " ${helm_file} " )
34- chart=$( yq ' . | select(.kind == "HelmRelease") | .spec.chart.spec.chart' " ${helm_file} " )
3562 values=$( yq ' . | select(.kind == "HelmRelease").spec.values' " ${helm_file} " )
36- echo " Chart version ${1} : $version ($chart from $url )" >&2
3763
38- # Syntax for chart repos is different from OCI repos
64+ # Let's see what information we got out about the chart...
65+ echo " ${ref} repo type: ${repo_type} " >&2
66+ echo " ${ref} repo name: ${repo_name} " >&2
67+ echo " ${ref} repo/chart URL: ${url} " >&2
68+ echo " ${ref} chart name: ${chart} " >&2
69+ echo " ${ref} chart version: ${version} " >&2
70+ echo " ${ref} release name: ${name} " >&2
71+ echo " ${ref} release namespace: ${namespace} " >&2
72+
73+ # Syntax for chart repos is different from OCI repos (as HelmRepo kind)
3974 if [[ " ${url} " = " oci://" * ]]; then
40- chart_args=(" ${url} / ${chart} " # treat as array, to avoid adding single-quotes
75+ chart_args=(" ${url} " # treat as array, to avoid adding single-quotes
4176 else
4277 chart_args=(" ${chart} " " ${url} "
4378 fi
@@ -46,19 +81,16 @@ helm_template() {
4681 template_out=$( helm template " ${name} " ${chart_args[@]} --version " ${version} " " ${namespace} " <( echo " ${values} " ) 2>&1 ) || {
4782 echo " $template_out "
4883 echo " $template_out " >&2
49- return 1
84+ return 2
5085 }
5186
5287 # Cleanup template, removing comments, output
5388 template_clean=$( yq -P ' sort_keys(..) comments=""' <( echo " ${template_out} " ) )
5489 echo " $template_clean "
55-
56- # Debug info
57- echo " Line count ${1} : values ($( echo " ${values} " | wc -l) $( echo " ${template_out} " | wc -l) $( echo " ${template_clean} " | wc -l) " >&2
5890}
5991
6092EOF=$( dd if=/dev/urandom bs=15 count=1 status=none | base64)
61- echo " markdown<<$EOF " >> " $GITHUB_OUTPUT "
93+ echo " markdown<<$EOF " > " $GITHUB_OUTPUT "
6294echo " ## Flux Helm diffs" >> " $GITHUB_OUTPUT "
6395
6496any_failed=0
@@ -70,26 +102,38 @@ for helm_file in "${helm_files[@]}"; do
70102
71103 # Template before
72104 return_code=0
73- before_out =$( helm_template before " base/${helm_file} " ) || return_code=1
74- if [ $return_code -ne 0 ]; then
105+ base_out =$( helm_template " base/${helm_file} " ) || return_code=$?
106+ if [ $return_code -eq 2 ]; then # Ignore files skipped
75107 {
76108 echo ' ```'
77- echo " ${before_out} "
109+ echo " Error rendering base ref:"
110+ echo " ${base_out} "
78111 echo ' ```'
79112 } >> " $GITHUB_OUTPUT "
80113 any_failed=1
81114 continue
82115 fi
83116
84117 # Template after
85- after_out=$( helm_template after " head/${helm_file} " ) || true
118+ return_code=0
119+ head_out=$( helm_template " head/${helm_file} " ) || return_code=$?
120+ if [ $return_code -ne 0 ]; then
121+ {
122+ echo ' ```'
123+ echo " Error rendering head ref:"
124+ echo " ${head_out} "
125+ echo ' ```'
126+ } >> " $GITHUB_OUTPUT "
127+ any_failed=1
128+ continue
129+ fi
86130
87131 # Template diff
88- diff_out=$( diff --unified=5 <( echo " ${before_out } " ) <( echo " ${after_out } " ) ) || true
132+ diff_out=$( diff --unified=5 <( echo " ${base_out } " ) <( echo " ${head_out } " ) ) || true
89133 echo " Diff has $( echo " $diff_out " | wc -l) "
90134 [ -z " ${diff_out} " && diff_out=" No changes"
91135 {
92- echo ' ```'
136+ echo ' ```diff '
93137 echo " ${diff_out} "
94138 echo ' ```'
95139 } >> " $GITHUB_OUTPUT "
99143 echo " $EOF "
100144 echo " any_failed=$any_failed "
101145} >> " $GITHUB_OUTPUT "
146+
147+ echo -e " \nAll done"
0 commit comments