[GitHub] [trafficcontrol] mhoppa opened a new pull request #4350: Up TP test wait time for key generation

2020-01-29 Thread GitBox
mhoppa opened a new pull request #4350: Up TP test wait time for key generation URL: https://github.com/apache/trafficcontrol/pull/4350 ## What does this PR (Pull Request) do? - [x] This PR is not related to any Issue We recently up the TP wait time on the key

[GitHub] [trafficcontrol] ocket8888 merged pull request #4327: Rewrite to_extension POST

2020-01-29 Thread GitBox
ocket merged pull request #4327: Rewrite to_extension POST URL: https://github.com/apache/trafficcontrol/pull/4327 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [trafficcontrol] ocket8888 closed issue #3859: Rewrite /riak/stats to Go

2020-01-29 Thread GitBox
ocket closed issue #3859: Rewrite /riak/stats to Go URL: https://github.com/apache/trafficcontrol/issues/3859 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [trafficcontrol] ocket8888 merged pull request #4349: Deprecate /riak/stats

2020-01-29 Thread GitBox
ocket merged pull request #4349: Deprecate /riak/stats URL: https://github.com/apache/trafficcontrol/pull/4349 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [trafficcontrol] rob05c commented on issue #4344: Fix TO atscfg missing max origin conns

2020-01-29 Thread GitBox
rob05c commented on issue #4344: Fix TO atscfg missing max origin conns URL: https://github.com/apache/trafficcontrol/pull/4344#issuecomment-579818240 @mitchell852 Ah, the Mid query had the same bug. I fixed that too, both bugs should be fixed now.

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4348: Deprecate GET jobs/{id}

2020-01-29 Thread GitBox
ocket commented on a change in pull request #4348: Deprecate GET jobs/{id} URL: https://github.com/apache/trafficcontrol/pull/4348#discussion_r372469327 ## File path: docs/source/api/jobs_id.rst ## @@ -18,6 +18,7 @@ *** ``jobs/{{ID}}`` ***

[GitHub] [trafficcontrol] mitchell852 merged pull request #4347: Fix internal server error in user current put

2020-01-29 Thread GitBox
mitchell852 merged pull request #4347: Fix internal server error in user current put URL: https://github.com/apache/trafficcontrol/pull/4347 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [trafficcontrol] mitchell852 closed issue #4241: Internal Server Error when updating user's password using /user/current endpoint

2020-01-29 Thread GitBox
mitchell852 closed issue #4241: Internal Server Error when updating user's password using /user/current endpoint URL: https://github.com/apache/trafficcontrol/issues/4241 This is an automated message from the Apache Git

[GitHub] [trafficcontrol] mitchell852 closed issue #3827: Rewrite /servers/checks to Go

2020-01-29 Thread GitBox
mitchell852 closed issue #3827: Rewrite /servers/checks to Go URL: https://github.com/apache/trafficcontrol/issues/3827 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [trafficcontrol] mitchell852 merged pull request #4325: Servers/checks go

2020-01-29 Thread GitBox
mitchell852 merged pull request #4325: Servers/checks go URL: https://github.com/apache/trafficcontrol/pull/4325 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [trafficcontrol] mitchell852 merged pull request #4350: Up TP test wait time for key generation

2020-01-29 Thread GitBox
mitchell852 merged pull request #4350: Up TP test wait time for key generation URL: https://github.com/apache/trafficcontrol/pull/4350 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [trafficcontrol] ocket8888 merged pull request #4173: Rewrite stats summary post to golang

2020-01-29 Thread GitBox
ocket merged pull request #4173: Rewrite stats summary post to golang URL: https://github.com/apache/trafficcontrol/pull/4173 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [trafficcontrol] mitchell852 opened a new issue #4351: Provide a better message on DS create/update/delete to indicate if a CDN snapshot and/or queue updates is required

2020-01-29 Thread GitBox
mitchell852 opened a new issue #4351: Provide a better message on DS create/update/delete to indicate if a CDN snapshot and/or queue updates is required URL: https://github.com/apache/trafficcontrol/issues/4351 ## I'm submitting a ... - [ ] bug report - [ ] new feature /

[GitHub] [trafficcontrol] mitchell852 merged pull request #4344: Fix TO atscfg missing max origin conns

2020-01-29 Thread GitBox
mitchell852 merged pull request #4344: Fix TO atscfg missing max origin conns URL: https://github.com/apache/trafficcontrol/pull/4344 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4173: Rewrite stats summary post to golang

2020-01-29 Thread GitBox
ocket commented on a change in pull request #4173: Rewrite stats summary post to golang URL: https://github.com/apache/trafficcontrol/pull/4173#discussion_r372490379 ## File path: CHANGELOG.md ## @@ -11,6 +11,7 @@ The format is based on [Keep a

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4333: Deprecate & Rewrite DELETE to_extension

2020-01-29 Thread GitBox
ocket commented on a change in pull request #4333: Deprecate & Rewrite DELETE to_extension URL: https://github.com/apache/trafficcontrol/pull/4333#discussion_r372502746 ## File path: docs/source/api/to_extensions_id.rst ## @@ -0,0 +1,75 @@ +.. +.. +.. Licensed under

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4333: Deprecate & Rewrite DELETE to_extension

2020-01-29 Thread GitBox
ocket commented on a change in pull request #4333: Deprecate & Rewrite DELETE to_extension URL: https://github.com/apache/trafficcontrol/pull/4333#discussion_r372501543 ## File path: docs/source/api/to_extensions_id_delete.rst ## @@ -18,6 +18,7 @@

[GitHub] [trafficcontrol] ocket8888 commented on issue #4239: Issues when updating a user's email using /user/current endpoint

2020-01-29 Thread GitBox
ocket commented on issue #4239: Issues when updating a user's email using /user/current endpoint URL: https://github.com/apache/trafficcontrol/issues/4239#issuecomment-579854126 Closed by #4346 This is an automated

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and renew API endpoints

2020-01-29 Thread GitBox
ocket commented on a change in pull request #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and renew API endpoints URL: https://github.com/apache/trafficcontrol/pull/3534#discussion_r372555732 ## File path:

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and renew API endpoints

2020-01-29 Thread GitBox
ocket commented on a change in pull request #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and renew API endpoints URL: https://github.com/apache/trafficcontrol/pull/3534#discussion_r372556417 ## File path:

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and renew API endpoints

2020-01-29 Thread GitBox
ocket commented on a change in pull request #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and renew API endpoints URL: https://github.com/apache/trafficcontrol/pull/3534#discussion_r372554861 ## File path: docs/source/api/letsencrypt_dnsrecords.rst

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and renew API endpoints

2020-01-29 Thread GitBox
ocket commented on a change in pull request #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and renew API endpoints URL: https://github.com/apache/trafficcontrol/pull/3534#discussion_r372557468 ## File path: docs/source/admin/traffic_router.rst

[GitHub] [trafficcontrol] mhoppa commented on issue #4173: Rewrite stats summary post to golang

2020-01-29 Thread GitBox
mhoppa commented on issue #4173: Rewrite stats summary post to golang URL: https://github.com/apache/trafficcontrol/pull/4173#issuecomment-579829952 @ocket once you get a chance can you re review?? :) This is an

[GitHub] [trafficcontrol] rob05c commented on a change in pull request #4343: traffic_ops_ort: fix for syncds having too many header rewrite false …

2020-01-29 Thread GitBox
rob05c commented on a change in pull request #4343: traffic_ops_ort: fix for syncds having too many header rewrite false … URL: https://github.com/apache/trafficcontrol/pull/4343#discussion_r372542415 ## File path: traffic_ops/ort/traffic_ops_ort.pl ## @@ -2652,6 +2658,23

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and renew API endpoints

2020-01-29 Thread GitBox
ocket commented on a change in pull request #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and renew API endpoints URL: https://github.com/apache/trafficcontrol/pull/3534#discussion_r372555958 ## File path:

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and renew API endpoints

2020-01-29 Thread GitBox
ocket commented on a change in pull request #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and renew API endpoints URL: https://github.com/apache/trafficcontrol/pull/3534#discussion_r372554279 ## File path:

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and renew API endpoints

2020-01-29 Thread GitBox
ocket commented on a change in pull request #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and renew API endpoints URL: https://github.com/apache/trafficcontrol/pull/3534#discussion_r372556953 ## File path: docs/source/admin/traffic_router.rst

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and renew API endpoints

2020-01-29 Thread GitBox
ocket commented on a change in pull request #3534: TP Delivery Service Generate SSL update, new letsencrypt generate and renew API endpoints URL: https://github.com/apache/trafficcontrol/pull/3534#discussion_r372555091 ## File path: docs/source/api/letsencrypt_autorenew.rst

[GitHub] [trafficcontrol] ocket8888 merged pull request #4348: Deprecate GET jobs/{id}

2020-01-29 Thread GitBox
ocket merged pull request #4348: Deprecate GET jobs/{id} URL: https://github.com/apache/trafficcontrol/pull/4348 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [trafficcontrol] ocket8888 closed issue #3833: Rewrite /stats_summary/create to Go

2020-01-29 Thread GitBox
ocket closed issue #3833: Rewrite /stats_summary/create to Go URL: https://github.com/apache/trafficcontrol/issues/3833 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [trafficcontrol] ocket8888 merged pull request #4186: Add deprecation notices to stats_summary/create

2020-01-29 Thread GitBox
ocket merged pull request #4186: Add deprecation notices to stats_summary/create URL: https://github.com/apache/trafficcontrol/pull/4186 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [trafficcontrol] mitchell852 merged pull request #4346: Fix user current update required email

2020-01-29 Thread GitBox
mitchell852 merged pull request #4346: Fix user current update required email URL: https://github.com/apache/trafficcontrol/pull/4346 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [trafficcontrol] mhoppa commented on issue #4173: Rewrite stats summary post to golang

2020-01-29 Thread GitBox
mhoppa commented on issue #4173: Rewrite stats summary post to golang URL: https://github.com/apache/trafficcontrol/pull/4173#issuecomment-579849903 https://github.com/apache/trafficcontrol/pull/4186 already posted waiting for this to merge let me remove the issue from the pull request

[GitHub] [trafficcontrol] mitchell852 closed issue #4240: Can update a user's username to contain spaces

2020-01-29 Thread GitBox
mitchell852 closed issue #4240: Can update a user's username to contain spaces URL: https://github.com/apache/trafficcontrol/issues/4240 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [trafficcontrol] mhoppa commented on a change in pull request #4333: Deprecate & Rewrite DELETE to_extension

2020-01-29 Thread GitBox
mhoppa commented on a change in pull request #4333: Deprecate & Rewrite DELETE to_extension URL: https://github.com/apache/trafficcontrol/pull/4333#discussion_r372511532 ## File path: docs/source/api/to_extensions_id.rst ## @@ -0,0 +1,75 @@ +.. +.. +.. Licensed under the

[GitHub] [trafficcontrol] mhoppa commented on a change in pull request #4333: Deprecate & Rewrite DELETE to_extension

2020-01-29 Thread GitBox
mhoppa commented on a change in pull request #4333: Deprecate & Rewrite DELETE to_extension URL: https://github.com/apache/trafficcontrol/pull/4333#discussion_r372511483 ## File path: docs/source/api/to_extensions_id_delete.rst ## @@ -18,6 +18,7 @@

[GitHub] [trafficcontrol] ocket8888 commented on issue #4290: Undocumented Traffic Ops Golang routes

2020-01-29 Thread GitBox
ocket commented on issue #4290: Undocumented Traffic Ops Golang routes URL: https://github.com/apache/trafficcontrol/issues/4290#issuecomment-579864890 `divisions/name/{{name}}` is deprecated, so it doesn't need docs This

[GitHub] [trafficcontrol] rob05c commented on a change in pull request #4343: traffic_ops_ort: fix for syncds having too many header rewrite false …

2020-01-29 Thread GitBox
rob05c commented on a change in pull request #4343: traffic_ops_ort: fix for syncds having too many header rewrite false … URL: https://github.com/apache/trafficcontrol/pull/4343#discussion_r372534463 ## File path: traffic_ops/ort/traffic_ops_ort.pl ## @@ -2652,6 +2658,23

[GitHub] [trafficcontrol] ocket8888 closed issue #3845: Rewrite /to_extensions/{{id}}/delete to Go

2020-01-29 Thread GitBox
ocket closed issue #3845: Rewrite /to_extensions/{{id}}/delete to Go URL: https://github.com/apache/trafficcontrol/issues/3845 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [trafficcontrol] ocket8888 merged pull request #4333: Deprecate & Rewrite DELETE to_extension

2020-01-29 Thread GitBox
ocket merged pull request #4333: Deprecate & Rewrite DELETE to_extension URL: https://github.com/apache/trafficcontrol/pull/4333 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [trafficcontrol] ocket8888 commented on issue #4240: Can update a user's username to contain spaces

2020-01-29 Thread GitBox
ocket commented on issue #4240: Can update a user's username to contain spaces URL: https://github.com/apache/trafficcontrol/issues/4240#issuecomment-579854649 Whoops, that PR referenced the wrong Issue. This is still open.

[GitHub] [trafficcontrol] ZugNZwang opened a new issue #4240: Can update a user's username to contain spaces

2020-01-29 Thread GitBox
ZugNZwang opened a new issue #4240: Can update a user's username to contain spaces URL: https://github.com/apache/trafficcontrol/issues/4240 ## I'm submitting a ... - [X] bug report - [ ] new feature / enhancement request - [ ] improvement request

[GitHub] [trafficcontrol] ocket8888 commented on issue #4239: Issues when updating a user's email using /user/current endpoint

2020-01-29 Thread GitBox
ocket commented on issue #4239: Issues when updating a user's email using /user/current endpoint URL: https://github.com/apache/trafficcontrol/issues/4239#issuecomment-579854287 Closed by #4346 This is an automated

[GitHub] [trafficcontrol] ocket8888 closed issue #4239: Issues when updating a user's email using /user/current endpoint

2020-01-29 Thread GitBox
ocket closed issue #4239: Issues when updating a user's email using /user/current endpoint URL: https://github.com/apache/trafficcontrol/issues/4239 This is an automated message from the Apache Git Service. To respond

[GitHub] [trafficcontrol] ocket8888 removed a comment on issue #4239: Issues when updating a user's email using /user/current endpoint

2020-01-29 Thread GitBox
ocket removed a comment on issue #4239: Issues when updating a user's email using /user/current endpoint URL: https://github.com/apache/trafficcontrol/issues/4239#issuecomment-579854126 Closed by #4346 This is an

[GitHub] [trafficcontrol] mhoppa commented on a change in pull request #4348: Deprecate GET jobs/{id}

2020-01-29 Thread GitBox
mhoppa commented on a change in pull request #4348: Deprecate GET jobs/{id} URL: https://github.com/apache/trafficcontrol/pull/4348#discussion_r372523507 ## File path: docs/source/api/jobs_id.rst ## @@ -18,6 +18,7 @@ *** ``jobs/{{ID}}`` *** +..

[GitHub] [trafficcontrol] traeak commented on a change in pull request #4343: traffic_ops_ort: fix for syncds having too many header rewrite false …

2020-01-29 Thread GitBox
traeak commented on a change in pull request #4343: traffic_ops_ort: fix for syncds having too many header rewrite false … URL: https://github.com/apache/trafficcontrol/pull/4343#discussion_r372537498 ## File path: traffic_ops/ort/traffic_ops_ort.pl ## @@ -2652,6 +2658,23

[GitHub] [trafficcontrol] mattjackson220 commented on a change in pull request #4339: cachegroupparameters rewrite

2020-01-29 Thread GitBox
mattjackson220 commented on a change in pull request #4339: cachegroupparameters rewrite URL: https://github.com/apache/trafficcontrol/pull/4339#discussion_r372646051 ## File path: traffic_ops/traffic_ops_golang/cachegroupparameter/parameters.go ## @@ -180,3 +185,120 @@

Build failed in Jenkins: trafficcontrol-PR #5208

2020-01-29 Thread Apache Jenkins Server
See Changes: -- GitHub pull request #3880 of commit 8ca6ecfb3e7757e77903028b41a5f18de8f1e904, no merge conflicts. Running as SYSTEM Setting status of

Build failed in Jenkins: trafficcontrol-PR #5210

2020-01-29 Thread Apache Jenkins Server
See Changes: [Michael_Hoppal] Deprecate ds/:id/state [github] Update Deliveryservice.pm -- [...truncated 2.95 MB...] traffic_portal_build_1 | | +--

[GitHub] [trafficcontrol] mattjackson220 commented on a change in pull request #4339: cachegroupparameters rewrite

2020-01-29 Thread GitBox
mattjackson220 commented on a change in pull request #4339: cachegroupparameters rewrite URL: https://github.com/apache/trafficcontrol/pull/4339#discussion_r372643832 ## File path: traffic_ops/traffic_ops_golang/cachegroupparameter/parameters.go ## @@ -180,3 +185,120 @@

Jenkins build is back to normal : trafficcontrol-PR #5211

2020-01-29 Thread Apache Jenkins Server
See

[GitHub] [trafficcontrol] mhoppa opened a new pull request #4352: Deprecate /deliveryservices/:id/state

2020-01-29 Thread GitBox
mhoppa opened a new pull request #4352: Deprecate /deliveryservices/:id/state URL: https://github.com/apache/trafficcontrol/pull/4352 ## What does this PR (Pull Request) do? - [x] This PR fixes #3857 ## Which Traffic Control components are affected by this PR?

[GitHub] [trafficcontrol] mhoppa opened a new pull request #4353: Deprecate /cdns/configs

2020-01-29 Thread GitBox
mhoppa opened a new pull request #4353: Deprecate /cdns/configs URL: https://github.com/apache/trafficcontrol/pull/4353 ## What does this PR (Pull Request) do? - [x] This PR fixes #3849 ## Which Traffic Control components are affected by this PR? -

Jenkins build is back to normal : trafficcontrol-PR #5209

2020-01-29 Thread Apache Jenkins Server
See

[GitHub] [trafficcontrol] mitchell852 commented on issue #4347: Fix internal server error in user current put

2020-01-29 Thread GitBox
mitchell852 commented on issue #4347: Fix internal server error in user current put URL: https://github.com/apache/trafficcontrol/pull/4347#issuecomment-579968413 @ocket - this looks like a good backport candidate for the 4.0.x branch

[GitHub] [trafficcontrol] mattjackson220 commented on a change in pull request #4339: cachegroupparameters rewrite

2020-01-29 Thread GitBox
mattjackson220 commented on a change in pull request #4339: cachegroupparameters rewrite URL: https://github.com/apache/trafficcontrol/pull/4339#discussion_r372644746 ## File path: lib/go-tc/cachegroup_parameter_associations.go ## @@ -0,0 +1,44 @@ +package tc + +/* + *

[GitHub] [trafficcontrol] mitchell852 commented on issue #4344: Fix TO atscfg missing max origin conns

2020-01-29 Thread GitBox
mitchell852 commented on issue #4344: Fix TO atscfg missing max origin conns URL: https://github.com/apache/trafficcontrol/pull/4344#issuecomment-579968071 @rob05c - reminder to backport this to 4.0.x branch This is an

[GitHub] [trafficcontrol] mitchell852 opened a new issue #4354: Server-side processing of TP tables

2020-01-29 Thread GitBox
mitchell852 opened a new issue #4354: Server-side processing of TP tables URL: https://github.com/apache/trafficcontrol/issues/4354 ## I'm submitting a ... - [ ] bug report - [x] new feature / enhancement request - [x] improvement request (usability, performance, tech debt,

[GitHub] [trafficcontrol] mhoppa edited a comment on issue #3848: Rewrite /cachegroup_fallbacks to Go

2020-01-29 Thread GitBox
mhoppa edited a comment on issue #3848: Rewrite /cachegroup_fallbacks to Go URL: https://github.com/apache/trafficcontrol/issues/3848#issuecomment-579969589 https://github.com/apache/trafficcontrol/pull/3766 accomplished this @ocket @mitchell852 @rawlinp can you close this?

[GitHub] [trafficcontrol] mhoppa commented on issue #3848: Rewrite /cachegroup_fallbacks to Go

2020-01-29 Thread GitBox
mhoppa commented on issue #3848: Rewrite /cachegroup_fallbacks to Go URL: https://github.com/apache/trafficcontrol/issues/3848#issuecomment-579969589 https://github.com/apache/trafficcontrol/pull/3766 accomplished this @ocket @mitchell852 @ocket can you close this?

[GitHub] [trafficcontrol] mattjackson220 commented on a change in pull request #4339: cachegroupparameters rewrite

2020-01-29 Thread GitBox
mattjackson220 commented on a change in pull request #4339: cachegroupparameters rewrite URL: https://github.com/apache/trafficcontrol/pull/4339#discussion_r372641848 ## File path: traffic_ops/traffic_ops_golang/cachegroupparameter/parameters.go ## @@ -180,3 +185,120 @@

[GitHub] [trafficcontrol] ocket8888 commented on issue #3848: Rewrite /cachegroup_fallbacks to Go

2020-01-29 Thread GitBox
ocket commented on issue #3848: Rewrite /cachegroup_fallbacks to Go URL: https://github.com/apache/trafficcontrol/issues/3848#issuecomment-580012836 Closed by #3766 - strangely, because that PR was merged before this issue was opened.

[GitHub] [trafficcontrol] ocket8888 closed issue #3848: Rewrite /cachegroup_fallbacks to Go

2020-01-29 Thread GitBox
ocket closed issue #3848: Rewrite /cachegroup_fallbacks to Go URL: https://github.com/apache/trafficcontrol/issues/3848 This is an automated message from the Apache Git Service. To respond to the message, please log on