ocket8888 commented on PR #7235: URL: https://github.com/apache/trafficcontrol/pull/7235#issuecomment-1343009472
> Also while your changes technically catches the error, I still think those should be fixed (or at least not pollute the console). Just changing the Content Routing Type causes a dozen of the label not found error. Changing anything will do that, just because it triggers a digest cycle. All of those errors are coming from DSCP labeling, because many DSCPs aren't labeled. Tell me, though, what does a DSCP of 10 mean? According to the "human-friendly" label, it means "10 - AF11". What's that? Our docs don't tell you, there's no tooltip that tells you. Even better, look at a DSCP of 37. What that value means - according to the label - is simply "37 - ". Insightful. I think these are proprietary labels that have no real meaning for the ATC project, and IMO we'd be better off without them. Numbers is numbers. That would stop the log spam. But that's also unrelated to this PR, could be done at a later date if we don't think it's safe to pull that trigger right now. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
