Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/1303
Status on this? is thing going to land for 7.1.x ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/1357
@shinrich @SolidWallOfCode Is this landing for 7.1.x?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user zwoop closed the pull request at:
https://github.com/apache/trafficserver/pull/1350
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user maskit commented on the issue:
https://github.com/apache/trafficserver/pull/888
@jaaju Unfortunately, this hasn't backported to 6.x yet, and it wouldn't be
in 6.2.1 because the release is almost there (RC0 passed a voting).
I'm going to try to backport in a couple of
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1373
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1390/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1373
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1284/ for details.
---
If your project is set up for it, you can reply to this email and have your
GitHub user bryancall opened a pull request:
https://github.com/apache/trafficserver/pull/1373
#1279 - CID 1367531: Uninitialized members (UNINIT_CTOR): proxy/main.cc
You can merge this pull request into a Git repository by running:
$ git pull
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1350
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1283/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1350
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1389/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1350
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1282/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1350
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1388/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user randall commented on the issue:
https://github.com/apache/trafficserver/pull/1350
@zwoop I believe I fixed the error on BSD.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user bryancall closed the pull request at:
https://github.com/apache/trafficserver/pull/1349
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1349
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1281/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1349
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1387/ for details.
---
If your project is set up for it, you can reply to this email and have
GitHub user vmamidi opened an issue:
https://github.com/apache/trafficserver/issues/1372
handle the traffic manager functionality in traffic server process
currently syncing of stats and other activities are handled in traffic
manager process. We can handle all those activities on
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1331
Linux build *failed*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1280/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1331
FreeBSD build *failed*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1386/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1332
Linux build *failed*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1279/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1332
FreeBSD build *failed*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1385/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user jablko closed the pull request at:
https://github.com/apache/trafficserver/pull/1342
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user jablko commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/1342#discussion_r97648817
--- Diff: plugins/experimental/money_trace/money_trace.cc ---
@@ -120,17 +120,13 @@ mt_check_request_header(TSHttpTxn txnp)
if (!hdr_value
Github user maskit commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/1342#discussion_r97648065
--- Diff: plugins/experimental/money_trace/money_trace.cc ---
@@ -120,17 +120,13 @@ mt_check_request_header(TSHttpTxn txnp)
if (!hdr_value
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1332
FreeBSD build *failed*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1384/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1332
Linux build *failed*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1278/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user jablko commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/1342#discussion_r97640188
--- Diff: plugins/experimental/money_trace/money_trace.cc ---
@@ -120,17 +120,13 @@ mt_check_request_header(TSHttpTxn txnp)
if (!hdr_value
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1332
Linux build *failed*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1277/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1371
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1276/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1371
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1383/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user zizhong commented on the issue:
https://github.com/apache/trafficserver/pull/351
Thanks, @bgaff . After reading the previous comments, I think @bryancall
suggested there should be a `limit` for the request body. And have you guys
done `API review` about this PR?
---
If
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1371
Linux build *failed*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1275/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1371
FreeBSD build *failed*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1382/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user jablko commented on the issue:
https://github.com/apache/trafficserver/pull/1328
I agree with @bgaff.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user jablko commented on the issue:
https://github.com/apache/trafficserver/pull/1355
> I'm marking this for 7.1.0, since it's mostly benign, and generally
useful.
Sounds good, thanks! Merged.
> Update: I think I see, it's merely a cleanup (no change in actual
Github user jablko closed the pull request at:
https://github.com/apache/trafficserver/pull/1355
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user jablko closed the issue at:
https://github.com/apache/trafficserver/issues/1353
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1313
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1274/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user jablko commented on the issue:
https://github.com/apache/trafficserver/pull/1309
Thanks @bryancall! Merged.
Yeah, until this PR landed, it was also part of #1313 (is there a way
around that?) so I guess applying both would conflict. Git is smart enough to
match
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1313
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1381/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user jablko closed the pull request at:
https://github.com/apache/trafficserver/pull/1309
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user jablko closed the issue at:
https://github.com/apache/trafficserver/issues/1307
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1371
FreeBSD build *failed*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1380/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1371
Linux build *failed*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1273/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
GitHub user jablko opened a pull request:
https://github.com/apache/trafficserver/pull/1371
Calling setrlimit() twice is redundant
Set .rlim_cur to either .rlim_max or RLIM_INFINITY.
Doing both is redundant.
This is a follow-up to #1362. The first commit belongs to that
Github user jablko commented on the issue:
https://github.com/apache/trafficserver/pull/1362
Come to think of it, why do we setrlimit() to .rlim_max and RLIM_INFINITY
in the same call? That seems redundant.
---
If your project is set up for it, you can reply to this email and have
Github user bgaff commented on the issue:
https://github.com/apache/trafficserver/pull/351
I can help, @Zizhong what of the previous asks haven't been completed?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
46 matches
Mail list logo