<div dir="ltr"><div dir="ltr"><div>David, thanks for driving this.</div><div><br></div><div>I agree with your suggestions Robin. <br></div><div><br></div><div>All currently present on [1] should have commit bit for those repos. <br></div><div><br></div><div>I think it is fine to the devs to have commit to the test repos, just to communicate QE in case of test changes. We already have a system in place on git.<br></div><div><br></div><div>[1] <a href="https://github.com/orgs/pulp/teams/qe">https://github.com/orgs/pulp/teams/qe</a></div><div><br></div><div>Thanks,<br></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jan 15, 2019 at 11:07 AM Robin Chan <<a href="mailto:rchan@redhat.com">rchan@redhat.com</a>> wrote:<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 dir="ltr"><div>A few suggestions.</div><div><br></div><div>#1. QE good with [1] - you all agree these are the folks with commit bit? In other words, you trust each other to do the merge with your own agreements of who has expertise and when things are ready - all the details?</div><div>#2. I would suggest we are suggesting QE have commit bit access to the specific subdirectories;</div><div>  a)<span class="gmail-m_8831046361131726255gmail-repo-root gmail-m_8831046361131726255gmail-js-repo-root"><span class="gmail-m_8831046361131726255gmail-js-path-segment"><span> pulp_file</span></span></span><span class="gmail-m_8831046361131726255gmail-separator">/</span><span class="gmail-m_8831046361131726255gmail-js-path-segment"><span>pulp_file</span></span><span class="gmail-m_8831046361131726255gmail-separator">/</span><span class="gmail-m_8831046361131726255gmail-js-path-segment"><span>tests</span></span><span class="gmail-m_8831046361131726255gmail-separator">/</span><span class="gmail-m_8831046361131726255gmail-final-path">functional</span><span class="gmail-m_8831046361131726255gmail-separator">/ (in pulp/pulp_file repo)</span></div><div><span class="gmail-m_8831046361131726255gmail-separator">  b) pulp/pulp_core/tests/functional/ (in pulp/pulp repo)<br></span></div><div><span class="gmail-m_8831046361131726255gmail-separator">I know this is not enforceable via the GIT settings, but helpful to be explicit about as we include this in agreement.<br></span></div><div><span class="gmail-m_8831046361131726255gmail-separator">#3. Shall we also agree that those not in [1] - in other words, the developers give up commit bit for #2. Can still contribute but don't need to be involved in #1 agreements.</span></div><div><span class="gmail-m_8831046361131726255gmail-separator"><br></span></div><div><span class="gmail-m_8831046361131726255gmail-separator">Fully supportive of this effort. I was one of the folks who gave my word prior to PUP-6 and see this as making sure the folks have what they need to get stuff done and keeping decision making with the folks closest to the work (i.e. QE makes decisions about all things QE.)</span></div><div><span class="gmail-m_8831046361131726255gmail-separator"><br></span></div><div><span class="gmail-m_8831046361131726255gmail-separator">Thanks,</span></div><div><span class="gmail-m_8831046361131726255gmail-separator">Robin<br></span>
  </div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jan 15, 2019 at 10:37 AM David Davis <<a href="mailto:daviddavis@redhat.com" target="_blank">daviddavis@redhat.com</a>> wrote:<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 dir="ltr"><div dir="ltr"><div dir="ltr">When we moved the pulp-smash tests out of the pulp-smash repository, we promised to give QE ownership of the smash tests within the Pulp repositories on github. I know we have a process in place to give the commit bit to contributors[0] but this promise predates PUP-6. <div><br></div><div><div dir="ltr" class="gmail-m_8831046361131726255gmail-m_-2970012245564294755gmail_signature"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Thus, I'd like to ask for feedback on giving the QE team in github[1] the commit bit to the following repositories in order to merge changes to smash tests:</div><div><br></div><div>pulp/pulp</div><div>pulp/pulp_file<br></div><div><br></div><div>I'd also like to ask plugin teams to consider giving QE commit access to their repositories if they have pulp-smash tests that are maintained by QE.</div><div><br></div><div>Feedback would be appreciated. I'll like to set an deadline of January 30th.</div><div><br></div><div>Thanks.</div><div><br></div><div>[0] <a href="https://github.com/pulp/pups/blob/master/pup-0006.md" target="_blank">https://github.com/pulp/pups/blob/master/pup-0006.md</a></div><div>[1] <a href="https://github.com/orgs/pulp/teams/qe" target="_blank">https://github.com/orgs/pulp/teams/qe</a></div><div><br></div><div>David<br><br></div></div></div></div></div></div></div></div></div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div>