zrhoffman opened a new issue #5098:
URL: https://github.com/apache/trafficcontrol/issues/5098


   <!--
   ************ STOP!! ************
   If this issue identifies a security vulnerability, DO NOT submit it! 
Instead, contact
   the Apache Traffic Control Security Team at 
[email protected] and follow the
   guidelines at https://www.apache.org/security/ regarding vulnerability 
disclosure.
   
   - For *SUPPORT QUESTIONS*, use the Traffic Control slack 
(https://s.apache.org/atc-slack)
   or Traffic Control mailing lists 
(https://trafficcontrol.apache.org/mailing_lists).
   - Before submitting, please **SEARCH GITHUB** for a similar issue or PR.
   -->
   
   ## I'm submitting a ...
   <!-- delete all those that don't apply -->
   <!--- security vulnerability (STOP!! - see above)-->
   -  improvement request (usability, performance, tech debt, etc.)
   
   ## Traffic Control components affected ...
   <!-- delete all those that don't apply -->
   -  Traffic Ops
   
   ## Current behavior:
   <!-- Describe how the current features are insufficient. -->
   We have no topologies validation when adding delivery service required 
capabilities
   
   ## New behavior:
   <!-- Describe how the feature would improve Traffic Control -->
   If a delivery service has a topology assigned, adding a delivery service 
required server capability must not invalidate the assigned topology. In other 
words, adding a delivery service required server capability must not alter the 
topology by excluding cache groups. All cache groups must be considered part of 
the topology.
   
   When a user adds a required server capability to a delivery service, the API 
must ensure that at least one cache in each of the topology's cache groups has 
that server capability to ensure the topology and all its cache groups are 
respected.
   
   ## Minimal reproduction of the problem with instructions:
   <!--
   If you can illustrate your feature request better with an example, please
   provide the *STEPS TO REPRODUCE* and include the applicable TC version.
   If not, feel free to delete this section.
   -->
   Add a required capability to a topology-based delivery service
   
   <!--
       Licensed to the Apache Software Foundation (ASF) under one
       or more contributor license agreements.  See the NOTICE file
       distributed with this work for additional information
       regarding copyright ownership.  The ASF licenses this file
       to you under the Apache License, Version 2.0 (the
       "License"); you may not use this file except in compliance
       with the License.  You may obtain a copy of the License at
   
       https://apache.org/licenses/LICENSE-2.0
   
       Unless required by applicable law or agreed to in writing,
       software distributed under the License is distributed on an
       "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
       KIND, either express or implied.  See the License for the
       specific language governing permissions and limitations
       under the License.
   -->
   


----------------------------------------------------------------
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:
[email protected]


Reply via email to