openedx / wg-devops

Issue repository for the DevOps Working Group
1 stars 1 forks source link

Can we converge on one set up official upstream images? #38

Open kdmccormick opened 2 years ago

kdmccormick commented 2 years ago

There are at least three sets of Open edX docker images:

  1. The Ansible-based ones under edxops/ that devstack uses,
  2. The in-repo-Dockerfile ones under openedx/ that nothing uses, and
  3. The ones built by Tutor and its plugins.

Set 1 could be dropped after Devstack deprecation & removal.

Could we drop Set 2, and make Set 3 the "official" set of Docker images? Would it make sense to host/mirror them under the openedx/ DockerHub org?

regisb commented 2 years ago

I am not aware of any application that uses the images from the in-repo Dockerfiles. The initial ambition was that they would replace the images in the Devstack, but that didn't work. You were the last one to make changes to edx-platform/Dockerfile, so I guess that you know more about it.

In the long run, Tutor could probably make use of the upstream images. But for that to happen there needs to be a deliberate effort to create, test and maintain these images. My impression is that the Dockerfiles were created with the hope that they would be properly maintained, but they are not because they are not part of anyone's workflow -- I might be wrong here of course, because I don't know if there are people actually using these images.

Thus I agree that the Dockerfiles should be removed, because they might create confusion and do not serve any particular purpose.

Would it make sense to host/mirror them under the openedx/ DockerHub org?

From my perspective, the organization that hosts a project is also tasked with its maintenance. ("great powers, great responsibilities") So I'm perfectly fine with 2U hosting mirror images, but I don't want to pull images from a different org.

For the record, pushing images to a different org is done quite easily with:

tutor config save --set DOCKER_IMAGE_OPENEDX='2u/openedx:{{ TUTOR_VERSION }}'
tutor images build openedx
tutor images push openedx
kdmccormick commented 1 month ago

This is potentially solved by: