[GitHub] mitchell852 commented on issue #2099: TP changelog data enhancement

2018-04-11 Thread GitBox
mitchell852 commented on issue #2099: TP changelog data enhancement URL: https://github.com/apache/incubator-trafficcontrol/issues/2099#issuecomment-380466042 I think the whole change log needs to be revisited in my opinion. It is not robust enough but yes, at the very least, the current

[GitHub] dewrich opened a new pull request #2115: Created API tests for /phys_locations

2018-04-11 Thread GitBox
dewrich opened a new pull request #2115: Created API tests for /phys_locations URL: https://github.com/apache/incubator-trafficcontrol/pull/2115 This is an automated message from the Apache Git Service. To respond to the

[GitHub] dewrich commented on issue #2111: updated the changelog format for TC issue: #2099

2018-04-11 Thread GitBox
dewrich commented on issue #2111: updated the changelog format for TC issue: #2099 URL: https://github.com/apache/incubator-trafficcontrol/pull/2111#issuecomment-380522148 @ezelkow1 This look good to you? This is an

[GitHub] mitchell852 closed pull request #2113: Created the API Tests for /statuses

2018-04-11 Thread GitBox
mitchell852 closed pull request #2113: Created the API Tests for /statuses URL: https://github.com/apache/incubator-trafficcontrol/pull/2113 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

[GitHub] dewrich opened a new pull request #2112: Updated the DST test cases to make remote calls for foreign key needs

2018-04-11 Thread GitBox
dewrich opened a new pull request #2112: Updated the DST test cases to make remote calls for foreign key needs URL: https://github.com/apache/incubator-trafficcontrol/pull/2112 This is an automated message from the Apache

[GitHub] dewrich opened a new pull request #2111: updated the changelog format for TC issue: #2099

2018-04-11 Thread GitBox
dewrich opened a new pull request #2111: updated the changelog format for TC issue: #2099 URL: https://github.com/apache/incubator-trafficcontrol/pull/2111 Fixes #2099 This is an automated message from the Apache Git

[GitHub] dewrich opened a new pull request #2113: Created the API Tests for /statuses

2018-04-11 Thread GitBox
dewrich opened a new pull request #2113: Created the API Tests for /statuses URL: https://github.com/apache/incubator-trafficcontrol/pull/2113 This is an automated message from the Apache Git Service. To respond to the

[GitHub] mitchell852 closed pull request #2115: Created API tests for /phys_locations

2018-04-11 Thread GitBox
mitchell852 closed pull request #2115: Created API tests for /phys_locations URL: https://github.com/apache/incubator-trafficcontrol/pull/2115 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] MattMills opened a new issue #2109: Feature Request: Traffic Stats should tag data with tenant

2018-04-11 Thread GitBox
MattMills opened a new issue #2109: Feature Request: Traffic Stats should tag data with tenant URL: https://github.com/apache/incubator-trafficcontrol/issues/2109 Traffic stats should tag data with tenant names so we can aggregate data by tenant.

[GitHub] dewrich opened a new pull request #2114: cleaned up the fixtures to align with the cdns more appropriately

2018-04-11 Thread GitBox
dewrich opened a new pull request #2114: cleaned up the fixtures to align with the cdns more appropriately URL: https://github.com/apache/incubator-trafficcontrol/pull/2114 This is an automated message from the Apache Git

[GitHub] dangogh opened a new pull request #2116: TO go: `go get` prior to running unit tests for the golang.org/x dependencies

2018-04-11 Thread GitBox
dangogh opened a new pull request #2116: TO go: `go get` prior to running unit tests for the golang.org/x dependencies URL: https://github.com/apache/incubator-trafficcontrol/pull/2116 unit tests in jenkins started failing when `golang.org/x` dependencies were removed from the vendor dir.

[GitHub] dangogh opened a new pull request #2110: TO go: allow orderby to be multiple columns separated by ", "

2018-04-11 Thread GitBox
dangogh opened a new pull request #2110: TO go: allow orderby to be multiple columns separated by "," URL: https://github.com/apache/incubator-trafficcontrol/pull/2110 still in progress -- testing needed. Also need to add missing json id to column mappings that are missing on many of

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

2018-04-11 Thread GitBox
rawlinp commented on issue #2029: [Issue 1907] TO API for backup edge cachegroup URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-380512898 @Vijay-1 - I just wanted to make sure you were aware that the cachegroup API was recently rewritten in Go if you wanted

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

2018-04-11 Thread Apache Jenkins Server
See Changes: [mitchell852] created the PhysLocations API Tests [mitchell852] updated to attach the RegionID [mitchell852] cleaned up the remaining inserts because API tests exist now

[GitHub] dewrich opened a new pull request #2118: cleaned up the teardown functions so each test case works as a unit

2018-04-11 Thread GitBox
dewrich opened a new pull request #2118: cleaned up the teardown functions so each test case works as a unit URL: https://github.com/apache/incubator-trafficcontrol/pull/2118 This is an automated message from the Apache Git

[GitHub] asfgit commented on issue #2114: cleaned up the fixtures to align with the cdns more appropriately

2018-04-11 Thread GitBox
asfgit commented on issue #2114: cleaned up the fixtures to align with the cdns more appropriately URL: https://github.com/apache/incubator-trafficcontrol/pull/2114#issuecomment-380538947 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 closed pull request #2112: Updated the DSR test cases to make remote calls for foreign key needs

2018-04-11 Thread GitBox
mitchell852 closed pull request #2112: Updated the DSR test cases to make remote calls for foreign key needs URL: https://github.com/apache/incubator-trafficcontrol/pull/2112 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below

[GitHub] mitchell852 closed issue #2099: TO changelog data enhancement

2018-04-11 Thread GitBox
mitchell852 closed issue #2099: TO changelog data enhancement URL: https://github.com/apache/incubator-trafficcontrol/issues/2099 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] mitchell852 closed pull request #2111: updated the changelog format for TC issue: #2099

2018-04-11 Thread GitBox
mitchell852 closed pull request #2111: updated the changelog format for TC issue: #2099 URL: https://github.com/apache/incubator-trafficcontrol/pull/2111 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] asfgit commented on issue #2113: Created the API Tests for /statuses

2018-04-11 Thread GitBox
asfgit commented on issue #2113: Created the API Tests for /statuses URL: https://github.com/apache/incubator-trafficcontrol/pull/2113#issuecomment-380563955 Refer to this link for build results (access rights to CI server needed):

[GitHub] dewrich opened a new pull request #2117: removed status comments and fixed the test cases

2018-04-11 Thread GitBox
dewrich opened a new pull request #2117: removed status comments and fixed the test cases URL: https://github.com/apache/incubator-trafficcontrol/pull/2117 This is an automated message from the Apache Git Service. To

[GitHub] asfgit commented on issue #2110: TO go: allow orderby to be multiple columns separated by ", "

2018-04-11 Thread GitBox
asfgit commented on issue #2110: TO go: allow orderby to be multiple columns separated by "," URL: https://github.com/apache/incubator-trafficcontrol/pull/2110#issuecomment-380527792 Refer to this link for build results (access rights to CI server needed):

[GitHub] ezelkow1 commented on issue #2111: updated the changelog format for TC issue: #2099

2018-04-11 Thread GitBox
ezelkow1 commented on issue #2111: updated the changelog format for TC issue: #2099 URL: https://github.com/apache/incubator-trafficcontrol/pull/2111#issuecomment-380529492 Looks good, should at least get us information and which server had a change happen. We can revisit more changelog

[GitHub] ezelkow1 commented on issue #2111: updated the changelog format for TC issue: #2099

2018-04-11 Thread GitBox
ezelkow1 commented on issue #2111: updated the changelog format for TC issue: #2099 URL: https://github.com/apache/incubator-trafficcontrol/pull/2111#issuecomment-380529492 Looks good, should at least get us information on which server had a change happen. We can revisit more changelog

[GitHub] mitchell852 closed pull request #2114: cleaned up the fixtures to align with the cdns more appropriately

2018-04-11 Thread GitBox
mitchell852 closed pull request #2114: cleaned up the fixtures to align with the cdns more appropriately URL: https://github.com/apache/incubator-trafficcontrol/pull/2114 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

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

2018-04-11 Thread Apache Jenkins Server
See Changes: [mitchell852] cleaned up the fixtures to align with the cdns more appropriately [mitchell852] updated the changelog format for TC issue: #2099 [mitchell852] updated to

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

2018-04-11 Thread GitBox
rob05c commented on issue #2124: Add TO Go GET deliveryservices routes URL: https://github.com/apache/incubator-trafficcontrol/pull/2124#issuecomment-380616330 @DylanVolz What do you think of this versioning scheme? The highlights are: Structs are versioned via anonymous

[GitHub] asfgit commented on issue #2115: Created API tests for /phys_locations

2018-04-11 Thread GitBox
asfgit commented on issue #2115: Created API tests for /phys_locations URL: https://github.com/apache/incubator-trafficcontrol/pull/2115#issuecomment-380622743 Refer to this link for build results (access rights to CI server needed):

[GitHub] rob05c opened a new pull request #2124: Add TO Go GET deliveryservices routes

2018-04-11 Thread GitBox
rob05c opened a new pull request #2124: Add TO Go GET deliveryservices routes URL: https://github.com/apache/incubator-trafficcontrol/pull/2124 Also versions the deliveryservice structs and API. This is an automated message

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

2018-04-11 Thread GitBox
rob05c commented on issue #2124: Add TO Go GET deliveryservices routes URL: https://github.com/apache/incubator-trafficcontrol/pull/2124#issuecomment-380616330 @DylanVolz What do you think of this versioning scheme? The highlights are: Structs are versioned via anonymous

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

2018-04-11 Thread GitBox
rob05c commented on issue #2124: Add TO Go GET deliveryservices routes URL: https://github.com/apache/incubator-trafficcontrol/pull/2124#issuecomment-380616330 @DylanVolz What do you think of this versioning scheme? The highlights are: Structs are versioned via anonymous

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

2018-04-11 Thread GitBox
rob05c commented on issue #2124: Add TO Go GET deliveryservices routes URL: https://github.com/apache/incubator-trafficcontrol/pull/2124#issuecomment-380616330 @DylanVolz What do you think of this versioning scheme? The highlights are: Structs are versioned via anonymous

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

2018-04-11 Thread GitBox
rob05c commented on issue #2124: Add TO Go GET deliveryservices routes URL: https://github.com/apache/incubator-trafficcontrol/pull/2124#issuecomment-380617535 Tested against our production delivery services. I started to write API tests, but the API test framework doesn't look

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

2018-04-11 Thread GitBox
rob05c commented on issue #2124: Add TO Go GET deliveryservices routes URL: https://github.com/apache/incubator-trafficcontrol/pull/2124#issuecomment-380616330 @DylanVolz What do you think of this versioning scheme? The highlights are: Structs are versioned via anonymous

[GitHub] asfgit commented on issue #2112: Updated the DSR test cases to make remote calls for foreign key needs

2018-04-11 Thread GitBox
asfgit commented on issue #2112: Updated the DSR test cases to make remote calls for foreign key needs URL: https://github.com/apache/incubator-trafficcontrol/pull/2112#issuecomment-380573533 Refer to this link for build results (access rights to CI server needed):

[GitHub] dewrich closed pull request #2120: Adds fqpacingrate key to ds structs

2018-04-11 Thread GitBox
dewrich closed pull request #2120: Adds fqpacingrate key to ds structs URL: https://github.com/apache/incubator-trafficcontrol/pull/2120 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

[GitHub] dewrich closed issue #2013: ds.fqPacingRate value not accounted for with delivery service requests

2018-04-11 Thread GitBox
dewrich closed issue #2013: ds.fqPacingRate value not accounted for with delivery service requests URL: https://github.com/apache/incubator-trafficcontrol/issues/2013 This is an automated message from the Apache Git

[GitHub] mitchell852 opened a new pull request #2120: Adds fqpacingrate key to ds structs

2018-04-11 Thread GitBox
mitchell852 opened a new pull request #2120: Adds fqpacingrate key to ds structs URL: https://github.com/apache/incubator-trafficcontrol/pull/2120 fixes #2013 This is an automated message from the Apache Git Service. To

[GitHub] rawlinp opened a new pull request #2123: Fix broken link in MSO quick-how-to doc

2018-04-11 Thread GitBox
rawlinp opened a new pull request #2123: Fix broken link in MSO quick-how-to doc URL: https://github.com/apache/incubator-trafficcontrol/pull/2123 This is an automated message from the Apache Git Service. To respond to the

[GitHub] mitchell852 closed pull request #2118: cleaned up the teardown functions so each test case works as a unit

2018-04-11 Thread GitBox
mitchell852 closed pull request #2118: cleaned up the teardown functions so each test case works as a unit URL: https://github.com/apache/incubator-trafficcontrol/pull/2118 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[GitHub] mitchell852 closed pull request #2119: Fixed the DSR tests by attaching CDNs and Types to the DSRs

2018-04-11 Thread GitBox
mitchell852 closed pull request #2119: Fixed the DSR tests by attaching CDNs and Types to the DSRs URL: https://github.com/apache/incubator-trafficcontrol/pull/2119 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] mitchell852 closed pull request #2121: fixed the package reference for CacheGroups

2018-04-11 Thread GitBox
mitchell852 closed pull request #2121: fixed the package reference for CacheGroups URL: https://github.com/apache/incubator-trafficcontrol/pull/2121 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-traffic_ops-test #536

2018-04-11 Thread Apache Jenkins Server
See Changes: [mitchell852] cleaned up the teardown functions so each test case works as a unit [mitchell852] attached CDNs and Types to the DSRs [dewrich] alphabetizes ds stuct because

[GitHub] dewrich opened a new pull request #2122: Created Swagger documentation for /phys_locations

2018-04-11 Thread GitBox
dewrich opened a new pull request #2122: Created Swagger documentation for /phys_locations URL: https://github.com/apache/incubator-trafficcontrol/pull/2122 This is an automated message from the Apache Git Service. To

[GitHub] asfgit commented on issue #2111: updated the changelog format for TC issue: #2099

2018-04-11 Thread GitBox
asfgit commented on issue #2111: updated the changelog format for TC issue: #2099 URL: https://github.com/apache/incubator-trafficcontrol/pull/2111#issuecomment-380601467 Refer to this link for build results (access rights to CI server needed):

[GitHub] dewrich opened a new pull request #2119: Fixed the DSR tests by attaching CDNs and Types to the DSRs

2018-04-11 Thread GitBox
dewrich opened a new pull request #2119: Fixed the DSR tests by attaching CDNs and Types to the DSRs URL: https://github.com/apache/incubator-trafficcontrol/pull/2119 This is an automated message from the Apache Git

[GitHub] dewrich opened a new pull request #2121: fixed the package reference for CacheGroups

2018-04-11 Thread GitBox
dewrich opened a new pull request #2121: fixed the package reference for CacheGroups URL: https://github.com/apache/incubator-trafficcontrol/pull/2121 This is an automated message from the Apache Git Service. To respond to

[GitHub] asfgit commented on issue #2117: removed status comments and fixed the test cases

2018-04-11 Thread GitBox
asfgit commented on issue #2117: removed status comments and fixed the test cases URL: https://github.com/apache/incubator-trafficcontrol/pull/2117#issuecomment-380647911 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2116: TO go: `go get` prior to running unit tests for the golang.org/x dependencies

2018-04-11 Thread GitBox
asfgit commented on issue #2116: TO go: `go get` prior to running unit tests for the golang.org/x dependencies URL: https://github.com/apache/incubator-trafficcontrol/pull/2116#issuecomment-380638860 Refer to this link for build results (access rights to CI server needed):

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

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

[GitHub] asfgit commented on issue #2121: fixed the package reference for CacheGroups

2018-04-11 Thread GitBox
asfgit commented on issue #2121: fixed the package reference for CacheGroups URL: https://github.com/apache/incubator-trafficcontrol/pull/2121#issuecomment-380675067 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2121: fixed the package reference for CacheGroups

2018-04-11 Thread GitBox
asfgit commented on issue #2121: fixed the package reference for CacheGroups URL: https://github.com/apache/incubator-trafficcontrol/pull/2121#issuecomment-380675239 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2119: Fixed the DSR tests by attaching CDNs and Types to the DSRs

2018-04-11 Thread GitBox
asfgit commented on issue #2119: Fixed the DSR tests by attaching CDNs and Types to the DSRs URL: https://github.com/apache/incubator-trafficcontrol/pull/2119#issuecomment-380674973 Refer to this link for build results (access rights to CI server needed):

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

2018-04-11 Thread Apache Jenkins Server
See -- GitHub pull request #2121 of commit ca20c837321b37765ef5354621e888edda76b38b, no merge conflicts. [EnvInject] - Loading node environment variables. Building remotely

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

2018-04-11 Thread Apache Jenkins Server
See -- GitHub pull request #2121 of commit 89dbac98a8defe313570971aabe315aaac9fe33e, no merge conflicts. [EnvInject] - Loading node environment variables. Building remotely

[GitHub] asfgit commented on issue #2121: fixed the package reference for CacheGroups

2018-04-11 Thread GitBox
asfgit commented on issue #2121: fixed the package reference for CacheGroups URL: https://github.com/apache/incubator-trafficcontrol/pull/2121#issuecomment-380675692 Refer to this link for build results (access rights to CI server needed):

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

2018-04-11 Thread Apache Jenkins Server
See

[GitHub] asfgit commented on issue #2122: Created Swagger documentation for /phys_locations

2018-04-11 Thread GitBox
asfgit commented on issue #2122: Created Swagger documentation for /phys_locations URL: https://github.com/apache/incubator-trafficcontrol/pull/2122#issuecomment-380676125 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2123: Fix broken link in MSO quick-how-to doc

2018-04-11 Thread GitBox
asfgit commented on issue #2123: Fix broken link in MSO quick-how-to doc URL: https://github.com/apache/incubator-trafficcontrol/pull/2123#issuecomment-380676517 Refer to this link for build results (access rights to CI server needed):

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

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

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

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

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

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

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

2018-04-11 Thread Apache Jenkins Server
See -- Started by upstream project "incubator-trafficcontrol-PR" build number 1399 originally caused by: GitHub pull request #2124 of commit

[GitHub] asfgit commented on issue #2103: Add a Location API

2018-04-11 Thread GitBox
asfgit commented on issue #2103: Add a Location API URL: https://github.com/apache/incubator-trafficcontrol/pull/2103#issuecomment-380678524 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-trafficcontrol-PR/1400/

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

2018-04-11 Thread Apache Jenkins Server
See

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

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

[GitHub] asfgit commented on issue #2118: cleaned up the teardown functions so each test case works as a unit

2018-04-11 Thread GitBox
asfgit commented on issue #2118: cleaned up the teardown functions so each test case works as a unit URL: https://github.com/apache/incubator-trafficcontrol/pull/2118#issuecomment-380668299 Refer to this link for build results (access rights to CI server needed):