[GitHub] trafficserver pull request: TS-4163 changes for ts_lua plugin docu...

2016-02-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/451 --- 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 pull request: TS-4176: s3_auth add support for matri...

2016-02-05 Thread robguima
Github user robguima commented on the pull request: https://github.com/apache/trafficserver/pull/455#issuecomment-180702368 @zwoop done thanks! --- 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 pull request: TS-4176: s3_auth add support for matri...

2016-02-05 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/455#issuecomment-180669497 Please squash this down to one commit IMO. --- 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 p

[GitHub] trafficserver pull request: TS-4176: s3_auth add support for matri...

2016-02-05 Thread robguima
Github user robguima commented on the pull request: https://github.com/apache/trafficserver/pull/455#issuecomment-180640259 @shukitchan just to undo my formatting change from older patch - think someone did that between 5.3 and 6.x --- If your project is set up for it, you can reply

Re: [VOTE] Release v6.1.1

2016-02-05 Thread Leif Hedstrom
> On Feb 2, 2016, at 9:27 PM, Leif Hedstrom wrote: > > Hi all, > > I've prepared a release candidate for v6.1.1, which includes: > > * [TS-4142] - Compile errors on OmniOS > * [TS-4165] - Logging breaks if changing log format > * [TS-4168] - Only close inactive and active connectio

[GitHub] trafficserver pull request: TS-4176: s3_auth add support for matri...

2016-02-05 Thread shukitchan
Github user shukitchan commented on the pull request: https://github.com/apache/trafficserver/pull/455#issuecomment-180587604 looks good now. --- 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 t

[GitHub] trafficserver pull request: TS-4178: Memory leak in SplitDNSConfig

2016-02-05 Thread bryancall
GitHub user bryancall opened a pull request: https://github.com/apache/trafficserver/pull/457 TS-4178: Memory leak in SplitDNSConfig You can merge this pull request into a Git repository by running: $ git pull https://github.com/bryancall/trafficserver ts-4178 Alternatively y

[GitHub] trafficserver pull request: TS-4177: Memory leak in LogFieldAliasT...

2016-02-05 Thread bryancall
Github user bryancall commented on the pull request: https://github.com/apache/trafficserver/pull/456#issuecomment-180567023 No, I didn't run clang-format on the pull request before pushing it to github. I have a script that automates committing pull requests and it will format it be

[GitHub] trafficserver pull request: TS-4177: Memory leak in LogFieldAliasT...

2016-02-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/456 --- 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 pull request: TS-4177: Memory leak in LogFieldAliasT...

2016-02-05 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/456#issuecomment-180551999 +1. Btw, is that really how clang-format indented it? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] trafficserver pull request: TS-4177: Memory leak in LogFieldAliasT...

2016-02-05 Thread bryancall
GitHub user bryancall opened a pull request: https://github.com/apache/trafficserver/pull/456 TS-4177: Memory leak in LogFieldAliasTable You can merge this pull request into a Git repository by running: $ git pull https://github.com/bryancall/trafficserver ts-4177 Alternative

[GitHub] trafficserver pull request: TS-4176: s3_auth add support for matri...

2016-02-05 Thread shukitchan
Github user shukitchan commented on the pull request: https://github.com/apache/trafficserver/pull/455#issuecomment-180535806 @jpeach , can you take a look 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

[GitHub] trafficserver pull request: TS-4176: s3_auth add support for matri...

2016-02-05 Thread shukitchan
Github user shukitchan commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/455#discussion_r52065719 --- Diff: plugins/experimental/s3_auth/s3_auth.cc --- @@ -346,26 +349,48 @@ S3Request::authorize(S3Config *s3) // If the configuration is a "v

[GitHub] trafficserver pull request: TS-4176: s3_auth add support for matri...

2016-02-05 Thread robguima
GitHub user robguima opened a pull request: https://github.com/apache/trafficserver/pull/455 TS-4176: s3_auth add support for matrix params @shukitchan please review https://issues.apache.org/jira/browse/TS-4176 You can merge this pull request into a Git repository by runn

Re: [VOTE] Release v6.1.1

2016-02-05 Thread Bryan Call
+ 1 - Running in production on RHEL 7 with ASAN build and tested on Fedora 23. -Bryan > On Feb 2, 2016, at 8:27 PM, Leif Hedstrom wrote: > > Hi all, > > I've prepared a release candidate for v6.1.1, which includes: > >* [TS-4142] - Compile errors on OmniOS >* [TS-4165] - Logging br

Re: [VOTE] Release v6.1.1

2016-02-05 Thread James Peach
+1 build and regressions on OS X and Fedora 23. > On Feb 2, 2016, at 8:27 PM, Leif Hedstrom wrote: > > Hi all, > > I've prepared a release candidate for v6.1.1, which includes: > > * [TS-4142] - Compile errors on OmniOS > * [TS-4165] - Logging breaks if changing log format > * [TS

Re: [VOTE] Release v6.1.1

2016-02-05 Thread Eric Sproul
On Tue, Feb 2, 2016 at 11:27 PM, Leif Hedstrom wrote: > Hi all, > > I've prepared a release candidate for v6.1.1, which includes: > > * [TS-4142] - Compile errors on OmniOS > * [TS-4165] - Logging breaks if changing log format > * [TS-4168] - Only close inactive and active connections

[GitHub] trafficserver pull request: TS-4172: Cleanup debug logs of Http2Co...

2016-02-05 Thread masaori335
Github user masaori335 commented on the pull request: https://github.com/apache/trafficserver/pull/454#issuecomment-180258234 I agree with we should keep consistency. So, I'm going to keep current names and debug tag in Http2ClientSession. The problem is same name (DebugHttp2S