[GitHub] [trafficcontrol] ocket8888 commented on issue #4097: Emulated vault

2019-11-08 Thread GitBox
ocket commented on issue #4097: Emulated vault URL: https://github.com/apache/trafficcontrol/pull/4097#issuecomment-552075804 Please fill out the Pull Request template to the best of your ability, and avoid deleting it entirely.

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4097: Emulated vault

2019-11-08 Thread GitBox
ocket commented on a change in pull request #4097: Emulated vault URL: https://github.com/apache/trafficcontrol/pull/4097#discussion_r344433313 ## File path: experimental/emulated_vault/README ## @@ -0,0 +1,23 @@ +The emulated_vault module supplies a HTTP server

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4097: Emulated vault

2019-11-08 Thread GitBox
ocket commented on a change in pull request #4097: Emulated vault URL: https://github.com/apache/trafficcontrol/pull/4097#discussion_r344433234 ## File path: experimental/emulated_vault/README ## @@ -0,0 +1,23 @@ +The emulated_vault module supplies a HTTP server

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4097: Emulated vault

2019-11-08 Thread GitBox
ocket commented on a change in pull request #4097: Emulated vault URL: https://github.com/apache/trafficcontrol/pull/4097#discussion_r344433248 ## File path: experimental/emulated_vault/README ## @@ -0,0 +1,23 @@ +The emulated_vault module supplies a HTTP server

[GitHub] [trafficcontrol] asf-ci commented on issue #4105: Federation resolvers id go

2019-11-08 Thread GitBox
asf-ci commented on issue #4105: Federation resolvers id go URL: https://github.com/apache/trafficcontrol/pull/4105#issuecomment-552075317 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/4765/

Build failed in Jenkins: trafficcontrol-PR #4765

2019-11-08 Thread Apache Jenkins Server
See Changes: -- GitHub pull request #4105 of commit ce5254003794d3bf34631669a9fb34c287b8a246, no merge conflicts. Running as SYSTEM Setting status of

[GitHub] [trafficcontrol] asf-ci commented on issue #4105: Federation resolvers id go

2019-11-08 Thread GitBox
asf-ci commented on issue #4105: Federation resolvers id go URL: https://github.com/apache/trafficcontrol/pull/4105#issuecomment-552075034 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/4764/

Build failed in Jenkins: trafficcontrol-PR #4764

2019-11-08 Thread Apache Jenkins Server
See Changes: -- GitHub pull request #4105 of commit 3c74f1a9b224a65ea24d2c46fb55425884a2babf, no merge conflicts. Running as SYSTEM Setting status of

[GitHub] [trafficcontrol] asf-ci commented on issue #4105: Federation resolvers id go

2019-11-08 Thread GitBox
asf-ci commented on issue #4105: Federation resolvers id go URL: https://github.com/apache/trafficcontrol/pull/4105#issuecomment-552053561 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/4763/

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

2019-11-08 Thread Apache Jenkins Server
See

Build failed in Jenkins: trafficcontrol-master-build #1600

2019-11-08 Thread Apache Jenkins Server
See Changes: [jeffrey_elsloo] Separate execution of required zone generation from optional priming -- Started by an SCM change Started by an SCM change

Build failed in Jenkins: trafficcontrol-PR #4762

2019-11-08 Thread Apache Jenkins Server
See Changes: -- GitHub pull request #4104 of commit 4ebc5d11046113d4ca65bde4fca82deda84be8b1, no merge conflicts. Running as SYSTEM [EnvInject] - Loading node environment variables.

[GitHub] [trafficcontrol] asf-ci commented on issue #4104: Separate execution of required zone generation from optional priming

2019-11-08 Thread GitBox
asf-ci commented on issue #4104: Separate execution of required zone generation from optional priming URL: https://github.com/apache/trafficcontrol/pull/4104#issuecomment-552046856 Refer to this link for build results (access rights to CI server needed):

[GitHub] [trafficcontrol] elsloo merged pull request #4104: Separate execution of required zone generation from optional priming

2019-11-08 Thread GitBox
elsloo merged pull request #4104: Separate execution of required zone generation from optional priming URL: https://github.com/apache/trafficcontrol/pull/4104 This is an automated message from the Apache Git Service. To

[GitHub] [trafficcontrol] asf-ci commented on issue #4104: Separate execution of required zone generation from optional priming

2019-11-08 Thread GitBox
asf-ci commented on issue #4104: Separate execution of required zone generation from optional priming URL: https://github.com/apache/trafficcontrol/pull/4104#issuecomment-552040900 Refer to this link for build results (access rights to CI server needed):

[GitHub] [trafficcontrol] ocket8888 opened a new pull request #4105: Federation resolvers id go

2019-11-08 Thread GitBox
ocket opened a new pull request #4105: Federation resolvers id go URL: https://github.com/apache/trafficcontrol/pull/4105 ## What does this PR (Pull Request) do? - [x] This PR fixes #3797 Rewrites the `/federation_resolvers/{{ID}}` endpoint to Go. Also adds a `DELETE`

[GitHub] [trafficcontrol] asf-ci commented on issue #4103: [WIP] Add a bounded queue to DNS request processing

2019-11-08 Thread GitBox
asf-ci commented on issue #4103: [WIP] Add a bounded queue to DNS request processing URL: https://github.com/apache/trafficcontrol/pull/4103#issuecomment-552031759 Refer to this link for build results (access rights to CI server needed):

Jenkins build is back to normal : trafficcontrol-master-build #1599

2019-11-08 Thread Apache Jenkins Server
See

[GitHub] [trafficcontrol] rawlinp opened a new pull request #4104: Separate execution of required zone generation from optional priming

2019-11-08 Thread GitBox
rawlinp opened a new pull request #4104: Separate execution of required zone generation from optional priming URL: https://github.com/apache/trafficcontrol/pull/4104 ## What does this PR (Pull Request) do? Generating zones is a required step that cannot be timed out safely. Priming

[GitHub] [trafficcontrol] elsloo opened a new pull request #4103: Add a bounded queue to DNS request processing

2019-11-08 Thread GitBox
elsloo opened a new pull request #4103: Add a bounded queue to DNS request processing URL: https://github.com/apache/trafficcontrol/pull/4103 ## What does this PR (Pull Request) do? - [x] This PR is not related to any Issue ## Which Traffic Control components are

[GitHub] [trafficcontrol] guzzijason edited a comment on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests

2019-11-08 Thread GitBox
guzzijason edited a comment on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests URL: https://github.com/apache/trafficcontrol/issues/3965#issuecomment-552014356 Nope. See my examples of the current GET/HEAD responses in the issue description. GET responds with

[GitHub] [trafficcontrol] guzzijason commented on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests

2019-11-08 Thread GitBox
guzzijason commented on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests URL: https://github.com/apache/trafficcontrol/issues/3965#issuecomment-552014356 Nope. See my examples of the current GET/HEAD responses in the issue description.

[GitHub] [trafficcontrol] ocket8888 commented on issue #4102: TP: Delivery Service URLs section shouldn't be displayed for ANY_MAP DS type

2019-11-08 Thread GitBox
ocket commented on issue #4102: TP: Delivery Service URLs section shouldn't be displayed for ANY_MAP DS type URL: https://github.com/apache/trafficcontrol/issues/4102#issuecomment-552012292 IDK what difference this makes, but those URLs are generated by the API, so technically TP is

[GitHub] [trafficcontrol] asf-ci commented on issue #3598: Removed restangular from UserService

2019-11-08 Thread GitBox
asf-ci commented on issue #3598: Removed restangular from UserService URL: https://github.com/apache/trafficcontrol/pull/3598#issuecomment-552012055 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/4759/

[GitHub] [trafficcontrol] asf-ci commented on issue #3761: Improvements to Server-to-Delivery Service assignments

2019-11-08 Thread GitBox
asf-ci commented on issue #3761: Improvements to Server-to-Delivery Service assignments URL: https://github.com/apache/trafficcontrol/pull/3761#issuecomment-552010761 Refer to this link for build results (access rights to CI server needed):

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3598: Removed restangular from UserService

2019-11-08 Thread GitBox
ocket commented on a change in pull request #3598: Removed restangular from UserService URL: https://github.com/apache/trafficcontrol/pull/3598#discussion_r344384173 ## File path: traffic_portal/app/src/common/api/UserService.js ## @@ -17,143 +17,152 @@ * under the

[GitHub] [trafficcontrol] mitchell852 commented on issue #4101: TP/TP: Unable to assign DSRC to DS of type CLIENT_STEERING

2019-11-08 Thread GitBox
mitchell852 commented on issue #4101: TP/TP: Unable to assign DSRC to DS of type CLIENT_STEERING URL: https://github.com/apache/trafficcontrol/issues/4101#issuecomment-551996853 ok, then i think you need to come up with a new title and description for this issue if you're going to keep it

[GitHub] [trafficcontrol] lbathina edited a comment on issue #4101: TP/TP: Unable to assign DSRC to DS of type CLIENT_STEERING

2019-11-08 Thread GitBox
lbathina edited a comment on issue #4101: TP/TP: Unable to assign DSRC to DS of type CLIENT_STEERING URL: https://github.com/apache/trafficcontrol/issues/4101#issuecomment-551996517 I think 2 should go in separate issue after we discuss it.

[GitHub] [trafficcontrol] lbathina commented on issue #4101: TP/TP: Unable to assign DSRC to DS of type CLIENT_STEERING

2019-11-08 Thread GitBox
lbathina commented on issue #4101: TP/TP: Unable to assign DSRC to DS of type CLIENT_STEERING URL: https://github.com/apache/trafficcontrol/issues/4101#issuecomment-551996517 This can be one issue - 2 PR could be submitted against it if its going to be in different PR.

[GitHub] [trafficcontrol] mitchell852 commented on issue #4101: TP/TP: Unable to assign DSRC to DS of type CLIENT_STEERING

2019-11-08 Thread GitBox
mitchell852 commented on issue #4101: TP/TP: Unable to assign DSRC to DS of type CLIENT_STEERING URL: https://github.com/apache/trafficcontrol/issues/4101#issuecomment-551995570 @lbathina - do you want to just close this issue and create 2 more for those 2 items?

[GitHub] [trafficcontrol] lbathina commented on issue #4101: TP/TP: Unable to assign DSRC to DS of type CLIENT_STEERING

2019-11-08 Thread GitBox
lbathina commented on issue #4101: TP/TP: Unable to assign DSRC to DS of type CLIENT_STEERING URL: https://github.com/apache/trafficcontrol/issues/4101#issuecomment-551995426 thanks jeremy. I am posting our chat in here. following should be fixed. 1. should not be able to assign

[GitHub] [trafficcontrol] lbathina commented on issue #4101: TP/TP: Unable to assign DSRC to DS of type CLIENT_STEERING

2019-11-08 Thread GitBox
lbathina commented on issue #4101: TP/TP: Unable to assign DSRC to DS of type CLIENT_STEERING URL: https://github.com/apache/trafficcontrol/issues/4101#issuecomment-551994571 although we cannot assign server to a CLIENT_STEERING target from TP, it seems that we could do it via TO. which

Build failed in Jenkins: trafficcontrol-master-build #1598

2019-11-08 Thread Apache Jenkins Server
See Changes: [ocket] changes search to xmlId rather than displayname (#4084) -- Started by an SCM change Running as SYSTEM [EnvInject] - Loading node

[GitHub] [trafficcontrol] ocket8888 closed issue #3753: unable to filter delivery services by xml-id on TP dashboard

2019-11-08 Thread GitBox
ocket closed issue #3753: unable to filter delivery services by xml-id on TP dashboard URL: https://github.com/apache/trafficcontrol/issues/3753 This is an automated message from the Apache Git Service. To respond to

[GitHub] [trafficcontrol] ocket8888 merged pull request #4084: TP: delivery service widget - changes search to ds xmlId rather than display name

2019-11-08 Thread GitBox
ocket merged pull request #4084: TP: delivery service widget - changes search to ds xmlId rather than display name URL: https://github.com/apache/trafficcontrol/pull/4084 This is an automated message from the Apache Git

Build failed in Jenkins: trafficcontrol-master-build #1597

2019-11-08 Thread Apache Jenkins Server
See Changes: [mitchell852] Removed restangular from CacheGroupParameterService (#3603) [mitchell852] Rewrote /federation_resolvers - POST (#4079) -- Started

[GitHub] [trafficcontrol] mitchell852 closed issue #3796: Rewrite /federation_resolvers to Go

2019-11-08 Thread GitBox
mitchell852 closed issue #3796: Rewrite /federation_resolvers to Go URL: https://github.com/apache/trafficcontrol/issues/3796 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [trafficcontrol] mitchell852 merged pull request #4079: Rewrote /federation_resolvers - POST

2019-11-08 Thread GitBox
mitchell852 merged pull request #4079: Rewrote /federation_resolvers - POST URL: https://github.com/apache/trafficcontrol/pull/4079 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [trafficcontrol] mitchell852 commented on a change in pull request #3598: Removed restangular from UserService

2019-11-08 Thread GitBox
mitchell852 commented on a change in pull request #3598: Removed restangular from UserService URL: https://github.com/apache/trafficcontrol/pull/3598#discussion_r344363500 ## File path: traffic_portal/app/src/common/api/UserService.js ## @@ -17,143 +17,152 @@ * under

[GitHub] [trafficcontrol] lbathina opened a new issue #4101: TP/TP: Unable to assign DSRC to DS of type CLIENT_STEERING

2019-11-08 Thread GitBox
lbathina opened a new issue #4101: TP/TP: Unable to assign DSRC to DS of type CLIENT_STEERING URL: https://github.com/apache/trafficcontrol/issues/4101 ## I'm submitting a ... - [X] bug report - [ ] new feature / enhancement request - [ ] improvement

[GitHub] [trafficcontrol] mitchell852 merged pull request #3603: Removed restangular from CacheGroupParameterService

2019-11-08 Thread GitBox
mitchell852 merged pull request #3603: Removed restangular from CacheGroupParameterService URL: https://github.com/apache/trafficcontrol/pull/3603 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [trafficcontrol] mitchell852 merged pull request #3618: Removed restangular from EndpointService

2019-11-08 Thread GitBox
mitchell852 merged pull request #3618: Removed restangular from EndpointService URL: https://github.com/apache/trafficcontrol/pull/3618 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [trafficcontrol] ocket8888 commented on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests

2019-11-08 Thread GitBox
ocket commented on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests URL: https://github.com/apache/trafficcontrol/issues/3965#issuecomment-551958799 Is GET not chunked? This is an automated message

[GitHub] [trafficcontrol] guzzijason edited a comment on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests

2019-11-08 Thread GitBox
guzzijason edited a comment on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests URL: https://github.com/apache/trafficcontrol/issues/3965#issuecomment-551955244 But why do we bother to send `Transfer-Encoding: chunked` on a HEAD response with no body? Even if the

[GitHub] [trafficcontrol] awilliams commented on a change in pull request #4100: Docs misc fixes

2019-11-08 Thread GitBox
awilliams commented on a change in pull request #4100: Docs misc fixes URL: https://github.com/apache/trafficcontrol/pull/4100#discussion_r344327095 ## File path: docs/source/overview/traffic_router.rst ## @@ -63,4 +63,4 @@ In this case Traffic Router has access to more

[GitHub] [trafficcontrol] guzzijason commented on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests

2019-11-08 Thread GitBox
guzzijason commented on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests URL: https://github.com/apache/trafficcontrol/issues/3965#issuecomment-551955244 But why do we bother to send `Transfer-Encoding: chunked` on a HEAD response with no body? Even if the spec says

[GitHub] [trafficcontrol] awilliams commented on a change in pull request #4100: Docs misc fixes

2019-11-08 Thread GitBox
awilliams commented on a change in pull request #4100: Docs misc fixes URL: https://github.com/apache/trafficcontrol/pull/4100#discussion_r344326702 ## File path: docs/source/overview/traffic_monitor.rst ## @@ -33,12 +33,12 @@ Traffic Monitor polls all :term:`cache

[GitHub] [trafficcontrol] awilliams opened a new pull request #4100: Docs misc fixes

2019-11-08 Thread GitBox
awilliams opened a new pull request #4100: Docs misc fixes URL: https://github.com/apache/trafficcontrol/pull/4100 ## What does this PR (Pull Request) do? - [x] Fixes minor issues (grammar, typos, formatting) in the documentation. ## Which Traffic Control components are

[GitHub] [trafficcontrol] asf-ci commented on issue #4100: Docs misc fixes

2019-11-08 Thread GitBox
asf-ci commented on issue #4100: Docs misc fixes URL: https://github.com/apache/trafficcontrol/pull/4100#issuecomment-551953774 Can one of the admins verify this patch? This is an automated message from the Apache Git

[GitHub] [trafficcontrol] mhoppa commented on a change in pull request #4079: Rewrote /federation_resolvers - POST

2019-11-08 Thread GitBox
mhoppa commented on a change in pull request #4079: Rewrote /federation_resolvers - POST URL: https://github.com/apache/trafficcontrol/pull/4079#discussion_r344313530 ## File path: docs/source/api/federation_resolvers.rst ## @@ -19,6 +19,87 @@ ``federation_resolvers``

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4079: Rewrote /federation_resolvers - POST

2019-11-08 Thread GitBox
ocket commented on a change in pull request #4079: Rewrote /federation_resolvers - POST URL: https://github.com/apache/trafficcontrol/pull/4079#discussion_r344302052 ## File path: docs/source/api/federation_resolvers.rst ## @@ -19,6 +19,87 @@ ``federation_resolvers``

[GitHub] [trafficcontrol] ocket8888 commented on issue #4079: Rewrote /federation_resolvers - POST

2019-11-08 Thread GitBox
ocket commented on issue #4079: Rewrote /federation_resolvers - POST URL: https://github.com/apache/trafficcontrol/pull/4079#issuecomment-551933845 Perl let you do the same thing. It's probably as a direct consequence of never validating the "Type". Honestly, that should never

[GitHub] [trafficcontrol] mitchell852 merged pull request #3620: Removed restangular from FederationService

2019-11-08 Thread GitBox
mitchell852 merged pull request #3620: Removed restangular from FederationService URL: https://github.com/apache/trafficcontrol/pull/3620 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [trafficcontrol] mitchell852 merged pull request #3619: Removed restangular from FederationResolverService

2019-11-08 Thread GitBox
mitchell852 merged pull request #3619: Removed restangular from FederationResolverService URL: https://github.com/apache/trafficcontrol/pull/3619 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [trafficcontrol] asf-ci commented on issue #3619: Removed restangular from FederationResolverService

2019-11-08 Thread GitBox
asf-ci commented on issue #3619: Removed restangular from FederationResolverService URL: https://github.com/apache/trafficcontrol/pull/3619#issuecomment-551926399 Refer to this link for build results (access rights to CI server needed):

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

2019-11-08 Thread Apache Jenkins Server
See

[GitHub] [trafficcontrol] asf-ci commented on issue #3620: Removed restangular from FederationService

2019-11-08 Thread GitBox
asf-ci commented on issue #3620: Removed restangular from FederationService URL: https://github.com/apache/trafficcontrol/pull/3620#issuecomment-551923825 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/4756/

Build failed in Jenkins: trafficcontrol-PR #4756

2019-11-08 Thread Apache Jenkins Server
See Changes: -- GitHub pull request #3620 of commit 7f39f8c0a35d24fe6499420be6ce5c5614dd1564, no merge conflicts. Running as SYSTEM Setting status of

[GitHub] [trafficcontrol] ocket8888 merged pull request #4099: Correctly handle assigning org servers to ds that have req caps

2019-11-08 Thread GitBox
ocket merged pull request #4099: Correctly handle assigning org servers to ds that have req caps URL: https://github.com/apache/trafficcontrol/pull/4099 This is an automated message from the Apache Git Service. To

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #4099: Correctly handle assigning org servers to ds that have req caps

2019-11-08 Thread GitBox
ocket commented on a change in pull request #4099: Correctly handle assigning org servers to ds that have req caps URL: https://github.com/apache/trafficcontrol/pull/4099#discussion_r344285041 ## File path: docs/source/api/deliveryservices_id_servers_eligible.rst ##

[GitHub] [trafficcontrol] ocket8888 edited a comment on issue #4093: Admin can map Federal Resolver with Invalid mapping parameter or without mapping parameter

2019-11-08 Thread GitBox
ocket edited a comment on issue #4093: Admin can map Federal Resolver with Invalid mapping parameter or without mapping parameter URL: https://github.com/apache/trafficcontrol/issues/4093#issuecomment-551916760 > _" Admin can map Federal Resolver with Invalid mapping parameter or

[GitHub] [trafficcontrol] ocket8888 edited a comment on issue #4093: Admin can map Federal Resolver with Invalid mapping parameter or without mapping parameter

2019-11-08 Thread GitBox
ocket edited a comment on issue #4093: Admin can map Federal Resolver with Invalid mapping parameter or without mapping parameter URL: https://github.com/apache/trafficcontrol/issues/4093#issuecomment-551916760 > _" Admin can map Federal Resolver with Invalid mapping parameter or

[GitHub] [trafficcontrol] ocket8888 edited a comment on issue #4093: Admin can map Federal Resolver with Invalid mapping parameter or without mapping parameter

2019-11-08 Thread GitBox
ocket edited a comment on issue #4093: Admin can map Federal Resolver with Invalid mapping parameter or without mapping parameter URL: https://github.com/apache/trafficcontrol/issues/4093#issuecomment-551916760 > _" Admin can map Federal Resolver with Invalid mapping parameter or

[GitHub] [trafficcontrol] mitchell852 commented on a change in pull request #3620: Removed restangular from FederationService

2019-11-08 Thread GitBox
mitchell852 commented on a change in pull request #3620: Removed restangular from FederationService URL: https://github.com/apache/trafficcontrol/pull/3620#discussion_r344282898 ## File path: experimental/traffic-portal-angular7/package.json ## @@ -32,6 +32,7 @@

[GitHub] [trafficcontrol] lactose commented on a change in pull request #4099: Correctly handle assigning org servers to ds that have req caps

2019-11-08 Thread GitBox
lactose commented on a change in pull request #4099: Correctly handle assigning org servers to ds that have req caps URL: https://github.com/apache/trafficcontrol/pull/4099#discussion_r344278514 ## File path: docs/source/api/deliveryservices_id_servers_eligible.rst ## @@

[GitHub] [trafficcontrol] ocket8888 commented on issue #4093: Admin can map Federal Resolver with Invalid mapping parameter or without mapping parameter

2019-11-08 Thread GitBox
ocket commented on issue #4093: Admin can map Federal Resolver with Invalid mapping parameter or without mapping parameter URL: https://github.com/apache/trafficcontrol/issues/4093#issuecomment-551916760 > _" Admin can map Federal Resolver with Invalid mapping parameter or without

[GitHub] [trafficcontrol] lactose commented on a change in pull request #4099: Correctly handle assigning org servers to ds that have req caps

2019-11-08 Thread GitBox
lactose commented on a change in pull request #4099: Correctly handle assigning org servers to ds that have req caps URL: https://github.com/apache/trafficcontrol/pull/4099#discussion_r344279424 ## File path:

[GitHub] [trafficcontrol] asf-ci commented on issue #3620: Removed restangular from FederationService

2019-11-08 Thread GitBox
asf-ci commented on issue #3620: Removed restangular from FederationService URL: https://github.com/apache/trafficcontrol/pull/3620#issuecomment-551912535 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/trafficcontrol-PR/4755/

[GitHub] [trafficcontrol] rob05c closed issue #4098: Parent.Config generats incorrect parent when DS share same origin

2019-11-08 Thread GitBox
rob05c closed issue #4098: Parent.Config generats incorrect parent when DS share same origin URL: https://github.com/apache/trafficcontrol/issues/4098 This is an automated message from the Apache Git Service. To respond to

[GitHub] [trafficcontrol] rob05c commented on issue #4098: Parent.Config generats incorrect parent when DS share same origin

2019-11-08 Thread GitBox
rob05c commented on issue #4098: Parent.Config generats incorrect parent when DS share same origin URL: https://github.com/apache/trafficcontrol/issues/4098#issuecomment-551903980 Duplicate of #3537 > would nullify the very purpose of have capabilities added. Well, if you

[GitHub] [trafficcontrol] rob05c edited a comment on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests

2019-11-08 Thread GitBox
rob05c edited a comment on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests URL: https://github.com/apache/trafficcontrol/issues/3965#issuecomment-551876550 I don't object to the change itself, but I have concerns about what's driving it. The HTTP Spec says

[GitHub] [trafficcontrol] rob05c commented on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests

2019-11-08 Thread GitBox
rob05c commented on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests URL: https://github.com/apache/trafficcontrol/issues/3965#issuecomment-551876550 I don't object to the change itself, but I have concerns about what's driving it. The HTTP Spec says servers

[GitHub] [trafficcontrol] smalenfant commented on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests

2019-11-08 Thread GitBox
smalenfant commented on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests URL: https://github.com/apache/trafficcontrol/issues/3965#issuecomment-551768262 This seems to be a big deal.. More than half our requests are HEAD requests... That should change the TCP stats

[GitHub] [trafficcontrol] smalenfant commented on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests

2019-11-08 Thread GitBox
smalenfant commented on issue #3965: Traffic Router inconsistent behavior on HTTP HEAD requests URL: https://github.com/apache/trafficcontrol/issues/3965#issuecomment-551717490 I haven't verified hits PR, but what would be the Content-Length on a Steering delivery services with