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 worker status #8350

Merged
merged 33 commits into from
Dec 19, 2023
Merged

add worker status #8350

merged 33 commits into from
Dec 19, 2023

Conversation

tcp
Copy link
Collaborator

@tcp tcp commented Dec 15, 2023

Description

Please include a summary of the change, the motivation, and any additional context that will help others understand your PR. If it closes one or more open issues, please tag them as described here.

Affected Dependencies

List any dependencies that are required for this change.

How has this been tested?

  • Describe the tests that you ran to verify your changes.
  • Provide instructions so we can reproduce.
  • List any relevant details for your test configuration.

Checklist

tcp and others added 18 commits December 15, 2023 10:36
@tcp
feat: split worker_service.py into worker, worker_stash, and worker_s…
ecbaaee 
…ervice
@tcp
feat: add worker stash to node client
364bc5f
@tcp
fix: consumsers typo
732f16e
@tcp
feat: improve worker repr
38811d9
@tcp
fix: rename service method for fetching container name
bb85bf3
@tcp
feat: upgrade DockerWorker with container_name
54e454a
@tcp
feat: add status to 0mq consumer
d77f812
@tcp
feat: update call to DockerWorker in worker_service.py
231634a
@tcp
feat: set job status to complete after it successfully runs
43fcf0b
@tcp
feat: type job and worker stashes in node
da81b40
@tcp
feat: add consumer_id to handle_message
6d0c953
@tcp
feat: add consumer_id to queue handle_message
43c478a
@tcp
fix: stash imports in node
df08e89
@tcp
feat: add consumer and worker properties to job
306f45b
@tcp
feat: add worker id and consumer id to job
020d5b0
@tcp
feat: update DockerWorker protocol version
65edbc5
@tcp
feat: update Job protocol version
eaa7562
@tcp
Merge branch 'helm-workerstatus' into helm-workerstatus-consumer-report
b4f3e68
@tcp tcp marked this pull request as draft December 15, 2023 13:58
@review-notebook-app Review Notebook App
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@koenvanderveen koenvanderveen changed the title feat: add worker status add worker status Dec 19, 2023
@koenvanderveen koenvanderveen marked this pull request as ready for review December 19, 2023 17:38
@koenvanderveen koenvanderveen merged commit 0c9da5b into dev Dec 19, 2023
32 checks passed
@koenvanderveen koenvanderveen deleted the helm-workerstatus branch December 19, 2023 19:20
shubham3121 added a commit that referenced this pull request Dec 21, 2023
@shubham3121
integrate changes from PR #8350
3cea958
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@koenvanderveen koenvanderveen koenvanderveen approved these changes

Assignees
No one assigned
Labels
0.8.4 Green Team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@tcp @koenvanderveen