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

mitchell852 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-386650324 @elsloo - @Vijay-1 and I discussed/agreed on these API routes but I don't think the changes have been made

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

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

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

elsloo commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-38663 Hey @Vijay-1 and @mitchell852 – where are we on getting these changes into the PR?

[GitHub] asfgit commented on issue #2108: create roles crud endpoints in golang

asfgit commented on issue #2108: create roles crud endpoints in golang URL: https://github.com/apache/incubator-trafficcontrol/pull/2108#issuecomment-386698482 Refer to this link for build results (access rights to CI server needed):

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

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

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

mitchell852 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-386650324 @Vijay-1 and I discussed these routes but I don't think the changes have been made yet to support them: GET

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

mitchell852 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-386650324 @Vijay-1 and I discussed/agreed on these API routes but I don't think the changes have been made yet to support

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

mitchell852 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-386650324 @Vijay-1 and I discussed/agreed on these API routes but I don't think the changes have been made yet to support

[GitHub] mitchell852 closed pull request #2225: Fix Traffic Ops Golang CRConfig to match Perl's magic LogRequestHeaders parameter handling

mitchell852 closed pull request #2225: Fix Traffic Ops Golang CRConfig to match Perl's magic LogRequestHeaders parameter handling URL: https://github.com/apache/incubator-trafficcontrol/pull/2225 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it

[GitHub] PeterRyder opened a new pull request #2226: Anonymous IP TO Implementation

PeterRyder opened a new pull request #2226: Anonymous IP TO Implementation URL: https://github.com/apache/incubator-trafficcontrol/pull/2226 @rivasj New PR for Anonymous IP Traffic Ops implementation after rebase. Refer to

[GitHub] mitchell852 commented on issue #1979: DO NOT MERGE - Junk commit to test prbuilder

mitchell852 commented on issue #1979: DO NOT MERGE - Junk commit to test prbuilder URL: https://github.com/apache/incubator-trafficcontrol/pull/1979#issuecomment-386648792 @jhg03a - can this be closed? This is an automated

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

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

[GitHub] PeterRyder commented on issue #1643: Anonymous IP TO Implementation

PeterRyder commented on issue #1643: Anonymous IP TO Implementation URL: https://github.com/apache/incubator-trafficcontrol/pull/1643#issuecomment-38980 Closing - refer to new PR https://github.com/apache/incubator-trafficcontrol/pull/2226 for 2.3.x rebase

[GitHub] mitchell852 commented on a change in pull request #2222: Add TO Go 1.1 routes

mitchell852 commented on a change in pull request #: Add TO Go 1.1 routes URL: https://github.com/apache/incubator-trafficcontrol/pull/#discussion_r186128347 ## File path: traffic_ops/traffic_ops_golang/routes.go ## @@ -77,106 +77,107 @@ func Routes(d ServerData)

[GitHub] mitchell852 commented on a change in pull request #2222: Add TO Go 1.1 routes

mitchell852 commented on a change in pull request #: Add TO Go 1.1 routes URL: https://github.com/apache/incubator-trafficcontrol/pull/#discussion_r186127827 ## File path: traffic_ops/traffic_ops_golang/routes.go ## @@ -77,106 +77,107 @@ func Routes(d ServerData)

[GitHub] asfgit commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes

asfgit commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes URL: https://github.com/apache/incubator-trafficcontrol/pull/2229#issuecomment-386760085 Refer to this link for build results (access rights to CI server needed):

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

See

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

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

[GitHub] asfgit commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes

asfgit commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes URL: https://github.com/apache/incubator-trafficcontrol/pull/2229#issuecomment-386769915 Refer to this link for build results (access rights to CI server needed):

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

See -- GitHub pull request #2229 of commit b3bd2a69deb797fc6277849167c95a391c2af37e, no merge conflicts. Setting status of b3bd2a69deb797fc6277849167c95a391c2af37e to

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

See -- GitHub pull request #2229 of commit ce5bb38e55dbf01d20941b11985f61bd8c207fc2, no merge conflicts. Setting status of ce5bb38e55dbf01d20941b11985f61bd8c207fc2 to

[GitHub] asfgit commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes

asfgit commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes URL: https://github.com/apache/incubator-trafficcontrol/pull/2229#issuecomment-386766718 Refer to this link for build results (access rights to CI server needed):

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

See

[GitHub] mitchell852 opened a new pull request #2227: ensures that the state changes when the url query param changes for c…

mitchell852 opened a new pull request #2227: ensures that the state changes when the url query param changes for c… URL: https://github.com/apache/incubator-trafficcontrol/pull/2227 …ustom menu items fixes a bug in TP's custom menu found by @guzzijason

[GitHub] rob05c commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes

rob05c commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes URL: https://github.com/apache/incubator-trafficcontrol/pull/2229#issuecomment-386754750 This is ready, in terms of functionality. I'm not sure how to write API tests for it though. Everything is ID based,

[GitHub] rob05c commented on a change in pull request #2222: Add TO Go 1.1 routes

rob05c commented on a change in pull request #: Add TO Go 1.1 routes URL: https://github.com/apache/incubator-trafficcontrol/pull/#discussion_r186188020 ## File path: traffic_ops/traffic_ops_golang/routes.go ## @@ -77,106 +77,107 @@ func Routes(d ServerData)

[GitHub] rob05c commented on a change in pull request #2222: Add TO Go 1.1 routes

rob05c commented on a change in pull request #: Add TO Go 1.1 routes URL: https://github.com/apache/incubator-trafficcontrol/pull/#discussion_r186187979 ## File path: traffic_ops/traffic_ops_golang/routes.go ## @@ -77,106 +77,107 @@ func Routes(d ServerData)

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

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

[GitHub] asfgit commented on issue #2222: Add TO Go 1.1 routes

asfgit commented on issue #: Add TO Go 1.1 routes URL: https://github.com/apache/incubator-trafficcontrol/pull/#issuecomment-386733440 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1520/

[GitHub] ajschmidt opened a new pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint

ajschmidt opened a new pull request #2228: Added the Get API in Go for the DeliveryserviceServers endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2228 This is the first of 6 methods in the API. More to come.

[GitHub] asfgit commented on issue #2227: ensures that the state changes when the url query param changes for c…

asfgit commented on issue #2227: ensures that the state changes when the url query param changes for c… URL: https://github.com/apache/incubator-trafficcontrol/pull/2227#issuecomment-386714004 Refer to this link for build results (access rights to CI server needed):

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

asfgit commented on issue #2228: Added the Get API in Go for the DeliveryserviceServers endpoint URL: https://github.com/apache/incubator-trafficcontrol/pull/2228#issuecomment-386743973 Can one of the admins verify this patch?

[GitHub] asfgit commented on issue #2108: create roles crud endpoints in golang

asfgit commented on issue #2108: create roles crud endpoints in golang URL: https://github.com/apache/incubator-trafficcontrol/pull/2108#issuecomment-386753582 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 closed pull request #2222: Add TO Go 1.1 routes

mitchell852 closed pull request #: Add TO Go 1.1 routes URL: https://github.com/apache/incubator-trafficcontrol/pull/ 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

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

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

[GitHub] asfgit commented on issue #2124: Add TO Go deliveryservices routes

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

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

See Changes: [mitchell852] Add TO Go 1.1 routes [rob05c] ensures that the state changes when the url query param changes for -- [...truncated

[GitHub] rob05c opened a new pull request #2229: Add Traffic Ops Golang deliveryservicesregexes

rob05c opened a new pull request #2229: Add Traffic Ops Golang deliveryservicesregexes URL: https://github.com/apache/incubator-trafficcontrol/pull/2229 This is an automated message from the Apache Git Service. To respond

[GitHub] rob05c closed pull request #2227: ensures that the state changes when the url query param changes for c…

rob05c closed pull request #2227: ensures that the state changes when the url query param changes for c… URL: https://github.com/apache/incubator-trafficcontrol/pull/2227 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[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_r186202885 ## File path: traffic_ops/traffic_ops_golang/deliveryservice/deliveryservicesv12.go ## @@ -0,0

[GitHub] rob05c commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes

rob05c commented on issue #2229: Add Traffic Ops Golang deliveryservicesregexes URL: https://github.com/apache/incubator-trafficcontrol/pull/2229#issuecomment-386754750 This is ready, in terms of functionality. I'm not sure how to write API tests for it though. Everything is ID based,

[GitHub] dewrich closed pull request #2221: Fix TO Go CRConfig to omit null missLocations

dewrich closed pull request #2221: Fix TO Go CRConfig to omit null missLocations URL: https://github.com/apache/incubator-trafficcontrol/pull/2221 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: