Skip to content

Docker worker updates #1880

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
Merged
Didayolo merged 1 commit into from
Jul 15, 2025
Merged

Docker worker updates #1880

Didayolo merged 1 commit into from
Jul 15, 2025

Conversation

@ObadaS
Copy link
Collaborator

@ObadaS ObadaS commented Jun 18, 2025 •
edited by Didayolo
Loading

A brief description of the purpose of the changes contained in this PR.

  • Changed the Dockerfile.compute_worker to use Fedora as a base image instead of python, reducing the overall image size and making it on par with the podman Containerfile (easier to maintain in the future)
  • Changed the Dockerfile.compute_worker_gpu to use the CPU image as base, making the image build faster by removing duplicate lines.

A checklist for hand testing

  • Test a results submission
  • Test a code submission
  • Test locally and on staging server

Checklist

  • Code review by me
  • Hand tested by me
  • I'm proud of my work
  • Code review by reviewer
  • Hand tested by reviewer
  • CircleCi tests are passing
  • Ready to merge

Changed base image from python3.9 to Fedora 42, reducing size from 1.…
e67088a 
…6GB to 1GB; Made Dockerfile.compute_worker_gpu base image the build CPU image, adding only the necessary things to build it faster
@Didayolo Didayolo self-assigned this Jul 15, 2025
@Didayolo Didayolo mentioned this pull request Jul 15, 2025
Merged
@Didayolo Didayolo merged commit d98abf9 into develop Jul 15, 2025
1 check passed
@Didayolo Didayolo deleted the workerUpdates branch July 15, 2025 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@Didayolo Didayolo

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ObadaS @Didayolo