Reviewer/Maintainer Danger rules for products
I think it's time to add these to omnibus-gitlab and charts/gitlab. I don't think we need to implement the whole reviewer roulette system that gitlab-Xe uses yet.
I think still just printing out the list of available folks is the way to go (rather than auto-assign), and I don't think we need to implement some sort of scheduling algorithm yet to ensure the load is distributed evenly.
We could
- Pull in the list from the team page
- There may be some historical inclusions in this that don't want to actively be reviewers or maintainers. (Marin, Gabriel, etc).
- For now, just hard code in the list
- DJ, Balu and myself for omnibus-gitlab
- DJ and Jason as Maintainers for charts, myself as reviewer
Thoughts? /cc @gitlab-org/distribution