Skip to content

Commit

Permalink
Merge pull request #29 from lalithkota/develop
Browse files Browse the repository at this point in the history
Helm charts: Fixed dependency builds
  • Loading branch information
pjoshi751 authored Jul 11, 2024
2 parents 1135e23 + d791c5f commit 007ce71
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 8 deletions.
18 changes: 12 additions & 6 deletions .github/workflows/push_trigger.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,16 +46,22 @@ jobs:
echo "::warning::No Charts to publish";
echo "SKIP=TRUE" >> $GITHUB_ENV
else
for chartpath in charts/*/; do
if [ -f ${chartpath}Chart.yaml ]; then
helm dep up $chartpath
fi
done
RANCHER_CHARTS=()
while IFS= read -r chartpath; do
echo "chartpath: $chartpath"
chartname=$(basename "$chartpath")
echo "Chartname: $chartname"
helm dep up charts/$chartpath
helm package charts/$chartpath
is_rancher_chart=$(grep "$RANCHER_CHART_FILTER" charts/${chartpath%*/}/Chart.yaml || true)
if [ -n "$is_rancher_chart" ]; then
RANCHER_CHARTS+=("$chartname")
if [ -f charts/${chartname}/Chart.yaml ]; then
echo "Chartname: $chartname"
helm package charts/$chartpath
is_rancher_chart=$(grep "$RANCHER_CHART_FILTER" charts/${chartpath%*/}/Chart.yaml || true)
if [ -n "$is_rancher_chart" ]; then
RANCHER_CHARTS+=("$chartname")
fi
fi
done < charts-list.txt
echo "RANCHER_CHARTS=${RANCHER_CHARTS[@]}" >> $GITHUB_ENV
Expand Down
1 change: 0 additions & 1 deletion charts/openg2p-social-registry-p2/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,6 @@ esignet:
postgresInit:
commonAnnotations:
"helm.sh/hook-weight": "0"

keygen:
commonAnnotations:
"helm.sh/hook-weight": "1"
Expand Down
1 change: 0 additions & 1 deletion charts/openg2p-social-registry/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -222,7 +222,6 @@ loggingOpensearch:
_meta:
type: "config"
config_version: 2
config:
dynamic:
http:
Expand Down

0 comments on commit 007ce71

Please sign in to comment.