«
»

Sheriffing @ Mozilla – checkin-needed

January 26th, 2017 by cbook
Hi,
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 repositories. Some are great new features and improvements and some are bugfixes of existing bugs etc.
While a lot of checkins are done by the developers themselves, also sheriffs are involved in this.
We not only monitor the Mozilla repositories (aka the tree) we also do checkins for people who don’t have the appropriate level of permission to check in changes (for example new community members).
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 for example finished their first patch.
To request checkin-needed people use this keyword in Bugzilla 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 contributor and you help them getting the patch landed. That‘s also a good opportunity to say “thanks for contributing to Mozilla” to the new community member, this is great motivation and recognition! 
How we work :
    
We have a wiki page with a bug query and some basic information for the Sheriff on Duty [1]. We use this query to get a overview what bugs need checkins.
For bugs with a patch attached that is not on 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 bugs 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 preferred way of doing checkin-neededs since autoland is an 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. In fact, you can land them yourself with autoland. In the future, checkin-needed will only be allowed on security bugs.
  • -> Make sure that you have a passing try run. It is a waste of the sheriffs time to come look at a checkin-needed and it has failures in the try run.
  • -> 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 dependencies 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!
Cheers,
– Tomcat
[1] https://wiki.mozilla.org/Sheriffing/How:To:Landing_checkin-needed_patches

This entry was posted on Thursday, January 26th, 2017 at 2:53 am and is filed under sheriffing. You can follow any responses to this entry through the RSS 2.0 feed. You can leave a response, or trackback from your own site.

166 Responses to “Sheriffing @ Mozilla – checkin-needed”

  1. Alise Foersterling Says:
    December 15th, 2021 at 12:59 pm

    Alise Foersterling

    I found a great…

  2. Maile Waldschmidt Says:
    December 16th, 2021 at 1:55 am

    Maile Waldschmidt

    I found a great…

  3. Willie Guith Says:
    December 17th, 2021 at 4:55 pm

    Willie Guith

    I found a great…

  4. Aubrey Cartaya Says:
    December 18th, 2021 at 3:45 am

    Aubrey Cartaya

    I found a great…

  5. Allan Cassavaugh Says:
    December 20th, 2021 at 12:17 pm

    Allan Cassavaugh

    I found a great…

  6. Stanley Domebo Says:
    December 23rd, 2021 at 12:14 pm

    Stanley Domebo

    I found a great…

  7. Buyprivateproxies Says:
    January 7th, 2022 at 2:16 am

    Buyprivateproxies

    I found a great…

  8. Private Proxies Txt Says:
    January 7th, 2022 at 9:11 am

    Private Proxies Txt

    I found a great…

  9. Order Proxies Says:
    January 27th, 2022 at 4:45 pm

    Order Proxies

    I found a great…

  10. Buy Private Proxies Says:
    February 4th, 2022 at 12:43 am

    Buy Private Proxies

    I found a great…

  11. Jame Scarnato Says:
    May 14th, 2022 at 5:48 am

    Jame Scarnato

    I found a great…

  12. Bulk Private Proxies Says:
    May 28th, 2022 at 8:26 am

    Bulk Private Proxies

    I found a great…

  13. Facebook Proxies Says:
    September 7th, 2022 at 2:26 pm

    Facebook Proxies

    I found a great…

  14. The Best Proxy Says:
    September 9th, 2022 at 10:02 am

    The Best Proxy

    I found a great…

  15. Willene Vallance Says:
    September 11th, 2022 at 10:00 am

    Willene Vallance

    I found a great…

  16. erp kosta consulting Says:
    March 15th, 2023 at 1:08 am

    thanks for sharing

Leave a Reply