<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hi guys,<div class=""><br class=""></div><div class="">As Steven noted, activity from my side has dropped significantly, and with +2 comes a certain responsibility of at the very least keeping track of the codebase. Various reasons keep me from even doing that so this seems the logical outcome.</div><div class=""><br class=""></div><div class="">Thanks for your support, it’s been a great ride :)<br class=""><div class=""><br class=""></div><div class="">see you in #kolla!</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">Op 15 jan. 2016, om 20:10 heeft Steven Dake (stdake) <<a href="mailto:stdake@cisco.com" class="">stdake@cisco.com</a>> het volgende geschreven:</div><br class="Apple-interchange-newline"><div class="">
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii" class="">
<div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; font-size: 14px; font-family: Calibri, sans-serif;" class="">
<div class="">I counted 6 votes in favor of removal. We have 10 people on our core team. A majority has been met and I have removed Harm from the core reviewer team.</div>
<div class=""><br class="">
</div>
<div class="">Harm,</div>
<div class=""><br class="">
</div>
<div class="">Thanks again for your helpful reviews and remember, your always welcome back in the future if your availability changes.</div>
<div class=""><br class="">
</div>
<div class="">For the record, the core reviewers that voted for removal were:</div>
<div class="">Steven Dake</div>
<div class="">Jeff Peeler</div>
<div class="">Paul Bourke</div>
<div class="">Michal Jastrzebski</div>
<div class="">Ryan Hallisey</div>
<div class="">Michal Rostecki</div>
<div class=""><br class="">
</div>
<div class="">Regards,</div>
<div class="">-steve</div>
<div class=""><br class="">
</div>
<div class=""><br class="">
</div>
<span id="OLK_SRC_BODY_SECTION" class="">
<div style="font-family: Calibri; font-size: 11pt; text-align: left; border-width: 1pt medium medium; border-style: solid none none; padding: 3pt 0in 0in; border-top-color: rgb(181, 196, 223);" class="">
<span style="font-weight:bold" class="">From: </span>Steven Dake <<a href="mailto:stdake@cisco.com" class="">stdake@cisco.com</a>><br class="">
<span style="font-weight:bold" class="">Reply-To: </span>openstack-dev <<a href="mailto:openstack-dev@lists.openstack.org" class="">openstack-dev@lists.openstack.org</a>><br class="">
<span style="font-weight:bold" class="">Date: </span>Thursday, January 14, 2016 at 5:12 PM<br class="">
<span style="font-weight:bold" class="">To: </span>openstack-dev <<a href="mailto:openstack-dev@lists.openstack.org" class="">openstack-dev@lists.openstack.org</a>><br class="">
<span style="font-weight:bold" class="">Subject: </span>[openstack-dev] [kolla] Please vote -> Removal of Harm Weites from the core reviewer team<br class="">
</div>
<div class=""><br class="">
</div>
<blockquote id="MAC_OUTLOOK_ATTRIBUTION_BLOCKQUOTE" style="BORDER-LEFT: #b5c4df 5 solid; PADDING:0 0 0 5; MARGIN:0 0 0 5;" class="">
<div class="">
<div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; font-size: 14px; font-family: Calibri, sans-serif;" class="">
<div class="">Hi fellow core reviewers,</div>
<div class=""><br class="">
</div>
<div class="">Harm joined Kolla early on with great enthusiasm and did a bang-up job for several months working on Kolla. We voted unanimously to add him to the core team. Over the last 6 months Harm hasn't really made much contribution to Kolla. I have spoken to
him about it in the past, and he indicated his work and other activities keep him from being able to execute the full job of a core reviewer and nothing environmental is changing in the near term that would improve things.</div>
<div class=""><br class="">
</div>
<div class="">I faced a similar work/life balance problem when working on Magnum as a core reviewer and also serving as PTL for Kolla. My answer there was to step down from the Magnum core reviewer team [1] because Kolla needed a PTL more then Magnum needed a core
reviewer. I would strongly prefer if folks don't have the time available to serve as a Kolla core reviewer, to step down as was done in the above example. Folks that follow this path will always be welcome back as a core reviewer in the future once they
become familiar with the code base, people, and the project.</div>
<div class=""><br class="">
</div>
<div class="">The other alternative to stepping down is for the core reviewer team to vote to remove an individual from the core review team if that is deemed necessary. For future reference, if you as a core reviewer have concerns about a fellow core reviewer's performance,
please contact the current PTL who will discuss the issue with you.</div>
<div class=""><br class="">
</div>
<div class="">I propose removing Harm from the core review team. Please vote:</div>
<div class=""><br class="">
</div>
<div class="">+1 = remove Harm from the core review team</div>
<div class="">-1 = don't remove Harm from the core review team</div>
<div class=""><br class="">
</div>
<div class="">Note folks that are voted off the core review team are always welcome to rejoin the core team in the future once they become familiar with the code base, people, and the project. Harm is a great guy, and I hope in the future he has more time available
to rejoin the Kolla core review team assuming this vote passes with simple majority.</div>
<div class=""><br class="">
</div>
<div class="">It is important to explain why, for some folks that may be new to a core reviewer role (which many of our core reviewers are), why a core reviewer should have their +2/-2 voting rights removed when they become inactive or their activity drops below an
acceptable threshold for extended or permanent periods. This hasn't happened in Harm's case, but it is possible that a core reviewer could approve a patch that is incorrect because they lack sufficient context with the code base. Our core reviewers are the
most important part of ensuring quality software. If the individual has lost context with the code base, their voting may be suspect, and more importantly the other core reviewers may not trust the individual's votes. Trust is the cornerstone of a software
review process, so we need to maximize trust on a technical level between our core team members. That is why maintaining context with the code base is critical and why I am proposing a vote to remove Harm from the core reviewer team.</div>
<div class=""><br class="">
</div>
<div class="">On a final note, folks should always know, joining the core review team is never "permanent". Folks are free to move on if their interests take them into other areas or their availability becomes limited. Core Reviewers can also be removed by majority
vote. If there is any core reviewer's performance you are concerned with, please contact the current PTL to first work on improving performance, or alternatively initiating a core reviewer removal voting process.</div>
<div class=""><br class="">
</div>
<div class="">On a more personal note, I want to personally thank Harm for his many and significant contributions to Kolla and especially going above and beyond by taking on the responsibility of a core reviewer. Harm's reviews were always very thorough and very high
quality, and I really do hope in the future Harm will rejoin the Kolla core team.</div>
<div class=""><br class="">
</div>
<div class="">Regards,</div>
<div class="">-steve</div>
<div class=""><br class="">
</div>
<div class="">[1] <a href="http://lists.openstack.org/pipermail/openstack-dev/2015-October/077844.html" class="">http://lists.openstack.org/pipermail/openstack-dev/2015-October/077844.html</a></div>
</div>
</div>
</blockquote>
</span>
</div>
__________________________________________________________________________<br class="">OpenStack Development Mailing List (not for usage questions)<br class="">Unsubscribe: <a href="mailto:OpenStack-dev-request@lists.openstack.org" class="">OpenStack-dev-request@lists.openstack.org</a>?subject:unsubscribe<br class=""><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" class="">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br class=""></div></blockquote></div><br class=""></div></div></body></html>