<div dir="ltr">+1<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jan 16, 2019 at 8:32 PM Brian Bouterse <<a href="mailto:bbouters@redhat.com">bbouters@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">This all sounds good to me.<br></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jan 15, 2019 at 2:25 PM Kersom <<a href="mailto:kersom@redhat.com" target="_blank">kersom@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">Robin, yeap. Exactly what you described it.<br></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jan 15, 2019 at 2:22 PM Robin Chan <<a href="mailto:rchan@redhat.com" target="_blank">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>Great. I withdraw:</div><div><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_-7058785004878037360gmail-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_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_-7058785004878037360gmail-m_8831046361131726255gmail-separator"><br></span></div><div><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_-7058785004878037360gmail-m_8831046361131726255gmail-separator">And to re-iterate and be very clear, Kersom's ", just to communicate QE in case of test changes. We already have a system in place on git." looks like getting an approved code review from someone in [1].</span></div><div><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_-7058785004878037360gmail-m_8831046361131726255gmail-separator"><br></span></div><div><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_-7058785004878037360gmail-m_8831046361131726255gmail-separator">That works for me and I appreciate the clarifications.</span></div><div><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_-7058785004878037360gmail-m_8831046361131726255gmail-separator">Robin<br></span></div><div><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_-7058785004878037360gmail-m_8831046361131726255gmail-separator"><br></span></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jan 15, 2019 at 2:15 PM 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">I agree. I think devs can merge changes to pulp-smash tests in pulp repos but they should get it reviewed by QE before merging--which, as Kersom says, we've been doing.<br clear="all"><div><div dir="ltr" class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><br></div><div>David<br></div></div></div></div></div></div></div></div><br></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jan 15, 2019 at 2:11 PM Kersom <<a href="mailto:kersom@redhat.com" target="_blank">kersom@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>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" target="_blank">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" target="_blank">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_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-m_8831046361131726255gmail-repo-root gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-m_8831046361131726255gmail-js-repo-root"><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-m_8831046361131726255gmail-js-path-segment"><span> pulp_file</span></span></span><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-m_8831046361131726255gmail-separator">/</span><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-m_8831046361131726255gmail-js-path-segment"><span>pulp_file</span></span><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-m_8831046361131726255gmail-separator">/</span><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-m_8831046361131726255gmail-js-path-segment"><span>tests</span></span><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-m_8831046361131726255gmail-separator">/</span><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-m_8831046361131726255gmail-final-path">functional</span><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-m_8831046361131726255gmail-separator">/ (in pulp/pulp_file repo)</span></div><div><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-m_8831046361131726255gmail-separator">  b) pulp/pulp_core/tests/functional/ (in pulp/pulp repo)<br></span></div><div><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-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_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-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_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-m_8831046361131726255gmail-separator"><br></span></div><div><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-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_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-m_8831046361131726255gmail-separator"><br></span></div><div><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-m_8831046361131726255gmail-separator">Thanks,</span></div><div><span class="gmail-m_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-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_8970498877981713297gmail-m_-468598375496791762gmail-m_3026890023738202244gmail-m_4252068830067340674gmail-m_-5623756234048087318gmail-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>
</blockquote></div>
</blockquote></div>
</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>
_______________________________________________<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>