<div dir="ltr">Thanks! Another thought after reading with the updates:<div><br></div><div>It's easy to assume in our discussion that "having the commit bit" == "being responsible for maintaining a thing". But the latter is a state a developer can be in, while the former is one of multiple potential ways to designate or assign that state. While the former does capture the policy being proposed, it does not necessarily reflect the current state of affairs, nor should we assume that someone reading this problem statement will already have that equivalence in mind. Thus I suggest changing this:</div><div><br></div><div>"Currently the contributors do not treat all members of the single developer team as if they have committer rights to all plugins and core."</div><div><br></div><div>to this:</div><div><br></div><div>"Currently the contributors do not treat all members of the single developer team as if they are responsible for maintaining all plugins and core."</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 27, 2017 at 2:01 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">Thanks for the feedback. I think I addressed all of it but let me know if I missed something.</div><div class="gmail_extra"><span class="HOEnZb"><font color="#888888"><br clear="all"><div><div class="m_4798616124841141054gmail_signature" data-smartmail="gmail_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></font></span><div><div class="h5">
<br><div class="gmail_quote">On Thu, Oct 26, 2017 at 11:38 AM, Michael Hrivnak <span dir="ltr"><<a href="mailto:mhrivnak@redhat.com" target="_blank">mhrivnak@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">This looks good. I made a couple of small in-line suggestions by doing strike-through followed by a replacement. Broader suggestions follow:<div><br></div><div>For me, I think the first point could be read as describing a logistical problem of giving someone a commit bit on a single repo. "you have to give that person committer rights to all plugins and Pulp core at once." Once we decide to give someone commit access to a repo (that's the hard part), actually doing it is very easy on github. An example is that we gave @mibanescu commit access only to pulp_deb. I think the intent of the first point is not to make such a logistical argument, but to describe undesirable outcomes of the current policy. It may be more clear to change this:</div><div><br></div><div>"First, having a single developer team means that in order to give someone committer rights to a plugin you have to give that person committer rights to all plugins and Pulp core at once."</div><div><br></div><div>to this:</div><div><br></div><div>"First, when a new developer is added to the single team, they are given committer rights to core and all plugins, regardless of which areas they intend to focus on."</div><div><br></div><div>The latter describes the undesirable outcome, whereas the former could be read as "The policy is bad because $POLICY."</div><div><br></div><div>The second point ("Second, there are ...") could be divided into two points. The first half talks about why it's valuable to know who has responsibility for a plugin. (This looks a lot like the third point.) The second half talks about the benefits to an individual of being able to focus on a specific area. I suggest either having 4 total points, or move the first half of point 2 to be part of point 3.</div></div><div class="gmail_extra"><div><div class="m_4798616124841141054h5"><br><div class="gmail_quote">On Tue, Oct 24, 2017 at 10:44 AM, Robin Chan <span dir="ltr"><<a href="mailto:rchan@redhat.com" target="_blank">rchan@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Looks good. I did add a statement between the "<>" in the next<br>
sentence.  Please feel free to update - just trying to help articulate<br>
the un-stated for clarity and evaluation of the proposed solution.<br>
<br>
Without clear ownership, feature implementation, bugfixes, and code<br>
review all become more difficult because< it is unclear who needs to<br>
be involved in the resolution of concerns and who has the final say in<br>
disputes (?).><br>
<div class="m_4798616124841141054m_327261391071971575HOEnZb"><div class="m_4798616124841141054m_327261391071971575h5"><br>
On Tue, Oct 24, 2017 at 9:43 AM, David Davis <<a href="mailto:daviddavis@redhat.com" target="_blank">daviddavis@redhat.com</a>> wrote:<br>
> Thanks Robin. I’ve updated the PUP based on your feedback and with the help<br>
> of some other folks on IRC. Please feel free to look it over again.<br>
><br>
> I’ll wait until about Monday October 30th before I proceed with working on<br>
> the rest of the PUP.<br>
><br>
><br>
> David<br>
><br>
> On Mon, Oct 23, 2017 at 4:22 PM, Robin Chan <<a href="mailto:rchan@redhat.com" target="_blank">rchan@redhat.com</a>> wrote:<br>
>><br>
>> Hi David,<br>
>> I've added some comments - trying to mostly articulate what I heard<br>
>> you guys saying last week.  I agree this is a good summary, just would<br>
>> like to work on stating the obvious a little more obviously.<br>
>> My comments are in red (or whatever this weird muddy pink color is.<br>
>> Mauve?)<br>
>> Thanks for getting us started on this.<br>
>> Robin<br>
>><br>
>><br>
>> On Thu, Oct 19, 2017 at 4:54 PM, David Davis <<a href="mailto:daviddavis@redhat.com" target="_blank">daviddavis@redhat.com</a>><br>
>> wrote:<br>
>> > Last week we talked about forming teams for each Pulp plugin and I<br>
>> > wanted to<br>
>> > open up a community discussion around that idea. @bmbouter and I have<br>
>> > come<br>
>> > up with a proposed problem statement [0] to kick off this discussion.<br>
>> ><br>
>> > We’re looking for feedback from everyone so please feel free to check<br>
>> > the<br>
>> > statement out and make edits. Or respond to this email with any thoughts<br>
>> > about it.<br>
>> ><br>
>> > After about a week or so, I’ll try to create a PUP based on any feedback<br>
>> > we<br>
>> > get.<br>
>> ><br>
>> > [0] <a href="http://pad-theforeman.rhcloud.com/p/pulp-plugin-pup" rel="noreferrer" target="_blank">http://pad-theforeman.rhcloud.<wbr>com/p/pulp-plugin-pup</a><br>
>> ><br>
>> > David<br>
>> ><br>
>> > ______________________________<wbr>_________________<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<wbr>/listinfo/pulp-dev</a><br>
>> ><br>
><br>
><br>
<br>
______________________________<wbr>_________________<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<wbr>/listinfo/pulp-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div></div></div><span class="m_4798616124841141054HOEnZb"><font color="#888888">-- <br><div class="m_4798616124841141054m_327261391071971575gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><p style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important"><span style="margin:0px!important;padding:0px!important">Michael</span> <span style="margin:0px!important;padding:0px!important">Hrivnak</span></p><p style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important"></p><span style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important"><span style="margin:0px!important;padding:0px!important">Principal Software Engineer</span><span style="margin:0px!important;padding:0px!important">, <span style="margin:0px!important;padding:0px!important">RHCE</span></span> </span><span style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px"></span><br style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important"><p style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important">Red Hat</p></div></div>
</font></span></div>
</blockquote></div><br></div></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><p style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important"><span style="margin:0px!important;padding:0px!important">Michael</span> <span style="margin:0px!important;padding:0px!important">Hrivnak</span></p><p style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important"></p><span style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important"><span style="margin:0px!important;padding:0px!important">Principal Software Engineer</span><span style="margin:0px!important;padding:0px!important">, <span style="margin:0px!important;padding:0px!important">RHCE</span></span> </span><span style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px"></span><br style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important"><p style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important">Red Hat</p></div></div>
</div>