Simplify the comments

599be28c · Shinya Maeda · 58dc574a · 599be28c
Commit 599be28c authored Jul 04, 2018 by Shinya Maeda
Show whitespace changes
Inline Side-by-side

Showing with 2 additions and 5 deletions

lib/gitlab/ci/trace.rb lib/gitlab/ci/trace.rb +2 -5

No files found.
--- a/lib/gitlab/ci/trace.rb
+++ b/lib/gitlab/ci/trace.rb
@@ -100,14 +100,11 @@ module Gitlab
      def erase!
        ##
-        # Erase an archived traces
+        # Erase the archived trace
-        # This removes both a database-row and a real file in either a file storage or a object storage
        trace_artifact&.destroy!
        ##
-        # Erase a live trace
+        # Erase the live trace
-        # Basically, jobs have _one_ of the following live traces, but it might be able to happen by a race condition
-        # Therefore, we remove all type of live traces.
        job.trace_chunks.fast_destroy_all # Destroy chunks of a live trace
        FileUtils.rm_f(current_path) if current_path # Remove a trace file of a live trace
        job.erase_old_trace! if job.has_old_trace? # Remove a trace in database of a live trace