summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKamil Trzcinski <ayufan@ayufan.eu>2016-10-19 14:03:31 +0200
committerKamil Trzcinski <ayufan@ayufan.eu>2016-10-19 14:04:09 +0200
commit67b96255d89de8e31234d68a3408adb8b9779dd9 (patch)
tree98999ab9a0901cebea1af02925f889ca17694b1b
parenta96756008a634961171e968d76faea68ab2bcb95 (diff)
downloadgitlab-ce-dont-touch-fs-on-pipeline-save.tar.gz
Keep around commits only on pipeline createdont-touch-fs-on-pipeline-save
-rw-r--r--CHANGELOG.md1
-rw-r--r--app/models/ci/pipeline.rb2
2 files changed, 2 insertions, 1 deletions
diff --git a/CHANGELOG.md b/CHANGELOG.md
index f4d317fd1db..44c0b80bf6d 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -16,6 +16,7 @@ Please view this file on the master branch, on stable branches it's out of date.
- Ability to resolve merge request conflicts with editor !6374
- Add `/projects/visible` API endpoint (Ben Boeckel)
- Fix centering of custom header logos (Ashley Dumaine)
+ - Keep around commits only pipeline creation as pipeline data doesn't change over time
- ExpireBuildArtifactsWorker query builds table without ordering enqueuing one job per build to cleanup
- Add an example for testing a phoenix application with Gitlab CI in the docs (Manthan Mallikarjun)
- Cancelled pipelines could be retried. !6927
diff --git a/app/models/ci/pipeline.rb b/app/models/ci/pipeline.rb
index e75fe6c222b..e84c91b417d 100644
--- a/app/models/ci/pipeline.rb
+++ b/app/models/ci/pipeline.rb
@@ -19,7 +19,7 @@ module Ci
validates_presence_of :status, unless: :importing?
validate :valid_commit_sha, unless: :importing?
- after_save :keep_around_commits, unless: :importing?
+ after_create :keep_around_commits, unless: :importing?
delegate :stages, to: :statuses