<div dir="ltr">Hi Ramesh,<div class="gmail_extra"><br><div class="gmail_quote">On 15 October 2015 at 11:53, Ramakrishnan G <span dir="ltr"><<a href="mailto:rameshg87.openstack@gmail.com" target="_blank">rameshg87.openstack@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><div><br></div><div><br></div><div>So what all are the problems ?</div><div>1) Ability to update the driver documentation not-related to Ironic easily without waiting. </div><div>2) To save some core reviewers time who might not be familiar with the hardware.</div><div><br></div><div>To solve the actual problem of updating the documentation easily while keeping it in-tree, I checked with infra folks if a subset of a repository can be +2ed/+Aed by another group.  They confirmed it's not possible (unless there was a communication gap in that conversation, folks can correct me if I am wrong).</div><div><br></div><div>The following are the options that I can think of to address this:</div><div><br></div><div>1) Easy approvals for patches solely related to driver documentation. Once the driver team feels the documentation is ready, it can be +Aed by a core team member skipping the normal process of review. Of course, fixing any comments that come by, but not waiting for the normal rule of 2x+2s.</div></div></div></blockquote><div><br></div><div>To date, when there is a driver documentation patch that is submitted, does the driver team review them? Are there past cases where this has occurred and there wasn't any 'useful' feedback from other reviewers before it got the +A?</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><div><br></div><div>2) A separate repository for driver documentation controller by driver developers (a bad idea ??)</div></div></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><div><br></div><div>3) Allow to push driver documentation to wiki for those who wish to.</div></div></div></blockquote><div><br></div><div>My preference is for the driver documentation to be outside any ironic repository that I feel responsible for reviewing. I.e., I want to reduce the number of patches that need to be reviewed :) I would love if the driver documentation was outside, reviewed by the driver folks (however they want to review it) and their own tech writers or whatever.</div><div><br></div><div>I took a look at Jim's comments on that patch and I'll copy some of it here (hope you don't mind Jim):</div><div><br></div><div>-----</div><div><p style="margin-top:0px;margin-bottom:0px;padding-top:0.5em;padding-bottom:0.5em;color:rgb(0,0,0);font-family:sans-serif">Totally opposed to documentation on the wiki. Docs should be reviewed (anyone with an Launchpad account can edit the wiki without review).</p><p style="margin-top:0px;margin-bottom:0px;padding-top:0.5em;padding-bottom:0.5em;color:rgb(0,0,0);font-family:sans-serif">Also, in-tree docs are so much prettier, and can be tied to a release if we decide to do so. :)</p><p style="margin-top:0px;margin-bottom:0px;padding-top:0.5em;padding-bottom:0.5em;color:rgb(0,0,0);font-family:sans-serif">If there's too much overhead with keeping docs in tree, let's solve *that* problem rather than just removing the docs.</p><p style="margin-top:0px;margin-bottom:0px;padding-top:0.5em;padding-bottom:0.5em;color:rgb(0,0,0);font-family:sans-serif">------</p><p style="margin-top:0px;margin-bottom:0px;padding-top:0.5em;padding-bottom:0.5em;color:rgb(0,0,0);font-family:sans-serif">Ramesh -- assuming I'm the odd person out and most people want the documentation in-tree, let's try to look at and address the overhead with keeping docs in tree. Perhaps you could elaborate on the problems (the 2 points you mention in your email). Eg, do people feel like there are too many nits or other unnecessary comments that cause too many revisions for very-little-benefit, or that no one 'ever' looks at the patch, or that even a 1-day delay in getting a patch merged is too long, or what?</p><p style="margin-top:0px;margin-bottom:0px;padding-top:0.5em;padding-bottom:0.5em;color:rgb(0,0,0);font-family:sans-serif">--ruby</p></div><div><br></div></div><br></div></div>