[GitHub] [trafficcontrol] rawlinp commented on issue #6963: `POST /api/3.1/servers` does not set `updPending`

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

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

   I don't necessarily agree with this note in the docs:
   > In the vast majority of cases, it is advisable that the PUT method of the [servers/{{ID}}](https://traffic-control-cdn.readthedocs.io/en/latest/api/v3/servers_id.html#to-api-v3-servers-id) endpoint be used instead.
   
   IMO queueing updates should be its own separate action -- being able to create a server in TO with updates already queued does not really buy us much. And for updates, it makes sense to queue updates separately without having to include all the unrelated server fields, because it's way more common to queue updates on a server than it is to actually change fields of a server.
   
   So, while this is technically a regression, I actually think it's better this way. Just my 2 cents.


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

To unsubscribe, e-mail: issues-unsubscribe@trafficcontrol.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org