[GitHub] trafficserver issue #1441: Mark the new span metrics as gauges in Epic plugi...

2017-02-13 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1441 clang-analyzer build *successful*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/104/ for details. --- If your project is set up for it, you can reply to this email

[GitHub] trafficserver pull request #1440: Remove nullptr instances that were introdu...

2017-02-13 Thread PSUdaemon
Github user PSUdaemon closed the pull request at: https://github.com/apache/trafficserver/pull/1440 --- 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 #1441: Mark the new span metrics as gauges in Epic plugi...

2017-02-13 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1441 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/freebsd-github/1540/ for details. --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver issue #1427: Crash in ATSConsistentHash::lookup_by_hashval

2017-02-13 Thread jrushford
Github user jrushford commented on the issue: https://github.com/apache/trafficserver/issues/1427 Randall, I'm unable to duplicate in my environment, CentOS 7.2 and gcc 4.8.5-4. What OS are you using and can you attach your parent.config? --- If your project is set up for it, you

[GitHub] trafficserver pull request #1441: Mark the new span metrics as gauges in Epi...

2017-02-13 Thread zwoop
Github user zwoop closed the pull request at: https://github.com/apache/trafficserver/pull/1441 --- 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 #1441: Mark the new span metrics as gauges in Epic plugi...

2017-02-13 Thread gtenev
Github user gtenev commented on the issue: https://github.com/apache/trafficserver/pull/1441 👍 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 enabled and wishes

[GitHub] trafficserver pull request #1441: Mark the new span metrics as gauges in Epi...

2017-02-13 Thread zwoop
GitHub user zwoop opened a pull request: https://github.com/apache/trafficserver/pull/1441 Mark the new span metrics as gauges in Epic plugin You can merge this pull request into a Git repository by running: $ git pull https://github.com/zwoop/trafficserver EpicSpanMetrics

[GitHub] trafficserver issue #1441: Mark the new span metrics as gauges in Epic plugi...

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

[GitHub] trafficserver issue #1331: MemView is a utility to create read only views of...

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

[GitHub] trafficserver pull request #1442: Doc: Add an example of disabled HTTP/2 ove...

2017-02-13 Thread masaori335
GitHub user masaori335 opened a pull request: https://github.com/apache/trafficserver/pull/1442 Doc: Add an example of disabled HTTP/2 over TLS You can merge this pull request into a Git repository by running: $ git pull https://github.com/masaori335/trafficserver doc-proto

[GitHub] trafficserver issue #1442: Doc: Add an example of disabled HTTP/2 over TLS

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

[GitHub] trafficserver issue #1442: Doc: Add an example of disabled HTTP/2 over TLS

2017-02-13 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1442 clang-analyzer build *successful*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/106/ for details. --- If your project is set up for it, you can reply to this email

[GitHub] trafficserver issue #1401: Segfault in write_to_net_io with 7.1.x

2017-02-13 Thread shinrich
Github user shinrich commented on the issue: https://github.com/apache/trafficserver/issues/1401 I've finally got my environment running and I see the same stack very quickly as well. In the cases I've seen it looks like there was a write error, but for some reason the write vio has

[GitHub] trafficserver issue #1442: Doc: Add an example of disabled HTTP/2 over TLS

2017-02-13 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1442 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/freebsd-github/1542/ for details. --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver issue #1331: MemView is a utility to create read only views of...

2017-02-13 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1331 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/freebsd-github/1541/ for details. --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver issue #1331: MemView is a utility to create read only views of...

2017-02-13 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1331 clang-analyzer build *failed*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/105/ for details. --- If your project is set up for it, you can reply to this email and

[GitHub] trafficserver issue #1401: Segfault in write_to_net_io with 7.1.x

2017-02-13 Thread PSUdaemon
Github user PSUdaemon commented on the issue: https://github.com/apache/trafficserver/issues/1401 I think I am [seeing this](https://ci.trafficserver.apache.org/job/ubuntu_14_04-6.2.x/compiler=clang,label=ubuntu_14_04,type=release/143/console) in 6.2.x as well: ```

[GitHub] trafficserver pull request #1420: Improve error message for custom log field...

2017-02-13 Thread SolidWallOfCode
Github user SolidWallOfCode commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1420#discussion_r100805070 --- Diff: proxy/logging/LogBuffer.cc --- @@ -615,10 +618,14 @@ LogBuffer::resolve_custom_entry(LogFieldList *fieldlist, char *printf_str,

[GitHub] trafficserver pull request #1440: Remove nullptr instances that were introdu...

2017-02-13 Thread PSUdaemon
GitHub user PSUdaemon opened a pull request: https://github.com/apache/trafficserver/pull/1440 Remove nullptr instances that were introduced from upstream backports… … and replace them with NULL. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] trafficserver pull request #1440: Remove nullptr instances that were introdu...

2017-02-13 Thread jablko
Github user jablko commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1440#discussion_r100845545 --- Diff: lib/raft/raft_impl.h --- @@ -338,8 +338,8 @@ template class RaftImpl : public Raft if (!i_am_leader()) return;

[GitHub] trafficserver issue #1331: MemView is a utility to create read only views of...

2017-02-13 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1331 FreeBSD build *failed*! See https://ci.trafficserver.apache.org/job/freebsd-github/1537/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1331: MemView is a utility to create read only views of...

2017-02-13 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1331 FreeBSD build *failed*! See https://ci.trafficserver.apache.org/job/freebsd-github/1539/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1331: MemView is a utility to create read only views of...

2017-02-13 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1331 clang-analyzer build *successful*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/101/ for details. --- If your project is set up for it, you can reply to this email

[GitHub] trafficserver issue #1331: MemView is a utility to create read only views of...

2017-02-13 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1331 FreeBSD build *failed*! See https://ci.trafficserver.apache.org/job/freebsd-github/1538/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1331: MemView is a utility to create read only views of...

2017-02-13 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1331 Linux build *failed*! See https://ci.trafficserver.apache.org/job/linux-github/1432/ for details. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] trafficserver issue #1331: MemView is a utility to create read only views of...

2017-02-13 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1331 clang-analyzer build *successful*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/103/ for details. --- If your project is set up for it, you can reply to this email

[GitHub] trafficserver issue #1440: Remove nullptr instances that were introduced fro...

2017-02-13 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1440 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/freebsd-github/1536/ for details. --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver issue #1440: Remove nullptr instances that were introduced fro...

2017-02-13 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1440 clang-analyzer build *successful*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/100/ for details. --- If your project is set up for it, you can reply to this email

[GitHub] trafficserver issue #1331: MemView is a utility to create read only views of...

2017-02-13 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1331 Linux build *failed*! See https://ci.trafficserver.apache.org/job/linux-github/1431/ for details. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] trafficserver issue #1331: MemView is a utility to create read only views of...

2017-02-13 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1331 Linux build *failed*! See https://ci.trafficserver.apache.org/job/linux-github/1430/ for details. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] trafficserver issue #1440: Remove nullptr instances that were introduced fro...

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

[GitHub] trafficserver issue #1331: MemView is a utility to create read only views of...

2017-02-13 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1331 clang-analyzer build *failed*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/102/ for details. --- If your project is set up for it, you can reply to this email and

[GitHub] trafficserver pull request #1440: Remove nullptr instances that were introdu...

2017-02-13 Thread PSUdaemon
Github user PSUdaemon commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1440#discussion_r100867264 --- Diff: lib/raft/raft_impl.h --- @@ -338,8 +338,8 @@ template class RaftImpl : public Raft if (!i_am_leader()) return;