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

2018-04-04 Thread GitBox
dangogh commented on a change in pull request #2010: Go login URL: https://github.com/apache/incubator-trafficcontrol/pull/2010#discussion_r179322637 ## File path: traffic_ops/install/bin/_postinstall ## @@ -262,13 +262,15 @@ sub generateLdapConf { } } -

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

2018-04-04 Thread GitBox
Vijay-1 commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-378573752 @DylanVolz I have tried to address your comments. Request your review on the same.

[GitHub] dewrich commented on issue #2080: Add cachegroup generic crud to traffic_ops_golang

2018-04-04 Thread GitBox
dewrich commented on issue #2080: Add cachegroup generic crud to traffic_ops_golang URL: https://github.com/apache/incubator-trafficcontrol/pull/2080#issuecomment-378616233 We've been versioning our `lib/go-tc/v13` structs going forward

[GitHub] asfgit commented on issue #2081: Go support for /profileparameters

2018-04-04 Thread GitBox
asfgit commented on issue #2081: Go support for /profileparameters URL: https://github.com/apache/incubator-trafficcontrol/pull/2081#issuecomment-378657430 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2081: Go support for /profileparameters

2018-04-04 Thread GitBox
asfgit commented on issue #2081: Go support for /profileparameters URL: https://github.com/apache/incubator-trafficcontrol/pull/2081#issuecomment-378628220 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2082: Fix extra license headings for removed x libs

2018-04-04 Thread GitBox
asfgit commented on issue #2082: Fix extra license headings for removed x libs URL: https://github.com/apache/incubator-trafficcontrol/pull/2082#issuecomment-378630735 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2080: Add cachegroup generic crud to traffic_ops_golang

2018-04-04 Thread GitBox
asfgit commented on issue #2080: Add cachegroup generic crud to traffic_ops_golang URL: https://github.com/apache/incubator-trafficcontrol/pull/2080#issuecomment-378634660 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2081: Go support for /profileparameters

2018-04-04 Thread GitBox
asfgit commented on issue #2081: Go support for /profileparameters URL: https://github.com/apache/incubator-trafficcontrol/pull/2081#issuecomment-378655912 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2083: updates to the README.md and INSTALL.md for the latest with the Golang proxy

2018-04-04 Thread GitBox
asfgit commented on issue #2083: updates to the README.md and INSTALL.md for the latest with the Golang proxy URL: https://github.com/apache/incubator-trafficcontrol/pull/2083#issuecomment-378667368 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 commented on a change in pull request #2081: Go support for /profileparameters

2018-04-04 Thread GitBox
mitchell852 commented on a change in pull request #2081: Go support for /profileparameters URL: https://github.com/apache/incubator-trafficcontrol/pull/2081#discussion_r179200263 ## File path: traffic_ops/traffic_ops_golang/routes.go ## @@ -191,6 +192,12 @@ func Routes(d

[GitHub] asfgit commented on issue #2080: Add cachegroup generic crud to traffic_ops_golang

2018-04-04 Thread GitBox
asfgit commented on issue #2080: Add cachegroup generic crud to traffic_ops_golang URL: https://github.com/apache/incubator-trafficcontrol/pull/2080#issuecomment-378658264 Refer to this link for build results (access rights to CI server needed):

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

2018-04-04 Thread Apache Jenkins Server
See Changes: [mitchell852] updated the compare tool for v13 -- Started by an SCM change [EnvInject] - Loading node environment variables. Building

[GitHub] asfgit commented on issue #2081: Go support for /profileparameters

2018-04-04 Thread GitBox
asfgit commented on issue #2081: Go support for /profileparameters URL: https://github.com/apache/incubator-trafficcontrol/pull/2081#issuecomment-378660923 Refer to this link for build results (access rights to CI server needed):

[GitHub] dewrich closed pull request #1911: Adds roles and capabilities tables to TP

2018-04-04 Thread GitBox
dewrich closed pull request #1911: Adds roles and capabilities tables to TP URL: https://github.com/apache/incubator-trafficcontrol/pull/1911 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] dewrich opened a new pull request #2083: updates to the README.md and INSTALL.md for the latest with the Golang proxy

2018-04-04 Thread GitBox
dewrich opened a new pull request #2083: updates to the README.md and INSTALL.md for the latest with the Golang proxy URL: https://github.com/apache/incubator-trafficcontrol/pull/2083 updated for the command line This is an

[GitHub] dewrich opened a new pull request #2084: Updates compare for v13

2018-04-04 Thread GitBox
dewrich opened a new pull request #2084: Updates compare for v13 URL: https://github.com/apache/incubator-trafficcontrol/pull/2084 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] dewrich closed pull request #2084: Updates compare for v13

2018-04-04 Thread GitBox
dewrich closed pull request #2084: Updates compare for v13 URL: https://github.com/apache/incubator-trafficcontrol/pull/2084 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 #1911: Adds roles and capabilities tables to TP

2018-04-04 Thread GitBox
asfgit commented on issue #1911: Adds roles and capabilities tables to TP URL: https://github.com/apache/incubator-trafficcontrol/pull/1911#issuecomment-378663136 Refer to this link for build results (access rights to CI server needed):

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

2018-04-04 Thread Apache Jenkins Server
See Changes: [dewrich] adds roles and capabilities tables to TP -- Started by an SCM change [EnvInject] - Loading node environment variables.

[GitHub] dewrich opened a new pull request #2085: updated the compare tool for v13

2018-04-04 Thread GitBox
dewrich opened a new pull request #2085: updated the compare tool for v13 URL: https://github.com/apache/incubator-trafficcontrol/pull/2085 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] asfgit commented on issue #2085: updated the compare tool for v13

2018-04-04 Thread GitBox
asfgit commented on issue #2085: updated the compare tool for v13 URL: https://github.com/apache/incubator-trafficcontrol/pull/2085#issuecomment-378677497 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2086: Api 1.3 cleanup per api guidelines

2018-04-04 Thread GitBox
asfgit commented on issue #2086: Api 1.3 cleanup per api guidelines URL: https://github.com/apache/incubator-trafficcontrol/pull/2086#issuecomment-378705506 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 opened a new pull request #2086: Api 1.3 cleanup per api guidelines

2018-04-04 Thread GitBox
mitchell852 opened a new pull request #2086: Api 1.3 cleanup per api guidelines URL: https://github.com/apache/incubator-trafficcontrol/pull/2086 This is an automated message from the Apache Git Service. To respond to the

[GitHub] mitchell852 commented on a change in pull request #2086: Api 1.3 cleanup per api guidelines

2018-04-04 Thread GitBox
mitchell852 commented on a change in pull request #2086: Api 1.3 cleanup per api guidelines URL: https://github.com/apache/incubator-trafficcontrol/pull/2086#discussion_r179276674 ## File path: traffic_ops/traffic_ops_golang/routes.go ## @@ -71,55 +71,35 @@ func Routes(d

[GitHub] asfgit commented on issue #2086: Api 1.3 cleanup per api guidelines

2018-04-04 Thread GitBox
asfgit commented on issue #2086: Api 1.3 cleanup per api guidelines URL: https://github.com/apache/incubator-trafficcontrol/pull/2086#issuecomment-378740421 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2081: Go support for /profileparameters

2018-04-04 Thread GitBox
asfgit commented on issue #2081: Go support for /profileparameters URL: https://github.com/apache/incubator-trafficcontrol/pull/2081#issuecomment-378725857 Refer to this link for build results (access rights to CI server needed):

[GitHub] dangogh commented on issue #1889: Add TO Golang CRConfig generation

2018-04-04 Thread GitBox
dangogh commented on issue #1889: Add TO Golang CRConfig generation URL: https://github.com/apache/incubator-trafficcontrol/pull/1889#issuecomment-378741178 I get compile errors: missing a file, maybe? ``` $ go build #

[GitHub] asfgit commented on issue #1889: Add TO Golang CRConfig generation

2018-04-04 Thread GitBox
asfgit commented on issue #1889: Add TO Golang CRConfig generation URL: https://github.com/apache/incubator-trafficcontrol/pull/1889#issuecomment-378744495 Refer to this link for build results (access rights to CI server needed):