Add subsidized cost factor for Open Source plan
What does this MR do and why?
- Related to #346399 (closed) (main issue)
- Related to #343134 (closed) (related because I've added
Open Source program
to the list of SaaS plans inPlan
model)
With the new Open Source plan been added specifically for those namespaces member of the Open Source program we are now able to distinguish open source projects from those in either Free or paid plans.
As we aim to harden CI minutes limits for all public projects we need to first ensure we don't impact public projects that are part of the Open Source program. For this reason, in this MR we are lowering the cost factor to 0.008
for those open source projects, making the Open Source program more appealing for open source projects and ensuring we don't impact them when we are going to raise the cost factor to 1.0
for public projects in upcoming milestones.
Lowering the cost factor means that CI minutes consumption moves at a slower rate (currently 125x slower than on private projects), effectively allowing a namespace to use more CI minutes. The limit cost factor 0.008
is low enough to ensure that a namespace can use 125x more CI minutes than their monthly allowance, but it'a also a non-zero value which means that the consumption is still capped.
Screenshots or screen recordings
These are strongly recommended to assist reviewers and reduce the time to merge your change.
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.
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.
Merge request reports
Activity
changed milestone to %14.7
assigned to @fabiopitino
- Resolved by Fabio Pitino
requested review from @marcel.amirault
- Resolved by Fabio Pitino
requested review from @shreyasagarwal
- A deleted user
added documentation label
1 Message This merge request adds or changes documentation files. A review from the Technical Writing team before you merge is recommended. Reviews can happen after you merge. Documentation review
The following files require a review from a technical writer:
doc/ci/pipelines/cicd_minutes.md
The review does not need to block merging this merge request. See the:
-
Metadata for the
*.md
files that you've changed. The first few lines of each*.md
file identify the stage and group most closely associated with your docs change. - The Technical Writer assigned for that stage and group.
- Documentation workflows for information on when to assign a merge request for review.
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 Drew Blessing ( @dblessing
) (UTC-6, 6 hours behind@fabiopitino
)Vasilii Iakliushin ( @vyaklushin
) (UTC+1, 1 hour ahead of@fabiopitino
)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.
If needed, you can retry the
danger-review
job that generated this comment.Generated by
Dangermarked the checklist item I have evaluated the MR acceptance checklist for this MR. as completed
- Resolved by Fabio Pitino
Setting label(s) devopsverify sectionops based on grouppipeline execution.
added devopsverify sectionops labels
added Technical Writing docsfeature labels
removed review request for @marcel.amirault
added 1 commit
- 34189eef - Add subsidized cost factor for Open Source plan
@marcel.amirault
, 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:
requested review from @vij and removed review request for @shreyasagarwal
- Resolved by Fabio Pitino
- Resolved by Fabio Pitino
- Resolved by Vijay Hawoldar
@fabiopitino this LGTM! I've left a few non-blocking comments so I'll pass back to you incase you wanted to take a look, but approving anyway!
removed review request for @vij
added 1 commit
- d68fd0d2 - Add subsidized cost factor for Open Source plan
- Resolved by Marius Bobin
@mbobin could you please review this?
requested review from @mbobin
- Resolved by Fabio Pitino
- Resolved by Fabio Pitino
added 1 commit
- f71cc952 - Add subsidized cost factor for Open Source plan
mentioned in issue #350325 (closed)
Thanks, @fabiopitino! Looks good!
enabled an automatic merge when the pipeline for 2920f360 succeeds
mentioned in commit 6f09c2fa
added workflowstaging-canary label
added workflowstaging label and removed workflowstaging-canary label
added workflowcanary label and removed workflowstaging label
added workflowproduction label and removed workflowcanary label
added releasedcandidate label
added releasedpublished label and removed releasedcandidate label
mentioned in merge request kubitus-project/kubitus-installer!562 (merged)