From 54dc1ac91776a6ff21f6e8b80e85d32cb957f701 Mon Sep 17 00:00:00 2001 From: "Richard Kuo (Danswer)" Date: Thu, 14 Nov 2024 11:14:12 -0800 Subject: [PATCH] unnecessary python setup --- .github/workflows/pr-helm-chart-testing.yml | 31 +++++++++++---------- 1 file changed, 16 insertions(+), 15 deletions(-) diff --git a/.github/workflows/pr-helm-chart-testing.yml b/.github/workflows/pr-helm-chart-testing.yml index 5b00303010d..f26ab43e780 100644 --- a/.github/workflows/pr-helm-chart-testing.yml +++ b/.github/workflows/pr-helm-chart-testing.yml @@ -23,21 +23,6 @@ jobs: with: version: v3.14.4 - - name: Set up Python - uses: actions/setup-python@v5 - with: - python-version: '3.11' - cache: 'pip' - cache-dependency-path: | - backend/requirements/default.txt - backend/requirements/dev.txt - backend/requirements/model_server.txt - - run: | - python -m pip install --upgrade pip - pip install --retries 5 --timeout 30 -r backend/requirements/default.txt - pip install --retries 5 --timeout 30 -r backend/requirements/dev.txt - pip install --retries 5 --timeout 30 -r backend/requirements/model_server.txt - - name: Set up chart-testing uses: helm/chart-testing-action@v2.6.1 @@ -52,6 +37,22 @@ jobs: echo "changed=true" >> "$GITHUB_OUTPUT" fi +# rkuo: I don't think we need python? +# - name: Set up Python +# uses: actions/setup-python@v5 +# with: +# python-version: '3.11' +# cache: 'pip' +# cache-dependency-path: | +# backend/requirements/default.txt +# backend/requirements/dev.txt +# backend/requirements/model_server.txt +# - run: | +# python -m pip install --upgrade pip +# pip install --retries 5 --timeout 30 -r backend/requirements/default.txt +# pip install --retries 5 --timeout 30 -r backend/requirements/dev.txt +# pip install --retries 5 --timeout 30 -r backend/requirements/model_server.txt + # lint all charts if any changes were detected - name: Run chart-testing (lint) if: steps.list-changed.outputs.changed == 'true'