[GitHub] trafficserver issue #1502: issue #1501: reconstruct to load the default valu...

2017-03-14 Thread zhilhuan
Github user zhilhuan commented on the issue: https://github.com/apache/trafficserver/pull/1502 @zwoop rebased to current master and run "clang-format -i" on this file. Would you please to run the CI again? --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver pull request #1502: issue #1501: reconstruct to load the defau...

2017-03-14 Thread zhilhuan
Github user zhilhuan commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1502#discussion_r106082140 --- Diff: proxy/ControlMatcher.cc --- @@ -220,6 +220,7 @@ HostMatcher::NewEntry(matcher_line *line_info) if

Build failed in Jenkins: debian_8-master » gcc,debian_8,debug #1360

2017-03-14 Thread jenkins
See -- [...truncated 1.55 MB...] make[1]: Entering directory

Build failed in Jenkins: ubuntu_16_04-master » gcc,ubuntu_16_04,debug #630

2017-03-14 Thread jenkins
See -- [...truncated 1.59 MB...] make[1]: Entering directory

Build failed in Jenkins: centos_6-master » gcc,centos_6,debug #2493

2017-03-14 Thread jenkins
See -- [...truncated 1.56 MB...] make[1]: Leaving directory

Build failed in Jenkins: freebsd_11-master » clang,freebsd_11,debug #24

2017-03-14 Thread jenkins
See -- [...truncated 1.65 MB...] gmake[1]: Leaving directory

Build failed in Jenkins: ubuntu_16_10-master » clang,ubuntu_16_10,debug #180

2017-03-14 Thread jenkins
See -- [...truncated 1.69 MB...] make[1]: Entering directory

Build failed in Jenkins: osx-master » clang,osx,debug #1443

2017-03-14 Thread jenkins
See Changes: [amc] Doc: Better TSContSchedule page. [amc] PR #1209: Remove unused classes from Map.h [amc] Add static type checking to configuration override API

Build failed in Jenkins: freebsd_10-master » clang,freebsd_10,debug #1489

2017-03-14 Thread jenkins
See -- [...truncated 1.66 MB...] gmake[1]: Leaving directory

Build failed in Jenkins: ubuntu_12_04-master » gcc,ubuntu_12_04,debug #2268

2017-03-14 Thread jenkins
See -- [...truncated 1.62 MB...] make[1]: Leaving directory

Build failed in Jenkins: fedora_24-master » gcc,fedora_24,debug #398

2017-03-14 Thread jenkins
See -- [...truncated 1.52 MB...] make[1]: Entering directory

Build failed in Jenkins: fedora_25-master » gcc,fedora_25,debug #125

2017-03-14 Thread jenkins
See -- [...truncated 1.52 MB...] make[1]: Entering directory

Build failed in Jenkins: centos_7-master » gcc,centos_7,debug #2272

2017-03-14 Thread jenkins
See -- [...truncated 1.51 MB...] make[1]: Leaving directory

Build failed in Jenkins: ubuntu_14_04-master » gcc,ubuntu_14_04,debug #2496

2017-03-14 Thread jenkins
See -- [...truncated 1.60 MB...] make[1]: Leaving directory

[GitHub] trafficserver pull request #1479: Add static type checking to configuration ...

2017-03-14 Thread SolidWallOfCode
Github user SolidWallOfCode closed the pull request at: https://github.com/apache/trafficserver/pull/1479 --- 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

[GitHub] trafficserver pull request #1209: Remove unused and never to be used classes...

2017-03-14 Thread SolidWallOfCode
Github user SolidWallOfCode closed the pull request at: https://github.com/apache/trafficserver/pull/1209 --- 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

[GitHub] trafficserver issue #1502: issue #1501: reconstruct to load the default valu...

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1502 clang-analyzer build *successful*! https://ci.trafficserver.apache.org/job/clang-analyzer-github/321/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1502: issue #1501: reconstruct to load the default valu...

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1502 Intel CC build *successful*! https://ci.trafficserver.apache.org/job/icc-github/189/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1502: issue #1501: reconstruct to load the default valu...

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1502 FreeBSD11 build *successful*! https://ci.trafficserver.apache.org/job/freebsd-github/1758/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1502: issue #1501: reconstruct to load the default valu...

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1502 Linux build *successful*! https://ci.trafficserver.apache.org/job/linux-github/1652/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1502: issue #1501: reconstruct to load the default valu...

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1502 Please rebase with current master, and also run "make clang-format" on this. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] trafficserver pull request #1502: issue #1501: reconstruct to load the defau...

2017-03-14 Thread zwoop
Github user zwoop commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1502#discussion_r106057816 --- Diff: proxy/ControlMatcher.cc --- @@ -220,6 +220,7 @@ HostMatcher::NewEntry(matcher_line *line_info) if (error.failed())

[GitHub] trafficserver issue #1502: issue #1501: reconstruct to load the default valu...

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1502 clang format *failed*! https://ci.trafficserver.apache.org/job/clang-format-github/63/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1502: issue #1501: reconstruct to load the default valu...

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1502 RAT check *successful*! https://ci.trafficserver.apache.org/job/RAT-github/76/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] trafficserver issue #1502: issue #1501: reconstruct to load the default valu...

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1502 AU check *successful*! https://ci.trafficserver.apache.org/job/autest-github/60/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] trafficserver issue #1502: issue #1501: reconstruct to load the default valu...

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1502 This sounds like a potential candidate for 7.1.0 as well. --- 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

[GitHub] trafficserver issue #1502: issue #1501: reconstruct to load the default valu...

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1502 [approve ci] --- 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,

[GitHub] trafficserver issue #1558: redundant url encoding for logging field "cquuc"

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/issues/1558 You should upgrade to 6.2.x, if that doesn't have this fix, you can request this to be back ported to 6.2.2, since 6.2 is an LTS version. --- If your project is set up for it, you can reply

[GitHub] trafficserver issue #1568: TS-4195: double free on exit

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1568 Once landed, I think we should contemplate this for a 7.1.0 cherry-pick. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] trafficserver issue #1565: Fix Assertion failure in the regex_revalidate plu...

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1565 Merged to 7.1.0. --- 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

[GitHub] trafficserver pull request #1565: Fix Assertion failure in the regex_revalid...

2017-03-14 Thread zwoop
Github user zwoop closed the pull request at: https://github.com/apache/trafficserver/pull/1565 --- 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] trafficserver issue #1209: Remove unused and never to be used classes from M...

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1209 Time to land this yet? --- 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] trafficserver issue #1557: brotli support in gzip plugin

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1557 See also ##776. --- 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

[GitHub] trafficserver issue #1557: brotli support in gzip plugin

2017-03-14 Thread shukitchan
Github user shukitchan commented on the issue: https://github.com/apache/trafficserver/pull/1557 My feedbacks 1) Perhaps we are overloading the "Accept-Encoding" header in the request. Let's say we create another header called "X-Normalized-Accept-Encoding". Its value will

[GitHub] trafficserver issue #1567: [RFC] Diags scrubbing mechanism

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1567 clang-analyzer build *successful*! https://ci.trafficserver.apache.org/job/clang-analyzer-github/320/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1567: [RFC] Diags scrubbing mechanism

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1567 Intel CC build *successful*! https://ci.trafficserver.apache.org/job/icc-github/188/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1567: [RFC] Diags scrubbing mechanism

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1567 FreeBSD11 build *successful*! https://ci.trafficserver.apache.org/job/freebsd-github/1757/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1567: [RFC] Diags scrubbing mechanism

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1567 AU check *failed*! https://ci.trafficserver.apache.org/job/autest-github/59/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] trafficserver issue #1299: TS-2888: remove vararg and format parameters from...

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1299 clang-analyzer build *successful*! https://ci.trafficserver.apache.org/job/clang-analyzer-github/319/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1567: [RFC] Diags scrubbing mechanism

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1567 Linux build *successful*! https://ci.trafficserver.apache.org/job/linux-github/1651/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1567: [RFC] Diags scrubbing mechanism

2017-03-14 Thread danobi
Github user danobi commented on the issue: https://github.com/apache/trafficserver/pull/1567 @SolidWallOfCode This patch is done on my end and ready for review. If I find some time tonight I'll stare at the code and see if I can catch any bugs. It doesn't have any AFAICT. --- If

[GitHub] trafficserver issue #1567: [RFC] Diags scrubbing mechanism

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1567 clang format *successful*! https://ci.trafficserver.apache.org/job/clang-format-github/62/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1567: [RFC] Diags scrubbing mechanism

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1567 RAT check *successful*! https://ci.trafficserver.apache.org/job/RAT-github/75/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] trafficserver issue #1299: TS-2888: remove vararg and format parameters from...

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1299 Linux build *successful*! https://ci.trafficserver.apache.org/job/linux-github/1650/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1299: TS-2888: remove vararg and format parameters from...

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1299 FreeBSD11 build *successful*! https://ci.trafficserver.apache.org/job/freebsd-github/1756/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1299: TS-2888: remove vararg and format parameters from...

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1299 Intel CC build *successful*! https://ci.trafficserver.apache.org/job/icc-github/187/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1371: Calling setrlimit() twice is redundant

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1371 clang-analyzer build *successful*! https://ci.trafficserver.apache.org/job/clang-analyzer-github/318/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1299: TS-2888: remove vararg and format parameters from...

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1299 AU check *successful*! https://ci.trafficserver.apache.org/job/autest-github/58/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] trafficserver issue #1299: TS-2888: remove vararg and format parameters from...

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1299 RAT check *successful*! https://ci.trafficserver.apache.org/job/RAT-github/74/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] trafficserver issue #1299: TS-2888: remove vararg and format parameters from...

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1299 clang format *successful*! https://ci.trafficserver.apache.org/job/clang-format-github/61/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1575: 5xx response instead of stale content

2017-03-14 Thread mlibbey
Github user mlibbey commented on the issue: https://github.com/apache/trafficserver/issues/1575 - in whatever fix we do, should make sure that the CC:must-revalidate does return the 5xx error (eg, the must-revalidate does not allow for serving stale --- If your project is set up for

[GitHub] trafficserver issue #1565: Fix Assertion failure in the regex_revalidate plu...

2017-03-14 Thread SolidWallOfCode
Github user SolidWallOfCode commented on the issue: https://github.com/apache/trafficserver/pull/1565 @jpeach I'll take care of updating the docs in a direct commit, it's not worth a PR. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1550: rectify a minor error in healthchecks.c

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1550 clang-analyzer build *successful*! https://ci.trafficserver.apache.org/job/clang-analyzer-github/317/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request #1552: rectify a minor error in file test_interce...

2017-03-14 Thread shukitchan
Github user shukitchan closed the pull request at: https://github.com/apache/trafficserver/pull/1552 --- 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] trafficserver issue #1552: rectify a minor error in file test_intercept.lua

2017-03-14 Thread shukitchan
Github user shukitchan commented on the issue: https://github.com/apache/trafficserver/pull/1552 it looks good. 👍 --- 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

[GitHub] trafficserver issue #1371: Calling setrlimit() twice is redundant

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1371 Linux build *successful*! https://ci.trafficserver.apache.org/job/linux-github/1649/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1371: Calling setrlimit() twice is redundant

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1371 FreeBSD11 build *successful*! https://ci.trafficserver.apache.org/job/freebsd-github/1755/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1552: rectify a minor error in file test_intercept.lua

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1552 clang-analyzer build *successful*! https://ci.trafficserver.apache.org/job/clang-analyzer-github/316/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1550: rectify a minor error in healthchecks.c

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1550 Linux build *successful*! https://ci.trafficserver.apache.org/job/linux-github/1648/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1552: rectify a minor error in file test_intercept.lua

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1552 Linux build *successful*! https://ci.trafficserver.apache.org/job/linux-github/1647/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1550: rectify a minor error in healthchecks.c

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1550 FreeBSD11 build *successful*! https://ci.trafficserver.apache.org/job/freebsd-github/1754/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1552: rectify a minor error in file test_intercept.lua

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1552 FreeBSD11 build *successful*! https://ci.trafficserver.apache.org/job/freebsd-github/1753/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1371: Calling setrlimit() twice is redundant

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1371 RAT check *successful*! https://ci.trafficserver.apache.org/job/RAT-github/73/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] trafficserver issue #1580: change command to use python3

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1580 clang-analyzer build *successful*! https://ci.trafficserver.apache.org/job/clang-analyzer-github/315/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1371: Calling setrlimit() twice is redundant

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1371 clang format *successful*! https://ci.trafficserver.apache.org/job/clang-format-github/60/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1371: Calling setrlimit() twice is redundant

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1371 Intel CC build *failed*! https://ci.trafficserver.apache.org/job/icc-github/186/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] trafficserver issue #1550: rectify a minor error in healthchecks.c

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1550 clang format *successful*! https://ci.trafficserver.apache.org/job/clang-format-github/59/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1550: rectify a minor error in healthchecks.c

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1550 RAT check *successful*! https://ci.trafficserver.apache.org/job/RAT-github/72/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] trafficserver issue #1371: Calling setrlimit() twice is redundant

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1371 AU check *successful*! https://ci.trafficserver.apache.org/job/autest-github/57/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] trafficserver issue #1552: rectify a minor error in file test_intercept.lua

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1552 RAT check *successful*! https://ci.trafficserver.apache.org/job/RAT-github/71/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] trafficserver pull request #1557: brotli support in gzip plugin

2017-03-14 Thread shukitchan
Github user shukitchan commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1557#discussion_r106037813 --- Diff: plugins/gzip/gzip.cc --- @@ -350,30 +435,89 @@ gzip_transform_finish(GzipData *data) if (data->downstream_length !=

[GitHub] trafficserver issue #1552: rectify a minor error in file test_intercept.lua

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1552 Intel CC build *successful*! https://ci.trafficserver.apache.org/job/icc-github/184/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1550: rectify a minor error in healthchecks.c

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1550 Intel CC build *successful*! https://ci.trafficserver.apache.org/job/icc-github/185/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1552: rectify a minor error in file test_intercept.lua

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1552 clang format *successful*! https://ci.trafficserver.apache.org/job/clang-format-github/58/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1552: rectify a minor error in file test_intercept.lua

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1552 AU check *failed*! https://ci.trafficserver.apache.org/job/autest-github/55/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] trafficserver issue #1550: rectify a minor error in healthchecks.c

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1550 AU check *failed*! https://ci.trafficserver.apache.org/job/autest-github/56/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...

2017-03-14 Thread persiaAziz
Github user persiaAziz commented on the issue: https://github.com/apache/trafficserver/pull/1503 7.1.x has good amount of works on client certs, so yes this should go in 7.1.x too :) . Like I said, it is unnecessary to check for client cert in the remap function when the cert is only

[GitHub] trafficserver issue #1371: Calling setrlimit() twice is redundant

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1371 [approve ci] --- 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,

[GitHub] trafficserver issue #1503: client cert should be added to netvcoptions only ...

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1503 What's the word on this PR? 1) Should we land it? 2) Do we still want this to 7.1.x? Note sure why it's been sitting for so long. --- If your project is set up for it, you can reply to this

[GitHub] trafficserver pull request #1580: change command to use python3

2017-03-14 Thread zwoop
Github user zwoop closed the pull request at: https://github.com/apache/trafficserver/pull/1580 --- 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] trafficserver issue #1497: Doc: change .svg to .png.

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1497 @SolidWallOfCode Why is this not merged? --- 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] trafficserver issue #1565: Fix Assertion failure in the regex_revalidate plu...

2017-03-14 Thread jrushford
Github user jrushford commented on the issue: https://github.com/apache/trafficserver/pull/1565 @zwoop there was a change, https://issues.apache.org/jira/browse/TS-4387, made that requires a mutex when using TSContSchedule(). With the PR, all plugins using TSContSchedule were

[GitHub] trafficserver issue #1517: Remove BisonHeaderToC++.sed

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1517 Note that it has to succeed on CentOS6, which is indeed running ``` [root@centos6 ~]# bison --version bison (GNU Bison) 2.4.1 ``` --- If your project is set up for it,

[GitHub] trafficserver issue #1548: Insure that parent health stats are updated prope...

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1548 @jrushford Do you want this in 7.1.0? If not, move it out to 7.2.0 please. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] trafficserver issue #1552: rectify a minor error in file test_intercept.lua

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1552 [approve ci] --- 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,

[GitHub] trafficserver issue #1552: rectify a minor error in file test_intercept.lua

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1552 @shukitchan You ok with this? --- 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

[GitHub] trafficserver issue #1550: rectify a minor error in healthchecks.c

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1550 [approve ci] to see if the FreeBSD builds succeed. --- 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] trafficserver issue #1580: change command to use python3

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1580 Intel CC build *successful*! https://ci.trafficserver.apache.org/job/icc-github/183/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1581: TS-4976: Regularize plugins - file

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1581 clang-analyzer build *successful*! https://ci.trafficserver.apache.org/job/clang-analyzer-github/314/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1580: change command to use python3

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1580 Linux build *successful*! https://ci.trafficserver.apache.org/job/linux-github/1646/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1582: Make regex_revalidate.c use ref-counted config da...

2017-03-14 Thread zwoop
GitHub user zwoop opened an issue: https://github.com/apache/trafficserver/issues/1582 Make regex_revalidate.c use ref-counted config data Instead of scheduling a "delete" of the data sometime in the future, a more reliable pattern is to ref-count the configurations, and acquire /

[GitHub] trafficserver issue #1580: change command to use python3

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1580 FreeBSD11 build *successful*! https://ci.trafficserver.apache.org/job/freebsd-github/1752/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver pull request #1565: Fix Assertion failure in the regex_revalid...

2017-03-14 Thread zwoop
Github user zwoop commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1565#discussion_r106033633 --- Diff: plugins/regex_revalidate/regex_revalidate.c --- @@ -362,7 +362,7 @@ config_handler(TSCont cont, TSEvent event ATS_UNUSED, void *edata

[GitHub] trafficserver issue #1580: change command to use python3

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1580 AU check *successful*! https://ci.trafficserver.apache.org/job/autest-github/54/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] trafficserver issue #1581: TS-4976: Regularize plugins - file

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1581 Linux build *successful*! https://ci.trafficserver.apache.org/job/linux-github/1645/ --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver issue #1580: change command to use python3

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1580 RAT check *successful*! https://ci.trafficserver.apache.org/job/RAT-github/70/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] trafficserver issue #1580: change command to use python3

2017-03-14 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1580 clang format *successful*! https://ci.trafficserver.apache.org/job/clang-format-github/57/ --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] trafficserver issue #1579: Clean up h2client.py to be a general testing tool

2017-03-14 Thread persiaAziz
Github user persiaAziz commented on the issue: https://github.com/apache/trafficserver/issues/1579 I am working (read "hacking hyper") on the h2 module of Traffic-Replay. Once that gets open sourced, we will have a better h2 client. The current h2client is meant just for basic h2

[GitHub] trafficserver pull request #1576: Add option to choose interface

2017-03-14 Thread zwoop
Github user zwoop closed the pull request at: https://github.com/apache/trafficserver/pull/1576 --- 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] trafficserver issue #1580: change command to use python3

2017-03-14 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1580 [approve ci] --- 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,

  1   2   >