rawlinp commented on a change in pull request #4700:
URL: https://github.com/apache/trafficcontrol/pull/4700#discussion_r433277459



##########
File path: infrastructure/cdn-in-a-box/enroller/Dockerfile
##########
@@ -20,7 +20,7 @@ FROM golang:1.14.2 AS enroller-builder
 # enroller source and dependencies
 COPY ./lib/ /go/src/github.com/apache/trafficcontrol/lib/
 COPY ./vendor/ /go/src/github.com/apache/trafficcontrol/vendor/
-COPY ./traffic_ops/client/ 
/go/src/github.com/apache/trafficcontrol/traffic_ops/v2-client/
+COPY ./traffic_ops/v2-client/ 
/go/src/github.com/apache/trafficcontrol/traffic_ops/v2-client/

Review comment:
       Your situation is not unique, I'm just saying there has been at least 
one other person who hit this issue, and it would be better to fix the bug for 
everyone rather than tie it to a massive new feature. Separate bugfix PRs are 
also much easier to backport if necessary.




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