[GitHub] [trafficcontrol] rawlinp commented on issue #5382: Incorrect description of delivery service max dns answers in TP and documentation

[GitBox <gi...@apache.org>]

rawlinp commented on issue #5382:
URL: https://github.com/apache/trafficcontrol/issues/5382#issuecomment-754823702


   That description that @mitchell852 proposed does sound sufficient for DNS delivery services. For HTTP delivery services, I think it only overrides the `edge.http.limit` parameter when both are set: 
   ```
   edge.http.limit | CRConfig.json | Integer that controls the default number of records returned when edge.http.routing is set to true; this can be overridden by the maxDnsAnswers delivery service setting
   ```
   - https://traffic-control-cdn.readthedocs.io/en/latest/admin/traffic_router.html?highlight=edge.dns.limit
   
   So maybe a good description for maxDnsAnswers in the HTTP DS form is:
   ```
   If the Traffic Router profile parameter "edge.http.limit" is set, setting this to a non-zero value will override that parameter for this delivery service, limiting the number of Traffic Router IP addresses (A records) that are included in responses to DNS requests for this delivery service.
   ```


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