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

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_r186620663 ## File path: traffic_ops/app/lib/API/CachegroupFallback.pm ## @@ -0,0 +1,283 @@

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

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_r186620311 ## File path: traffic_ops/app/lib/API/CachegroupFallback.pm ## @@ -0,0 +1,283 @@

[GitHub] rawlinp commented on a change in pull request #2124: Add TO Go deliveryservices routes

rawlinp commented on a change in pull request #2124: Add TO Go deliveryservices routes URL: https://github.com/apache/incubator-trafficcontrol/pull/2124#discussion_r186584997 ## File path: traffic_ops/traffic_ops_golang/deliveryservice/deliveryservicesv12.go ## @@ -0,0

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

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

[GitHub] asfgit commented on issue #2236: update portal info bubbles

asfgit commented on issue #2236: update portal info bubbles URL: https://github.com/apache/incubator-trafficcontrol/pull/2236#issuecomment-387225250 Refer to this link for build results (access rights to CI server needed):

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

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

[GitHub] rob05c commented on issue #1236: Delivery Service APIs: CRUD

rob05c commented on issue #1236: Delivery Service APIs: CRUD URL: https://github.com/apache/incubator-trafficcontrol/issues/1236#issuecomment-387223244 https://github.com/apache/incubator-trafficcontrol/pull/2124 This is an

[GitHub] ashish-timilsina commented on issue #2236: update portal info bubbles

ashish-timilsina commented on issue #2236: update portal info bubbles URL: https://github.com/apache/incubator-trafficcontrol/pull/2236#issuecomment-387221380 @mitchell852 This is an automated message from the Apache Git

[GitHub] ashish-timilsina opened a new pull request #2236: update portal info bubbles

ashish-timilsina opened a new pull request #2236: update portal info bubbles URL: https://github.com/apache/incubator-trafficcontrol/pull/2236 update portal info bubbles from Traffic control site ipv6 should be true by default

[GitHub] asfgit commented on issue #2234: Add TO Go staticdnsentries endpoint

asfgit commented on issue #2234: Add TO Go staticdnsentries endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2234#issuecomment-387215917 Refer to this link for build results (access rights to CI server needed):

[GitHub] dneuman64 opened a new issue #2235: Ability to blacklist IPs

dneuman64 opened a new issue #2235: Ability to blacklist IPs URL: https://github.com/apache/incubator-trafficcontrol/issues/2235 Traffic Control should support the ability to blacklist individual IPs. Currently we can limit by country codes and by using our CZF, but there isn't a good

[GitHub] rob05c opened a new pull request #2234: Add TO Go staticdnsentries endpoint

rob05c opened a new pull request #2234: Add TO Go staticdnsentries endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2234 Looks like it isn't possible to write API Tests for this, because it's only a GET. Might have to wait until the API/Portal supports creating static

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

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

[GitHub] asfgit commented on issue #2010: Go login

asfgit commented on issue #2010: Go login URL: https://github.com/apache/incubator-trafficcontrol/pull/2010#issuecomment-387206309 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1530/ Test

[GitHub] DylanVolz commented on a change in pull request #2010: Go login

DylanVolz commented on a change in pull request #2010: Go login URL: https://github.com/apache/incubator-trafficcontrol/pull/2010#discussion_r186549373 ## File path: traffic_ops/traffic_ops_golang/auth/login.go ## @@ -0,0 +1,156 @@ +package auth + +/* + * Licensed to the

[GitHub] DylanVolz commented on a change in pull request #2010: Go login

DylanVolz commented on a change in pull request #2010: Go login URL: https://github.com/apache/incubator-trafficcontrol/pull/2010#discussion_r186548894 ## File path: traffic_ops/traffic_ops_golang/config/config.go ## @@ -142,6 +156,16 @@ func LoadConfig(cdnConfPath

[GitHub] mitchell852 commented on issue #1338: Role APIs: CRUD

mitchell852 commented on issue #1338: Role APIs: CRUD URL: https://github.com/apache/incubator-trafficcontrol/issues/1338#issuecomment-387192948 duplicate of #2033 This is an automated message from the Apache Git Service.

[GitHub] mitchell852 closed issue #1338: Role APIs: CRUD

mitchell852 closed issue #1338: Role APIs: CRUD URL: https://github.com/apache/incubator-trafficcontrol/issues/1338 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] dewrich closed issue #1214: Profile APIs: CRUD

dewrich closed issue #1214: Profile APIs: CRUD URL: https://github.com/apache/incubator-trafficcontrol/issues/1214 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

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

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

[GitHub] asfgit commented on issue #2233: Fixed the Go import package alignment because the github project moved

asfgit commented on issue #2233: Fixed the Go import package alignment because the github project moved URL: https://github.com/apache/incubator-trafficcontrol/pull/2233#issuecomment-387188635 Refer to this link for build results (access rights to CI server needed):

[GitHub] dewrich opened a new pull request #2233: Fixed the Go import package alignment because the github project moved

dewrich opened a new pull request #2233: Fixed the Go import package alignment because the github project moved URL: https://github.com/apache/incubator-trafficcontrol/pull/2233 Fixed the Go to point to `github.com/lestrrat-go/jwx` which prevents `go get -v` from working in the Docker

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

See Changes: [rob05c] fixed missing 'lastUpdate' field in Types and removed Types handler in [rob05c] reverted changes removing PERL api --

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

mitchell852 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_r186530252 ## File path: traffic_ops/app/lib/API/CachegroupFallback.pm ## @@ -0,0 +1,283

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

mitchell852 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_r186530205 ## File path: traffic_ops/app/lib/API/CachegroupFallback.pm ## @@ -0,0 +1,283

[GitHub] rob05c commented on a change in pull request #2010: Go login

rob05c commented on a change in pull request #2010: Go login URL: https://github.com/apache/incubator-trafficcontrol/pull/2010#discussion_r186523821 ## File path: traffic_ops/traffic_ops_golang/auth/login.go ## @@ -0,0 +1,156 @@ +package auth + +/* + * Licensed to the

[GitHub] rob05c commented on a change in pull request #2010: Go login

rob05c commented on a change in pull request #2010: Go login URL: https://github.com/apache/incubator-trafficcontrol/pull/2010#discussion_r186522372 ## File path: traffic_ops/traffic_ops_golang/config/config.go ## @@ -142,6 +156,16 @@ func LoadConfig(cdnConfPath string,

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

See -- Started by upstream project "incubator-trafficcontrol-master-build" build number 753 originally caused by: Started by an SCM change [EnvInject] - Loading node

[GitHub] rob05c closed issue #2153: Read method on api/1.3/types does not return 'lastUpdated'

rob05c closed issue #2153: Read method on api/1.3/types does not return 'lastUpdated' URL: https://github.com/apache/incubator-trafficcontrol/issues/2153 This is an automated message from the Apache Git Service. To respond

[GitHub] rob05c commented on issue #2153: Read method on api/1.3/types does not return 'lastUpdated'

rob05c commented on issue #2153: Read method on api/1.3/types does not return 'lastUpdated' URL: https://github.com/apache/incubator-trafficcontrol/issues/2153#issuecomment-387171514 Fixed by #2155 in 58ab7c2aeb6e7c7a97a5afbdaf463562ea00e20d

[GitHub] rob05c closed pull request #2155: Fixed missing 'lastUpdate' field in Types Go API

rob05c closed pull request #2155: Fixed missing 'lastUpdate' field in Types Go API URL: https://github.com/apache/incubator-trafficcontrol/pull/2155 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:

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

See -- Started by upstream project "incubator-trafficcontrol-master-build" build number 752 originally caused by: Started by an SCM change [EnvInject] - Loading node

[GitHub] rob05c commented on issue #2228: Added the Get API in Go for the DeliveryserviceServers endpoint

rob05c commented on issue #2228: Added the Get API in Go for the DeliveryserviceServers endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2228#issuecomment-387167737 Looks good. Tested against a prod database, produces the same output as Perl

[GitHub] rob05c closed pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint

rob05c closed pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2228 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint

rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2228#discussion_r186515615 ## File path:

[GitHub] rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint

rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2228#discussion_r186515364 ## File path:

[GitHub] rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint

rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2228#discussion_r186515275 ## File path:

[GitHub] rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint

rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2228#discussion_r186515078 ## File path:

[GitHub] rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint

rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2228#discussion_r186514602 ## File path:

[GitHub] rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint

rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2228#discussion_r186514365 ## File path:

[GitHub] rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint

rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2228#discussion_r186514365 ## File path:

[GitHub] rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint

rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2228#discussion_r186514290 ## File path:

[GitHub] rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint

rob05c commented on a change in pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2228#discussion_r186514141 ## File path: lib/go-tc/deliveryservice_servers.go ## @@ -0,0 +1,80

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

See

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

See -- Started by upstream project "incubator-trafficcontrol-master-build" build number 751 originally caused by: Started by an SCM change Started by an SCM change

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

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

[GitHub] asfgit commented on issue #2231: Fix TO Go ASNs unit tests for renamed symbols

asfgit commented on issue #2231: Fix TO Go ASNs unit tests for renamed symbols URL: https://github.com/apache/incubator-trafficcontrol/pull/2231#issuecomment-387153848 Refer to this link for build results (access rights to CI server needed):

[GitHub] rob05c opened a new issue #2232: Rewrite Traffic Ops CRConfig Table Endpoints in Go

rob05c opened a new issue #2232: Rewrite Traffic Ops CRConfig Table Endpoints in Go URL: https://github.com/apache/incubator-trafficcontrol/issues/2232 One of the first steps to Self Service Change Integrity is to change the CRConfig to select the latest timestamps for each delivery

[GitHub] dewrich closed pull request #2231: Fix TO Go ASNs unit tests for renamed symbols

dewrich closed pull request #2231: Fix TO Go ASNs unit tests for renamed symbols URL: https://github.com/apache/incubator-trafficcontrol/pull/2231 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] rob05c opened a new pull request #2231: Fix TO Go ASNs unit tests for renamed symbols

rob05c opened a new pull request #2231: Fix TO Go ASNs unit tests for renamed symbols URL: https://github.com/apache/incubator-trafficcontrol/pull/2231 This is an automated message from the Apache Git Service. To respond to

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

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

[GitHub] asfgit commented on issue #2010: Go login

asfgit commented on issue #2010: Go login URL: https://github.com/apache/incubator-trafficcontrol/pull/2010#issuecomment-387114915 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1527/ Test

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

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

[GitHub] asfgit commented on issue #2010: Go login

asfgit commented on issue #2010: Go login URL: https://github.com/apache/incubator-trafficcontrol/pull/2010#issuecomment-387112785 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1526/ Test

[GitHub] rob05c opened a new issue #2230: Portal Servers page is missing Traffic Monitor link

rob05c opened a new issue #2230: Portal Servers page is missing Traffic Monitor link URL: https://github.com/apache/incubator-trafficcontrol/issues/2230 The old UI has a button on Traffic Monitor Server rows, that links to that monitor's FQDN (web GUI). The Portal has no easy way

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

Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-386972753 @mitchell852 @elsloo Will upload the changes in couple of days. Caught up in some thing else.