Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add CI testing using API keys on merges to master/dev #100

Merged
merged 53 commits into from
Nov 22, 2023
Merged

Add CI testing using API keys on merges to master/dev #100

merged 53 commits into from
Nov 22, 2023

Conversation

elliottower
Copy link
Contributor

@elliottower elliottower commented Nov 22, 2023 •
edited
Loading

This is from two other PRs in dev, just wanted to confirm the CI passed there before merging to main.

I've also updated the repo settings to disallow merge commits--I had previously just done merge rather than squash and merge, hence all these commits.

@elliottower
Update Umshini environments to use new OpenAI 1.0.0 API, clean up code
8c8b977
@elliottower
Add workflows for automated CI testing of Umshini environments
075ba5d
@elliottower
Fix environment variable names for AzureOpenAI workflow
8dc87e7
@elliottower
Add test_umshini_environments file and exclude by default from pytest…
e0d6c8f 
… (new contributors need not worry about them)
@elliottower
Remove coverage due to errors
1f0b534
@elliottower
Fix typos
13456f6
@elliottower
Add missing dependency colorama
f883c17
@elliottower
Remove old client='' for langchain examples (hasn't been around for a…
c6fa050 
… while)
@elliottower
Restore backup mdoel code from Umshini environments (gpt-35-turbo ins…
f8aedda 
…tead of gpt-4)
@elliottower
Add checks that the env vars are done correctly
8dd2df6
@elliottower
Remove xvfb because not needed for these tests
ec3f135
@elliottower
Fix logic for env variable checks
075b41e
@elliottower
Fix wording slightly
1ef8b82
@elliottower
Change umshini env test to only install dependencies for it
581c1b8
@elliottower
Remove pytest xdist because it might mess up env vars
9bd5153
@elliottower
Add testing req as well as umshini for umshini tests
8a0bd35
@elliottower
Add pettingzoo classic reqs to umshini as well, not ideal but have to…
b0df5ae 
… bc project structure
@elliottower
Remove unittest skipif because it seems to not work properly for umsh…
a0388fe 
…ini tests
@elliottower
Specify httpx version as there's an OpenAI issue in CI with 0.25 version
6ac655b
@elliottower
Add explicit openai_api_key param, shouldn't change it but maybe
897dcf2
@elliottower
Specify openai version as most recent seems to break it
40520de
@elliottower
Loosen restriction on bardapi (requests subdependency which messes th…
481511e 
…ings up)
@elliottower
Make regular tests go first to see if they work
4850e28
@elliottower
Make same requirements for azure and regular openai workflows for con…
0864ed1 
…sistency
@elliottower
Add test debugging statements to see if the key is available in pytho…
b753370 
…n os.environ
@elliottower
Use to make environment variables visible to python
43ffe63
@elliottower
Use env var instead of secret directly in bash
1a207d5
@elliottower
Add troubleshooting to test if deployment is visible
e4344ca
@elliottower
Add same troubleshooting and echoing to in other workflow
a0ca659
@elliottower
Install dependencies first before doing environment variables
10ac5c3
@elliottower
Add env to individual tests as well
d203cab
@elliottower
Make secrets case sensitive allcaps to match on GitHub
f9aabd4
@elliottower
Change name to OPENAI_API_KEY for env var
8fd75d1
@elliottower
Add printing to regular openai ci
6a1a667
@elliottower
Test
ba2ce6a
@elliottower
Add more testing
a17c688
@elliottower
Test secret
8d8843f
@elliottower
test secret
80651de
@elliottower
Test secret fix typo
b622940
@elliottower
Change where env is defined?
759cce7
@elliottower
Remove test secret part
63bde2f
@elliottower
Remove test secret part
823d738
@elliottower
Remove all testing/debug, only run on merge to master
8e7b299
@elliottower
Revert unnecessary changes which were made for testing
cb0cb2a
@elliottower
Merge remote-tracking branch 'upstream/main' into umshini-openai-v1-u…
4d283ab 
…pdate

# Conflicts:
#	pyproject.toml
@elliottower
Merge master
7e8bee1
@elliottower
Add on pull request target to test the CIs without merging to master
f5e2f43
@elliottower
Fix syntax
419491a
@elliottower
Fix syntax to run regularly in this PR
99b8712
@elliottower
Make workflow trigger on pushes to main and dev
4990b55
@elliottower
Umshini: update environments to use OpenAI 1.0.0 API, add CI testing (#…
edc7f7d 
…98)
@elliottower
Add back pytest xdist to the CI for openai (takes 4 minutes sequentia…
1e297fa 
…lly)
@elliottower
Fix slow CI tests (using pytest-xdist) (#99)
b96d989
@elliottower elliottower merged commit e7f7865 into main Nov 22, 2023
15 checks passed
@elliottower elliottower deleted the dev branch November 22, 2023 22:17
@elliottower elliottower restored the dev branch November 22, 2023 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@elliottower