[GitHub] dewrich opened a new pull request #2093: Added functionality to attach parameters to the profile response if an id is sent

2018-04-06 Thread GitBox
dewrich opened a new pull request #2093: Added functionality to attach parameters to the profile response if an id is sent URL: https://github.com/apache/incubator-trafficcontrol/pull/2093 This is an automated message from

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

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

[GitHub] asfgit commented on issue #2093: Added functionality to attach parameters to the profile response if an id is sent

2018-04-06 Thread GitBox
asfgit commented on issue #2093: Added functionality to attach parameters to the profile response if an id is sent URL: https://github.com/apache/incubator-trafficcontrol/pull/2093#issuecomment-379361973 Refer to this link for build results (access rights to CI server needed):

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

2018-04-06 Thread Apache Jenkins Server
See -- GitHub pull request #2093 of commit 2fabbabb37b47ea0bacd3647e0fbca7ee5c3a30b, no merge conflicts. Setting status of 2fabbabb37b47ea0bacd3647e0fbca7ee5c3a30b to

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

2018-04-06 Thread Apache Jenkins Server
See Changes: [mitchell852] updates to the README.md and INSTALL.md for the latest changes -- Started by an SCM change Started by an SCM change

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

2018-04-06 Thread Apache Jenkins Server
See Changes: [mitchell852] work in progress until CRUD Interfaces support compound keys [mitchell852] hooked in the profileparameters routes with a successful Read [mitchell852] fixed

[GitHub] dewrich closed pull request #2089: Makes api calls for ds requests using query params instead of route p…

2018-04-06 Thread GitBox
dewrich closed pull request #2089: Makes api calls for ds requests using query params instead of route p… URL: https://github.com/apache/incubator-trafficcontrol/pull/2089 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 #527

2018-04-06 Thread Apache Jenkins Server
See Changes: [dewrich] makes api calls for ds requests using query params instead of route -- Started by an SCM change [EnvInject] - Loading node

[GitHub] asfgit commented on issue #2093: Added functionality to attach parameters to the profile response if an id is sent

2018-04-06 Thread GitBox
asfgit commented on issue #2093: Added functionality to attach parameters to the profile response if an id is sent URL: https://github.com/apache/incubator-trafficcontrol/pull/2093#issuecomment-379357827 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 closed pull request #2092: Add db last_updated not null

2018-04-06 Thread GitBox
mitchell852 closed pull request #2092: Add db last_updated not null URL: https://github.com/apache/incubator-trafficcontrol/pull/2092 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

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

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

[GitHub] mitchell852 opened a new pull request #2094: removes path params from ds request endpoints per api guidelines

2018-04-06 Thread GitBox
mitchell852 opened a new pull request #2094: removes path params from ds request endpoints per api guidelines URL: https://github.com/apache/incubator-trafficcontrol/pull/2094 This is an automated message from the Apache

[GitHub] mitchell852 commented on a change in pull request #2094: removes path params from ds request endpoints per api guidelines

2018-04-06 Thread GitBox
mitchell852 commented on a change in pull request #2094: removes path params from ds request endpoints per api guidelines URL: https://github.com/apache/incubator-trafficcontrol/pull/2094#discussion_r179878201 ## File path: traffic_ops/traffic_ops_golang/routes.go ## @@

[GitHub] rob05c opened a new pull request #2092: Add db last_updated not null

2018-04-06 Thread GitBox
rob05c opened a new pull request #2092: Add db last_updated not null URL: https://github.com/apache/incubator-trafficcontrol/pull/2092 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] asfgit commented on issue #2093: Added functionality to attach parameters to the profile response if an id is sent

2018-04-06 Thread GitBox
asfgit commented on issue #2093: Added functionality to attach parameters to the profile response if an id is sent URL: https://github.com/apache/incubator-trafficcontrol/pull/2093#issuecomment-379363045 Refer to this link for build results (access rights to CI server needed):

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

2018-04-06 Thread Apache Jenkins Server
See -- GitHub pull request #2093 of commit d8cae075255d7e95d8e4952be525c83db566d3d3, has merge conflicts. !!! PR mergeability status has changed !!! PR now has NO merge

[GitHub] asfgit commented on issue #2093: Added functionality to attach parameters to the profile response if an id is sent

2018-04-06 Thread GitBox
asfgit commented on issue #2093: Added functionality to attach parameters to the profile response if an id is sent URL: https://github.com/apache/incubator-trafficcontrol/pull/2093#issuecomment-379366807 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 closed pull request #2081: Go support for /profileparameters

2018-04-06 Thread GitBox
mitchell852 closed pull request #2081: Go support for /profileparameters URL: https://github.com/apache/incubator-trafficcontrol/pull/2081 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] mitchell852 commented on a change in pull request #2093: Added functionality to attach parameters to the profile response if an id is sent

2018-04-06 Thread GitBox
mitchell852 commented on a change in pull request #2093: Added functionality to attach parameters to the profile response if an id is sent URL: https://github.com/apache/incubator-trafficcontrol/pull/2093#discussion_r179871772 ## File path: traffic_ops/traffic_ops_golang/routes.go

[GitHub] asfgit commented on issue #2092: Add db last_updated not null

2018-04-06 Thread GitBox
asfgit commented on issue #2092: Add db last_updated not null URL: https://github.com/apache/incubator-trafficcontrol/pull/2092#issuecomment-379336308 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 closed pull request #2083: updates to the README.md and INSTALL.md for the latest with the Golang proxy

2018-04-06 Thread GitBox
mitchell852 closed 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 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

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

2018-04-06 Thread Apache Jenkins Server
See Changes: [mitchell852] Add db last_updated not null -- Started by an SCM change [EnvInject] - Loading node environment variables. Building

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

2018-04-06 Thread Apache Jenkins Server
See

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

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

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

2018-04-06 Thread Apache Jenkins Server
See -- GitHub pull request #2081 of commit a1ca8e1a1f27be9f63d80617b8770b37a588d17d, has merge conflicts. !!! PR mergeability status has changed !!! PR now has NO merge

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

2018-04-06 Thread Apache Jenkins Server
See -- GitHub pull request #2093 of commit deec24fd89384645a770a5ebb1a65a9686cef668, no merge conflicts. Setting status of deec24fd89384645a770a5ebb1a65a9686cef668 to

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

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

[GitHub] asfgit commented on issue #2093: Added functionality to attach parameters to the profile response if an id is sent

2018-04-06 Thread GitBox
asfgit commented on issue #2093: Added functionality to attach parameters to the profile response if an id is sent URL: https://github.com/apache/incubator-trafficcontrol/pull/2093#issuecomment-379363706 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2095: ORT updates .cer files if generated file would be different, including order

2018-04-06 Thread GitBox
asfgit commented on issue #2095: ORT updates .cer files if generated file would be different, including order URL: https://github.com/apache/incubator-trafficcontrol/pull/2095#issuecomment-379415118 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-06 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-379391403 Refer to this link for build results (access rights to CI server needed):

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

2018-04-06 Thread Apache Jenkins Server
See -- GitHub pull request #2080 of commit 8d8c204ebe722234cad9e82fb4bd1d2e0d103f4f, has merge conflicts. Setting status of 8d8c204ebe722234cad9e82fb4bd1d2e0d103f4f to

[GitHub] asfgit commented on issue #2094: removes path params from ds request endpoints per api guidelines

2018-04-06 Thread GitBox
asfgit commented on issue #2094: removes path params from ds request endpoints per api guidelines URL: https://github.com/apache/incubator-trafficcontrol/pull/2094#issuecomment-379395777 Refer to this link for build results (access rights to CI server needed):

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

2018-04-06 Thread Apache Jenkins Server
See

[GitHub] dangogh opened a new pull request #2095: ORT updates .cer files if generated file would be different, including order

2018-04-06 Thread GitBox
dangogh opened a new pull request #2095: ORT updates .cer files if generated file would be different, including order URL: https://github.com/apache/incubator-trafficcontrol/pull/2095 This fixes #2032 @jt `xxx.cer` files and `logs_xml.config` must be regenerated if generated file

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

2018-04-06 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-379399229 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2089: Makes api calls for ds requests using query params instead of route p…

2018-04-06 Thread GitBox
asfgit commented on issue #2089: Makes api calls for ds requests using query params instead of route p… URL: https://github.com/apache/incubator-trafficcontrol/pull/2089#issuecomment-379389398 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-06 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-379427260 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 opened a new pull request #2089: TP - makes api calls for ds requests using query params instead of route p…

2018-04-06 Thread GitBox
mitchell852 opened a new pull request #2089: TP - makes api calls for ds requests using query params instead of route p… URL: https://github.com/apache/incubator-trafficcontrol/pull/2089 …arams This is an automated message

[GitHub] mitchell852 opened a new pull request #2090: TO API - organizes routes by version

2018-04-06 Thread GitBox
mitchell852 opened a new pull request #2090: TO API - organizes routes by version URL: https://github.com/apache/incubator-trafficcontrol/pull/2090 also, introduces 2.0 routes that dont include route params This is an

[GitHub] asfgit commented on issue #2090: TO API - organizes routes by version

2018-04-06 Thread GitBox
asfgit commented on issue #2090: TO API - organizes routes by version URL: https://github.com/apache/incubator-trafficcontrol/pull/2090#issuecomment-379278127 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2090: TO API - organizes routes by version

2018-04-06 Thread GitBox
asfgit commented on issue #2090: TO API - organizes routes by version URL: https://github.com/apache/incubator-trafficcontrol/pull/2090#issuecomment-379288961 Refer to this link for build results (access rights to CI server needed):

[GitHub] mitchell852 closed pull request #2086: Api 1.3 cleanup per api guidelines

2018-04-06 Thread GitBox
mitchell852 closed pull request #2086: Api 1.3 cleanup per api guidelines URL: https://github.com/apache/incubator-trafficcontrol/pull/2086 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] asfgit commented on issue #2089: TP - makes api calls for ds requests using query params instead of route p…

2018-04-06 Thread GitBox
asfgit commented on issue #2089: TP - makes api calls for ds requests using query params instead of route p… URL: https://github.com/apache/incubator-trafficcontrol/pull/2089#issuecomment-379271823 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2090: TO API - organizes routes by version

2018-04-06 Thread GitBox
asfgit commented on issue #2090: TO API - organizes routes by version URL: https://github.com/apache/incubator-trafficcontrol/pull/2090#issuecomment-379303403 Refer to this link for build results (access rights to CI server needed):

[GitHub] rob05c commented on issue #2091: Fix TM to warn not err on matchset without regex

2018-04-06 Thread GitBox
rob05c commented on issue #2091: Fix TM to warn not err on matchset without regex URL: https://github.com/apache/incubator-trafficcontrol/pull/2091#issuecomment-379303308 This specifically fixes the Monitor to keep going, when there's a data bug in Traffic Ops / the CRConfig. We've

[GitHub] asfgit commented on issue #2091: Fix TM to warn not err on matchset without regex

2018-04-06 Thread GitBox
asfgit commented on issue #2091: Fix TM to warn not err on matchset without regex URL: https://github.com/apache/incubator-trafficcontrol/pull/2091#issuecomment-379304649 Refer to this link for build results (access rights to CI server needed):

[GitHub] rob05c opened a new pull request #2091: Fix TM to warn not err on matchset without regex

2018-04-06 Thread GitBox
rob05c opened a new pull request #2091: Fix TM to warn not err on matchset without regex URL: https://github.com/apache/incubator-trafficcontrol/pull/2091 Changes the Monitor to warn instead of erroring, if a delivery service has a null matchlist in the CRConfig, but has other valid

[GitHub] asfgit commented on issue #2090: TO API - organizes routes by version

2018-04-06 Thread GitBox
asfgit commented on issue #2090: TO API - organizes routes by version URL: https://github.com/apache/incubator-trafficcontrol/pull/2090#issuecomment-379319889 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #2090: TO API - organizes routes by version

2018-04-06 Thread GitBox
asfgit commented on issue #2090: TO API - organizes routes by version URL: https://github.com/apache/incubator-trafficcontrol/pull/2090#issuecomment-379329840 Refer to this link for build results (access rights to CI server needed):