<div dir="ltr"><div>Using the tests for Pulp 3 as example, since the code basis is hosted in Pulp repos, I think that could be created a QE group to be involved in any new code changes related to the tests. <br></div><div><br></div><div>My suggestion is at least one QE person should review changes to the code basis
 related to the tests. This will help us later on to assure that a test should behave in a certain way and that certain feature was already tested.<br></div><div><br></div><div>But I guess that discussion going on here is more broad.<br></div><div> <br></div><div><br></div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 15, 2018 at 12:36 PM, David Davis <span dir="ltr"><<a href="mailto:daviddavis@redhat.com" target="_blank">daviddavis@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Doing some research on code owners, I found[0] that I was wrong in that code owners have to have the commit bit:<div><br></div><div><div><i>The people you choose as code owners must have write permissions for the repository.</i></div><div><br></div><div><div dir="ltr" class="m_1285131244357952184m_949101636729261770gmail_signature"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>I think this makes the code owners feature less useful to us. More importantly though, it means that we need to have a way for people to obtain the commit bit before they can become code owners.</div><div><br></div><div>I’d propose that we table this discussion for now and instead focus on the process for obtaining the commit bit. @ipanova is working on a new draft of the commit bit PUP.</div><div><br></div><div>[0] <a href="https://help.github.com/articles/about-codeowners/" target="_blank">https://help.github.com/<wbr>articles/about-codeowners/</a></div><span class="HOEnZb"><font color="#888888"><div><br></div><div>David<br></div></font></span></div></div></div></div></div><br></div></div><div class="HOEnZb"><div class="h5"><br><div class="gmail_quote"><div dir="ltr">On Tue, Aug 14, 2018 at 8:58 PM Bryan Kearney <<a href="mailto:bkearney@redhat.com" target="_blank">bkearney@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 08/14/2018 11:00 AM, Milan Kovacik wrote:<br>
> hard unless the SME (and/or commit bit ownership) status is tied<br>
> formally to review quality and rate.<br>
> <br>
I can see nagging the owners, but doing reviews of other parts of code<br>
you dont know is a great way to (1) Learn it and (2) help the project.<br>
<br>
-- bk<br>
<br>
<br>
</blockquote></div>
</div></div><br>______________________________<wbr>_________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/pulp-dev</a><br>
<br></blockquote></div><br></div>