Skip to content

Commit

Permalink
Fixup workflows
Browse files Browse the repository at this point in the history
  • Loading branch information
sd109 committed Nov 3, 2024
1 parent 0483b04 commit 871c560
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 7 deletions.
6 changes: 2 additions & 4 deletions .github/workflows/test-pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,7 @@
name: Test pull request

on:
# We use pull_request_target so that dependabot-created workflows can run
pull_request_target:
pull_request:
types:
- opened
- synchronize
Expand All @@ -24,8 +23,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: PR must be from a branch in the stackhpc/azimuth-llm repo
run: |
exit ${{ github.event.pull_request.head.repo.full_name == 'stackhpc/azimuth-llm' && '0' || '1' }}
run: exit ${{ github.event.pull_request.head.repo.full_name == 'stackhpc/azimuth-llm' && '0' || '1' }}

publish_images:
needs: [fail_on_remote]
Expand Down
5 changes: 2 additions & 3 deletions .github/workflows/update-dependencies.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ jobs:
contents: write
pull-requests: write
steps:

- name: Checkout the config repo
uses: actions/checkout@v4
with:
Expand Down Expand Up @@ -43,8 +42,8 @@ jobs:
# Update yaml in-place with yq
yq e -i '.api.image.version = strenv(NEW_VLLM_TAG)' $CHART_VALUES
# Can't use in-place editing with jq
jq --indent 4 --arg tag $NEW_VLLM_TAG '.properties.api.properties.image.properties.version.default = $tag' chart/values.schema.json > chart/values.schema.json.new
mv chart/values.schema.json{.new,}
jq --indent 4 --arg tag $NEW_VLLM_TAG '.properties.api.properties.image.properties.version.default = $tag' charts/azimuth-llm/values.schema.json > charts/azimuth-llm/values.schema.json.new
mv charts/azimuth-llm/values.schema.json{.new,}
fi
- name: Create Pull Request
Expand Down
2 changes: 2 additions & 0 deletions charts/azimuth-llm/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -92,6 +92,8 @@ ui:
# available configuration options.
appSettings:
model_name: *model-name
model_instruction: |
You are a helpful AI assistant. Please respond appropriately.
# Use local system fonts by default to avoid GDPR issues
# with Gradio's defaults fonts which require fetching from
# the Google fonts API. To restore default Gradio theme
Expand Down

0 comments on commit 871c560

Please sign in to comment.