You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficcontrol.apache.org by GitBox <gi...@apache.org> on 2021/01/22 18:51:36 UTC

[GitHub] [trafficcontrol] mitchell852 opened a new issue #5460: Provide operations+ users with the ability to add a CDN-level message/alert via TP/TO API

mitchell852 opened a new issue #5460:
URL: https://github.com/apache/trafficcontrol/issues/5460


   ## I'm submitting a ...
   -  new feature / enhancement request
   
   ## Traffic Control components affected ...
   -  Traffic Ops
   -  Traffic Portal
   
   ## Current behavior:
   In Traffic Control, 2 actions are global to a CDN:
   
   - Snapshotting a CDN
   - Queuing servers updates for a CDN
   
   Because of this, incomplete changes (i.e. delivery service changes) can accidentally be pushed to the TM/TR via a "cdn snapshot" or to caches via a "cdn queue updates".
   
   ## New behavior:
   Provide a mechanism to create a cdn-level alert to prevent users from unknowingly committing incomplete changes via a snapshot / queue updates. This alert should show who created it and why and should be displayed in TP.
   
   ## Anything else:
   The alert should not "lock" the cdn in any way but rather provide a way to create an alert for each cdn that gets displayed in TP.
   
   <!--
       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:
users@infra.apache.org

[GitHub] [trafficcontrol] mitchell852 commented on issue #5460: Provide operations+ users with the ability to add a CDN-level message/alert via TP/TO API

Posted by GitBox <gi...@apache.org>.
mitchell852 commented on issue #5460:
URL: https://github.com/apache/trafficcontrol/issues/5460#issuecomment-769173916


   @jhg03a - i changed the description of this issue. it's simply a way to display an alert/message - created by humans, used by humans. nothing more. it's not really intended to be used with automation or serve as a mutex.


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

[GitHub] [trafficcontrol] jhg03a commented on issue #5460: Provide operations+ users with the ability to add a CDN-level message/alert via TP/TO API

Posted by GitBox <gi...@apache.org>.
jhg03a commented on issue #5460:
URL: https://github.com/apache/trafficcontrol/issues/5460#issuecomment-768664981


   Given that the input here is arbitrary and driven by humans, I don't believe this is satisfactory for automation outside the scope of ATC to accommodate for safely handling the global mutexes of queue/snap.
   
   If the goal is to add more programmatic access around those two mutexes, it would be better served by a lock and unlock method in the API taking an arbitrary or enumerated string as to which lock you were attempting to acquire either as a human or non-human.


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

[GitHub] [trafficcontrol] mitchell852 commented on issue #5460: Provide operations+ users with the ability to add a CDN-level message/alert via TP/TO API

Posted by GitBox <gi...@apache.org>.
mitchell852 commented on issue #5460:
URL: https://github.com/apache/trafficcontrol/issues/5460#issuecomment-829428967


   this was implemented with #5179 


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

[GitHub] [trafficcontrol] mitchell852 closed issue #5460: Provide operations+ users with the ability to add a CDN-level message/alert via TP/TO API

Posted by GitBox <gi...@apache.org>.
mitchell852 closed issue #5460:
URL: https://github.com/apache/trafficcontrol/issues/5460


   


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