Skip to content

Commit 18929e1

Browse files
fix: another small fix to helm charts in release CI (Mellanox#1658)
maintenance op chart has a different name pattern in the network operator, need to account for that
2 parents 9910271 + a36b464 commit 18929e1

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

.github/workflows/release.yaml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -259,7 +259,12 @@ jobs:
259259
continue
260260
fi
261261
262-
CHART_LOCATION_IN_NETWORK_OPERATOR=deployment/network-operator/charts/$SOURCE_REPO
262+
CHART_NAME=$(yq ".${component}.chartName" hack/release.yaml)
263+
# Use SOURCE_REPO if chartName is null or empty
264+
if [ "$CHART_NAME" = "null" ] || [ -z "$CHART_NAME" ]; then
265+
CHART_NAME=$SOURCE_REPO
266+
fi
267+
CHART_LOCATION_IN_NETWORK_OPERATOR=deployment/network-operator/charts/$CHART_NAME
263268
rm -rf $CHART_LOCATION_IN_NETWORK_OPERATOR/*
264269
cp -r ../$SOURCE_REPO/$CHART_LOCATION/* $CHART_LOCATION_IN_NETWORK_OPERATOR
265270

hack/release.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -113,6 +113,7 @@ maintenanceOperator:
113113
sourceRepository: maintenance-operator
114114
version: network-operator-v25.7.0-beta.3
115115
chartLocation: deployment/maintenance-operator-chart
116+
chartName: maintenance-operator-chart
116117
spectrumXOperator:
117118
image: spectrum-x-operator
118119
repository: nvcr.io/nvstaging/mellanox

0 commit comments

Comments
 (0)