Fix helm repo addition and dockerhub path context (#1155)

terryyylim · web-flow · commit dc6cdb8b551e · 2020-11-11T20:24:47.000+08:00
* Fix helm repo addition and dockerhub path context

Signed-off-by: Terence &lt;terencelimxp@gmail.com&gt;

* Fix env variable access and context

Signed-off-by: Terence &lt;terencelimxp@gmail.com&gt;
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -80,10 +80,11 @@ jobs:
           RELEASE_VERSION: ${{ needs.get-version.outputs.release_version }}
         with:
           push: true
-          file: infra/docker/${{ matrix.component }}/Dockerfile
+          context: .
+          file: ./infra/docker/${{ matrix.component }}/Dockerfile
           tags: feastdev/feast-${{ matrix.component }}:${{ needs.get-version.outputs.release_version }}
           build-args: |
-            REVISION=${RELEASE_VERSION}
+            REVISION=$RELEASE_VERSION
       - name: Build and push latest
         uses: docker/build-push-action@v2
         env:
@@ -93,10 +94,11 @@ jobs:
         with:
           if: ${VERSION_WITHOUT_PREFIX} == ${HIGHEST_SEMVER_TAG:1}
           push: true
-          file: infra/docker/${{ matrix.component }}/Dockerfile
+          context: .
+          file: ./infra/docker/${{ matrix.component }}/Dockerfile
           tags: feastdev/feast-${{ matrix.component }}:latest
           build-args: |
-            REVISION=${RELEASE_VERSION}
+            REVISION=$RELEASE_VERSION
 
   publish-helm-charts:
     runs-on:  ubuntu-latest
diff --git a/infra/scripts/sync-helm-charts.sh b/infra/scripts/sync-helm-charts.sh
@@ -30,7 +30,7 @@ fi
 
 exit_code=0
 
-helm repo add bitnami bitnami https://charts.bitnami.com/bitnami
+helm repo add bitnami https://charts.bitnami.com/bitnami
 
 for dir in "$repo_dir"/*; do
     if  helm dep update "$dir" && helm dep build "$dir"; then
