<div dir="ltr">On Mon, Feb 10, 2014 at 4:09 PM, Sara Dickinson <span dir="ltr"><<a href="mailto:sara@sinodun.com" target="_blank">sara@sinodun.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
I have added a few extra bullet points and heading to the workflow page - please review my changes and make sure they all make sense!<br></blockquote><div><br>"<span><span class="">however remember that this will they are dependant on each other for being accepted into the upstream repository."<br>
</span></span></div><div><span><span class="">..that this will they are.. ? :)<br></span></span></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Jerry - that would be great - thank you. I think this is how most people will work so I think we need this asap. For example, if I have commits I made in subversion that I will  port after we move I will have to cherry pick right?<br>
</blockquote><div><br></div><div>Yes.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Question: Right now who has permission to run what bot commands?<br>
</blockquote><div><br></div><div>Should be all of us.<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
In the beginning I think I hear that the plan is that Jerry/Jakob will run every test on every pull request (not the developer)? Is this right? Will you do this immediately on receiving the request or only after review? How long does this take for the production releases - 2 hours I think for both DBs?<br>
</blockquote><div><br></div><div>Currently:<br></div><div>- After review.<br></div><div>- Only smoke.<br></div><div>- Only SQLite.<br></div><div>- 10-20mins.<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

Have I understood correctly that the aim (for pull requests that come from the development team) is to move quite quickly to a point where each developer starts (at least) a build on submitting a pull request and deals with any build/test failures before getting anyone else involved (reviewers or release manager)? This seems like one of the big wins to me that as a developer I can do this step by myself without the code actually going into the repo? Then it is the developer that co-ordinates the review and so the 'Jerry/Jakob/release manager' role is essentially just to do whatever extra review/testing is necessary as a sanity check before merging. Is this how people see this process developing?<br>
</blockquote><div><br></div><div>I can't answer for people but we have the option do allow developers to build and test their feature branch. Doing this does not really remove the requirement to have a separate set of eyes to look at the code.<br>
<br></div></div>-- <br>Jerry Lundström - OpenDNSSEC Developer<br><a href="http://www.opendnssec.org/" target="_blank">http://www.opendnssec.org/</a><br>
</div></div>