zrhoffman commented on a change in pull request #4821:
URL: https://github.com/apache/trafficcontrol/pull/4821#discussion_r444609281



##########
File path: infrastructure/cdn-in-a-box/docker-compose.yml
##########
@@ -58,10 +58,10 @@ services:
   # defined below
   trafficops:
     build:
-      context: ../..
-      dockerfile: infrastructure/cdn-in-a-box/traffic_ops/Dockerfile-go
+      context: .
+      dockerfile: traffic_ops/Dockerfile-go
       args:
-        TRAFFIC_OPS_RPM: 
infrastructure/cdn-in-a-box/traffic_ops/traffic_ops.rpm
+        TRAFFIC_OPS_RPM: traffic_ops/traffic_ops.rpm

Review comment:
       These path changes also need to be made to the `trafficops-go-nondebug` 
service in `/infrastructure/cdn-in-a-box/optional/docker-compose.debugging.yml`

##########
File path: infrastructure/cdn-in-a-box/traffic_ops/Dockerfile-go
##########
@@ -45,18 +45,18 @@ FROM    trafficops-common-deps
 
 # Override TRAFFIC_OPS_RPM arg to use a different one using --build-arg 
TRAFFIC_OPS_RPM=...  Can be local file or http://...
 #
-ARG     TRAFFIC_OPS_RPM=infrastructure/cdn-in-a-box/traffic_ops/traffic_ops.rpm
-
-COPY    $TRAFFIC_OPS_RPM /
-RUN     to_rpm_filename="$(basename $TRAFFIC_OPS_RPM)" && \
-        rpm --install --nodeps --verbose --hash "$to_rpm_filename" && \
-        rm "$to_rpm_filename"
-
-COPY    infrastructure/cdn-in-a-box/enroller/server_template.json \
-        infrastructure/cdn-in-a-box/traffic_ops/config.sh \
-        infrastructure/cdn-in-a-box/traffic_ops/run-go.sh \
-        infrastructure/cdn-in-a-box/traffic_ops/to-access.sh \
-        /
+ARG TRAFFIC_OPS_RPM=traffic_ops/traffic_ops.rpm
+
+COPY $TRAFFIC_OPS_RPM /
+RUN to_rpm_filename="$(basename $TRAFFIC_OPS_RPM)" && \
+    rpm --install --nodeps --verbose --hash "$to_rpm_filename" && \
+    rm "$to_rpm_filename"
+
+COPY enroller/server_template.json \
+     traffic_ops/config.sh \
+     traffic_ops/run-go.sh \
+     traffic_ops/to-access.sh \
+     /

Review comment:
       These path changes need to be made in 
`/infrastructure/cdn-in-a-box/traffic_ops/Dockerfile-go.dockerignore` for any 
Docker BuildKit users




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to