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

Monitor manager collab #12

Open
wants to merge 36 commits into
base: develop
Choose a base branch
from
Open

Monitor manager collab #12

wants to merge 36 commits into from

Conversation

bgunnar5
Copy link
Owner

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.
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