Skip to content

Commit 3d5255c

Browse files
authored
Fix.image.tag (#115)
* Fix image tag * readme nits
1 parent d4ed9aa commit 3d5255c

2 files changed

Lines changed: 11 additions & 8 deletions

File tree

.github/workflows/helm-images.yaml

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -52,22 +52,25 @@ jobs:
5252
5353
# Set chart version - use tag name if available, otherwise use semver format
5454
if [[ "${{ github.ref_type }}" == "tag" ]]; then
55-
CHART_VERSION="${{ github.ref_name }}"
56-
# Remove 'v' prefix if present
57-
CHART_VERSION="${CHART_VERSION#v}"
55+
TAG_NAME="${{ github.ref_name }}"
56+
# Chart version must not have 'v' prefix (OCI/semver convention)
57+
CHART_VERSION="${TAG_NAME#v}"
58+
# App version keeps 'v' prefix to match container image tags
59+
APP_VERSION="${TAG_NAME}"
5860
else
5961
CHART_VERSION="0.0.0-${{ github.sha }}"
62+
APP_VERSION="${CHART_VERSION}"
6063
fi
61-
64+
6265
# Package and push each chart in deploy/charts/
6366
for chart_dir in deploy/charts/*/; do
6467
if [ -f "${chart_dir}Chart.yaml" ]; then
6568
chart_name=$(basename "$chart_dir")
6669
echo "Processing chart: $chart_name"
67-
70+
6871
# Update chart version and appVersion in Chart.yaml
6972
sed -i "s/^version:.*/version: ${CHART_VERSION}/" "${chart_dir}Chart.yaml"
70-
sed -i "s/^appVersion:.*/appVersion: ${CHART_VERSION}/" "${chart_dir}Chart.yaml"
73+
sed -i "s/^appVersion:.*/appVersion: \"${APP_VERSION}\"/" "${chart_dir}Chart.yaml"
7174
7275
# Package the chart
7376
helm package "$chart_dir" --version "${CHART_VERSION}"

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,8 +45,8 @@ The hub needs to be reachable by agents from anywhere — it's the only componen
4545
4. **Install the hub via Helm:**
4646

4747
```bash
48-
helm install kedge-hub oci://ghcr.io/faroshq/charts/kedge-hub \
49-
--namespace kedge --create-namespace \
48+
helm install kedge oci://ghcr.io/faroshq/charts/kedge-hub \
49+
--namespace kedge-system --create-namespace \
5050
--set hub.hubExternalURL=https://kedge.example.com
5151
```
5252

0 commit comments

Comments
 (0)