search

citusdata / docker

:ship: Docker images and configuration for Citus
Apache License 2.0
245 stars 100 forks source link

Bump docker to version 11.0.4 #305

Closed gurkanindibay closed 2 years ago

gurkanindibay commented 2 years ago

Changes related to version bump look good. However there are 2 changes that I'd like to see here.

  1. Please remove git submodule for tools repo. It causes fatal warnings on builds.
  2. Why do we update postgres-12/Dockerfile with a version that does not support PG12? I suggest we remove that file completely. We already removed it from publish_docker_images_on_push matrix so we already stopped building images using that on CI.
  1. Removed
  2. https://github.com/citusdata/docker/issues/306 I will fix with that issue