Skip to content
Snippets Groups Projects

Re-enable gitaly migration for ref_name_for_sha after bugfixes

Merged username-removed-367626 requested to merge reenable-gitaly-ref-name into master
All threads resolved!

Conversation: https://gitlab.com/gitlab-org/gitaly/issues/221

What does this MR do?

Re-enable gitaly migration for ref_name_for_sha after bugfixes

Are there points in the code the reviewer needs to double check?

No

Why was this MR needed?

The migration had to be disabled because of https://gitlab.com/gitlab-org/gitaly/issues/180, which has been fixed now.

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • username-removed-487110 approved this merge request

    approved this merge request

  • Other than the comment it LGTM :)

  • added 1 commit

    • 6b5923d4 - Re-enable gitaly migration for ref_name_for_sha after bugfixes

    Compare with previous version

  • Andrew Newdigate changed milestone to %9.2

    changed milestone to %9.2

  • added Gitaly label

  • mentioned in issue gitaly#221

  • @eReGeBe LGTM, please address Jacob's comment.

  • assigned to @eReGeBe

  • added 272 commits

    Compare with previous version

  • @rspeicher regarding ee-compat-check, it seems that EE master is somewhat outdated: it has an older GITALY_SERVER_VERSION and GitalyClient code. The conflict is very straightforward to fix, can we resolve it on the CE Upstream merge?

  • Robert Speicher resolved all discussions

    resolved all discussions

  • username-removed-487110 approved this merge request

    approved this merge request

  • Robert Speicher mentioned in commit e66b811d

    mentioned in commit e66b811d

  • Picked into 9-2-stable will go into 9-2 RC6

  • Robert Speicher mentioned in commit a72a3cc2

    mentioned in commit a72a3cc2

  • Please register or sign in to reply
    Loading