Build failed in Jenkins: incubator-trafficcontrol-traffic_ops-test #640

2018-05-21 Thread Apache Jenkins Server
See Changes: [mitchell852] Add TO Go staticdnsentries endpoint -- [...truncated 97.88 KB...] unit_golang_1 | --- PASS: TestFuncs (0.00s)

Build failed in Jenkins: incubator-trafficcontrol-traffic_ops-test #639

2018-05-21 Thread Apache Jenkins Server
See Changes: [mitchell852] Add TO Go profile parameters routes -- [...truncated 96.20 KB...] unit_golang_1 | --- PASS: TestFuncs (0.00s)

[GitHub] mitchell852 commented on issue #2291: update portal ds fields

2018-05-21 Thread GitBox
mitchell852 commented on issue #2291: update portal ds fields URL: https://github.com/apache/incubator-trafficcontrol/pull/2291#issuecomment-390840975 you probably want to just open a new PR, right? This is an automated

[GitHub] mitchell852 closed pull request #2256: Add TO Go profile parameters routes

2018-05-21 Thread GitBox
mitchell852 closed pull request #2256: Add TO Go profile parameters routes URL: https://github.com/apache/incubator-trafficcontrol/pull/2256 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] asfgit commented on issue #2088: TO Go: implement tenants CRUD

2018-05-21 Thread GitBox
asfgit commented on issue #2088: TO Go: implement tenants CRUD URL: https://github.com/apache/incubator-trafficcontrol/pull/2088#issuecomment-390807661 Refer to this link for build results (access rights to CI server needed):

Jenkins build is back to normal : incubator-trafficcontrol-PR #1632

2018-05-21 Thread Apache Jenkins Server
See

[GitHub] asfgit commented on issue #2314: Remove deliveryservice.org_server_fqdn column/compute it from Origin table

2018-05-21 Thread GitBox
asfgit commented on issue #2314: Remove deliveryservice.org_server_fqdn column/compute it from Origin table URL: https://github.com/apache/incubator-trafficcontrol/pull/2314#issuecomment-390801463 Refer to this link for build results (access rights to CI server needed):

Build failed in Jenkins: incubator-trafficcontrol-PR #1631

2018-05-21 Thread Apache Jenkins Server
See -- GitHub pull request #2314 of commit e7dda421a9115105bb7fd29e5858f11708a003fa, no merge conflicts. Setting status of e7dda421a9115105bb7fd29e5858f11708a003fa to

[GitHub] rawlinp commented on issue #2247: Add an Origin API

2018-05-21 Thread GitBox
rawlinp commented on issue #2247: Add an Origin API URL: https://github.com/apache/incubator-trafficcontrol/pull/2247#issuecomment-390789307 I believe this PR is no longer a WIP. I have tested it pretty well and am now looking for reviews. Note: I also have #2314 which depends on this PR.

[GitHub] rawlinp commented on issue #2314: Remove deliveryservice.org_server_fqdn column/compute it from Origin table

2018-05-21 Thread GitBox
rawlinp commented on issue #2314: Remove deliveryservice.org_server_fqdn column/compute it from Origin table URL: https://github.com/apache/incubator-trafficcontrol/pull/2314#issuecomment-390786527 NOTE: this depends on #2247 which would need to be merged before this.

[GitHub] rawlinp opened a new pull request #2314: Remove deliveryservice.org_server_fqdn column/compute it from Origin table

2018-05-21 Thread GitBox
rawlinp opened a new pull request #2314: Remove deliveryservice.org_server_fqdn column/compute it from Origin table URL: https://github.com/apache/incubator-trafficcontrol/pull/2314 With the Origin table now, the deliveryservice.org_server_fqdn column can be removed. First, populate the

[GitHub] rob05c commented on a change in pull request #2282: Deliveryservice_Server API conversion to Go - formerly #2269

2018-05-21 Thread GitBox
rob05c commented on a change in pull request #2282: Deliveryservice_Server API conversion to Go - formerly #2269 URL: https://github.com/apache/incubator-trafficcontrol/pull/2282#discussion_r189707585 ## File path: traffic_ops/traffic_ops_golang/deliveryservice/servers/servers.go

[GitHub] rob05c commented on a change in pull request #2282: Deliveryservice_Server API conversion to Go - formerly #2269

2018-05-21 Thread GitBox
rob05c commented on a change in pull request #2282: Deliveryservice_Server API conversion to Go - formerly #2269 URL: https://github.com/apache/incubator-trafficcontrol/pull/2282#discussion_r189707285 ## File path: lib/go-tc/deliveryservice_servers.go ## @@ -78,3 +106,63

[GitHub] rob05c commented on a change in pull request #2282: Deliveryservice_Server API conversion to Go - formerly #2269

2018-05-21 Thread GitBox
rob05c commented on a change in pull request #2282: Deliveryservice_Server API conversion to Go - formerly #2269 URL: https://github.com/apache/incubator-trafficcontrol/pull/2282#discussion_r189707065 ## File path: lib/go-tc/deliveryservice_servers.go ## @@ -19,21 +21,47

[GitHub] rob05c commented on a change in pull request #2282: Deliveryservice_Server API conversion to Go - formerly #2269

2018-05-21 Thread GitBox
rob05c commented on a change in pull request #2282: Deliveryservice_Server API conversion to Go - formerly #2269 URL: https://github.com/apache/incubator-trafficcontrol/pull/2282#discussion_r189706518 ## File path: traffic_ops/traffic_ops_golang/deliveryservice/servers/servers.go

[GitHub] rob05c commented on a change in pull request #2282: Deliveryservice_Server API conversion to Go - formerly #2269

2018-05-21 Thread GitBox
rob05c commented on a change in pull request #2282: Deliveryservice_Server API conversion to Go - formerly #2269 URL: https://github.com/apache/incubator-trafficcontrol/pull/2282#discussion_r189707974 ## File path: traffic_ops/traffic_ops_golang/deliveryservice/servers/servers.go

Jenkins build is back to normal : incubator-trafficcontrol-PR-rat #911

2018-05-21 Thread Apache Jenkins Server
See

Build failed in Jenkins: incubator-trafficcontrol-traffic_ops-test #638

2018-05-21 Thread Apache Jenkins Server
See Changes: [mitchell852] Fix TO old Perl UI to get diff CRConfig from Go -- [...truncated 96.29 KB...] unit_golang_1 | --- PASS: TestFuncs

Build failed in Jenkins: incubator-trafficcontrol-PR-rat #910

2018-05-21 Thread Apache Jenkins Server
See -- Started by upstream project "incubator-trafficcontrol-PR" build number 1629 originally caused by: GitHub pull request #2291 of commit

[GitHub] mitchell852 commented on issue #2312: New Endpoint /api//tenants/children/:id

2018-05-21 Thread GitBox
mitchell852 commented on issue #2312: New Endpoint /api//tenants/children/:id URL: https://github.com/apache/incubator-trafficcontrol/issues/2312#issuecomment-390756744 I think the shortest path is this: GET api/1.3/deliveryservices?tenant=60=true (or something like that) to fetch

[GitHub] jhg03a commented on issue #2312: New Endpoint /api//tenants/children/:id

2018-05-21 Thread GitBox
jhg03a commented on issue #2312: New Endpoint /api//tenants/children/:id URL: https://github.com/apache/incubator-trafficcontrol/issues/2312#issuecomment-390754855 Another option would be to add a new parameter ?tenentchildren=true or ?astenant= to applicable endpoints ideally, but that

Build failed in Jenkins: incubator-trafficcontrol-traffic_ops-test #637

2018-05-21 Thread Apache Jenkins Server
See Changes: [dewrich] reset role_id_seq and do not specify id on inserts this removes test -- [...truncated 95.81 KB...] unit_golang_1 | === RUN

[GitHub] jhg03a opened a new issue #2312: New Endpoint /api//tenants/children/:id

2018-05-21 Thread GitBox
jhg03a opened a new issue #2312: New Endpoint /api//tenants/children/:id URL: https://github.com/apache/incubator-trafficcontrol/issues/2312 Can you please add a new TO API endpoint /api//tenants/children/:id which returns a list of all tenant ids that are children and requested id. This

[GitHub] dewrich closed pull request #2311: reset role_id_seq and do not specify id on inserts

2018-05-21 Thread GitBox
dewrich closed pull request #2311: reset role_id_seq and do not specify id on inserts URL: https://github.com/apache/incubator-trafficcontrol/pull/2311 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] asfgit commented on issue #2311: reset role_id_seq and do not specify id on inserts

2018-05-21 Thread GitBox
asfgit commented on issue #2311: reset role_id_seq and do not specify id on inserts URL: https://github.com/apache/incubator-trafficcontrol/pull/2311#issuecomment-390727393 Refer to this link for build results (access rights to CI server needed):

[GitHub] DylanVolz opened a new pull request #2311: reset role_id_seq and do not specify id on inserts

2018-05-21 Thread GitBox
DylanVolz opened a new pull request #2311: reset role_id_seq and do not specify id on inserts URL: https://github.com/apache/incubator-trafficcontrol/pull/2311 this removes test failures from when role_id_seq is an id already inserted.

Build failed in Jenkins: incubator-trafficcontrol-traffic_ops-test #636

2018-05-21 Thread Apache Jenkins Server
See Changes: [rob05c] Added a docker-compose script to create a dev environment for [rob05c] Added apache license to new Docker Compose files --

[GitHub] rob05c closed pull request #2269: Adds Docker Compose configuration for creating a Traffic_Ops Dev environment

2018-05-21 Thread GitBox
rob05c closed pull request #2269: Adds Docker Compose configuration for creating a Traffic_Ops Dev environment URL: https://github.com/apache/incubator-trafficcontrol/pull/2269 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below

[GitHub] Vijay-1 commented on a change in pull request #2029: [Issue 1907] TO API for backup edge cachegroup

2018-05-21 Thread GitBox
Vijay-1 commented on a change in pull request #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#discussion_r189627482 ## File path: traffic_ops/app/db/migrations/2018032100_cache_group_fallback.sql

Jenkins build is back to normal : incubator-trafficcontrol-PR-rat #908

2018-05-21 Thread Apache Jenkins Server
See

[GitHub] rob05c opened a new pull request #2310: Add TO Go cdns/metric_types route 501

2018-05-21 Thread GitBox
rob05c opened a new pull request #2310: Add TO Go cdns/metric_types route 501 URL: https://github.com/apache/incubator-trafficcontrol/pull/2310 The cdns/metric_types route is documented in the API, and has a Perl route, but the Perl function it points to doesn't exist. As such, this

[GitHub] rob05c commented on issue #2309: Implement /api/1.1/cdns/metric_types

2018-05-21 Thread GitBox
rob05c commented on issue #2309: Implement /api/1.1/cdns/metric_types URL: https://github.com/apache/incubator-trafficcontrol/issues/2309#issuecomment-390683350 Readers, be aware: this route is not currently a priority, and not scheduled to be implemented any time soon. If this route is

[GitHub] tyagian opened a new issue #2308: missing snapshot in change logs on TO, TP notifications

2018-05-21 Thread GitBox
tyagian opened a new issue #2308: missing snapshot in change logs on TO, TP notifications URL: https://github.com/apache/incubator-trafficcontrol/issues/2308 After few snapshots, none of them were visible on TO change logs or TP notifications but traffic dropped on servers so snapshot