reenable building what is required only

Author: def-

bin/ci-builder

@@ -167,10 +167,9 @@ print(hash.decode())
 cache_tag=cache-$flavor-$rust_version-$arch_go
 
 image_registry="materialize"
-# TODO: Reenable
-# if is_truthy "${CI:-0}"; then
-#     image_registry="ghcr.io/materializeinc/materialize"
-# fi
+if is_truthy "${CI:-0}"; then
+    image_registry="ghcr.io/materializeinc/materialize"
+fi
 
 case "$cmd" in
     build)

ci/mkpipeline.py

@@ -183,8 +183,7 @@ def fetch_hashes() -> None:
     check_depends_on(pipeline, args.pipeline)
     add_version_to_preflight_tests(pipeline)
     trim_builds_prep_thread.join()
-    # TODO: Reenable
-    # trim_builds(pipeline, hash_check)
+    trim_builds(pipeline, hash_check)
     add_cargo_test_dependency(
         pipeline,
         args.pipeline,

misc/python/materialize/mzbuild.py

@@ -857,12 +857,10 @@ def try_pull(self, max_retries: int) -> bool:
 
     def is_published_if_necessary(self) -> bool:
         """Report whether the image exists on Docker Hub if it is publishable."""
-        # TODO: Put back the original implementation
-        return self.publish
-        # if self.publish and is_docker_image_pushed(self.spec()):
-        #     ui.say(f"{self.spec()} already exists")
-        #     return True
-        # return False
+        if self.publish and is_docker_image_pushed(self.spec()):
+            ui.say(f"{self.spec()} already exists")
+            return True
+        return False
 
     def run(
         self,
@@ -1084,9 +1082,7 @@ def ensure(self, pre_build: Callable[[list[ResolvedImage]], None] | None = None)
                     )
                 )
 
-            # TODO: Reenable
-            # deps_to_build = [dep for dep, should_build in futures if should_build]
-            deps_to_build = [dep for dep, should_build in futures]
+            deps_to_build = [dep for dep, should_build in futures if should_build]
 
         prep = self._prepare_batch(deps_to_build)
         if pre_build: