Refactor the usage quotas -> ci/cd minutes page

What does this MR do and why?

Adds visibility for shared runner monthly usage to the usage quota page. Adds tabs for switching between the two bits of data.

#345795 (closed)

Screenshots or screen recordings

Usage by month

CI/CD minutes usage	Shared runner usage

Usage by project

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Preferences > Usage Quotas

MR acceptance checklist

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

• I have evaluated the MR acceptance checklist for this MR.

changed milestone to %14.10

assigned to @jivanvl

added Category:Continuous Integration Compute minutes FY23Q1 GitLab Free GitLab Premium GitLab Ultimate OKR-FY24Q2 Technical Writing UI text UX VerifyP3 [deprecated] Accepting merge requests devopsverify documentation frontend grouppipeline execution sectionops severity1 typefeature workflowin dev + 1 deleted label

Reviewer roulette

Changes that require review have been detected!

Please refer to the table below for assigning reviewers and maintainers suggested by Danger in the specified category:

Category	Reviewer	Maintainer
frontend	Himanshu Kapoor (@himkp) (UTC+1, 1 hour ahead of @jivanvl)	Miguel Rincon (@mrincon) (UTC+1, 1 hour ahead of @jivanvl)

To spread load more evenly across eligible reviewers, Danger has picked a candidate for each review slot, based on their timezone. Feel free to override these selections if you think someone else would be better-suited or use the GitLab Review Workload Dashboard to find other available reviewers.

To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines. Please consider assigning a reviewer or maintainer who is a domain expert in the area of the merge request.

Once you've decided who will review this merge request, assign them as a reviewer! Danger does not automatically notify them for you.

Generated by Danger

Bundle size analysis [beta]

This compares changes in bundle size for entry points between the commits ce90b934 and 8a03b35d

Special assets

Entrypoint / Name	Size before	Size after	Diff	Diff in percent
average	3.42 MB	3.42 MB	-	0.0 %
mainChunk	1.91 MB	1.91 MB	-	0.0 %

---

Please look at the full report for more details

---

Read more about how this report works.

Generated by Danger

Allure report

allure-report-publisher generated test report!

review-qa-reliable: test report
review-qa-smoke: test report
review-qa-blocking: test report

review-qa-blocking: test report for de333d0b

+-------------------------------------------------------------------+
|                          suites summary                           |
+----------------------+--------+--------+---------+-------+--------+
|                      | passed | failed | skipped | flaky | result |
+----------------------+--------+--------+---------+-------+--------+
| Plan                 | 41     | 0      | 1       | 0     | ✅     |
| Manage               | 28     | 0      | 2       | 0     | ✅     |
| Verify               | 12     | 0      | 1       | 0     | ✅     |
| Create               | 15     | 0      | 2       | 0     | ✅     |
| Package Registry     | 6      | 0      | 0       | 0     | ✅     |
| Protect              | 3      | 0      | 0       | 0     | ✅     |
| Package              | 0      | 0      | 1       | 0     | ➖     |
| Configure            | 0      | 0      | 1       | 0     | ➖     |
| SSH keys support     | 2      | 0      | 0       | 0     | ✅     |
| Version sanity check | 0      | 0      | 1       | 0     | ➖     |
+----------------------+--------+--------+---------+-------+--------+
| Total                | 107    | 0      | 9       | 0     | ✅     |
+----------------------+--------+--------+---------+-------+--------+

review-qa-blocking: test report for de333d0b

expand test summary

+-------------------------------------------------------------------+
|                          suites summary                           |
+----------------------+--------+--------+---------+-------+--------+
|                      | passed | failed | skipped | flaky | result |
+----------------------+--------+--------+---------+-------+--------+
| Create               | 15     | 0      | 2       | 0     | ✅     |
| SSH keys support     | 2      | 0      | 0       | 0     | ✅     |
| Protect              | 3      | 0      | 0       | 0     | ✅     |
| Manage               | 28     | 0      | 2       | 0     | ✅     |
| Plan                 | 41     | 0      | 1       | 0     | ✅     |
| Verify               | 12     | 0      | 1       | 0     | ✅     |
| Configure            | 0      | 0      | 1       | 0     | ➖     |
| Package              | 0      | 0      | 1       | 0     | ➖     |
| Package Registry     | 6      | 0      | 0       | 0     | ✅     |
| Version sanity check | 0      | 0      | 1       | 0     | ➖     |
+----------------------+--------+--------+---------+-------+--------+
| Total                | 107    | 0      | 9       | 0     | ✅     |
+----------------------+--------+--------+---------+-------+--------+

removed [deprecated] Accepting merge requests label

added 438 commits

• 73d8c863...a8e64e16 - 437 commits from branch master
• 00e2be67 - Refactor the usage quotas -> ci/cd minutes page

Compare with previous version

	6 Warnings
	This merge request is quite big (888 lines changed), please consider splitting it into multiple merge requests.
	a144c5dc: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.
	8ea22611: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.
	0204bd1b: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.
	ddb754e8: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.
	749c8689: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.

If needed, you can retry the danger-review job that generated this comment.

Reviewer roulette

Changes that require review have been detected!

Please refer to the table below for assigning reviewers and maintainers suggested by Danger in the specified category:

Category	Reviewer	Maintainer
backend	Tiger Watson (@tigerwnz) (UTC+10, 10 hours ahead of @jivanvl)	Sincheol (David) Kim (@dskim_gitlab) (UTC+9.5, 9.5 hours ahead of @jivanvl)
frontend	Justin Ho (@justin_ho) (UTC+0, same timezone as @jivanvl)	Miguel Rincon (@mrincon) (UTC+2, 2 hours ahead of @jivanvl)
test Quality for spec/features/*	Tiffany Rea (@treagitlab) (UTC-7, 7 hours behind @jivanvl)	Maintainer review is optional for test Quality for spec/features/*

To spread load more evenly across eligible reviewers, Danger has picked a candidate for each review slot, based on their timezone. Feel free to override these selections if you think someone else would be better-suited or use the GitLab Review Workload Dashboard to find other available reviewers.

To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines. Please consider assigning a reviewer or maintainer who is a domain expert in the area of the merge request.

Once you've decided who will review this merge request, assign them as a reviewer! Danger does not automatically notify them for you.

Generated by Danger

added 1 commit

• 53c1b177 - Refactor the usage quotas -> ci/cd minutes page

Compare with previous version

changed the description

added 254 commits

• 53c1b177...ce90b934 - 253 commits from branch master
• 8a03b35d - Refactor the usage quotas -> ci/cd minutes page

Compare with previous version

added 543 commits

• 8a03b35d...9eca3088 - 542 commits from branch master
• 5cc489ee - Refactor the usage quotas -> ci/cd minutes page

Compare with previous version

Bundle size analysis [beta]

This compares changes in bundle size for entry points between the commits a69794b0 and de333d0b

Special assets

Entrypoint / Name	Size before	Size after	Diff	Diff in percent
mainChunk	1.93 MB	1.92 MB	-2.18 KB	-0.1 %
average	3.45 MB	3.45 MB	-	0.0 %

Significant Growth: 8

Expand

Entrypoint / Name	Size before	Size after	Diff	Diff in percent
pages.groups.settings.ci_cd.show	530.69 KB	1.36 MB	+859.16 KB	161.9 %
pages.projects.show	1.68 MB	1.89 MB	+211 KB	12.2 %
pages.projects.security.policies.index	2.59 MB	2.78 MB	+198.62 KB	7.5 %
pages.groups.security.policies.index	2.45 MB	2.64 MB	+198.42 KB	7.9 %
pages.projects.security.policies.edit	2.52 MB	2.62 MB	+110.18 KB	4.3 %
pages.projects.security.policies.new	2.52 MB	2.62 MB	+110.18 KB	4.3 %
pages.groups.security.policies.new	2.35 MB	2.46 MB	+109.98 KB	4.6 %
pages.admin.broadcast_messages	21.11 KB	22.55 KB	+1.44 KB	6.8 %

Significant Reduction: 4

Expand

Entrypoint / Name	Size before	Size after	Diff	Diff in percent
pages.projects.edit	2.83 MB	2.34 MB	-501.69 KB	-17.3 %
pages.projects.environments.show	1.4 MB	1.3 MB	-98.2 KB	-6.9 %
pages.projects.work_items	935.2 KB	884.46 KB	-50.74 KB	-5.4 %
sandboxed_mermaid	1.08 MB	1.05 MB	-30.35 KB	-2.7 %

---

Your MR has at least one entrypoint growing significantly (more > 1 KB or 2%). If you write new or extend existing features, this is expected and there is nothing to worry about.

Please consider pinging someone from the FE Foundations (@dmishunov, @justin_ho, @mikegreiling or @nmezzopera) for review, if you are unsure about the size increase.

Note: We do not have exact data for a69794b0. So we have used data from: 78964950.
The intended commit has no webpack pipeline, so we chose the last commit with one before it.

Please look at the full report for more details

---

Read more about how this report works.

Generated by Danger

added 799 commits

• 5cc489ee...d2ca4276 - 798 commits from branch master
• 4d979390 - Refactor the usage quotas -> ci/cd minutes page

Compare with previous version

assigned to @pburdette

removed workflowin dev label

Hey @v_mishra I'm helping take this over for @jivanvl

Can you give it a UX review so I can see what needs to be done before I help out with specs etc. It seems similar to #335968 (closed) but maybe we should wait for this issue before introducing an incomplete year selection ? Since it looks to be very limited now since we don't have BE in place yet. Thank you

requested review from @v_mishra

removed review request for @v_mishra

mentioned in issue #345795 (closed)

added 1708 commits

• 4d979390...1310bff4 - 1707 commits from branch master
• a1ce789d - Refactor the usage quotas -> ci/cd minutes page

Compare with previous version

Suggested Reviewers (beta)

This is an experimental ML-based code reviewer recommendation system created by ~"group::applied ml".

The individuals below may be good candidates to participate in the review based on various factors.

After you review all recommendations, please assign reviewers manually, as this is not done automatically.

You can use slash commands in comments to quickly assign /assign_reviewer @user1.

Reviewers
@jivanvl, @peterhegman, @oregand, @mrincon, @psimyn

If you do not believe these recommendations are useful or if you do not want to use any of the suggestions, please apply the label Bad Suggested Reviewer. You can also provide feedback for this feature on this issue: https://gitlab.com/gitlab-org/gitlab/-/issues/357923.

Automatically generated by Suggested Reviewers Bot

added 1 commit

• 749c8689 - Apply UX changes

Compare with previous version

requested review from @v_mishra

added 1 commit

• ddb754e8 - Fix failing specs and refactor

Compare with previous version

changed milestone to %15.0

added 1 commit

• 0204bd1b - Add new test coverage

Compare with previous version

added 1 commit

• b23003a6 - Adjust the margins

Compare with previous version

marked this merge request as ready

changed the description

@richard.chong can you review the test coverage please?

requested review from @richard.chong

Hey @tristan.read I'm helping @jivanvl out on this one.

Can you do the first review please?

requested review from @tristan.read

mentioned in issue #355666 (closed)

added 1 commit

• ee31ba78 - Apply margin to other headers

Compare with previous version

added 1 commit

• 18c9bb3b - Add padding left

Compare with previous version

approved this merge request

removed review request for @v_mishra

@v_mishra, thanks for approving this merge request.

This is the first time the merge request is approved. To ensure full test coverage, a new pipeline has been started.

For more info, please refer to the following links:

• rspec minimal jobs
• jest minimal jobs
• merging a merge request.

added 1 commit

• 8ea22611 - Address frontend review

Compare with previous version

approved this merge request

removed review request for @richard.chong

approved this merge request

requested review from @mrincon

removed review request for @mrincon

added 1 commit

• a144c5dc - Apply maintainer feedback

Compare with previous version

added 1 commit

• d6e6c6ec - Add back formatter

Compare with previous version

requested review from @mrincon and removed review request for @tristan.read

@mrincon I will send MR back to you shortly, I would like to test the group implementation of these components first.

removed review request for @mrincon

added 1 commit

• de333d0b - Fix groups quotas

Compare with previous version

requested review from @mrincon

approved this merge request

resolved all threads

enabled an automatic merge when the pipeline for db28a9bc succeeds

merged

mentioned in commit bdf5fc56

mentioned in issue #360805 (closed)

added workflowstaging-canary label

mentioned in issue #243722 (closed)

added workflowcanary label and removed workflowstaging-canary label

added workflowstaging label and removed workflowcanary label

added workflowproduction label and removed workflowstaging label

mentioned in issue #335968 (closed)

mentioned in issue #361343 (closed)

added releasedcandidate label

mentioned in merge request kubitus-project/kubitus-installer!1040 (merged)

mentioned in issue #357909 (closed)

mentioned in issue #366704 (closed)

removed documentation label