<br><br><div class="gmail_quote">On Thu, Oct 11, 2012 at 7:40 PM, Anish Mangal <span dir="ltr"><<a href="mailto:anishmangal2002@gmail.com" target="_blank">anishmangal2002@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<p>I guess one thing to remember here is that we have a dedicated quality assurance team.</p>
<p>As a developer it makd s sense to have a fast dev cycle<br>
As a PM it makes sense to have quality release<br>
As a qa person it makes sense to be thd last person to signoff before a package is declared golden<br>
As a customer any bug that passes through devel to golden untested is a big failure on ACs part</p>
<p>So<br>
We need to think not only in terms of developers being able to come up with quick changes but qa also being able to test them. Those processes need to bw delinked as much as possoble so they dont block on each other unnecessarily</p>
</blockquote><div><br>+1. That's why Rafa point makes absolute sense, to have a devel-branch before the final release (in case development needs to start on new feaures, that ought NOT to go in the release).<br> <br></div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<p>Also I dont buy the argument that incentivizing developers to test more is going to solve our testing requirements. There is a reason companies invest in q.a. developers test pieces of code. Q.a. tests the functionality and its correctness. That is ultimately what is most important to the customer.</p>
</blockquote><div><br>+10.<br>Here, by testing I meant just a confirmation that the patch worked in the "broad" sense, and did not render the system/UI usable (this is a part of on-going discussions with Rafa, as to how to enable code-review, so that there are more eyes for the code, and developers in general remain in sync).<br>
<br>I compleltely agree that the finer/polishing touches need to be given by the QA, as they don't face the mental-blocks that developers tend to have (at least I do, when I am in that "developer" zone :D ).<br>
<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<p>Just my 2c</p><div class="HOEnZb"><div class="h5">
<div class="gmail_quote">On Oct 11, 2012 9:40 AM, "Ajay Garg" <<a href="mailto:ajay@activitycentral.com" target="_blank">ajay@activitycentral.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br><br><div class="gmail_quote">On Thu, Oct 11, 2012 at 6:54 PM, Ruben Rodríguez <span dir="ltr"><<a href="mailto:ruben@activitycentral.com" target="_blank">ruben@activitycentral.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


2012/10/11 Ajay Garg <<a href="mailto:ajay@activitycentral.com" target="_blank">ajay@activitycentral.com</a>>:<br>
<div>> Having so many branches is an overkill; it is too efforts-intensive,<br>
> error-prone and generally daunting.<br>
<br>
</div>It may be a bigger effort when you compare it with how we work now,<br>
but there are good reasons for that scheme:<br>
<br>
* A lot more provision against failure. Think of what would happen if<br>
we ever push a rpm update that unexpectedly makes the machines stop to<br>
boot or causes some other big trouble. I've seen that happen in<br>
several distros, and it is a project killer. At least we need to<br>
separate what code is golden, what is being tested by the QA team and<br>
what is fresh from the coders.<br></blockquote><div><br>Well, I think the onus lies on the developer then, since he/she was responsible for committing a fix that caused the catastrophe.<br>Moreover, I think this is a blessing in disguise (if I may say), as this will cause such catastrophes to be caught much earlier.<br>


<br><br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
* Most of that work is supposed to be automated or to be done by the<br>
RM. Code going up the branches should always be fast-forward.<br></blockquote><div><br>Again, another candidate for time-wastage. Why not work in one branch itself?  That would prevent un-necessary duplicate testing by the RM.<br>


<br>It makes MUCH, MUCH, MUCH more sense to concentrate all testing for one codebase (i.e. for one code, corresponding to one branch).<br>(and this applies to everyone, not just the RM).<br><br><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">



* The developers only work in their personal clones of the devel<br>
branch,</blockquote><div><br><br>That is understood. As long as developers don't commit, they are free to  do anything they want to do on the branch, as long as they don't commit. A code is "effectively" in the branch, only and only after it is remotely-pushed.<br>


<br><br><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> so from their perspective there is only *one* branch that<br>
matters. All the rest is risk management. Of course this is a<br>
provision for more developers working on the code in the future, so I<br>
understand it can look overkill now that you do most of the code alone<br>
:)<br>
* Some steps in the scheme are optional and can be removed, like the<br>
master and the production branches being different. As with any<br>
proposal this needs to be tuned up.<br></blockquote><div><br><br>All this I am speaking from personal experience, as giving commit-access to the develper was a big step in saving time (courtesy Anish).<br><br>Earlier, before committing, me and Anish had to go through the testing process individually (meaning double effort; surely no gain). Instead, Anish then started to test on a RPM-level (i.e. building rpms, and deploying on the XO). That served the following purposes ::<br>


<br>a)<br>End-to-End testing<br><br>b)<br>Testing at a rpm-level, not patch-level.<br><br><br><br>Anyways, now that the code-review would be in-place soon, the committer (other than the developer) will anyways test the patch once before committing (thus providing more security against catastrophes).<br>


<br><br>In summary, I am not in favour of multiple branches. As Rafa said, we may split into a devel-branch sometime just before the final release; but even that too if necessary.<br><br>Let's conceentrate on just one branch - meaning one branch to test.<br>


<br><br><br><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div><div><br>
--<br>
Rubén Rodríguez<br>
CTO<br>
Activity Central: <a href="http://activitycentral.com" target="_blank">http://activitycentral.com</a><br>
<br>
Facebook: <a href="https://activitycentral.com/facebook" target="_blank">https://activitycentral.com/facebook</a><br>
Google+: <a href="https://activitycentral.com/googleplus" target="_blank">https://activitycentral.com/googleplus</a><br>
Twitter: <a href="https://activitycentral.com/twitter" target="_blank">https://activitycentral.com/twitter</a><br>
_______________________________________________<br>
Dextrose mailing list<br>
<a href="mailto:Dextrose@lists.sugarlabs.org" target="_blank">Dextrose@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/dextrose" target="_blank">http://lists.sugarlabs.org/listinfo/dextrose</a><br>
</div></div></blockquote></div><br><br clear="all"><br><font face="arial, sans-serif">Regards,<br><br>Ajay Garg</font><br style="font-size:13px;font-family:arial,sans-serif"><font face="arial, sans-serif">Dextrose Developer</font><br style="font-size:13px;font-family:arial,sans-serif">


<span style="font-size:13px;font-family:arial,sans-serif">Activity Central: </span><a href="http://activitycentral.com/" style="font-size:13px;font-family:arial,sans-serif" target="_blank">http://activitycentral.com</a><br>



<br>_______________________________________________<br>
Dextrose mailing list<br>
<a href="mailto:Dextrose@lists.sugarlabs.org" target="_blank">Dextrose@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/dextrose" target="_blank">http://lists.sugarlabs.org/listinfo/dextrose</a><br>
<br></blockquote></div>
</div></div></blockquote></div><br><br clear="all"><br><font face="arial, sans-serif">Regards,<br><br>Ajay Garg</font><br style="font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)"><font face="arial, sans-serif">Dextrose Developer</font><br style="font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">
<span style="font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">Activity Central: </span><a href="http://activitycentral.com/" style="font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)" target="_blank">http://activitycentral.com</a><br>