Import notes from Bitbucket
What does this MR do and why?
Makes the BitBucket Cloud importer parallel instead of running sequentially in order to fix migration timeouts. This follows the same framework as the GitHub importer and the BitBucket Server importer.
The work is split into the following MRs:
- Import repository and wiki and mark the import as complete (feature flagged)
- Import pull requests: Import Bitbucket pull requests (!131134 - merged)
- Import issues: Import Bitbucket issues (!131138 - merged)
-
Import notes
👈 this MR - Refactoring MR(s) for reusing shared code between BitBucket Cloud and BitBucket Server
The feature is behind bitbucket_parallel_importer
feature flag so that it is safe to continue releasing bits without affecting the importer. The feature flag is controlled in lib/gitlab/import_sources.rb
.
Important notes
- This MR imports MR and issue notes
- Issue and MR refs are converted in
lib/gitlab/bitbucket_import/importer.rb
How to set up and validate locally
- Follow https://docs.gitlab.com/ee/integration/bitbucket.html to setup OAuth for BitBucket Cloud. You will need an account on https://bitbucket.org/. Make sure you add the bitbucket configuration in the
development
section of yourconfig/gitlab.yml
as shown at https://gitlab.com/gitlab-org/gitlab-development-kit/-/blob/main/doc/howto/gitlab-oauth2.md#set-up-gdk - Create a project and repo on BitBucket. Create a few issues and MRs with notes.
- Enable the feature flag:
Feature.enable(:bitbucket_parallel_importer)
- On your gdk server, create a new project > click on
Import project
>Bitbucket Cloud
> follow instructions to connect to https://bitbucket.org/. - Import the project created in step 2.
- Verify that the notes were imported correctly on both MRs and issues.
- You can also view the importer logs to see when each step was executed or if there are errors:
tail -f log/importer.log
Bitbucket | Gitlab | |
---|---|---|
MR notes with issue and MR refs | ||
Issue note with MR ref |
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #412614 (closed)
Edited by Madelein van Niekerk