ocket8888 opened a new pull request #3283: ORT.py now implements all the same command line flags as the Perl script URL: https://github.com/apache/trafficcontrol/pull/3283 ## What does this PR do? Adds an implementation of the previously unused command-line parameters: * `--rev_proxy_disable` * `--dispersion DISP` * `--login_dispersion DISP` * `--wait_for_parents INT` * `--retries RETRIES` The default values are the same as the Perl script _in its usage output_. For example, the usage output of `traffic_ops_ort.pl` claims the default value for `--retries RETRIES` is 3, but it actually defaults to 5 - ORT.py honors the documented 3. Note that with the exception of `--rev_proxy_disable`, all arguments can be used the same way as with the Perl script; e.g. `--wait_for_parents=0` (although in the case of `--wait_for_parents` **any** integer that is not `0` will be considered a "Truth-y" value as opposed to the Perl behavior that required exactly `1`). Like the Perl script, the bypass of reverse proxies is disabled by default, but _unlike_ the Perl script, the mere presence of `--rev_proxy_disable` changes this behavior; no value is required, or indeed accepted. ## Which TC components are affected by this PR? - [x] Documentation - [ ] Grove - [ ] Traffic Analytics - [ ] Traffic Monitor - [ ] Traffic Ops - [x] Traffic Ops ORT (Python) - [ ] Traffic Portal - [ ] Traffic Router - [ ] Traffic Stats - [ ] Traffic Vault - [ ] Other _________ ## What is the best way to verify this PR? For most of them, a CDN-in-a-Box will suffice. I honestly don't know how I'd go about testing the reverse proxy usage, but the code that implements it is straightforward enough that I'm confident it will work. ## Check all that apply - [ ] This PR includes tests - [x] This PR includes documentation updates - [ ] This PR includes an update to CHANGELOG.md - [ ] This PR includes all required license headers - [ ] This PR includes a database migration (ensure that migration sequence is correct) - [ ] This PR fixes a serious security flaw. Read more: [www.apache.org/security](http://www.apache.org/security/)
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
