[GitHub] [trafficcontrol] rob05c edited a comment on issue #3746: Fix DS "ACTIVE" flag to work as expected

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

rob05c edited a comment on issue #3746: Fix DS "ACTIVE" flag to work as expected
URL: https://github.com/apache/trafficcontrol/issues/3746#issuecomment-514232434
 
 
   Big +1. This is a terrible foot-gun. It's incredibly unintuitive, and further, if the wrong DS were left inadvertently serving at the edge, it could easily be a security issue.
   
   I suggest we add an additional DS flag, "active-caches," and rename this in the UI to reflect that it is "active-routed." Of course, with documentation and UI tooltips indicating exactly what each does.
   
   The "active-caches" flag to remove a DS from cache configs should be a tiny amount of work, it's basically a single if-statement in remap.config generation, unless I'm mistaken.
   
   I'm also +1 on labelling this 'bug' and 'major'. It may have been by-design, but it's a dangerous and deceptive design, and effectively a user interface bug.

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