Skip to content

Add Fulfillment::Utilization to CodeOwners file

Sheldon Led requested to merge led/create-utilization-codeowners-list into master

What does this MR do and why?

After the recent changes in the Pipelines tab in the Usage Quotas Page, a discussion started on the last MR (!105880 (comment 1205644552)) Regarding:

  1. Where the Vue code should leave
    1. So far we've decided to move back to ee/app/assets/javascripts/usage_quotas/pipelines. I've added the move change to this MR.
  2. How we update .gitlab/CODEOWNERS file:
    1. Option 1: Add [Fulfillment::Utilization] section to explicitly say we own seats and storage but not pipelines.
    2. Option 2: Just update ^[Verify] section adding /ee/app/assets/javascripts/usage_quotas/pipelines/

On item 2 I currently implemented Option 1.

Related items:

  • Move frontend code managed by the verify frontend team to a single directory: &8943
  • .gitlab/CODEOWNERS
    # When adding a group as a code owner, make sure to invite the group to the
    # project here: https://gitlab.com/gitlab-org/gitlab/-/project_members
    # As described in https://docs.gitlab.com/ee/user/project/code_owners.html
  • Code Owners docs: https://docs.gitlab.com/ee/user/project/code_owners.html

Screenshots or screen recordings

N/A

How to set up and validate locally

Navigate to one of:

Make sure the tab is rendered correctly

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Sheldon Led

Merge request reports