Module: Gitlab::Import::MergeRequestHelpers

Includes:
DatabaseHelpers
Included in:
BitbucketServerImport::Importers::PullRequestNotes::ApprovedEvent, BitbucketServerImport::Importers::PullRequestNotes::DeclinedEvent, BitbucketServerImport::Importers::PullRequestNotes::MergeEvent, BitbucketServerImport::Importers::PullRequestNotesImporter, GithubImport::Importer::PullRequestImporter, GithubImport::Importer::PullRequests::ReviewImporter, MergeRequestCreator
Defined in:
lib/gitlab/import/merge_request_helpers.rb

Instance Method Summary collapse

Methods included from DatabaseHelpers

#insert_and_return_id

Instance Method Details

#add_approval_system_note!(project_id, merge_request_id, user_id, submitted_at) ⇒ Object 



99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
 
# File 'lib/gitlab/import/merge_request_helpers.rb', line 99

def add_approval_system_note!(project_id, merge_request_id, user_id, submitted_at)
  attributes = {
    importing: true,
    noteable_id: merge_request_id,
    noteable_type: 'MergeRequest',
    project_id: project_id,
    author_id: user_id,
    note: 'approved this merge request',
    system: true,
    system_note_metadata: SystemNoteMetadata.new(action: 'approved'),
    created_at: submitted_at,
    updated_at: submitted_at
  }

  Note.create!(attributes)
end

#create_approval!(project_id, merge_request_id, user_id, submitted_at) ⇒ Object 



81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
 
# File 'lib/gitlab/import/merge_request_helpers.rb', line 81

def create_approval!(project_id, merge_request_id, user_id, submitted_at)
  approval = Approval.create(
    merge_request_id: merge_request_id,
    user_id: user_id,
    created_at: submitted_at,
    updated_at: submitted_at,
    importing: true
  )

  return unless approval.persisted?

  note = add_approval_system_note!(project_id, merge_request_id, user_id, submitted_at)

  [approval, note]
rescue ActiveRecord::RecordNotUnique, PG::UniqueViolation
  # Multiple approvals mapped to the same user (e.g. personal namespace owner) can cause validation race condition
end

#create_merge_request_metrics(attributes) ⇒ Object

Returns MergeRequest::Metrics.

Parameters:

  • attributes (Hash)

Returns:

  • MergeRequest::Metrics



10
11
12
13
14
 
# File 'lib/gitlab/import/merge_request_helpers.rb', line 10

def create_merge_request_metrics(attributes)
  metric = MergeRequest::Metrics.find_or_initialize_by(merge_request: merge_request) # rubocop: disable CodeReuse/ActiveRecord -- no need to move this to ActiveRecord model
  metric.update(attributes)
  metric
end

#create_merge_request_without_hooks(project, attributes, iid) ⇒ Object

rubocop: disable CodeReuse/ActiveRecord



17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
 
# File 'lib/gitlab/import/merge_request_helpers.rb', line 17

def create_merge_request_without_hooks(project, attributes, iid)
  # This work must be wrapped in a transaction as otherwise we can leave
  # behind incomplete data in the event of an error. This can then lead
  # to duplicate key errors when jobs are retried.
  MergeRequest.transaction do
    # When creating merge requests there are a lot of hooks that may
    # run, for many different reasons. Many of these hooks (e.g. the
    # ones used for rendering Markdown) are completely unnecessary and
    # may even lead to transaction timeouts.
    #
    # To ensure importing pull requests has a minimal impact and can
    # complete in a reasonable time we bypass all the hooks by inserting
    # the row and then retrieving it. We then only perform the
    # additional work that is strictly necessary.
    merge_request_id = insert_and_return_id(attributes, project.merge_requests)

    merge_request = project.merge_requests.reset.find(merge_request_id)

    [merge_request, false]
  end
rescue ActiveRecord::InvalidForeignKey
  # It's possible the project has been deleted since scheduling this
  # job. In this case we'll just skip creating the merge request.
  []
rescue ActiveRecord::RecordNotUnique
  # It's possible we previously created the MR, but failed when updating
  # the Git data. In this case we'll just continue working on the
  # existing row.
  [project.merge_requests.find_by(iid: iid), true]
end

#create_reviewer!(merge_request_id, user_id, submitted_at) ⇒ Object 



116
117
118
119
120
121
122
123
124
125
126
 
# File 'lib/gitlab/import/merge_request_helpers.rb', line 116

def create_reviewer!(merge_request_id, user_id, submitted_at)
  ::MergeRequestReviewer.create!(
    merge_request_id: merge_request_id,
    user_id: user_id,
    state: ::MergeRequestReviewer.states['reviewed'],
    created_at: submitted_at
  )
rescue ActiveRecord::RecordNotUnique
  # multiple reviews from single person could make a SQL concurrency issue here
  nil
end

#insert_merge_request_reviewers(merge_request, reviewers) ⇒ Object 



74
75
76
77
78
79
 
# File 'lib/gitlab/import/merge_request_helpers.rb', line 74

def insert_merge_request_reviewers(merge_request, reviewers)
  return unless reviewers.present?

  rows = reviewers.map { |reviewer_id| { merge_request_id: merge_request.id, user_id: reviewer_id } }
  MergeRequestReviewer.insert_all(rows)
end

#insert_or_replace_git_data(merge_request, source_branch_sha, target_branch_sha, already_exists = false) ⇒ Object

rubocop: enable CodeReuse/ActiveRecord



49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
 
# File 'lib/gitlab/import/merge_request_helpers.rb', line 49

def insert_or_replace_git_data(merge_request, source_branch_sha, target_branch_sha, already_exists = false)
  # These fields are set so we can create the correct merge request
  # diffs.
  merge_request.source_branch_sha = source_branch_sha
  merge_request.target_branch_sha = target_branch_sha

  merge_request.keep_around_commit

  # We force to recreate all diffs to replace all existing data
  # We use `.all` as otherwise `dependent: :nullify` (the default)
  # takes an effect
  merge_request.merge_request_diffs.all.delete_all if already_exists

  # MR diffs normally use an "after_save" hook to pull data from Git.
  # All of this happens in the transaction started by calling
  # create/save/etc. This in turn can lead to these transactions being
  # held open for much longer than necessary. To work around this we
  # first save the diff, then populate it.
  diff = merge_request.merge_request_diffs.build
  diff.importing = true
  diff.save
  diff.save_git_content
  diff.set_as_latest_diff
end