Skip to content

Update Premium and Ultimate pipeline_triggers plan limit to 25k

Leaminn Ma requested to merge update-pipeline-triggers-limit-premium-ultimate into master

What does this MR do and why?

As seen in #350577 (comment 1890651451), the limits for pipeline_triggers are not consistent across all plans for GitLab.com. This is because either new plans were introduced or the limits were not migrated when the plan names were updated. This MR adds a migration to update the paid plans (Premium and Ultimate) to 25k. The other plans were updated in !145846 (merged).

We confirmed in #350577 (comment 1890651451) that customers under the relevant plans are all well under 25k. We also have a confirmation to proceed with this change here.

Resolves #350577 (closed).

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Query plan

Up

main: == [advisory_lock_connection] object_id: 124580, pg_backend_pid: 50947
main: == 20240502194436 UpdatePipelineTriggersPremiumUltimatePlanLimits: migrating ==
main: == 20240502194436 UpdatePipelineTriggersPremiumUltimatePlanLimits: migrated (0.0026s) 

main: == [advisory_lock_connection] object_id: 124580, pg_backend_pid: 50947
ci: == [advisory_lock_connection] object_id: 124820, pg_backend_pid: 50949
ci: == 20240502194436 UpdatePipelineTriggersPremiumUltimatePlanLimits: migrating ==
ci: -- The migration is skipped since it modifies the schemas: [:gitlab_main].
ci: -- This database can only apply migrations in one of the following schemas: [:gitlab_ci, :gitlab_internal, :gitlab_shared].
ci: == 20240502194436 UpdatePipelineTriggersPremiumUltimatePlanLimits: migrated (0.0064s) 

ci: == [advisory_lock_connection] object_id: 124820, pg_backend_pid: 50949

Down

main: == [advisory_lock_connection] object_id: 124200, pg_backend_pid: 51462
main: == 20240502194436 UpdatePipelineTriggersPremiumUltimatePlanLimits: reverting ==
main: == 20240502194436 UpdatePipelineTriggersPremiumUltimatePlanLimits: reverted (0.0034s) 

main: == [advisory_lock_connection] object_id: 124200, pg_backend_pid: 51462
ci: == [advisory_lock_connection] object_id: 124200, pg_backend_pid: 51879
ci: == 20240502194436 UpdatePipelineTriggersPremiumUltimatePlanLimits: reverting ==
ci: -- The migration is skipped since it modifies the schemas: [:gitlab_main].
ci: -- This database can only apply migrations in one of the following schemas: [:gitlab_ci, :gitlab_internal, :gitlab_shared].
ci: == 20240502194436 UpdatePipelineTriggersPremiumUltimatePlanLimits: reverted (0.0090s) 

ci: == [advisory_lock_connection] object_id: 124200, pg_backend_pid: 51879

Related to #350577 (closed)

Edited by Leaminn Ma

Merge request reports