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



##########
File path: traffic_ops/app/conf/cdn.conf
##########
@@ -36,7 +36,6 @@
         "oauth_client_secret": "",
         "routing_blacklist": {
             "ignore_unknown_routes": false,
-            "perl_routes": [],

Review comment:
       So, `postinstall` is unchanged. It even still calls `_postinstall` with 
the changes made in #4763. The postinstall.py is _added_ as an alternative 
only, but the default is to still use the Perl version.
   
   Installation is still `yum install trops.rpm` and then run `postinstall`. 
configuration is mostly untouched, just some things don't have any purpose 
anymore. I guess the biggest change is that some scripts may expect that by 
setting their `PERL5LIB` path (or whatever it's called) that their dependencies 
will be satisfied, but that won't actually be true for new installations. A lot 
of those aren't used anymore afaik, and many others have active PRs rewriting 
them. Oh, osversion.cfg is gone, only osversions.json is supported now.
   
   > Can you make a simlilar update to the ansible TO config templates?
   
   I can try, but I don't know much about Ansible - I kind of assume removing 
the Perl service might just break it all in half.




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