newbie question

2015-01-23 Thread Michael Herstine
Hi all, Just grabbed the source & wanted to use Vagrant to try it out without modifying my development machine. Then I noticed there’s already a Vagrantfile present, and it seems to do what I want (mounts the trafficserver source in the VM). But when I do a ‘vagrant up’ I get a lot of errors ab

Re: newbie question

2015-01-23 Thread Steve Malenfant
Guests additions are specific to Vagrant/Virtualbox, not to ATS. I use it for testing, but don't use the provided Vagrantfile. On Fri, Jan 23, 2015 at 1:15 PM, Michael Herstine < mherst...@linkedin.com.invalid> wrote: > Hi all, > > Just grabbed the source & wanted to use Vagrant to try it out wit

Fwd: New Defects reported by Coverity Scan for Apache Traffic Server

2015-01-23 Thread Leif Hedstrom
Anyone recognize these three new Coverity errors as part of their recent (~24h) commits? — Leif > Begin forwarded message: > > Date: January 23, 2015 at 2:59:10 AM MST > From: scan-ad...@coverity.com > To: zw...@apache.org > Subject: New Defects reported by Coverity Scan for Apache Traffic Ser

Re: Fwd: New Defects reported by Coverity Scan for Apache Traffic Server

2015-01-23 Thread Susan Hinrichs
I've touched HttpTransact.cc in the past 24 hours, but didn't change anything that would obviously trigger these new warnings. Looks like we should be null checking the results of find_field, but that logic doesn't seem to have changed. So don't know why coverity is complaining now. On 1/23

[ANNOUNCE} Apache Traffic Server Spring 2015 Summit!

2015-01-23 Thread Leif Hedstrom
Hi all, We’re pleased to announce our next ATS Summit! It’ll be held adjacent to the 2015 ApacheCon North America Conference, Thursday - Friday April 16-17th. In addition, we’ll do an informal Hackathon on Sunday April 12th, and possibly Saturday April 18th. Details on the Summit is still being

[GitHub] trafficserver pull request: New TSQA

2015-01-23 Thread jacksontj
Github user jacksontj commented on the pull request: https://github.com/apache/trafficserver/pull/163#issuecomment-71288426 @bgaff I squashed all of the commits into one. I've also changed it from traffic_cop to traffic_server while we wait for @jpeach 's commit to land. --- If your

[GitHub] trafficserver pull request: New TSQA

2015-01-23 Thread jacksontj
Github user jacksontj commented on the pull request: https://github.com/apache/trafficserver/pull/163#issuecomment-71289722 A point @bgaff brought up was that tsqa (the harness/framework) lives outside this source tree. This is intentional as the *tests* are tied to the trafficserver

[GitHub] trafficserver pull request: New TSQA

2015-01-23 Thread jacksontj
Github user jacksontj commented on the pull request: https://github.com/apache/trafficserver/pull/163#issuecomment-71289896 Also, do we need the Apache file headers on all test cases as well? (@zwoop would probably be the one to answer) --- If your project is set up for it, you can

[GitHub] trafficserver pull request: New TSQA

2015-01-23 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/163#issuecomment-71293965 Yes. > On Jan 23, 2015, at 5:29 PM, Thomas Jackson wrote: > > Also, do we need the Apache file headers on all test cases as well? (@zwoo

[GitHub] trafficserver pull request: New TSQA

2015-01-23 Thread bretep
Github user bretep commented on the pull request: https://github.com/apache/trafficserver/pull/163#issuecomment-71299489 @jacksontj how about adding it as a git submodule? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[GitHub] trafficserver pull request: New TSQA

2015-01-23 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/163#issuecomment-71299904 Not sure why you would prefer not to just merge it in, but I can with 99% certainty say that we would not include it as a git submodule. We would consider it as a gi

[GitHub] trafficserver pull request: New TSQA

2015-01-23 Thread bretep
Github user bretep commented on the pull request: https://github.com/apache/trafficserver/pull/163#issuecomment-7124 @zwoop I can get behind that. I'd much rather have it merged myself. --- If your project is set up for it, you can reply to this email and have your reply appear o