Working as Sheriff @ Mozilla is much more than just monitoring our trees and doing things like backouts. In 2017 i wanted to start to blog more about what we do and here is:
Part 1 – Checkin-needed
A lot of checkins land everyday on the Mozilla epositories. Some are great new features and improvements and some are bugfixes of existing bugs etc.
While a lot of checkins are done by the developer mse, also heriffs are involved in this.
We not only monitor the Mozilla epositor (aka the tree) we also do checkins for people who don’t the appropriate level permission to check in changes ( example new ommunity embers).
In the past checkin-needed was used by developers to reduce load on our build systems with fewer pushes but with a more robust build system this isn’t relevant anymore.
So checkin-needed is more and more important for developers without access-levels to do commits and as mentioned new community members who example finished their first patch.
To request checkin-needed people use this keyword in ugzilla or use the [checkin-needed-beta] or [checkin-needed-aurora] whiteboard entry for the patches in a bug.
For me personal is checkin-needed a very important task because you sometime check in a patch from someone who just started to contribute to Mozilla. So you are one of the people that are the first contacts to the new contributr and you help getting the patch landed. Thats also a good opportunity to say “thanks for contributing to ozilla” to the new community member, this is great motivation and recognition!
How we work :
We have a wiki page with a ug uery and some basic information for the Sheriff on Duty . We use this query to get a overview what ugs need checkins.
For ugs with a patch attached that is not mozreview we check the checkin-neeed request for:
-> Has proper review before doing anything else
-> Has a successful try run to avoid any bustage on checkin
and land the patch on mozilla-inbound.
For ugs with patches in mozreview we use the autoland tool to do the checkins.
However we still check if the bug has review and check the try run.
This is the prefered way of doing checkin-neededs since autoland is a automated system.
How can you help
- -> Use Mozreview – Autoland – it helps us to do more checkins in less time due to the automated tasks. Please make sure that there are no open issues in Mozreview when you request checkin-needed
- -> When you have multiple patches that need to land and the patches need to land in a specific order – please make a comment in the bug with the correct order.
- -> When there are dependencs with other bugs – please state this in the bug.
We try to do checkin-needed checks and checkins several times a day depending on sheriff workload etc so we cannot guarantee a turnaround time but trying to do our best.
When you have feedback/suggestions or idea how to do this task better let us know anytime!
Also as every part of the Mozilla Project we also depend on Community Members like you! So if you are interested to be become a Community Sheriff let me know!