Skip to content

Issue 81 Integrate Docker Api and Jobs#88

Merged
elwincheng merged 2 commits intomainfrom
issue_81_integrate_docker_api_and_jobs
Feb 13, 2026
Merged

Issue 81 Integrate Docker Api and Jobs#88
elwincheng merged 2 commits intomainfrom
issue_81_integrate_docker_api_and_jobs

Conversation

@elwincheng
Copy link
Collaborator

Integrate Docker API to start up, run and clean up a container within Supervisor job processing.

  • Updated the Enqueue method in the scheduler to return job IDs.
  • Added tests for container job execution and integration with Supervisor
  • Rename containerMgr to dockerMgr and serve_image.go to docker.go
  • Removed duplicate, unused supervisor status functions

Testing

  • Run src/docker/run_container_job_test.sh

Elwin Cheng added 2 commits February 12, 2026 11:37
… job processing.

- Updated the `Enqueue` method in the scheduler to return job IDs.
- Added tests for container job execution and integration with Supervisor
- Rename containerMgr to dockerMgr and serve_image.go to docker.go
- Removed duplicate unused supervisor status functions
Copy link
Collaborator

@blobcode blobcode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm - make sure to check the redis adapter version? We may be using v8 iirc.

@elwincheng elwincheng linked an issue Feb 13, 2026 that may be closed by this pull request
@elwincheng
Copy link
Collaborator Author

lgtm - make sure to check the redis adapter version? We may be using v8 iirc.

Just checked - we are using v9

@elwincheng elwincheng merged commit 32da220 into main Feb 13, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

integration between jobs and Docker API

2 participants