From 08334bc561642467b1fa85dc802f8a0eb6968d43 Mon Sep 17 00:00:00 2001
From: Balaji Arun <balaji@aptoslabs.com>
Date: Fri, 3 Mar 2023 09:59:53 -0800
Subject: [PATCH] [Dockerfile][Experimental] Remove cargo-chef and sequence
 builds (#6841)

---
 docker/experimental/builder.Dockerfile       | 21 --------------------
 docker/experimental/debian-base.Dockerfile   |  3 +--
 docker/experimental/docker-bake-rust-all.hcl |  3 ---
 3 files changed, 1 insertion(+), 26 deletions(-)

diff --git a/docker/experimental/builder.Dockerfile b/docker/experimental/builder.Dockerfile
index c9b90647f2145..383ff1c30597b 100644
--- a/docker/experimental/builder.Dockerfile
+++ b/docker/experimental/builder.Dockerfile
@@ -17,18 +17,6 @@ RUN --mount=type=cache,target=/var/cache/apt,sharing=locked \
         binutils \
         lld
 
-# install cargo chef to cache dependencies
-RUN --mount=type=cache,target=/root/.cargo <<EOT
-    cargo install cargo-chef@0.1.51
-    cargo install cargo-cache --no-default-features --features ci-autoclean
-EOT
-
-### Build Rust dependencies ###
-FROM rust-base as planner
-
-COPY --link . .
-RUN cargo chef prepare --recipe-path recipe.json
-
 ### Build Rust code ###
 FROM rust-base as builder-base
 
@@ -52,15 +40,6 @@ RUN ARCHITECTURE=$(uname -m | sed -e "s/arm64/arm_64/g" | sed -e "s/aarch64/aarc
     && chmod +x "/usr/local/bin/protoc" \
     && rm "protoc-21.5-linux-$ARCHITECTURE.zip"
 
-# Use cargo chef
-COPY  --link --from=planner /aptos/recipe.json recipe.json
-
-# Build dependencies - this is the caching Docker layer!
-RUN <<EOT
-    cargo chef cook --profile ${PROFILE} --workspace --recipe-path recipe.json
-    cargo cache
-EOT
-
 COPY --link . /aptos/
 
 FROM builder-base as aptos-node-builder
diff --git a/docker/experimental/debian-base.Dockerfile b/docker/experimental/debian-base.Dockerfile
index 891d13bfc0d35..fe201f1038ec5 100644
--- a/docker/experimental/debian-base.Dockerfile
+++ b/docker/experimental/debian-base.Dockerfile
@@ -5,6 +5,5 @@ FROM debian AS debian-base
 RUN rm -f /etc/apt/apt.conf.d/docker-clean; echo 'Binary::apt::APT::Keep-Downloaded-Packages "true";' > /etc/apt/apt.conf.d/keep-cache
 
 # Add Tini to make sure the binaries receive proper SIGTERM signals when Docker is shut down
-ADD https://github.com/krallin/tini/releases/download/v0.19.0/tini /tini
-RUN chmod +x /tini
+ADD --chmod=755 https://github.com/krallin/tini/releases/download/v0.19.0/tini /tini
 ENTRYPOINT ["/tini", "--"]
\ No newline at end of file
diff --git a/docker/experimental/docker-bake-rust-all.hcl b/docker/experimental/docker-bake-rust-all.hcl
index 773b9fa5e9a53..60d72517f2763 100644
--- a/docker/experimental/docker-bake-rust-all.hcl
+++ b/docker/experimental/docker-bake-rust-all.hcl
@@ -54,7 +54,6 @@ group "forge-images" {
 
 target "debian-base" {
   dockerfile = "docker/experimental/debian-base.Dockerfile"
-  context = "."
   contexts = {
     debian = "docker-image://debian:bullseye-20220912@sha256:3e82b1af33607aebaeb3641b75d6e80fd28d36e17993ef13708e9493e30e8ff9"
   }
@@ -81,7 +80,6 @@ target "builder-base" {
 target "aptos-node-builder" {
   dockerfile = "docker/experimental/builder.Dockerfile"
   target = "aptos-node-builder"
-  context = "."
   contexts = {
     rust = "docker-image://rust:1.64.0-bullseye@sha256:5cf09a76cb9baf4990d121221bbad64927cc5690ee54f246487e302ddc2ba300"
     builder-base = "target:builder-base"
@@ -100,7 +98,6 @@ target "aptos-node-builder" {
 target "tools-builder" {
   dockerfile = "docker/experimental/builder.Dockerfile"
   target = "tools-builder"
-  context = "."
   contexts = {
     rust = "docker-image://rust:1.64.0-bullseye@sha256:5cf09a76cb9baf4990d121221bbad64927cc5690ee54f246487e302ddc2ba300"
     builder-base =  "target:builder-base"