[GitHub] elsloo closed issue #2380: Traffic Stats build issues

2018-06-08 Thread GitBox
elsloo closed issue #2380: Traffic Stats build issues URL: https://github.com/apache/incubator-trafficcontrol/issues/2380 This is an automated message from the Apache Git Service. To respond to the message, please log on

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

2018-06-07 Thread GitBox
rawlinp 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_r193758109 ## File path: traffic_ops/traffic_ops_golang/crconfig/edgelocations.go ## @@ -32,7

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

2018-06-07 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_r193745582 ## File path:

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

2018-06-07 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_r193746661 ## File path: traffic_ops/traffic_ops_golang/crconfig/edgelocations.go ## @@ -32,7

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

2018-06-07 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_r193746661 ## File path: traffic_ops/traffic_ops_golang/crconfig/edgelocations.go ## @@ -32,7

[GitHub] dewrich closed pull request #2378: Master 2370

2018-06-07 Thread GitBox
dewrich closed pull request #2378: Master 2370 URL: https://github.com/apache/incubator-trafficcontrol/pull/2378 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 is a foreign pull request

[GitHub] asfgit commented on issue #2379: Update installation.rst

2018-06-07 Thread GitBox
asfgit commented on issue #2379: Update installation.rst URL: https://github.com/apache/incubator-trafficcontrol/pull/2379#issuecomment-395474370 Can one of the admins verify this patch? This is an automated message from the

[GitHub] asfgit commented on issue #2379: Update installation.rst

2018-06-07 Thread GitBox
asfgit commented on issue #2379: Update installation.rst URL: https://github.com/apache/incubator-trafficcontrol/pull/2379#issuecomment-395473034 Can one of the admins verify this patch? This is an automated message from the

[GitHub] villajo opened a new pull request #2379: Update installation.rst

2018-06-07 Thread GitBox
villajo opened a new pull request #2379: Update installation.rst URL: https://github.com/apache/incubator-trafficcontrol/pull/2379 Update "blue cdn" to "blue_cdn" on lines 170 and 171 to match what's requested in the input field. (String with no spaces)

[GitHub] dewrich closed pull request #2379: Update installation.rst

2018-06-07 Thread GitBox
dewrich closed pull request #2379: Update installation.rst URL: https://github.com/apache/incubator-trafficcontrol/pull/2379 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 is a foreign

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

2018-06-07 Thread GitBox
rawlinp 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_r193497898 ## File path: traffic_ops/traffic_ops_golang/crconfig/edgelocations.go ## @@ -32,7

[GitHub] rob05c commented on issue #2383: Traffic Monitor crashes when CRConfig is absent on TrafficOps

2018-06-08 Thread GitBox
rob05c commented on issue #2383: Traffic Monitor crashes when CRConfig is absent on TrafficOps URL: https://github.com/apache/incubator-trafficcontrol/issues/2383#issuecomment-395874807 Duplicate of https://github.com/apache/incubator-trafficcontrol/issues/2383 Should be fixed by

[GitHub] rob05c closed issue #2383: Traffic Monitor crashes when CRConfig is absent on TrafficOps

2018-06-08 Thread GitBox
rob05c closed issue #2383: Traffic Monitor crashes when CRConfig is absent on TrafficOps URL: https://github.com/apache/incubator-trafficcontrol/issues/2383 This is an automated message from the Apache Git Service. To

[GitHub] rob05c commented on issue #2383: Traffic Monitor crashes when CRConfig is absent on TrafficOps

2018-06-08 Thread GitBox
rob05c commented on issue #2383: Traffic Monitor crashes when CRConfig is absent on TrafficOps URL: https://github.com/apache/incubator-trafficcontrol/issues/2383#issuecomment-395890573 Ok, #2377 has been merged. Can you try building from master, and if it still happens, reopen this with

[GitHub] elsloo opened a new issue #2384: ORT incorrectly handling raw remap

2018-06-08 Thread GitBox
elsloo opened a new issue #2384: ORT incorrectly handling raw remap URL: https://github.com/apache/incubator-trafficcontrol/issues/2384 When a raw remap line is added to a regular delivery service (i.e.: not an `ANY_MAP` DS), the raw remap text is appended to the last remap rule. This is a

[GitHub] elsloo closed pull request #2377: Fix TM nil panic

2018-06-08 Thread GitBox
elsloo closed pull request #2377: Fix TM nil panic URL: https://github.com/apache/incubator-trafficcontrol/pull/2377 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 is a foreign pull

[GitHub] dneuman64 opened a new issue #2385: Router hostname should be searchable in the servers page

2018-06-08 Thread GitBox
dneuman64 opened a new issue #2385: Router hostname should be searchable in the servers page URL: https://github.com/apache/incubator-trafficcontrol/issues/2385 Often times there are maintenances that affect a certain router, it would be nice if you could search the server page by router

[GitHub] dangogh opened a new pull request #2381: [Issue 2380] add soft link to lib for traffic_stats build

2018-06-07 Thread GitBox
dangogh opened a new pull request #2381: [Issue 2380] add soft link to lib for traffic_stats build URL: https://github.com/apache/incubator-trafficcontrol/pull/2381 Fixes #2380 this solves the immediate issue, but should have something in place to ensure all dependencies are

[GitHub] asfgit commented on issue #2381: [Issue 2380] add soft link to lib for traffic_stats build

2018-06-07 Thread GitBox
asfgit commented on issue #2381: [Issue 2380] add soft link to lib for traffic_stats build URL: https://github.com/apache/incubator-trafficcontrol/pull/2381#issuecomment-395568590 Refer to this link for build results (access rights to CI server needed):

[GitHub] dangogh opened a new issue #2380: Traffic Stats build issues

2018-06-07 Thread GitBox
dangogh opened a new issue #2380: Traffic Stats build issues URL: https://github.com/apache/incubator-trafficcontrol/issues/2380 `traffic_stats/vendor` directory is missing the `lib/go-tc` directory (included by `traffic_ops/client`, which **is** in the vendor directory via a soft link).

[GitHub] asfgit commented on issue #2420: Fix TO Go logging unauthenticated endpoints

2018-06-18 Thread GitBox
asfgit commented on issue #2420: Fix TO Go logging unauthenticated endpoints URL: https://github.com/apache/trafficcontrol/pull/2420#issuecomment-398178769 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2421: Add TO Go deliveryservices/sslkeys/delete

2018-06-18 Thread GitBox
asfgit commented on issue #2421: Add TO Go deliveryservices/sslkeys/delete URL: https://github.com/apache/trafficcontrol/pull/2421#issuecomment-398180182 Refer to this link for build results (access rights to CI server needed):

[GitHub] serDrem opened a new pull request #2422: new certificate chain analyzer test

2018-06-18 Thread GitBox
serDrem opened a new pull request #2422: new certificate chain analyzer test URL: https://github.com/apache/trafficcontrol/pull/2422 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] rob05c opened a new pull request #2423: Fix TO API Test for DS enum change

2018-06-18 Thread GitBox
rob05c opened a new pull request #2423: Fix TO API Test for DS enum change URL: https://github.com/apache/trafficcontrol/pull/2423 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] dewrich closed pull request #2423: Fix TO API Test for DS enum change

2018-06-18 Thread GitBox
dewrich closed pull request #2423: Fix TO API Test for DS enum change URL: https://github.com/apache/trafficcontrol/pull/2423 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 is a foreign

[GitHub] asfgit commented on issue #2419: cdn-in-a-box includes traffic_portal

2018-06-18 Thread GitBox
asfgit commented on issue #2419: cdn-in-a-box includes traffic_portal URL: https://github.com/apache/trafficcontrol/pull/2419#issuecomment-398166252 Refer to this link for build results (access rights to CI server needed):

[GitHub] rob05c opened a new pull request #2421: Add TO Go deliveryservices/sslkeys/delete

2018-06-18 Thread GitBox
rob05c opened a new pull request #2421: Add TO Go deliveryservices/sslkeys/delete URL: https://github.com/apache/trafficcontrol/pull/2421 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] rob05c edited a comment on issue #2232: Rewrite Traffic Ops CRConfig Table Endpoints in Go

2018-06-18 Thread GitBox
rob05c edited a comment on issue #2232: Rewrite Traffic Ops CRConfig Table Endpoints in Go URL: https://github.com/apache/trafficcontrol/issues/2232#issuecomment-398070296 # Table Usage ``` get /api/1.1/cdns/:name/configs/routing - DB: Server, Profile, Cdn, Parameter,

[GitHub] rob05c opened a new pull request #2420: Fix TO Go logging unauthenticated endpoints

2018-06-18 Thread GitBox
rob05c opened a new pull request #2420: Fix TO Go logging unauthenticated endpoints URL: https://github.com/apache/trafficcontrol/pull/2420 Also adds info logging for requests, and adds a quasi-unique request ID to the APIInfo object for logging and other uses.

[GitHub] rawlinp commented on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. [WIP]

2018-06-18 Thread GitBox
rawlinp commented on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. [WIP] URL: https://github.com/apache/trafficcontrol/pull/2407#issuecomment-398192126 Ok, that driver makes more sense to me, but it seems like maybe we need a new delivery service type

[GitHub] asfgit commented on issue #2422: new certificate chain analyzer test

2018-06-18 Thread GitBox
asfgit commented on issue #2422: new certificate chain analyzer test URL: https://github.com/apache/trafficcontrol/pull/2422#issuecomment-398196256 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 closed pull request #2386: Fix TO Go DS POST to 400 on duplicate, not 5xx

2018-06-18 Thread GitBox
mitchell852 closed pull request #2386: Fix TO Go DS POST to 400 on duplicate, not 5xx URL: https://github.com/apache/trafficcontrol/pull/2386 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

[GitHub] asfgit commented on issue #2364: Fix Traffic Ops Go to use transactions for deliveryservices, sslkeys, urisigning

2018-06-18 Thread GitBox
asfgit commented on issue #2364: Fix Traffic Ops Go to use transactions for deliveryservices, sslkeys, urisigning URL: https://github.com/apache/trafficcontrol/pull/2364#issuecomment-398105656 Refer to this link for build results (access rights to CI server needed):

[GitHub] ashish-timilsina commented on issue #1934: Additional textbox in DS request form

2018-06-18 Thread GitBox
ashish-timilsina commented on issue #1934: Additional textbox in DS request form URL: https://github.com/apache/trafficcontrol/issues/1934#issuecomment-398115421 @mitchell852 yes that should work however, I can't seem to find the comment field. Is it "Long Description 2"?

[GitHub] rob05c commented on issue #2232: Rewrite Traffic Ops CRConfig Table Endpoints in Go

2018-06-18 Thread GitBox
rob05c commented on issue #2232: Rewrite Traffic Ops CRConfig Table Endpoints in Go URL: https://github.com/apache/trafficcontrol/issues/2232#issuecomment-398070296 # Table Usage ``` get /api/1.1/cdns/:name/configs/routing - DB: Server, Profile, Cdn, Parameter,

[GitHub] dangogh closed pull request #2417: adds db dump link to TP

2018-06-18 Thread GitBox
dangogh closed pull request #2417: adds db dump link to TP URL: https://github.com/apache/trafficcontrol/pull/2417 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 is a foreign pull request

[GitHub] mitchell852 closed pull request #2392: Fix Traffic Ops Delivery Service endpoints for anymap

2018-06-18 Thread GitBox
mitchell852 closed pull request #2392: Fix Traffic Ops Delivery Service endpoints for anymap URL: https://github.com/apache/trafficcontrol/pull/2392 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:

[GitHub] asfgit commented on issue #2419: cdn-in-a-box includes traffic_portal

2018-06-18 Thread GitBox
asfgit commented on issue #2419: cdn-in-a-box includes traffic_portal URL: https://github.com/apache/trafficcontrol/pull/2419#issuecomment-398125776 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2419: cdn-in-a-box includes traffic_portal

2018-06-18 Thread GitBox
asfgit commented on issue #2419: cdn-in-a-box includes traffic_portal URL: https://github.com/apache/trafficcontrol/pull/2419#issuecomment-398127533 Refer to this link for build results (access rights to CI server needed):

[GitHub] Vijay-1 commented on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. [WIP]

2018-06-18 Thread GitBox
Vijay-1 commented on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. [WIP] URL: https://github.com/apache/trafficcontrol/pull/2407#issuecomment-398082369 @rawlinp Eric has pointed out the drive behind this PR. Do you see a problem in the way it is

[GitHub] mitchell852 closed pull request #2401: Implement Origin UI in Traffic Portal

2018-06-18 Thread GitBox
mitchell852 closed pull request #2401: Implement Origin UI in Traffic Portal URL: https://github.com/apache/trafficcontrol/pull/2401 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 is a

[GitHub] asfgit commented on issue #2382: To go dnsseckeys generate

2018-06-18 Thread GitBox
asfgit commented on issue #2382: To go dnsseckeys generate URL: https://github.com/apache/trafficcontrol/pull/2382#issuecomment-398091725 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1827/ Test

[GitHub] rawlinp commented on a change in pull request #2401: Implement Origin UI in Traffic Portal

2018-06-18 Thread GitBox
rawlinp commented on a change in pull request #2401: Implement Origin UI in Traffic Portal URL: https://github.com/apache/trafficcontrol/pull/2401#discussion_r196107562 ## File path: traffic_portal/app/src/common/modules/form/origin/form.origin.tpl.html ## @@ -0,0 +1,147

[GitHub] rob05c commented on issue #2232: Rewrite Traffic Ops CRConfig Table Endpoints in Go

2018-06-18 Thread GitBox
rob05c commented on issue #2232: Rewrite Traffic Ops CRConfig Table Endpoints in Go URL: https://github.com/apache/trafficcontrol/issues/2232#issuecomment-398070296 # Table Usage ``` get /api/1.1/cdns/:name/configs/routing - DB: Server, Profile, Cdn, Parameter,

[GitHub] dewrich closed pull request #2419: cdn-in-a-box includes traffic_portal

2018-06-18 Thread GitBox
dewrich closed pull request #2419: cdn-in-a-box includes traffic_portal URL: https://github.com/apache/trafficcontrol/pull/2419 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 is a foreign

[GitHub] limited commented on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. [WIP]

2018-06-18 Thread GitBox
limited commented on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. [WIP] URL: https://github.com/apache/trafficcontrol/pull/2407#issuecomment-398221657 It only skips the mid tier if all mids are down. If mids are up, they are used as normally

[GitHub] rawlinp commented on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. [WIP]

2018-06-18 Thread GitBox
rawlinp commented on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. [WIP] URL: https://github.com/apache/trafficcontrol/pull/2407#issuecomment-398225128 So it should be dynamic? This PR would make it static per DS. What do you mean by the mids being all

[GitHub] rob05c opened a new pull request #2405: Add Traffic Ops Golang user/current

2018-06-12 Thread GitBox
rob05c opened a new pull request #2405: Add Traffic Ops Golang user/current URL: https://github.com/apache/incubator-trafficcontrol/pull/2405 This is an automated message from the Apache Git Service. To respond to the

[GitHub] asfgit commented on issue #2404: TO Go DS API: initialDispersion not required for DNS DSes

2018-06-12 Thread GitBox
asfgit commented on issue #2404: TO Go DS API: initialDispersion not required for DNS DSes URL: https://github.com/apache/incubator-trafficcontrol/pull/2404#issuecomment-396774378 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2405: Add Traffic Ops Golang user/current

2018-06-12 Thread GitBox
asfgit commented on issue #2405: Add Traffic Ops Golang user/current URL: https://github.com/apache/incubator-trafficcontrol/pull/2405#issuecomment-396772488 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2402: To go riak ping

2018-06-12 Thread GitBox
asfgit commented on issue #2402: To go riak ping URL: https://github.com/apache/incubator-trafficcontrol/pull/2402#issuecomment-396763696 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1797/

[GitHub] alficles commented on a change in pull request #2365: Add TO Go api helpers

2018-06-12 Thread GitBox
alficles commented on a change in pull request #2365: Add TO Go api helpers URL: https://github.com/apache/incubator-trafficcontrol/pull/2365#discussion_r194858032 ## File path: traffic_ops/traffic_ops_golang/api/api.go ## @@ -0,0 +1,304 @@ +package api + +/* + * Licensed

[GitHub] rob05c opened a new pull request #2400: To go cdns sslkeys

2018-06-12 Thread GitBox
rob05c opened a new pull request #2400: To go cdns sslkeys URL: https://github.com/apache/incubator-trafficcontrol/pull/2400 Branched from #2365 - recommend merging that first. This is an automated message from the Apache Git

[GitHub] limited opened a new pull request #2399: Add Dockerfile, docker-compose and build scripts for ATS and astats [WIP]

2018-06-12 Thread GitBox
limited opened a new pull request #2399: Add Dockerfile, docker-compose and build scripts for ATS and astats [WIP] URL: https://github.com/apache/incubator-trafficcontrol/pull/2399 This is an automated message from the

[GitHub] alficles commented on a change in pull request #2365: Add TO Go api helpers

2018-06-12 Thread GitBox
alficles commented on a change in pull request #2365: Add TO Go api helpers URL: https://github.com/apache/incubator-trafficcontrol/pull/2365#discussion_r194857103 ## File path: traffic_ops/traffic_ops_golang/api/api.go ## @@ -0,0 +1,304 @@ +package api + +/* + * Licensed

[GitHub] rawlinp opened a new pull request #2401: Implement Origin UI in Traffic Portal

2018-06-12 Thread GitBox
rawlinp opened a new pull request #2401: Implement Origin UI in Traffic Portal URL: https://github.com/apache/incubator-trafficcontrol/pull/2401 Provide the ability to list, create, edit, and delete Origins using Traffic Portal.

[GitHub] asfgit commented on issue #2399: Add Dockerfile, docker-compose and build scripts for ATS and astats [WIP]

2018-06-12 Thread GitBox
asfgit commented on issue #2399: Add Dockerfile, docker-compose and build scripts for ATS and astats [WIP] URL: https://github.com/apache/incubator-trafficcontrol/pull/2399#issuecomment-396699870 Can one of the admins verify this patch?

[GitHub] dangogh closed pull request #2403: Fix merge conflict for changed func signature

2018-06-12 Thread GitBox
dangogh closed pull request #2403: Fix merge conflict for changed func signature URL: https://github.com/apache/incubator-trafficcontrol/pull/2403 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:

[GitHub] asfgit commented on issue #2399: Add Dockerfile, docker-compose and build scripts for ATS and astats [WIP]

2018-06-12 Thread GitBox
asfgit commented on issue #2399: Add Dockerfile, docker-compose and build scripts for ATS and astats [WIP] URL: https://github.com/apache/incubator-trafficcontrol/pull/2399#issuecomment-396705290 Refer to this link for build results (access rights to CI server needed):

[GitHub] rob05c opened a new pull request #2402: To go riak ping

2018-06-12 Thread GitBox
rob05c opened a new pull request #2402: To go riak ping URL: https://github.com/apache/incubator-trafficcontrol/pull/2402 Branched from #2365 - recommend merging that first. This is an automated message from the Apache Git

[GitHub] asfgit commented on issue #2400: To go cdns sslkeys

2018-06-12 Thread GitBox
asfgit commented on issue #2400: To go cdns sslkeys URL: https://github.com/apache/incubator-trafficcontrol/pull/2400#issuecomment-396708178 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1795/

[GitHub] alficles commented on a change in pull request #2365: Add TO Go api helpers

2018-06-12 Thread GitBox
alficles commented on a change in pull request #2365: Add TO Go api helpers URL: https://github.com/apache/incubator-trafficcontrol/pull/2365#discussion_r194853155 ## File path: traffic_ops/traffic_ops_golang/api/api.go ## @@ -0,0 +1,304 @@ +package api + +/* + * Licensed

[GitHub] asfgit commented on issue #2395: To go ds sslkeys generate

2018-06-13 Thread GitBox
asfgit commented on issue #2395: To go ds sslkeys generate URL: https://github.com/apache/incubator-trafficcontrol/pull/2395#issuecomment-396983071 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2338: Add TO Go users

2018-06-13 Thread GitBox
asfgit commented on issue #2338: Add TO Go users URL: https://github.com/apache/incubator-trafficcontrol/pull/2338#issuecomment-396972562 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1805/

[GitHub] asfgit commented on issue #2405: Add Traffic Ops Golang user/current

2018-06-13 Thread GitBox
asfgit commented on issue #2405: Add Traffic Ops Golang user/current URL: https://github.com/apache/incubator-trafficcontrol/pull/2405#issuecomment-396975459 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2386: Fix TO Go DS POST to 400 on duplicate, not 5xx

2018-06-13 Thread GitBox
asfgit commented on issue #2386: Fix TO Go DS POST to 400 on duplicate, not 5xx URL: https://github.com/apache/incubator-trafficcontrol/pull/2386#issuecomment-396986742 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2369: To go deletednsseckeys

2018-06-13 Thread GitBox
asfgit commented on issue #2369: To go deletednsseckeys URL: https://github.com/apache/incubator-trafficcontrol/pull/2369#issuecomment-396960296 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1801/

[GitHub] asfgit commented on issue #2392: Fix Traffic Ops Delivery Service endpoints for anymap

2018-06-13 Thread GitBox
asfgit commented on issue #2392: Fix Traffic Ops Delivery Service endpoints for anymap URL: https://github.com/apache/incubator-trafficcontrol/pull/2392#issuecomment-396984910 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2402: To go riak ping

2018-06-13 Thread GitBox
asfgit commented on issue #2402: To go riak ping URL: https://github.com/apache/incubator-trafficcontrol/pull/2402#issuecomment-396979699 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1807/

[GitHub] asfgit commented on issue #2400: To go cdns sslkeys

2018-06-13 Thread GitBox
asfgit commented on issue #2400: To go cdns sslkeys URL: https://github.com/apache/incubator-trafficcontrol/pull/2400#issuecomment-396981356 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1808/

[GitHub] dewrich closed pull request #2404: TO Go DS API: initialDispersion not required for DNS DSes

2018-06-13 Thread GitBox
dewrich closed pull request #2404: TO Go DS API: initialDispersion not required for DNS DSes URL: https://github.com/apache/incubator-trafficcontrol/pull/2404 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 #2382: To go dnsseckeys generate

2018-06-13 Thread GitBox
asfgit commented on issue #2382: To go dnsseckeys generate URL: https://github.com/apache/incubator-trafficcontrol/pull/2382#issuecomment-396958436 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2356: Add TO Go Logs

2018-06-13 Thread GitBox
asfgit commented on issue #2356: Add TO Go Logs URL: https://github.com/apache/incubator-trafficcontrol/pull/2356#issuecomment-396961681 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1802/ Test

[GitHub] asfgit commented on issue #2353: Add TO Go regions/name

2018-06-13 Thread GitBox
asfgit commented on issue #2353: Add TO Go regions/name URL: https://github.com/apache/incubator-trafficcontrol/pull/2353#issuecomment-396968862 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1803/

[GitHub] asfgit commented on issue #2339: Add TO Go users/id/deliveryservices

2018-06-13 Thread GitBox
asfgit commented on issue #2339: Add TO Go users/id/deliveryservices URL: https://github.com/apache/incubator-trafficcontrol/pull/2339#issuecomment-396970817 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries

2018-06-13 Thread GitBox
asfgit commented on issue #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries URL: https://github.com/apache/trafficcontrol/pull/2406#issuecomment-397014647 Can one of the admins verify this patch?

[GitHub] Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup

2018-06-13 Thread GitBox
Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/trafficcontrol/pull/2029#issuecomment-397005255 Yes. The changes requested for edgeLocations.go for CRConfig were addressed.

[GitHub] mitchell852 commented on issue #2397: TP - empty value field for parameter not accepted

2018-06-13 Thread GitBox
mitchell852 commented on issue #2397: TP - empty value field for parameter not accepted URL: https://github.com/apache/trafficcontrol/issues/2397#issuecomment-397016213 To achieve this in TP: 1. remove the API validation on "Value:

[GitHub] asfgit commented on issue #2401: Implement Origin UI in Traffic Portal

2018-06-13 Thread GitBox
asfgit commented on issue #2401: Implement Origin UI in Traffic Portal URL: https://github.com/apache/trafficcontrol/pull/2401#issuecomment-397051464 Refer to this link for build results (access rights to CI server needed):

[GitHub] rivasj opened a new pull request #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries

2018-06-13 Thread GitBox
rivasj opened a new pull request #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries URL: https://github.com/apache/trafficcontrol/pull/2406 Re-added anonymous_blocking_enabled field to deliveryservice queries after deliveryservice endpoints implemented in go

[GitHub] rawlinp commented on a change in pull request #2314: Remove deliveryservice.org_server_fqdn column/compute it from Origin table

2018-06-13 Thread GitBox
rawlinp commented on a change in pull request #2314: Remove deliveryservice.org_server_fqdn column/compute it from Origin table URL: https://github.com/apache/trafficcontrol/pull/2314#discussion_r195185045 ## File path:

[GitHub] rob05c commented on a change in pull request #2314: Remove deliveryservice.org_server_fqdn column/compute it from Origin table

2018-06-13 Thread GitBox
rob05c commented on a change in pull request #2314: Remove deliveryservice.org_server_fqdn column/compute it from Origin table URL: https://github.com/apache/trafficcontrol/pull/2314#discussion_r195180097 ## File path:

[GitHub] rob05c commented on a change in pull request #2365: Add TO Go api helpers

2018-06-12 Thread GitBox
rob05c commented on a change in pull request #2365: Add TO Go api helpers URL: https://github.com/apache/incubator-trafficcontrol/pull/2365#discussion_r194768563 ## File path: traffic_ops/traffic_ops_golang/api/api.go ## @@ -0,0 +1,304 @@ +package api + +/* + * Licensed

[GitHub] dewrich closed issue #2387: GET /api/1.3/cachegroups doesn't support filtering by type

2018-06-12 Thread GitBox
dewrich closed issue #2387: GET /api/1.3/cachegroups doesn't support filtering by type URL: https://github.com/apache/incubator-trafficcontrol/issues/2387 This is an automated message from the Apache Git Service. To respond

[GitHub] asfgit commented on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops.

2018-06-13 Thread GitBox
asfgit commented on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. URL: https://github.com/apache/trafficcontrol/pull/2407#issuecomment-397053454 Can one of the admins verify this patch?

[GitHub] limited commented on a change in pull request #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries

2018-06-13 Thread GitBox
limited commented on a change in pull request #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries URL: https://github.com/apache/trafficcontrol/pull/2406#discussion_r195228980 ## File path: lib/go-tc/deliveryservices.go ## @@ -58,59 +58,60 @@ type

[GitHub] dewrich opened a new pull request #2408: Refactored the API Tests to run in a container

2018-06-13 Thread GitBox
dewrich opened a new pull request #2408: Refactored the API Tests to run in a container URL: https://github.com/apache/trafficcontrol/pull/2408 These are now compatible with the cdn-in-a-box docker containers This is an

[GitHub] asfgit commented on issue #2408: Refactored the API Tests to run in a Docker container

2018-06-13 Thread GitBox
asfgit commented on issue #2408: Refactored the API Tests to run in a Docker container URL: https://github.com/apache/trafficcontrol/pull/2408#issuecomment-397089096 Refer to this link for build results (access rights to CI server needed):

[GitHub] Vijay-1 commented on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops.

2018-06-13 Thread GitBox
Vijay-1 commented on issue #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. URL: https://github.com/apache/trafficcontrol/pull/2407#issuecomment-397054098 **Rawlin's comment on DS-es sharing same origin** If there is a scenario where we need two different

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

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

[GitHub] mitchell852 commented on a change in pull request #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries

2018-06-13 Thread GitBox
mitchell852 commented on a change in pull request #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries URL: https://github.com/apache/trafficcontrol/pull/2406#discussion_r195243156 ## File path: lib/go-tc/deliveryservices.go ## @@ -58,59 +58,60 @@

[GitHub] rivasj commented on a change in pull request #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries

2018-06-13 Thread GitBox
rivasj commented on a change in pull request #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries URL: https://github.com/apache/trafficcontrol/pull/2406#discussion_r195246642 ## File path: lib/go-tc/deliveryservices.go ## @@ -58,59 +58,60 @@ type

[GitHub] dineshkumar82 opened a new pull request #2407: Edge retrieval from origin - Configuring go_direct from traffic ops.

2018-06-13 Thread GitBox
dineshkumar82 opened a new pull request #2407: Edge retrieval from origin - Configuring go_direct from traffic ops. URL: https://github.com/apache/trafficcontrol/pull/2407 All DS-es sharing an origin server should have same value for go_direct (or) No two Delivery service shares same

[GitHub] rob05c commented on a change in pull request #2314: Remove deliveryservice.org_server_fqdn column/compute it from Origin table

2018-06-13 Thread GitBox
rob05c commented on a change in pull request #2314: Remove deliveryservice.org_server_fqdn column/compute it from Origin table URL: https://github.com/apache/trafficcontrol/pull/2314#discussion_r195205994 ## File path:

[GitHub] rob05c commented on a change in pull request #2314: Remove deliveryservice.org_server_fqdn column/compute it from Origin table

2018-06-13 Thread GitBox
rob05c commented on a change in pull request #2314: Remove deliveryservice.org_server_fqdn column/compute it from Origin table URL: https://github.com/apache/trafficcontrol/pull/2314#discussion_r195205994 ## File path:

[GitHub] rawlinp commented on a change in pull request #2314: Remove deliveryservice.org_server_fqdn column/compute it from Origin table

2018-06-13 Thread GitBox
rawlinp commented on a change in pull request #2314: Remove deliveryservice.org_server_fqdn column/compute it from Origin table URL: https://github.com/apache/trafficcontrol/pull/2314#discussion_r195215795 ## File path:

[GitHub] rob05c commented on a change in pull request #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries

2018-06-13 Thread GitBox
rob05c commented on a change in pull request #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries URL: https://github.com/apache/trafficcontrol/pull/2406#discussion_r195226665 ## File path: traffic_ops/experimental/server/api/deliveryservices.go ##

[GitHub] rob05c commented on a change in pull request #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries

2018-06-13 Thread GitBox
rob05c commented on a change in pull request #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries URL: https://github.com/apache/trafficcontrol/pull/2406#discussion_r195232902 ## File path: lib/go-tc/deliveryservices.go ## @@ -58,59 +58,60 @@ type

[GitHub] rob05c commented on a change in pull request #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries

2018-06-13 Thread GitBox
rob05c commented on a change in pull request #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries URL: https://github.com/apache/trafficcontrol/pull/2406#discussion_r195232902 ## File path: lib/go-tc/deliveryservices.go ## @@ -58,59 +58,60 @@ type

[GitHub] dewrich commented on a change in pull request #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries

2018-06-13 Thread GitBox
dewrich commented on a change in pull request #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries URL: https://github.com/apache/trafficcontrol/pull/2406#discussion_r195241031 ## File path: traffic_ops/experimental/server/api/deliveryservices.go ##

<    1   2   3   4   5   6   7   8   9   10   >