[GitHub] mitchell852 opened a new issue #1876: CDN snapshot diff always has a diff with tm_path

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

mitchell852 opened a new issue #1876: CDN snapshot diff always has a diff with tm_path
URL: https://github.com/apache/incubator-trafficcontrol/issues/1876
 
 
   When performing a snapshot diff in Traffic Portal, the tm_path value under the Stats tab is always different
   
   ![image](https://user-images.githubusercontent.com/251272/36172315-b9a52c9e-10c2-11e8-974e-3bf13b401734.png)
   
   This is because the "old" tm_path always includes the endpoint that was used to perform the current snapshot:
   
   /tools/write_crconfig/cdn1 (if snapshot was performed thru old TO UI)
   or
   /api/1.3/cdns/:id/snapshot (if snapshot was performed thru the API (or TP))
   
   and the "new" tm_path always includes the path used to fetch the "new" snapshot:
   
   /api/1.3/cdns/cdn1/snapshot/new
   
   hence, always a diff.
   
   my suggestion is that tm_path is removed altogether as it appears to not be used anywhere
   
   https://github.com/apache/incubator-trafficcontrol/blob/master/traffic_ops/app/lib/UI/Topology.pm#L64
   
   
   
   
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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