Skip to content

Ask Access team members to check for feature flags

Michelle Gill requested to merge m_gill/triage-ops:master into master

Context

In gitlab-com/www-gitlab-com#12276 (closed), it was decided that ~"group::access" has special circumstances that would benefit from being asked in an MR whether they should use a feature flag. We decided against using the MR template, since items often get skipped there and it does not prompt the author for reasoning. The thought is that if the answer is "No," there is an opportunity to explain why or for a reviewer/maintainer to challenge that. The intent of this MR is to check daily for merge requests created within the last day authored by ~group::access and post a comment about using a feature flag.

Open questions

  • Do we need to account for duplicate comments being posted via label or some other means? Or is the 1 day timeframe sufficient? 🤔
  • Are tests needed here?
  • Should the Triage Operations documentation be updated or cross-linked to the Manage handbook page?

End result

Screen_Shot_2021-08-13_at_9.01.54_AM

Action items

  • (If applicable) Add documentation to the handbook pages for Triage Operations =>
  • (If applicable) Identify the affected groups and how to communicate to them:
    • /cc @person_or_group =>
    • Relevant Slack channels =>
    • Week-in-review
Edited by Kyle Wiebers

Merge request reports