Skip to content

Commit

Permalink
Merge pull request #34 from Sefaria/fix-deploy
Browse files Browse the repository at this point in the history
fix: chart name
  • Loading branch information
BrendanGalloway authored Mar 25, 2024
2 parents 4c36154 + 0a295b7 commit cabfa92
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions .github/workflows/deploy.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
name: Deploy Production

on:
workflow_dispatch:
inputs:
Expand All @@ -11,7 +10,6 @@ on:
type: string
required: true
default: ""

jobs:
deploy:
runs-on: ubuntu-latest
Expand Down Expand Up @@ -49,7 +47,7 @@ jobs:
uses: frenck/action-setup-yq@v1
- name: Deploy Production
run: |
helm install llm "oci://us-east1-docker.pkg.dev/${{ secrets.PROD_GKE_PROJECT}}/containers/sefaria-llm/chart/Sefaria-LLM" \
helm install llm "oci://us-east1-docker.pkg.dev/${{ secrets.PROD_GKE_PROJECT}}/containers/sefaria-llm/chart/sefaria-llm" \
--version ${{ inputs.chart_version }} \
--set deployment.image.tag="${{ inputs.version }}" \
--set deployment.image.registry="us-east1-docker.pkg.dev/${{ secrets.PROD_GKE_PROJECT }}/containers"

0 comments on commit cabfa92

Please sign in to comment.