Skip to content

Commit c890fe4

Browse files
committed
fix: environment variables sent
1 parent 6c02a74 commit c890fe4

File tree

2 files changed

+58
-58
lines changed

2 files changed

+58
-58
lines changed

.github/workflows/dev.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ jobs:
7676
with:
7777
template-repository-name: "lambda-feedback/chat-function-boilerplate"
7878
# allow for developer to specify the environment variables that are used by the deployed AWS Lambda. Default to mock then admin can update.
79-
deployed-environment-variables: '["OPENAI_API_KEY","OPENAI_MODEL"]'
79+
deployed-environment-variables: '[\"OPENAI_API_KEY\",\"OPENAI_MODEL\",\"GOOGLE_AI_API_KEY\",\"GOOGLE_AI_MODEL\"]'
8080
permissions:
8181
contents: read
8282
id-token: write

.github/workflows/main.yml

Lines changed: 57 additions & 57 deletions
Original file line numberDiff line numberDiff line change
@@ -6,77 +6,77 @@ on:
66
workflow_dispatch:
77

88
jobs:
9-
# test:
10-
# name: Staging deployment tests
11-
# runs-on: ubuntu-latest
12-
# strategy:
13-
# fail-fast: false
14-
# matrix:
15-
# python-version: ["3.11"]
16-
# env:
17-
# OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
18-
# OPENAI_MODEL: ${{ vars.OPENAI_MODEL }}
19-
# GOOGLE_AI_API_KEY: ${{ secrets.GOOGLE_AI_API_KEY }}
20-
# GOOGLE_AI_MODEL: ${{ vars.GOOGLE_AI_MODEL }}
21-
# steps:
22-
# - name: Checkout Code
23-
# uses: actions/checkout@v4
9+
test:
10+
name: Staging deployment tests
11+
runs-on: ubuntu-latest
12+
strategy:
13+
fail-fast: false
14+
matrix:
15+
python-version: ["3.11"]
16+
env:
17+
OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
18+
OPENAI_MODEL: ${{ vars.OPENAI_MODEL }}
19+
GOOGLE_AI_API_KEY: ${{ secrets.GOOGLE_AI_API_KEY }}
20+
GOOGLE_AI_MODEL: ${{ vars.GOOGLE_AI_MODEL }}
21+
steps:
22+
- name: Checkout Code
23+
uses: actions/checkout@v4
2424

25-
# - name: Set up Python ${{ matrix.python-version }}
26-
# id: python-setup
27-
# uses: actions/setup-python@v5
28-
# with:
29-
# python-version: ${{ matrix.python-version }}
25+
- name: Set up Python ${{ matrix.python-version }}
26+
id: python-setup
27+
uses: actions/setup-python@v5
28+
with:
29+
python-version: ${{ matrix.python-version }}
3030

31-
# # - name: Load cached venv
32-
# # id: dependencies-cache
33-
# # uses: actions/cache@v3
34-
# # with:
35-
# # path: .venv
36-
# # key: venv-${{ runner.os }}-${{ steps.python-setup.outputs.python-version }}
31+
# - name: Load cached venv
32+
# id: dependencies-cache
33+
# uses: actions/cache@v3
34+
# with:
35+
# path: .venv
36+
# key: venv-${{ runner.os }}-${{ steps.python-setup.outputs.python-version }}
3737

38-
# - name: Create Venv if Cache not found
39-
# # if: steps.dependencies-cache.outputs.cache-hit != 'true'
40-
# run: |
41-
# python -m venv .venv
38+
- name: Create Venv if Cache not found
39+
# if: steps.dependencies-cache.outputs.cache-hit != 'true'
40+
run: |
41+
python -m venv .venv
4242
43-
# - name: Install dependencies
44-
# # if: steps.dependencies-cache.outputs.cache-hit != 'true'
45-
# run: |
46-
# pip install --upgrade pip
47-
# pip install -r requirements.txt
43+
- name: Install dependencies
44+
# if: steps.dependencies-cache.outputs.cache-hit != 'true'
45+
run: |
46+
pip install --upgrade pip
47+
pip install -r requirements.txt
4848
49-
# - name: Run tests
50-
# if: always()
51-
# run: |
52-
# source .venv/bin/activate
53-
# pytest --junit-xml=./reports/pytest.xml --tb=auto -v
49+
- name: Run tests
50+
if: always()
51+
run: |
52+
source .venv/bin/activate
53+
pytest --junit-xml=./reports/pytest.xml --tb=auto -v
5454
55-
# - name: Upload test results
56-
# uses: actions/upload-artifact@v4
57-
# if: always()
58-
# with:
59-
# name: test-results
60-
# path: ./reports/pytest.xml
61-
# if-no-files-found: warn
55+
- name: Upload test results
56+
uses: actions/upload-artifact@v4
57+
if: always()
58+
with:
59+
name: test-results
60+
path: ./reports/pytest.xml
61+
if-no-files-found: warn
6262

63-
# build:
64-
# name: Build Docker Image
65-
# uses: lambda-feedback/chat-function-workflows/.github/workflows/gh_build.yml@main
66-
# needs: test
67-
# permissions:
68-
# contents: read
69-
# id-token: write
70-
# packages: write
63+
build:
64+
name: Build Docker Image
65+
uses: lambda-feedback/chat-function-workflows/.github/workflows/gh_build.yml@main
66+
needs: test
67+
permissions:
68+
contents: read
69+
id-token: write
70+
packages: write
7171

7272
deploy:
7373
name: Deploy to Lambda Feedback
7474
uses: lambda-feedback/chat-function-workflows/.github/workflows/main_deploy.yml@main
75-
# needs: test
75+
needs: test
7676
with:
7777
template-repository-name: "lambda-feedback/chat-function-boilerplate"
7878
# allow for developer to specify the environment variables that are used by the deployed AWS Lambda. Default to mock then admin can update.
79-
deployed-environment-variables: '["OPENAI_API_KEY","OPENAI_MODEL","GOOGLE_AI_API_KEY","GOOGLE_AI_MODEL"]'
79+
deployed-environment-variables: '[\"OPENAI_API_KEY\",\"OPENAI_MODEL\",\"GOOGLE_AI_API_KEY\",\"GOOGLE_AI_MODEL\"]'
8080
permissions:
8181
contents: read
8282
id-token: write

0 commit comments

Comments
 (0)