From 0a8c69ab997d720bdc4a12dc5731d668e9d23c19 Mon Sep 17 00:00:00 2001 From: Danny Chiao Date: Thu, 4 Aug 2022 16:18:53 -0500 Subject: [PATCH 1/4] ci: Update feast-feature-server helm chart to be automatically released in each Feast version Signed-off-by: Danny Chiao --- infra/charts/feast-feature-server/Chart.yaml | 2 +- infra/charts/feast-feature-server/README.md | 46 +++++++++---------- infra/scripts/helm/push-helm-charts.sh | 2 + .../helm/validate-helm-chart-versions.sh | 2 +- infra/scripts/release/files_to_bump.txt | 3 ++ ui/package.json | 2 +- 6 files changed, 31 insertions(+), 26 deletions(-) diff --git a/infra/charts/feast-feature-server/Chart.yaml b/infra/charts/feast-feature-server/Chart.yaml index 6c1afc9540..32bc7ed334 100644 --- a/infra/charts/feast-feature-server/Chart.yaml +++ b/infra/charts/feast-feature-server/Chart.yaml @@ -2,7 +2,7 @@ apiVersion: v2 name: feast-feature-server description: Feast Feature Server in Go or Python type: application -version: 0.22.0 +version: 0.23.0 keywords: - machine learning - big data diff --git a/infra/charts/feast-feature-server/README.md b/infra/charts/feast-feature-server/README.md index a55451e788..6a811a9c11 100644 --- a/infra/charts/feast-feature-server/README.md +++ b/infra/charts/feast-feature-server/README.md @@ -1,6 +1,6 @@ # feast-feature-server -![Version: 0.22.0](https://img.shields.io/badge/Version-0.22.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) +![Version: 0.23.0](https://img.shields.io/badge/Version-0.23.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) Feast Feature Server in Go or Python @@ -12,28 +12,28 @@ Feast Feature Server in Go or Python ## Values -| Key | Type | Default | Description | -|-----|------|---------|-------------| -| affinity | object | `{}` | | -| fullnameOverride | string | `""` | | -| image.pullPolicy | string | `"IfNotPresent"` | | -| image.repository | string | `""` | | -| image.tag | string | `""` | | -| imagePullSecrets | list | `[]` | | -| livenessProbe.initialDelaySeconds | int | `30` | | -| livenessProbe.periodSeconds | int | `30` | | -| nameOverride | string | `""` | | -| nodeSelector | object | `{}` | | -| podAnnotations | object | `{}` | | -| podSecurityContext | object | `{}` | | -| readinessProbe.initialDelaySeconds | int | `20` | | -| readinessProbe.periodSeconds | int | `10` | | -| replicaCount | int | `1` | | -| resources | object | `{}` | | -| securityContext | object | `{}` | | -| service.port | int | `80` | | -| service.type | string | `"ClusterIP"` | | -| tolerations | list | `[]` | | +| Key | Type | Default | Description | +| ---------------------------------- | ------ | ---------------- | ----------- | +| affinity | object | `{}` | | +| fullnameOverride | string | `""` | | +| image.pullPolicy | string | `"IfNotPresent"` | | +| image.repository | string | `""` | | +| image.tag | string | `""` | | +| imagePullSecrets | list | `[]` | | +| livenessProbe.initialDelaySeconds | int | `30` | | +| livenessProbe.periodSeconds | int | `30` | | +| nameOverride | string | `""` | | +| nodeSelector | object | `{}` | | +| podAnnotations | object | `{}` | | +| podSecurityContext | object | `{}` | | +| readinessProbe.initialDelaySeconds | int | `20` | | +| readinessProbe.periodSeconds | int | `10` | | +| replicaCount | int | `1` | | +| resources | object | `{}` | | +| securityContext | object | `{}` | | +| service.port | int | `80` | | +| service.type | string | `"ClusterIP"` | | +| tolerations | list | `[]` | | ---------------------------------------------- Autogenerated from chart metadata using [helm-docs v1.11.0](https://github.com/norwoodj/helm-docs/releases/v1.11.0) diff --git a/infra/scripts/helm/push-helm-charts.sh b/infra/scripts/helm/push-helm-charts.sh index 08753adb3c..1c32ee985b 100755 --- a/infra/scripts/helm/push-helm-charts.sh +++ b/infra/scripts/helm/push-helm-charts.sh @@ -17,7 +17,9 @@ helm repo add feast-helm-chart-repo $bucket cd infra/charts helm package feast helm package feast-python-server +helm package feast-feature-server helm gcs push --public feast-${1}.tgz feast-helm-chart-repo --force helm gcs push --public feast-python-server-${1}.tgz feast-helm-chart-repo --force +helm gcs push --public feast-feature-server-${1}.tgz feast-helm-chart-repo --force rm -f ./*.tgz \ No newline at end of file diff --git a/infra/scripts/helm/validate-helm-chart-versions.sh b/infra/scripts/helm/validate-helm-chart-versions.sh index 0ba75bd744..aac79d9315 100755 --- a/infra/scripts/helm/validate-helm-chart-versions.sh +++ b/infra/scripts/helm/validate-helm-chart-versions.sh @@ -3,7 +3,7 @@ set -e # Amount of file locations that need to be bumped in unison when versions increment -UNIQUE_VERSIONS_COUNT=18 +UNIQUE_VERSIONS_COUNT=20 if [ $# -ne 1 ]; then echo "Please provide a single semver version (without a \"v\" prefix) to test the repository against, e.g 0.99.0" diff --git a/infra/scripts/release/files_to_bump.txt b/infra/scripts/release/files_to_bump.txt index a1e2d29623..dc8553cd2b 100644 --- a/infra/scripts/release/files_to_bump.txt +++ b/infra/scripts/release/files_to_bump.txt @@ -9,4 +9,7 @@ infra/charts/feast/charts/feature-server/values.yaml 8 infra/charts/feast/README.md 11 58 59 infra/charts/feast-python-server/Chart.yaml 5 infra/charts/feast-python-server/README.md 3 +infra/charts/feast-feature-server/Chart.yaml 5 +infra/charts/feast-feature-server/README.md 3 java/pom.xml 41 +ui/package.json diff --git a/ui/package.json b/ui/package.json index 22128cc968..c5eddbfd2e 100644 --- a/ui/package.json +++ b/ui/package.json @@ -1,6 +1,6 @@ { "name": "@feast-dev/feast-ui", - "version": "0.20.5", + "version": "0.23.0", "private": false, "files": [ "dist" From 40824e22f1699f5e477b4ed4b6107f5f844162d0 Mon Sep 17 00:00:00 2001 From: Danny Chiao Date: Thu, 4 Aug 2022 16:20:37 -0500 Subject: [PATCH 2/4] update files to bump Signed-off-by: Danny Chiao --- infra/scripts/release/files_to_bump.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/infra/scripts/release/files_to_bump.txt b/infra/scripts/release/files_to_bump.txt index dc8553cd2b..37f5393326 100644 --- a/infra/scripts/release/files_to_bump.txt +++ b/infra/scripts/release/files_to_bump.txt @@ -12,4 +12,4 @@ infra/charts/feast-python-server/README.md 3 infra/charts/feast-feature-server/Chart.yaml 5 infra/charts/feast-feature-server/README.md 3 java/pom.xml 41 -ui/package.json +ui/package.json 3 From 44ac31d5c610e11ba8396cfd63d94945dc67cb3d Mon Sep 17 00:00:00 2001 From: Danny Chiao Date: Thu, 4 Aug 2022 16:34:48 -0500 Subject: [PATCH 3/4] update ui automation Signed-off-by: Danny Chiao --- .github/workflows/publish.yml | 3 ++- ui/.npmrc | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) create mode 100644 ui/.npmrc diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml index 184fdb3cb6..ef4a0c79c8 100644 --- a/.github/workflows/publish.yml +++ b/.github/workflows/publish.yml @@ -196,4 +196,5 @@ jobs: working-directory: ./ui run: npm publish env: - NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}} + # This publish is working using an NPM automation token to bypass 2FA + NPM_TOKEN: ${{ secrets.NPM_TOKEN }} diff --git a/ui/.npmrc b/ui/.npmrc new file mode 100644 index 0000000000..bd3327ab5a --- /dev/null +++ b/ui/.npmrc @@ -0,0 +1 @@ +//registry.npmjs.org/:_authToken=${NPM_TOKEN} \ No newline at end of file From 3e7b76032819e80d2be44051c9d135d84abb4b70 Mon Sep 17 00:00:00 2001 From: Danny Chiao Date: Thu, 4 Aug 2022 16:36:32 -0500 Subject: [PATCH 4/4] update ui automation Signed-off-by: Danny Chiao --- .github/workflows/publish.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml index ef4a0c79c8..54051be385 100644 --- a/.github/workflows/publish.yml +++ b/.github/workflows/publish.yml @@ -130,7 +130,7 @@ jobs: publish-python-sdk: runs-on: ubuntu-latest - needs: [build_wheels] + needs: [build_wheels, publish-web-ui-npm] steps: - uses: actions/download-artifact@v2 with: