Hi, Before this change I had already been queuing the jobs myself as a reviewer. It also happens to be that many reviewers are committers. I wouldn't ask contributors to run the dtests/utests for any purpose other then so that they know the submission is done.
Even if they did and they pass it doesn't matter. It only matters if it passes in CI. If it fails in CI but passes on their desktop it's not good enough so we have to run in CI anyways. If a reviewer is not a committer. Well they can ask someone else to do it? I know we have issues with responsiveness, but I would make myself available for that. It shouldn't be a big problem because if someone is doing a lot of reviews they should be a committer right? Regards, Ariel On Thu, Mar 9, 2017, at 01:51 PM, Jason Brown wrote: > Hey all, > > A nice convention we've stumbled into wrt to patches submitted via Jira > is > to post the results of unit test and dtest runs to the ticket (to show > the > patch doesn't break things). Many contributors have used the > DataStax-provided cassci system, but that's not the best long term > solution. To that end, I'd like to start a conversation about what is the > best way to proceed going forward, and then add it to the "How to > contribute" docs. > > As an example, should contributors/committers run dtests and unit tests > on > *some* machine (publicly available or otherwise), and then post those > results to the ticket? This could be a link to a build system, like what > we > have with cassci, or just upload the output of the test run(s). > > I don't have any fixed notions, and am looking forward to hearing other's > ideas. > > Thanks, > > -Jason > > p.s. a big thank you to DataStax for providing the cassci system
