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



##########
File path: infrastructure/cdn-in-a-box/optional/docker-compose.debugging.yml
##########
@@ -32,13 +32,13 @@ services:
       - trafficmonitor-nondebug
   trafficops:
     build:
-      context: .
-      dockerfile: traffic_ops/Dockerfile-go-debug
-    image: trafficops-go-debug
+      context: ../..

Review comment:
       Context for this one can stay `.`. Otherwise, it won't find the 
Dockerfile

##########
File path: infrastructure/cdn-in-a-box/optional/docker-compose.debugging.yml
##########
@@ -62,13 +62,13 @@ services:
       args:
         TRAFFIC_MONITOR_RPM: traffic_monitor/traffic_monitor.rpm
     command: /usr/bin/true
-  # The trafficops-go-nondebug service ensures that the trafficops-go base 
image
-  # exists before trying to build trafficops-go-debug.
-  trafficops-go-nondebug:
-    image: trafficops-go
+  # The trafficops-nondebug service ensures that the trafficops base image
+  # exists before trying to build trafficops-debug.
+  trafficops-nondebug:
+    image: trafficops
     build:
-      context: .
-      dockerfile: traffic_ops/Dockerfile-go
+      context: ../../
+      dockerfile: traffic_ops/Dockerfile

Review comment:
       Building `trafficops-nondebug` fails:
   ```dockerfile
   Building trafficops-nondebug
   unable to prepare context: unable to evaluate symlinks in Dockerfile path: 
lstat 
/home/user/go/src/github.com/apache/trafficcontrol/traffic_ops/Dockerfile: no 
such file or directory
   Service 'trafficops-nondebug' failed to build
   ```
   
   You'll need to change the rpm path, too.




----------------------------------------------------------------
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