Skip to content
Snippets Groups Projects

Changes limit for lsif artifacts to 100MB

Merged Igor Drozdov requested to merge id-change-default-for-ci-max-artifact-size-lsif into master

What does this MR do?

Related issue: #249561 (closed)

After the recent fix of the in-memory processing, the memory usage reduced significantly for a large file, maybe we can try increasing the limit without waiting for gitlab-workhorse#270 (moved).

Up:

== 20201105135051 ChangeDefaultValueOfCiMaxArtifactSizeLsifOfPlanLimitsFrom20To100: migrating
-- change_column_default(:plan_limits, :ci_max_artifact_size_lsif, 100)
   -> 0.0133s
-- execute("UPDATE plan_limits SET ci_max_artifact_size_lsif = 100 WHERE ci_max_artifact_size_lsif = 20")
   -> 0.0063s
== 20201105135051 ChangeDefaultValueOfCiMaxArtifactSizeLsifOfPlanLimitsFrom20To100: migrated (0.0316s)

Down:

== 20201105135051 ChangeDefaultValueOfCiMaxArtifactSizeLsifOfPlanLimitsFrom20To100: reverting
-- change_column_default(:plan_limits, :ci_max_artifact_size_lsif, 20)
   -> 0.0072s
-- execute("UPDATE plan_limits SET ci_max_artifact_size_lsif = 20 WHERE ci_max_artifact_size_lsif = 100")
   -> 0.0007s
== 20201105135051 ChangeDefaultValueOfCiMaxArtifactSizeLsifOfPlanLimitsFrom20To100: reverted (0.0134s)
Edited by Igor Drozdov

Merge request reports

Merged results pipeline #213406889 passed

Merged results pipeline passed for 8ae7636c

Test coverage 70.97% from 3 jobs

Merged by Tiger WatsonTiger Watson 4 years ago (Nov 9, 2020 12:39am UTC)

Loading

Pipeline #213414266 passed

Pipeline passed for 5ee6edc1 on master

Test coverage 57.58% from 3 jobs
5 environments impacted.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading