[GitHub] dewrich closed pull request #2264: Add more documentation about the TO Client Libraries

2018-05-14 Thread GitBox
dewrich closed pull request #2264: Add more documentation about the TO Client Libraries URL: https://github.com/apache/incubator-trafficcontrol/pull/2264 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

Jenkins build is back to normal : incubator-trafficcontrol-master-build #758

2018-05-14 Thread Apache Jenkins Server
See

Jenkins build is back to normal : incubator-trafficcontrol-traffic_ops-test #617

2018-05-14 Thread Apache Jenkins Server
See

Build failed in Jenkins: incubator-trafficcontrol-rat #715

2018-05-14 Thread Apache Jenkins Server
See -- Started by upstream project "incubator-trafficcontrol-master-build" build number 759 originally caused by: Started by an SCM change [EnvInject] - Loading node

[GitHub] jhg03a opened a new pull request #2265: Fix formatting in Native Client Doc Readme

2018-05-14 Thread GitBox
jhg03a opened a new pull request #2265: Fix formatting in Native Client Doc Readme URL: https://github.com/apache/incubator-trafficcontrol/pull/2265 An error was found in the formatting of the new TO native client documentation after it was merged, so this addresses that error.

[GitHub] rob05c commented on issue #2249: Add CRConfig DS Modified fields

2018-05-14 Thread GitBox
rob05c commented on issue #2249: Add CRConfig DS Modified fields URL: https://github.com/apache/incubator-trafficcontrol/pull/2249#issuecomment-387898955 I'll also note, I strongly dislike SQL triggers, for myriad reasons; but in this case, it's the best, safest way to guarantee the

[GitHub] dewrich closed pull request #2265: Fix formatting in Native Client Doc Readme

2018-05-14 Thread GitBox
dewrich closed pull request #2265: Fix formatting in Native Client Doc Readme URL: https://github.com/apache/incubator-trafficcontrol/pull/2265 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

Build failed in Jenkins: incubator-trafficcontrol-rat #714

2018-05-14 Thread Apache Jenkins Server
See -- Started by upstream project "incubator-trafficcontrol-master-build" build number 758 originally caused by: Started by an SCM change [EnvInject] - Loading node

[GitHub] mitchell852 commented on a change in pull request #2245: Anonymous IP TP Implementation

2018-05-14 Thread GitBox
mitchell852 commented on a change in pull request #2245: Anonymous IP TP Implementation URL: https://github.com/apache/incubator-trafficcontrol/pull/2245#discussion_r188078542 ## File path: traffic_portal/app/src/traffic_portal_properties.json ## @@ -252,10 +260,10 @@

[GitHub] rivasj commented on a change in pull request #2245: Anonymous IP TP Implementation

2018-05-14 Thread GitBox
rivasj commented on a change in pull request #2245: Anonymous IP TP Implementation URL: https://github.com/apache/incubator-trafficcontrol/pull/2245#discussion_r188082216 ## File path: traffic_portal/app/src/traffic_portal_properties.json ## @@ -252,10 +260,10 @@

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

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

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

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

[GitHub] mitchell852 opened a new pull request #2268: adds ?orderby=name when fetching cache groups

2018-05-14 Thread GitBox
mitchell852 opened a new pull request #2268: adds ?orderby=name when fetching cache groups URL: https://github.com/apache/incubator-trafficcontrol/pull/2268 fixes #2260 This is an automated message from the Apache Git

[GitHub] asfgit commented on issue #2268: adds ?orderby=name when fetching cache groups

2018-05-14 Thread GitBox
asfgit commented on issue #2268: adds ?orderby=name when fetching cache groups URL: https://github.com/apache/incubator-trafficcontrol/pull/2268#issuecomment-388958125 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2149: TO and TODB in docker front-to-back

2018-05-14 Thread GitBox
asfgit commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-388961224 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 closed pull request #2245: Anonymous IP TP Implementation

2018-05-14 Thread GitBox
mitchell852 closed pull request #2245: Anonymous IP TP Implementation URL: https://github.com/apache/incubator-trafficcontrol/pull/2245 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] smalenfant commented on issue #2015: Traffic Portal: clone parameters (feature request)

2018-05-14 Thread GitBox
smalenfant commented on issue #2015: Traffic Portal: clone parameters (feature request) URL: https://github.com/apache/incubator-trafficcontrol/issues/2015#issuecomment-388902247 @mitchell852 Just have this exact problem at the moment. You have to take notes and find your parameter when

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

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

[GitHub] mitchell852 opened a new issue #2267: POST api/$version/deliveryservices/sslkeys/add should validate ssl keys

2018-05-14 Thread GitBox
mitchell852 opened a new issue #2267: POST api/$version/deliveryservices/sslkeys/add should validate ssl keys URL: https://github.com/apache/incubator-trafficcontrol/issues/2267 There is currently no validation when adding/uploading ssl keys to a delivery service. This can put Traffic

[GitHub] asfgit commented on issue #2266: ORT: Removing unnecessary errors

2018-05-14 Thread GitBox
asfgit commented on issue #2266: ORT: Removing unnecessary errors URL: https://github.com/apache/incubator-trafficcontrol/pull/2266#issuecomment-388890184 Refer to this link for build results (access rights to CI server needed):

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

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

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

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

[GitHub] asfgit commented on issue #2226: Anonymous IP TO Implementation

2018-05-14 Thread GitBox
asfgit commented on issue #2226: Anonymous IP TO Implementation URL: https://github.com/apache/incubator-trafficcontrol/pull/2226#issuecomment-38869 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2149: TO and TODB in docker front-to-back

2018-05-14 Thread GitBox
asfgit commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-388893389 Refer to this link for build results (access rights to CI server needed):

[GitHub] smalenfant opened a new pull request #2266: ORT: Removing unnecessary errors

2018-05-14 Thread GitBox
smalenfant opened a new pull request #2266: ORT: Removing unnecessary errors URL: https://github.com/apache/incubator-trafficcontrol/pull/2266 The diff was not showing the source/destination due to improper handling of notes. Fix will... - Prevent wrong ERROR printing on the

[GitHub] elsloo closed pull request #2184: Anonymous IP TR Implementation

2018-05-14 Thread GitBox
elsloo closed pull request #2184: Anonymous IP TR Implementation URL: https://github.com/apache/incubator-trafficcontrol/pull/2184 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

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

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

Build failed in Jenkins: incubator-trafficcontrol-rat #716

2018-05-14 Thread Apache Jenkins Server
See -- Started by upstream project "incubator-trafficcontrol-master-build" build number 760 originally caused by: Started by an SCM change [EnvInject] - Loading node

[GitHub] smalenfant commented on issue #1666: TP - option to create and link parameter in profile page

2018-05-14 Thread GitBox
smalenfant commented on issue #1666: TP - option to create and link parameter in profile page URL: https://github.com/apache/incubator-trafficcontrol/issues/1666#issuecomment-388902711 Here's how I see this that could work... - From the profile view (you selected a profile). -

[GitHub] asfgit commented on issue #2149: TO and TODB in docker front-to-back

2018-05-14 Thread GitBox
asfgit commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-388912284 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2149: TO and TODB in docker front-to-back

2018-05-14 Thread GitBox
asfgit commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-388915771 Refer to this link for build results (access rights to CI server needed):

[GitHub] dg4prez closed pull request #2266: ORT: Removing unnecessary errors

2018-05-14 Thread GitBox
dg4prez closed pull request #2266: ORT: Removing unnecessary errors URL: https://github.com/apache/incubator-trafficcontrol/pull/2266 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

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

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

Build failed in Jenkins: incubator-trafficcontrol-rat #717

2018-05-14 Thread Apache Jenkins Server
See -- Started by upstream project "incubator-trafficcontrol-master-build" build number 761 originally caused by: Started by an SCM change [EnvInject] - Loading node

[GitHub] mitchell852 commented on issue #2267: POST api/$version/deliveryservices/sslkeys/add should validate ssl keys

2018-05-14 Thread GitBox
mitchell852 commented on issue #2267: POST api/$version/deliveryservices/sslkeys/add should validate ssl keys URL: https://github.com/apache/incubator-trafficcontrol/issues/2267#issuecomment-388916262 Also, maybe TR should not blow up if/when it encounters a bad cert but hopefully with

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

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

[GitHub] dangogh closed pull request #2268: adds ?orderby=name when fetching cache groups

2018-05-14 Thread GitBox
dangogh closed pull request #2268: adds ?orderby=name when fetching cache groups URL: https://github.com/apache/incubator-trafficcontrol/pull/2268 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] dangogh closed issue #2260: TP: sorting of cache group list is no longer alphanumeric

2018-05-14 Thread GitBox
dangogh closed issue #2260: TP: sorting of cache group list is no longer alphanumeric URL: https://github.com/apache/incubator-trafficcontrol/issues/2260 This is an automated message from the Apache Git Service. To respond

[GitHub] dangogh commented on issue #2149: TO and TODB in docker front-to-back

2018-05-14 Thread GitBox
dangogh commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-388970888 closing for now while still WIP This is an automated message from

[GitHub] asfgit commented on issue #2269: Deliveryservice_Server API conversion to Go

2018-05-14 Thread GitBox
asfgit commented on issue #2269: Deliveryservice_Server API conversion to Go URL: https://github.com/apache/incubator-trafficcontrol/pull/2269#issuecomment-388975850 Can one of the admins verify this patch? This is an

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

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

[GitHub] rob05c commented on a change in pull request #2246: Add HitCount and FreshFor

2018-05-14 Thread GitBox
rob05c commented on a change in pull request #2246: Add HitCount and FreshFor URL: https://github.com/apache/incubator-trafficcontrol/pull/2246#discussion_r188119204 ## File path: grove/plugin/http_cacheinspector.go ## @@ -169,16 +170,17 @@ func cacheinspect(icfg

[GitHub] asfgit commented on issue #2270: Add TO Go Servers Details endpoint

2018-05-14 Thread GitBox
asfgit commented on issue #2270: Add TO Go Servers Details endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2270#issuecomment-388989863 Refer to this link for build results (access rights to CI server needed):

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

2018-05-14 Thread GitBox
asfgit commented on issue #2247: Add an Origin API URL: https://github.com/apache/incubator-trafficcontrol/pull/2247#issuecomment-388997317 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1569/

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

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

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

2018-05-14 Thread Apache Jenkins Server
See Changes: [mitchell852] added anonymousBlockingEnabled to TP properties [mitchell852] added Anonymous Blocking field to TP UI [mitchell852] cleanup formatting [dangogh] adds

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

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

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

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

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

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

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

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

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

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

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

2018-05-14 Thread GitBox
rob05c commented on a change in pull request #2269: Deliveryservice_Server API conversion to Go URL: https://github.com/apache/incubator-trafficcontrol/pull/2269#discussion_r188110723 ## File path: infrastructure/docker/traffic_ops/Dockerfile_psql ## @@ -0,0 +1,20 @@

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

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

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

2018-05-14 Thread GitBox
rob05c commented on a change in pull request #2269: Deliveryservice_Server API conversion to Go URL: https://github.com/apache/incubator-trafficcontrol/pull/2269#discussion_r188114096 ## File path: infrastructure/docker/traffic_ops/dbInit.sh ## @@ -0,0 +1,8 @@

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

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

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

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

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

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

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

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

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

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

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

2018-05-14 Thread GitBox
rob05c commented on a change in pull request #2269: Deliveryservice_Server API conversion to Go URL: https://github.com/apache/incubator-trafficcontrol/pull/2269#discussion_r188111222 ## File path: lib/go-tc/deliveryservice_servers.go ## @@ -78,3 +99,65 @@ type DssServer

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

2018-05-14 Thread GitBox
rob05c commented on a change in pull request #2269: Deliveryservice_Server API conversion to Go URL: https://github.com/apache/incubator-trafficcontrol/pull/2269#discussion_r188114148 ## File path: infrastructure/docker/traffic_ops/Dockerfile_psql ## @@ -0,0 +1,20 @@

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

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

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

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

[GitHub] asfgit commented on issue #2149: TO and TODB in docker front-to-back

2018-05-14 Thread GitBox
asfgit commented on issue #2149: TO and TODB in docker front-to-back URL: https://github.com/apache/incubator-trafficcontrol/pull/2149#issuecomment-388982939 Refer to this link for build results (access rights to CI server needed):

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

2018-05-14 Thread GitBox
rob05c commented on a change in pull request #2269: Deliveryservice_Server API conversion to Go URL: https://github.com/apache/incubator-trafficcontrol/pull/2269#discussion_r188114240 ## File path: lib/go-tc/deliveryservice_servers.go ## @@ -1,6 +1,8 @@ package tc

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

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

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

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

[GitHub] rob05c commented on a change in pull request #2246: Add HitCount and FreshFor

2018-05-14 Thread GitBox
rob05c commented on a change in pull request #2246: Add HitCount and FreshFor URL: https://github.com/apache/incubator-trafficcontrol/pull/2246#discussion_r188120447 ## File path: grove/web/util.go ## @@ -198,3 +200,145 @@ func ParseHTTPDate(d string) (time.Time, bool) {

[GitHub] rob05c commented on a change in pull request #2262: Part 1 of Range Request handling

2018-05-14 Thread GitBox
rob05c commented on a change in pull request #2262: Part 1 of Range Request handling URL: https://github.com/apache/incubator-trafficcontrol/pull/2262#discussion_r188121895 ## File path: grove/plugin/range_req_handler.go ## @@ -0,0 +1,225 @@ +package plugin + +/* +

[GitHub] rob05c commented on a change in pull request #2262: Part 1 of Range Request handling

2018-05-14 Thread GitBox
rob05c commented on a change in pull request #2262: Part 1 of Range Request handling URL: https://github.com/apache/incubator-trafficcontrol/pull/2262#discussion_r188121571 ## File path: grove/plugin/range_req_handler.go ## @@ -0,0 +1,225 @@ +package plugin + +/* +

[GitHub] rob05c commented on a change in pull request #2262: Part 1 of Range Request handling

2018-05-14 Thread GitBox
rob05c commented on a change in pull request #2262: Part 1 of Range Request handling URL: https://github.com/apache/incubator-trafficcontrol/pull/2262#discussion_r188121571 ## File path: grove/plugin/range_req_handler.go ## @@ -0,0 +1,225 @@ +package plugin + +/* +

[GitHub] rob05c commented on a change in pull request #2262: Part 1 of Range Request handling

2018-05-14 Thread GitBox
rob05c commented on a change in pull request #2262: Part 1 of Range Request handling URL: https://github.com/apache/incubator-trafficcontrol/pull/2262#discussion_r188121922 ## File path: grove/plugin/range_req_handler.go ## @@ -0,0 +1,225 @@ +package plugin + +/* +

[GitHub] rob05c commented on a change in pull request #2262: Part 1 of Range Request handling

2018-05-14 Thread GitBox
rob05c commented on a change in pull request #2262: Part 1 of Range Request handling URL: https://github.com/apache/incubator-trafficcontrol/pull/2262#discussion_r188122792 ## File path: grove/plugin/range_req_handler.go ## @@ -0,0 +1,225 @@ +package plugin + +/* +

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

2018-05-14 Thread GitBox
asfgit commented on issue #2247: Add an Origin API URL: https://github.com/apache/incubator-trafficcontrol/pull/2247#issuecomment-388998048 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1570/

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

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

[GitHub] asfgit commented on issue #2269: Deliveryservice_Server API conversion to Go

2018-05-14 Thread GitBox
asfgit commented on issue #2269: Deliveryservice_Server API conversion to Go URL: https://github.com/apache/incubator-trafficcontrol/pull/2269#issuecomment-388975605 Can one of the admins verify this patch? This is an

[GitHub] rob05c opened a new pull request #2270: Add TO Go Servers Details endpoint

2018-05-14 Thread GitBox
rob05c opened a new pull request #2270: Add TO Go Servers Details endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2270 This is an automated message from the Apache Git Service. To respond to the

Build failed in Jenkins: incubator-trafficcontrol-rat #718

2018-05-14 Thread Apache Jenkins Server
See -- Started by upstream project "incubator-trafficcontrol-master-build" build number 762 originally caused by: Started by an SCM change Started by an SCM change

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

2018-05-14 Thread GitBox
rob05c commented on issue #2232: Rewrite Traffic Ops CRConfig Table Endpoints in Go URL: https://github.com/apache/incubator-trafficcontrol/issues/2232#issuecomment-387444293 Ok, we have PRs for all major table routes, but many of them are only "CRUD", and missing additional endpoints.

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

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

[GitHub] ajschmidt opened a new pull request #2269: Deliveryservice_Server API conversion to Go

2018-05-14 Thread GitBox
ajschmidt opened a new pull request #2269: Deliveryservice_Server API conversion to Go URL: https://github.com/apache/incubator-trafficcontrol/pull/2269 This PR converts these API methods to Go: GETapi/1.1/deliveryserviceserver POST

[GitHub] rob05c commented on a change in pull request #2262: Part 1 of Range Request handling

2018-05-14 Thread GitBox
rob05c commented on a change in pull request #2262: Part 1 of Range Request handling URL: https://github.com/apache/incubator-trafficcontrol/pull/2262#discussion_r188121450 ## File path: grove/plugin/range_req_handler.go ## @@ -0,0 +1,225 @@ +package plugin + +/* +

[GitHub] rob05c commented on a change in pull request #2262: Part 1 of Range Request handling

2018-05-14 Thread GitBox
rob05c commented on a change in pull request #2262: Part 1 of Range Request handling URL: https://github.com/apache/incubator-trafficcontrol/pull/2262#discussion_r188121319 ## File path: grove/plugin/plugin.go ## @@ -91,6 +92,12 @@ type BeforeRespondData struct {

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

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

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

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

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

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

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

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