[GitHub] [trafficcontrol] asf-ci commented on issue #3919: TP: adds a better view for comparing/managing params of 2 profiles

2019-09-11 Thread GitBox
asf-ci commented on issue #3919: TP: adds a better view for comparing/managing params of 2 profiles URL: https://github.com/apache/trafficcontrol/pull/3919#issuecomment-530482662 Refer to this link for build results (access rights to CI server needed):

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

2019-09-11 Thread GitBox
ocket commented on issue #3848: Rewrite /cachegroup_fallbacks to Go URL: https://github.com/apache/trafficcontrol/issues/3848#issuecomment-530516837 the decision to either deprecate and/or rewrite the endpoint can't be made here, but once the decision is made this issue can be closed

[GitHub] [trafficcontrol] mitchell852 commented on issue #3845: Rewrite /to_extensions/{{id}}/delete to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3845: Rewrite /to_extensions/{{id}}/delete to Go URL: https://github.com/apache/trafficcontrol/issues/3845#issuecomment-530516765 for obvious reasons, i don't believe this should be rewritten to Go as-is. maybe create a `DELETE /to_extensions/{{id}}` instead

[GitHub] [trafficcontrol] ocket8888 commented on issue #3850: Rewrite /cdns/usage/overview to Go

2019-09-11 Thread GitBox
ocket commented on issue #3850: Rewrite /cdns/usage/overview to Go URL: https://github.com/apache/trafficcontrol/issues/3850#issuecomment-530516650 the decision to either deprecate and/or rewrite the endpoint can't be made here, but once it is made this issue can be closed (if

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

2019-09-11 Thread GitBox
ocket commented on issue #3845: Rewrite /to_extensions/{{id}}/delete to Go URL: https://github.com/apache/trafficcontrol/issues/3845#issuecomment-530517116 the decision to either deprecate and/or rewrite the endpoint can't be made here, but once the decision is made this issue can be

[GitHub] [trafficcontrol] ocket8888 commented on issue #3836: Rewrite /deliveryservice_user to Go

2019-09-11 Thread GitBox
ocket commented on issue #3836: Rewrite /deliveryservice_user to Go URL: https://github.com/apache/trafficcontrol/issues/3836#issuecomment-530520867 the decision to either deprecate and/or rewrite the endpoint can't be made here, but once the decision is made this issue can be closed

[GitHub] [trafficcontrol] mitchell852 commented on issue #3831: Rewrite /servercheck/aadata to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3831: Rewrite /servercheck/aadata to Go URL: https://github.com/apache/trafficcontrol/issues/3831#issuecomment-530520934 `aadata` is something used by the old TO UI. i think this should be deprecated (not rewritten to Go and a deprecation notice added to the

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

2019-09-11 Thread GitBox
ocket commented on issue #3833: Rewrite /stats_summary/create to Go URL: https://github.com/apache/trafficcontrol/issues/3833#issuecomment-530521012 the decision to either deprecate and/or rewrite the endpoint can't be made here, but once the decision is made this issue can be closed

[GitHub] [trafficcontrol] ocket8888 commented on issue #3831: Rewrite /servercheck/aadata to Go

2019-09-11 Thread GitBox
ocket commented on issue #3831: Rewrite /servercheck/aadata to Go URL: https://github.com/apache/trafficcontrol/issues/3831#issuecomment-530521109 the decision to either deprecate and/or rewrite the endpoint can't be made here, but once the decision is made this issue can be closed (if

[GitHub] [trafficcontrol] mitchell852 commented on issue #3808: Rewrite /cachegroup/{{parameterID}}/parameter to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3808: Rewrite /cachegroup/{{parameterID}}/parameter to Go URL: https://github.com/apache/trafficcontrol/issues/3808#issuecomment-530525759 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the response) because not

[GitHub] [trafficcontrol] asf-ci commented on issue #3919: TP: adds a better view for comparing/managing params of 2 profiles

2019-09-11 Thread GitBox
asf-ci commented on issue #3919: TP: adds a better view for comparing/managing params of 2 profiles URL: https://github.com/apache/trafficcontrol/pull/3919#issuecomment-530575156 Refer to this link for build results (access rights to CI server needed):

[GitHub] [trafficcontrol] mitchell852 commented on issue #3837: Rewrite /deliveryservice_user/{{DSID}}/{{userID}} to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3837: Rewrite /deliveryservice_user/{{DSID}}/{{userID}} to Go URL: https://github.com/apache/trafficcontrol/issues/3837#issuecomment-530518251 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the response) as

[GitHub] [trafficcontrol] asf-ci commented on issue #3919: TP: adds a better view for comparing/managing params of 2 profiles

2019-09-11 Thread GitBox
asf-ci commented on issue #3919: TP: adds a better view for comparing/managing params of 2 profiles URL: https://github.com/apache/trafficcontrol/pull/3919#issuecomment-530518271 Refer to this link for build results (access rights to CI server needed):

[GitHub] [trafficcontrol] mitchell852 commented on issue #3804: Rewrite /user/current/jobs to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3804: Rewrite /user/current/jobs to Go URL: https://github.com/apache/trafficcontrol/issues/3804#issuecomment-530526856 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the response) because I thought the new jobs

[GitHub] [trafficcontrol] mitchell852 commented on issue #3805: Rewrite /parameters/validate to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3805: Rewrite /parameters/validate to Go URL: https://github.com/apache/trafficcontrol/issues/3805#issuecomment-530526514 not sure if this is even used so imo this should be deprecated (not rewritten to Go and a deprecation notice added to the response)

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3811: Rewrite /cachegroups/{{parameterID}}/parameter/available to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3811: Rewrite /cachegroups/{{parameterID}}/parameter/available to Go URL: https://github.com/apache/trafficcontrol/issues/3811#issuecomment-530525336 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the perl

[GitHub] [trafficcontrol] mitchell852 commented on issue #3850: Rewrite /cdns/usage/overview to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3850: Rewrite /cdns/usage/overview to Go URL: https://github.com/apache/trafficcontrol/issues/3850#issuecomment-530515302 I think this should be deprecated (not rewritten). Not sure anyone uses it or if it even works.

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

2019-09-11 Thread GitBox
mitchell852 commented on issue #3848: Rewrite /cachegroup_fallbacks to Go URL: https://github.com/apache/trafficcontrol/issues/3848#issuecomment-530515754 I think this should be deprecated (not rewritten to perl and a deprecation message added to the response). This can all be handled

[GitHub] [trafficcontrol] mitchell852 commented on issue #3798: Rewrite /hwinfo to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3798: Rewrite /hwinfo to Go URL: https://github.com/apache/trafficcontrol/issues/3798#issuecomment-530527885 take the `-wip` off and it should work :) This is an automated message from the

[GitHub] [trafficcontrol] mitchell852 commented on issue #3799: Rewrite /hwinfo/dtdata to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3799: Rewrite /hwinfo/dtdata to Go URL: https://github.com/apache/trafficcontrol/issues/3799#issuecomment-530527663 the dtdata makes me think it was used by the old TO UI so `/hwinfo/dtdata` should be deprecated imo (not rewritten to Go and a deprecation

[GitHub] [trafficcontrol] asf-ci commented on issue #3919: TP: adds a better view for comparing/managing params of 2 profiles

2019-09-11 Thread GitBox
asf-ci commented on issue #3919: TP: adds a better view for comparing/managing params of 2 profiles URL: https://github.com/apache/trafficcontrol/pull/3919#issuecomment-530473993 Refer to this link for build results (access rights to CI server needed):

[GitHub] [trafficcontrol] mitchell852 commented on issue #3813: Rewrite /profiles/name/{{name}}/copy/{{fromName}} to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3813: Rewrite /profiles/name/{{name}}/copy/{{fromName}} to Go URL: https://github.com/apache/trafficcontrol/issues/3813#issuecomment-530524788 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the response)

[GitHub] [trafficcontrol] mitchell852 commented on issue #3812: Rewrite /regions/{{region}}/phys_locations to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3812: Rewrite /regions/{{region}}/phys_locations to Go URL: https://github.com/apache/trafficcontrol/issues/3812#issuecomment-530525064 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the response) because GET

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3816: Rewrite /parameters/{{id}}/profiles

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3816: Rewrite /parameters/{{id}}/profiles URL: https://github.com/apache/trafficcontrol/issues/3816#issuecomment-530524178 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the perl response) as you should be

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3813: Rewrite /profiles/name/{{name}}/copy/{{fromName}} to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3813: Rewrite /profiles/name/{{name}}/copy/{{fromName}} to Go URL: https://github.com/apache/trafficcontrol/issues/3813#issuecomment-530524788 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the perl response)

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3812: Rewrite /regions/{{region}}/phys_locations to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3812: Rewrite /regions/{{region}}/phys_locations to Go URL: https://github.com/apache/trafficcontrol/issues/3812#issuecomment-530525064 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the perl response)

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3817: Rewrite /parameters/{{id}}/unassigned_profiles to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3817: Rewrite /parameters/{{id}}/unassigned_profiles to Go URL: https://github.com/apache/trafficcontrol/issues/3817#issuecomment-530523836 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the perl response)

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3819: Rewrite /regions/name/{{name}} to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3819: Rewrite /regions/name/{{name}} to Go URL: https://github.com/apache/trafficcontrol/issues/3819#issuecomment-530522900 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the perl response) as you can do the

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3818: Rewrite /divisions/{{division}}/regions to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3818: Rewrite /divisions/{{division}}/regions to Go URL: https://github.com/apache/trafficcontrol/issues/3818#issuecomment-530523341 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the perl response) as you

[GitHub] [trafficcontrol] mitchell852 commented on a change in pull request #3894: Blueprint: new profile comparison/management view

2019-09-11 Thread GitBox
mitchell852 commented on a change in pull request #3894: Blueprint: new profile comparison/management view URL: https://github.com/apache/trafficcontrol/pull/3894#discussion_r323433081 ## File path: blueprints/profile-param-compare-manage.md ## @@ -0,0 +1,238 @@ + + +#

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3837: Rewrite /deliveryservice_user/{{DSID}}/{{userID}} to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3837: Rewrite /deliveryservice_user/{{DSID}}/{{userID}} to Go URL: https://github.com/apache/trafficcontrol/issues/3837#issuecomment-530518251 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the perl response)

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3839: Rewrite /user/current/update to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3839: Rewrite /user/current/update to Go URL: https://github.com/apache/trafficcontrol/issues/3839#issuecomment-530517588 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the perl response) as this is currently

[GitHub] [trafficcontrol] asf-ci commented on issue #3919: TP: adds a better view for comparing/managing params of 2 profiles

2019-09-11 Thread GitBox
asf-ci commented on issue #3919: TP: adds a better view for comparing/managing params of 2 profiles URL: https://github.com/apache/trafficcontrol/pull/3919#issuecomment-530488023 Refer to this link for build results (access rights to CI server needed):

[GitHub] [trafficcontrol] mitchell852 commented on issue #3819: Rewrite /regions/name/{{name}} to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3819: Rewrite /regions/name/{{name}} to Go URL: https://github.com/apache/trafficcontrol/issues/3819#issuecomment-530522900 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the response) as you can do the same thing

[GitHub] [trafficcontrol] mitchell852 commented on issue #3818: Rewrite /divisions/{{division}}/regions to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3818: Rewrite /divisions/{{division}}/regions to Go URL: https://github.com/apache/trafficcontrol/issues/3818#issuecomment-530523341 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the response) as you should be

[GitHub] [trafficcontrol] mitchell852 commented on issue #3789: Rewrite /divisions/name/{{name}} to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3789: Rewrite /divisions/name/{{name}} to Go URL: https://github.com/apache/trafficcontrol/issues/3789#issuecomment-530528320 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the perl response) as you can do this with:

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3810: Rewrite /cachegroupparameters/{{cachegroupID}}/{{parameterID}} to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3810: Rewrite /cachegroupparameters/{{cachegroupID}}/{{parameterID}} to Go URL: https://github.com/apache/trafficcontrol/issues/3810#issuecomment-530525607 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3809: Rewrite /cachegroupparameters to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3809: Rewrite /cachegroupparameters to Go URL: https://github.com/apache/trafficcontrol/issues/3809#issuecomment-530525664 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the perl response) because not sure

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3808: Rewrite /cachegroup/{{parameterID}}/parameter to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3808: Rewrite /cachegroup/{{parameterID}}/parameter to Go URL: https://github.com/apache/trafficcontrol/issues/3808#issuecomment-530525759 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the perl response)

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

2019-09-11 Thread Apache Jenkins Server
See

[GitHub] [trafficcontrol] asf-ci commented on issue #3894: Blueprint: new profile comparison/management view

2019-09-11 Thread GitBox
asf-ci commented on issue #3894: Blueprint: new profile comparison/management view URL: https://github.com/apache/trafficcontrol/pull/3894#issuecomment-530544487 Refer to this link for build results (access rights to CI server needed):

[GitHub] [trafficcontrol] mitchell852 commented on issue #3839: Rewrite /user/current/update to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3839: Rewrite /user/current/update to Go URL: https://github.com/apache/trafficcontrol/issues/3839#issuecomment-530517588 i think this should be deprecated (not rewritten to perl and a deprecation notice added to the response) as this is currently handled by

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3839: Rewrite /user/current/update to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3839: Rewrite /user/current/update to Go URL: https://github.com/apache/trafficcontrol/issues/3839#issuecomment-530517588 i think this should be deprecated (not rewritten to perl and a deprecation notice added to the response) as this is currently

[GitHub] [trafficcontrol] ocket8888 commented on issue #3837: Rewrite /deliveryservice_user/{{DSID}}/{{userID}} to Go

2019-09-11 Thread GitBox
ocket commented on issue #3837: Rewrite /deliveryservice_user/{{DSID}}/{{userID}} to Go URL: https://github.com/apache/trafficcontrol/issues/3837#issuecomment-530520704 the decision to either deprecate and/or rewrite the endpoint can't be made here, but once the decision is made this

[GitHub] [trafficcontrol] mitchell852 commented on issue #3833: Rewrite /stats_summary/create to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3833: Rewrite /stats_summary/create to Go URL: https://github.com/apache/trafficcontrol/issues/3833#issuecomment-530520602 i think this should not rewritten to Go as-is. Instead rewrite it as POST /stats_summary and add a deprecation notice to POST

[GitHub] [trafficcontrol] ocket8888 commented on issue #3839: Rewrite /user/current/update to Go

2019-09-11 Thread GitBox
ocket commented on issue #3839: Rewrite /user/current/update to Go URL: https://github.com/apache/trafficcontrol/issues/3839#issuecomment-530520597 the decision to either deprecate and/or rewrite the endpoint can't be made here, but once the decision is made this issue can be closed

[GitHub] [trafficcontrol] ocket8888 commented on issue #3834: Rewrite /types/trimmed to Go

2019-09-11 Thread GitBox
ocket commented on issue #3834: Rewrite /types/trimmed to Go URL: https://github.com/apache/trafficcontrol/issues/3834#issuecomment-530520479 the decision to either deprecate and/or rewrite the endpoint can't be made here, but once the decision is made this issue can be closed (if

Build failed in Jenkins: trafficcontrol-PR #4242

2019-09-11 Thread Apache Jenkins Server
See Changes: [mitchell852] adds a blueprint for the new proposed profile comparison view in TP [mitchell852] style changes per feedback from ocket and fixes license problem with

[GitHub] [trafficcontrol] asf-ci commented on issue #3894: Blueprint: new profile comparison/management view

2019-09-11 Thread GitBox
asf-ci commented on issue #3894: Blueprint: new profile comparison/management view URL: https://github.com/apache/trafficcontrol/pull/3894#issuecomment-530540306 Refer to this link for build results (access rights to CI server needed):

[GitHub] [trafficcontrol] ocket8888 merged pull request #3894: Blueprint: new profile comparison/management view

2019-09-11 Thread GitBox
ocket merged pull request #3894: Blueprint: new profile comparison/management view URL: https://github.com/apache/trafficcontrol/pull/3894 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [trafficcontrol] mitchell852 commented on issue #3836: Rewrite /deliveryservice_user to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3836: Rewrite /deliveryservice_user to Go URL: https://github.com/apache/trafficcontrol/issues/3836#issuecomment-530518542 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the response) as assigning delivery services

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3839: Rewrite /user/current/update to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3839: Rewrite /user/current/update to Go URL: https://github.com/apache/trafficcontrol/issues/3839#issuecomment-530517588 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the response) as this is currently

[GitHub] [trafficcontrol] mitchell852 commented on issue #3834: Rewrite /types/trimmed to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3834: Rewrite /types/trimmed to Go URL: https://github.com/apache/trafficcontrol/issues/3834#issuecomment-530519181 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the response) because: a. it's weird b. you can

[GitHub] [trafficcontrol] mitchell852 commented on issue #3810: Rewrite /cachegroupparameters/{{cachegroupID}}/{{parameterID}} to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3810: Rewrite /cachegroupparameters/{{cachegroupID}}/{{parameterID}} to Go URL: https://github.com/apache/trafficcontrol/issues/3810#issuecomment-530525607 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the

[GitHub] [trafficcontrol] mitchell852 commented on issue #3811: Rewrite /cachegroups/{{parameterID}}/parameter/available to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3811: Rewrite /cachegroups/{{parameterID}}/parameter/available to Go URL: https://github.com/apache/trafficcontrol/issues/3811#issuecomment-530525336 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the response)

[GitHub] [trafficcontrol] mitchell852 commented on issue #3809: Rewrite /cachegroupparameters to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3809: Rewrite /cachegroupparameters to Go URL: https://github.com/apache/trafficcontrol/issues/3809#issuecomment-530525664 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the response) because not sure it's used

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3834: Rewrite /types/trimmed to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3834: Rewrite /types/trimmed to Go URL: https://github.com/apache/trafficcontrol/issues/3834#issuecomment-530519181 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the perl response) because: a. it's weird

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3836: Rewrite /deliveryservice_user to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3836: Rewrite /deliveryservice_user to Go URL: https://github.com/apache/trafficcontrol/issues/3836#issuecomment-530518542 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the perl response) as assigning

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3831: Rewrite /servercheck/aadata to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3831: Rewrite /servercheck/aadata to Go URL: https://github.com/apache/trafficcontrol/issues/3831#issuecomment-530520934 `aadata` is something used by the old TO UI. i think this should be deprecated (not rewritten to Go and a deprecation notice added

[GitHub] [trafficcontrol] mitchell852 commented on issue #3816: Rewrite /parameters/{{id}}/profiles

2019-09-11 Thread GitBox
mitchell852 commented on issue #3816: Rewrite /parameters/{{id}}/profiles URL: https://github.com/apache/trafficcontrol/issues/3816#issuecomment-530524178 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the response) as you should be able to get

[GitHub] [trafficcontrol] mitchell852 commented on issue #3817: Rewrite /parameters/{{id}}/unassigned_profiles to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3817: Rewrite /parameters/{{id}}/unassigned_profiles to Go URL: https://github.com/apache/trafficcontrol/issues/3817#issuecomment-530523836 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the response) as: a) not

[GitHub] [trafficcontrol] mitchell852 commented on issue #3850: Rewrite /cdns/usage/overview to Go

2019-09-11 Thread GitBox
mitchell852 commented on issue #3850: Rewrite /cdns/usage/overview to Go URL: https://github.com/apache/trafficcontrol/issues/3850#issuecomment-530528825 understood. just adding my thoughts. This is an automated message from

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

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3848: Rewrite /cachegroup_fallbacks to Go URL: https://github.com/apache/trafficcontrol/issues/3848#issuecomment-530515754 I think this should be deprecated (not rewritten to perl and a deprecation message added to the perl response). This can all be

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3805: Rewrite /parameters/validate to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3805: Rewrite /parameters/validate to Go URL: https://github.com/apache/trafficcontrol/issues/3805#issuecomment-530526514 not sure if this is even used so imo this should be deprecated (not rewritten to Go and a deprecation notice added to the perl

[GitHub] [trafficcontrol] mitchell852 edited a comment on issue #3804: Rewrite /user/current/jobs to Go

2019-09-11 Thread GitBox
mitchell852 edited a comment on issue #3804: Rewrite /user/current/jobs to Go URL: https://github.com/apache/trafficcontrol/issues/3804#issuecomment-530526856 i think this should be deprecated (not rewritten to Go and a deprecation notice added to the perl response) because I thought the

[GitHub] [trafficcontrol] mitchell852 opened a new pull request #3919: TP: adds a better view for comparing/managing params of 2 profiles

2019-09-11 Thread GitBox
mitchell852 opened a new pull request #3919: TP: adds a better view for comparing/managing params of 2 profiles URL: https://github.com/apache/trafficcontrol/pull/3919 ## What does this PR (Pull Request) do? - [ ] This PR fixes #REPLACE_ME OR is not related to any Issue

[GitHub] [trafficcontrol] ocket8888 opened a new pull request #3918: Moved content invalidation docs into qht guide

2019-09-11 Thread GitBox
ocket opened a new pull request #3918: Moved content invalidation docs into qht guide URL: https://github.com/apache/trafficcontrol/pull/3918 ## What does this PR (Pull Request) do? - [x] This PR is not related to any Issue ## Which Traffic Control components are affected by

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

2019-09-11 Thread Apache Jenkins Server
See

[GitHub] [trafficcontrol] asf-ci commented on issue #3918: Moved content invalidation docs into qht guide

2019-09-11 Thread GitBox
asf-ci commented on issue #3918: Moved content invalidation docs into qht guide URL: https://github.com/apache/trafficcontrol/pull/3918#issuecomment-530447597 Refer to this link for build results (access rights to CI server needed):

[GitHub] [trafficcontrol] ocket8888 commented on issue #3914: TM: Report time for full body download for _astats

2019-09-11 Thread GitBox
ocket commented on issue #3914: TM: Report time for full body download for _astats URL: https://github.com/apache/trafficcontrol/pull/3914#issuecomment-530446265 Does this PR fix a security vulnerability or include a database migration?