[GitHub] rob05c closed pull request #2396: Fix TO Go "CRUDer" GetRefTypes to new, not shared

rob05c closed pull request #2396: Fix TO Go "CRUDer" GetRefTypes to new, not shared URL: https://github.com/apache/incubator-trafficcontrol/pull/2396 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] rob05c commented on a change in pull request #2365: Add TO Go api helpers

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

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

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

[GitHub] ezelkow1 opened a new issue #2397: TP - empty value field for parameter not accepted

ezelkow1 opened a new issue #2397: TP - empty value field for parameter not accepted URL: https://github.com/apache/incubator-trafficcontrol/issues/2397 With the old TO UI we do currently have parameters that have/use empty value fields, i.e. various ATS plugins. These were valid in the

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

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

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

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

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

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

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

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

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]

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

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] dangogh closed pull request #2403: Fix merge conflict for changed func signature

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]

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

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

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

See

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

See

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

See -- Started by upstream project "incubator-trafficcontrol-PR" build number 1794 originally caused by: GitHub pull request #2399 of commit

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

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] rob05c commented on a change in pull request #2365: Add TO Go api helpers

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

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

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

See -- GitHub pull request #2398 of commit 419f958c351e246d82535c2b7462d02492c51059, no merge conflicts. [EnvInject] - Loading node environment variables. Building remotely

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

See -- Started by an SCM change Started by an SCM change [EnvInject] - Loading node environment variables. Building remotely on H34 (ubuntu xenial) in workspace

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

[GitHub] rob05c opened a new pull request #2396: Fix TO Go "CRUDer" GetRefTypes to new, not shared

rob05c opened a new pull request #2396: Fix TO Go "CRUDer" GetRefTypes to new, not shared URL: https://github.com/apache/incubator-trafficcontrol/pull/2396 Using shared objects between threads is dangerous, and there's no reason for it, the cost to create these structs is insignificant.

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

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

[GitHub] mitchell852 commented on issue #2387: GET /api/1.3/cachegroups doesn't support filtering by type

mitchell852 commented on issue #2387: GET /api/1.3/cachegroups doesn't support filtering by type URL: https://github.com/apache/incubator-trafficcontrol/issues/2387#issuecomment-396620104 https:///#!/types//cache-groups calls this api: GET api/1.3/cachegroups?type=6

[GitHub] asfgit commented on issue #2396: Fix TO Go "CRUDer" GetRefTypes to new, not shared

asfgit commented on issue #2396: Fix TO Go "CRUDer" GetRefTypes to new, not shared URL: https://github.com/apache/incubator-trafficcontrol/pull/2396#issuecomment-396619738 Refer to this link for build results (access rights to CI server needed):

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

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

[GitHub] alficles commented on a change in pull request #2395: To go ds sslkeys generate

alficles commented on a change in pull request #2395: To go ds sslkeys generate URL: https://github.com/apache/incubator-trafficcontrol/pull/2395#discussion_r194776585 ## File path: traffic_ops/traffic_ops_golang/deliveryservice/gencert.go ## @@ -0,0 +1,121 @@ +package

[GitHub] rob05c commented on a change in pull request #2395: To go ds sslkeys generate

rob05c commented on a change in pull request #2395: To go ds sslkeys generate URL: https://github.com/apache/incubator-trafficcontrol/pull/2395#discussion_r194813254 ## File path: traffic_ops/traffic_ops_golang/deliveryservice/gencert.go ## @@ -0,0 +1,121 @@ +package

[GitHub] rob05c commented on a change in pull request #2395: To go ds sslkeys generate

rob05c commented on a change in pull request #2395: To go ds sslkeys generate URL: https://github.com/apache/incubator-trafficcontrol/pull/2395#discussion_r194813254 ## File path: traffic_ops/traffic_ops_golang/deliveryservice/gencert.go ## @@ -0,0 +1,121 @@ +package

[GitHub] dangogh opened a new pull request #2398: [Issue 2387] allow cachegroup select by type id

dangogh opened a new pull request #2398: [Issue 2387] allow cachegroup select by type id URL: https://github.com/apache/incubator-trafficcontrol/pull/2398 This fixes #2387 Now allows `.../api/1.3/cachegroups?type=4`, e.g.