You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficcontrol.apache.org by GitBox <gi...@apache.org> on 2020/10/29 21:00:54 UTC

[GitHub] [trafficcontrol] ocket8888 opened a new pull request #5226: Move clients out of empty directory

ocket8888 opened a new pull request #5226:
URL: https://github.com/apache/trafficcontrol/pull/5226


   ## What does this PR (Pull Request) do?
   - [x] This PR is not related to any Issue
   
   This PR just moves the Python and Java clients out of the annoying, empty, top-level directory `traffic_control`, and also updates references throughout the repository to correctly specify the client path.
   
   Draft because I don't want to change this for 5.0.0
   
   ## Which Traffic Control components are affected by this PR?
   - CDN in a Box
   - Documentation
   - Traffic Control Client (Python)
   - CI tests
   
   ## What is the best way to verify this PR?
   There should be no functionality changes, so just verify that the installation instructions work as now written.
   
   ## The following criteria are ALL met by this PR
   - [x] Tests are unnecessary when functionality isn't changing
   - [x] Documentation changes are unnecessary when functionality isn't changing
   - [x] An update to CHANGELOG.md is not necessary
   - [x] This PR includes any and all required license headers
   - [x] This PR does not include a database migration
   - [x] This PR **DOES NOT FIX A SERIOUS SECURITY VULNERABILITY**


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [trafficcontrol] rawlinp commented on pull request #5226: Move clients out of empty directory

Posted by GitBox <gi...@apache.org>.
rawlinp commented on pull request #5226:
URL: https://github.com/apache/trafficcontrol/pull/5226#issuecomment-719040965


   _Making a mental note to go fix the myriad things that will break because of this_


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [trafficcontrol] ocket8888 commented on pull request #5226: Move clients out of empty directory

Posted by GitBox <gi...@apache.org>.
ocket8888 commented on pull request #5226:
URL: https://github.com/apache/trafficcontrol/pull/5226#issuecomment-719062593


   Oh yeah. That's why it's a draft, I think people maintaining production deployments will need more than a couple days heads-up.


----------------------------------------------------------------
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:
users@infra.apache.org