[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3653: Add support for Cache Assignment Groups to DB and TO API

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

ocket8888 commented on a change in pull request #3653: Add support for Cache Assignment Groups to DB and TO API
URL: https://github.com/apache/trafficcontrol/pull/3653#discussion_r292010200
 
 

 ##########
 File path: docs/source/api/deliveryservices_id_servers.rst
 ##########
 @@ -23,7 +23,7 @@
 
 ``GET``
 =======
-Retrieves properties of Edge-Tier servers assigned to a :term:`Delivery Service`.
+Retrieves properties of Edge-Tier servers assigned to a :term:`Delivery Service`. This includes servers explicitly assigned to a Delivery Service and servers assigned via a :term:`Cache Assignment Group`.
 
 Review comment:
   If you're going to make `Cache Assignment Group` a `:term:` - and I think you should - then you need to add a definition for it to the Glossary. Otherwise this does nothing except generate a compilation warning.
   
   Also, terms should be linked to the glossary (via `:term:`) at least once per major section ("page" in an HTML context), so you'd need to do that on all the other "pages" where you mentioned CAGs before.

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


With regards,
Apache Git Services