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

Testing/manager #14

Open
wants to merge 72 commits into
base: feature/coverage
Choose a base branch
from
Open

Testing/manager #14

wants to merge 72 commits into from

Conversation

bgunnar5
Copy link
Owner

@bgunnar5 bgunnar5 commented Oct 2, 2024

No description provided.

ryannova and others added 30 commits July 23, 2024 12:29
@ryannova
Include celerymanager and update celeryadapter to check the status of…
78ef619 
… celery workers.
@ryannova
Fixed issue where the update status was outside of if statement for c…
8561a18 
…hecking workers
@ryannova
Include worker status stop and add template for merlin restart
1120dd7
@ryannova
Added comment to the CeleryManager init
f41938f
@ryannova
Increment db_num instead of being fixed
690115e
@ryannova
Added other subprocess parameters and created a linking system for re…
de4ffd0 
…dis to store env dict
@ryannova
Implemented stopping of celery workers and restarting workers properly
67e9268
@ryannova
Update stopped to stalled for when the worker doesn't respond to restart
406e4c2
@ryannova
Working merlin manager run but start and stop not working properly
78e4525
@ryannova
Made fix for subprocess to start new shell and fixed manager start an…
eca74ac 
…d stop
@ryannova
Added comments and update changelog
ec8aa78
@ryannova
Include style fixes
3f04d24
@ryannova
Fix style for black
5538f4b
@ryannova
Revert launch_job script that was edited when doing automated lint
b6bcd33
@ryannova
Move importing of CONFIG to be within redis_connection due to error o…
9b97f8b 
…f config not being created yet
@ryannova
Added space to fix style
c9dfd31
@ryannova
Revert launch_jobs.py:
a9bd865
@ryannova
Update import of all merlin.config to be in the function
ddc7614
@bgunnar5
suggested changes plus beginning work on monitor/manager collab
353a66b
@bgunnar5
move managers to their own folder and fix ssl problems
1a4d416
@bgunnar5
final PR touch ups
875f137
@ryannova
Merge pull request #2 from bgunnar5/monitor_manager_collab
9020aa0 
Merlin Manager x Merlin Monitor Integration
@ryannova
Fix lint style changes
58da9bc
@ryannova
Fixed issue with context manager
e75dcc2
@ryannova
Reset file that was incorrect changed
11f9e7c
@ryannova
Check for ssl cert before applying to Redis connection
7204e46
@ryannova
Comment out Active tests for celerymanager
53d8f32
@ryannova
Fix lint issue with unused import after commenting out Active celery …
a5ccb2d 
…tests
@ryannova
Fixed style for import
2b0e8a6
@ryannova
Fixed kwargs being modified when making a copy for saving to redis wo…
e49f378 
…rker args.
@bgunnar5
fix yaml syntax error
c136058
@bgunnar5
fix typo for getting runner os
56a6a05
@bgunnar5
fix test and add python version to CI cache
f45a798
@bgunnar5
add in common-setup step again with caches this time
290d350
@bgunnar5
run fix-style
8a1bc14
@bgunnar5
update CHANGELOG
58622ba
@bgunnar5
fix remaining style issues
917f8d7
@bgunnar5
run without caches to compare execution time of test suite
91c7505
@bgunnar5
resolve merge conflict
c7adb96
@bgunnar5
allow redis config to not use ssl
608e00e
@bgunnar5
remove stop-workers and query-workers tests from definitions.py
bf41a2d
@bgunnar5
create helper_funcs file with common testing functions
630c9c9
@bgunnar5
move query-workers to pytest and add base class w/ stop-workers tests
17889fd
@bgunnar5
update CHANGELOG
5c28b49
@bgunnar5
final changes for the stop-workers & query-workers tests
643b4d1
@bgunnar5
run fix-style
0f0264c
@bgunnar5
move stop and query workers tests to the same file
6340604
@bgunnar5
run fix-style
19c4bf7
@bgunnar5
go back to original cache setup
99257d8
@bgunnar5
try new cache for singularity install
f947eae
@bgunnar5
fix syntax issue in github workflow
beafb22
@bgunnar5
attempt to fix singularity cache
fac2892
@bgunnar5
remove ls statement that breaks workflow
c49660a
@bgunnar5
revert back to no common setup
ecb1762
@bgunnar5
remove unnecessary dependency
39d09d6
@bgunnar5
update github actions versions to use latest
5f4673b
@bgunnar5
update action versions that didn't save
70e540f
@bgunnar5
merge in new tests for stop and query workers commands
94f8f72
@bgunnar5
run fix-style
d2e85ec
@bgunnar5
move distributed test suite actions back to v2
9c4fca4
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
@bgunnar5 @ryannova