I've added some comments to this PR, Brennan. Nice work. I am curious if 3 states of Active (tr-routed/served from cache, not tr-routed but served from cache and neither tr-routed nor served from cache) is really something that is needed. I would be curious what others have to say about that..especially operators of Traffic Control.
Jeremy On Mon, Apr 27, 2020 at 4:05 PM ocket 8888 <[email protected]> wrote: > I opened a blueprint PR last week to change how the DS "Active" flag works > - specifically from a boolean to an enumerated string constant. But I > forgot to alert the 'list. > > It's a relatively small change, but if you have any concerns, feel free to > voice them here: https://github.com/apache/trafficcontrol/pull/4654 > > It's still a draft because the changes can't be made until API v3, but > don't let that discourage you from leaving comments. >
