That's an unfortunate +1, it's always sad to lose one of our own. Hope we'll see you back soon! Thanks for all the fis...reviews!
On 15 January 2016 at 06:59, Ryan Hallisey <rhall...@redhat.com> wrote: > Hello all, > > Harm, really appreciate all your reviews. You were always very thorough and > insightful. You'll always be welcome back to the core team in the future. > Thanks for everything Harm! > > +1 > -Ryan > > ----- Original Message ----- > From: "Michal Rostecki" <mroste...@mirantis.com> > To: openstack-dev@lists.openstack.org > Sent: Friday, January 15, 2016 6:46:43 AM > Subject: Re: [openstack-dev] [kolla] Please vote -> Removal of Harm Weites > from the core reviewer team > > On 01/15/2016 01:12 AM, Steven Dake (stdake) wrote: >> Hi fellow core reviewers, >> >> 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. >> >> 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. >> >> 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. >> >> I propose removing Harm from the core review team. Please vote: >> >> +1 = remove Harm from the core review team >> -1 = don't remove Harm from the core review team >> >> 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. >> >> 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. >> >> 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. >> >> 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. >> >> Regards, >> -steve >> >> [1] >> http://lists.openstack.org/pipermail/openstack-dev/2015-October/077844.html >> > > Hi all, > > First of all, thank you Harm for your work on Kolla in the Liberty > release. And especially for your good reviews and advices when I was > beginning my contribution to this project. > > For now, I'm +1 for removing Harm from core team. But I also hope that > in future it will be possible for him to rejoin our team and be as > active as before. > > Cheers, > Michal > > __________________________________________________________________________ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > > __________________________________________________________________________ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev __________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
