Skip to content

Conversation

@ObadaS
Copy link
Collaborator

@ObadaS ObadaS commented Jan 30, 2026

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

The goal of this PR is make it easier to maintain the repository by moving some files, removing useless code, reduce

  • Moved and renamed all Dockerfiles to packaging/container/Containerfile*
  • Use fedora instead of python image in Containerfile (for django and site_worker)
  • Replace poetry with uv (main project and compute worker)
  • Updated Container.builder image (node:10 -> node:lts-alpine3.23), smaller and more recent
  • Use official Flower image instead of building our own (much smaller, going from 1+GB to 85MB)
  • Remove "apt update" and "dnf update" in some of the Containerfiles since the base image is already CVE free and doing this brings in more packages (more chance of CVE) and installs latest python
  • Remove some unmaintained sections in the documentation (Fabric)

Issues this PR resolves

Closes #2063

A checklist for hand testing

The E2E tests already takes care of a lot of testing, but some parts of the code remains untested:

  • Create a Queue
  • Connect it to a compute worker (preferably with an HTTPS instance)
  • Create an Organization
  • Make a submission as an Organization
  • Check Analytics
  • Check Server Status Page

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

Obada Haddad added 2 commits January 30, 2026 15:39
…less updates inside containers; move/rename Dockerfiles; switch to uv from poetry
@ObadaS ObadaS changed the title Cleanup to reduce CVE and make folder more less convoluted Cleanup and Optimizations to reduce CVE and make folders less convoluted Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[compute_worker] replace poetry with uv

2 participants