You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@trafficcontrol.apache.org by rawlinp <gi...@git.apache.org> on 2017/08/23 22:46:07 UTC

[GitHub] incubator-trafficcontrol pull request #865: [TC-287] Add support for per-Del...

GitHub user rawlinp opened a pull request:

    https://github.com/apache/incubator-trafficcontrol/pull/865

    [TC-287] Add support for per-DeliveryService routing names

    Add a routing_name column to the DeliveryService table and populate it
    to 'tr' and 'edge' for HTTP and DNS delivery services, respectively.
    
    To make upgrading seamless for CDNs using a custom http.routing.name
    (configured in the TR), provide a temporary upgrade parameter to
    populate the routing name for HTTP Delivery Services. Otherwise HTTP
    DSes will get routing_name = "tr".
    
    Replace all hardcoded/assumed routing names of 'edge' or 'tr' in Traffic
    Ops with the proper routing name configured for that delivery service.
    
    Update the API/UI to support custom routing names for delivery services.
    To prevent breaking the API, "routingName" is an optional field (the DB
    schema provides an arbitrary default).

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/rawlinp/incubator-trafficcontrol per-cdn-routing-names_2

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-trafficcontrol/pull/865.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #865
    
----
commit 1925845dd9c5cc53a87dcff130d1e5843142023f
Author: Rawlin Peters <ra...@comcast.com>
Date:   2017-07-31T17:37:50Z

    [TC-287] Add support for per-DeliveryService routing names
    
    Add a routing_name column to the DeliveryService table and populate it
    to 'tr' and 'edge' for HTTP and DNS delivery services, respectively.
    
    To make upgrading seamless for CDNs using a custom http.routing.name
    (configured in the TR), provide a temporary upgrade parameter to
    populate the routing name for HTTP Delivery Services. Otherwise HTTP
    DSes will get routing_name = "tr".
    
    Replace all hardcoded/assumed routing names of 'edge' or 'tr' in Traffic
    Ops with the proper routing name configured for that delivery service.
    
    Update the API/UI to support custom routing names for delivery services.
    To prevent breaking the API, "routingName" is an optional field (the DB
    schema provides an arbitrary default).

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafficcontrol issue #865: [TC-287] Add support for per-DeliverySe...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit commented on the issue:

    https://github.com/apache/incubator-trafficcontrol/pull/865
  
    
    Refer to this link for build results (access rights to CI server needed): 
    https://builds.apache.org/job/incubator-trafficcontrol-PR/138/



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafficcontrol issue #865: [TC-287] Add support for per-DeliverySe...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit commented on the issue:

    https://github.com/apache/incubator-trafficcontrol/pull/865
  
    
    Refer to this link for build results (access rights to CI server needed): 
    https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/99/



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafficcontrol issue #865: [TC-287] Add support for per-DeliverySe...

Posted by dneuman64 <gi...@git.apache.org>.

Github user dneuman64 commented on the issue:

    https://github.com/apache/incubator-trafficcontrol/pull/865
  
    Thanks for squashing your commits, however a PR this size should probably have more than 1 commit.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---