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

Dev workflows #10

Closed
wants to merge 29 commits into from
Closed

Dev workflows #10

wants to merge 29 commits into from

Conversation

yotamloe
Copy link
Contributor

  • Add httpretty~=1.1.4 to requirements.txt
  • Separate workflow deploy.yaml and tests.yaml
  • Update workflow actions version

yotamloe and others added 29 commits March 12, 2024 12:51
@yotamloe
Add httpretty~=1.1.4
23bb1a4
@yotamloe
Update workflows
cd62914
@yotamloe
Update workflows
eb1e57d
@yotamloe
Update tests workflow
16cf53c
@yotamloe
Update workflows
40be8ec
@yotamloe
Add dev deps
1950b66
@yotamloe
add pytest debug logs
f283b58
@8naama
fix test config files
91f4e16
@8naama
try fix deadlock in test_sending_data test
525002d
@8naama
try another approach to fix tests
55fd527
@8naama
revert attempts
a2103ca
@8naama
try fix issue
d0a6c19
@8naama
try killing process different way
535fd25
@8naama
try another approach
65ae906
@8naama
add print for process state
dd716ce
@8naama
add return at ApisManager
b3ee7b2
@8naama
try new approach
6a7ccd3
@8naama
add missing param for new approach
462202b
@8naama
try False on queue get block
458e355
@8naama
verify deadlock in q.get
1943bcb
@8naama
update logger test to debug
8d4892f
@8naama
add checks to the functions that fill the queue
553e9fc
@8naama
add test param to run()func in api manager to try stop the process …
e2a47ac 
…for the test to work
@8naama
add test=True to the relevant tests
d555573
@8naama
add extra test log and reduce sleep before killing process
bddecbd
@8naama
run ApisManager().run as process to kill it
7f7f76f
@8naama
test another URL and restore prev api_managers
5c11a7e
@8naama
revert oath_api
f449376
@8naama
add test parameter to API manager to see if it will solve issue
76ef30d
@8naama 8naama mentioned this pull request Mar 28, 2024
Closed
@8naama 8naama mentioned this pull request Jun 10, 2024
Merged
@8naama 8naama closed this Jul 8, 2024
@8naama 8naama deleted the dev-workflows branch August 18, 2024 09:55
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.
2 participants
@yotamloe @8naama